[PATCH xserver] Add definitions for the AArch64 architecture

Alan Coopersmith alan.coopersmith at oracle.com
Thu Jan 3 13:30:28 PST 2013


On 01/ 3/13 07:33 AM, Thomas Petazzoni wrote:
> AArch64 is the new 64 bits architecture from ARM, for which a few
> definitions are needed in the X.org server to make it build properly.
> 
> Like for the ARM 32 bits architecture, we for now assume that AArch64
> will be used in Little Endian mode for Linux.
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> ---
>  include/servermd.h |    9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/include/servermd.h b/include/servermd.h
> index d6a9a3a..c2d7d70 100644
> --- a/include/servermd.h
> +++ b/include/servermd.h
> @@ -272,6 +272,15 @@ SOFTWARE.
>  
>  #endif                          /* linux/m68k */
>  
> +/* linux on AArch64 */
> +#if defined(linux) && defined(__aarch64__)
> +
> +#define IMAGE_BYTE_ORDER       LSBFirst
> +#define BITMAP_BIT_ORDER       LSBFirst
> +#define GLYPHPADBYTES          4
> +
> +#endif
> +
>  /* linux on ARM */
>  #if defined(linux) && defined(__arm__)
>  #define IMAGE_BYTE_ORDER	LSBFirst

Since you're defining the same values as the next section, why not just
make it #if defined(linux) && (defined(__arm__) || defined(__aarch64__))
instead of duplicating them?

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list