[U-Boot] [PATCH v2 1/7] ARM: prepare armv7.h to be included from assembly source

Masahiro Yamada yamada.m at jp.panasonic.com
Fri Jun 28 03:00:37 CEST 2013


Hello Andre,

> diff --git a/arch/arm/include/asm/armv7.h b/arch/arm/include/asm/armv7.h
> index a73630b..20caa7c 100644
> --- a/arch/arm/include/asm/armv7.h
> +++ b/arch/arm/include/asm/armv7.h
> @@ -23,7 +23,6 @@
>   */
>  #ifndef ARMV7_H
>  #define ARMV7_H
> -#include <linux/types.h>
>  
>  /* Cortex-A9 revisions */
>  #define MIDR_CORTEX_A9_R0P1	0x410FC091
> @@ -57,6 +56,9 @@
>  #define ARMV7_CLIDR_CTYPE_INSTRUCTION_DATA	3
>  #define ARMV7_CLIDR_CTYPE_UNIFIED		4
>  
> +#ifndef __ASSEMBLY__
> +#include <linux/types.h>
> +
>  /*
>   * CP15 Barrier instructions
>   * Please note that we have separate barrier instructions in ARMv7
> @@ -74,4 +76,6 @@ void v7_outer_cache_inval_all(void);
>  void v7_outer_cache_flush_range(u32 start, u32 end);
>  void v7_outer_cache_inval_range(u32 start, u32 end);
>  
> +#endif /* ! __ASSEMBLY__ */
> +
>  #endif


Instread of moving #include <linux/types.h> line,
I'd like to suggest to add #ifndef __ASSEMBLY__ guard
to include/linux/types.h.
I think this is a more correct way of fixing.


If I see Linux Kernel "include/linux/types.h",
typedefs are placed inside #ifndef  __ASSEMBLEY__ .. #endif.

On the other hand, "include/linux/types.h" of U-Boot
does not have #ifndef __ASSEMBLY__ guard.



Best Regards
Masahiro Yamada



More information about the U-Boot mailing list