[Buildroot] [PATCH] arch/Config.in.xtensa: provide BR2_ENDIAN symbol

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Tue May 5 15:45:00 UTC 2015


Dear Max Filippov,

On Tue,  5 May 2015 18:42:12 +0300, Max Filippov wrote:
> Signed-off-by: Max Filippov <jcmvbkbc at gmail.com>
> ---
>  arch/Config.in.xtensa | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)

Thanks for coming up so quickly with a patch for this!

I think it's important to mention that this is fixing an autobuilder
failure, and put the pointer to the corresponding failure.

> +choice
> +	prompt "Target Architecture Endianness"
> +	depends on BR2_XTENSA_CUSTOM
> +	default BR2_XTENSA_LITTLE_ENDIAN
> +config BR2_XTENSA_LITTLE_ENDIAN
> +	bool "Little endian"
> +config BR2_XTENSA_BIG_ENDIAN
> +	bool "Big endian"
> +endchoice

A little bit of spacing between options would be good I believe.

Other than that, looks good to me.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list