[PATCH u-boot-marvell 01/11] include/linux/byteorder: Fix compilation of __constant_cpu_to_be32()

[Stefan Roese]

On 11/3/21 03:02, Marek Behún wrote:
> From: Pali Rohár <pali at kernel.org>
> 
> The macro __constant_cpu_to_be32() uses ___constant_swab32(), which for
> some reason is not defined and causes the following error during
> compilation:
> 
>    include/linux/byteorder/little_endian.h:28:52: warning:
>      implicit declaration of function ‘___constant_swab32’;
>      did you mean ‘__builtin_bswap32’? [-Wimplicit-function-declaration]
>     #define __constant_cpu_to_be32(x) ((__force __be32)___constant_swab32((x)))
> 
> Declare all ___constant_swabXX() macros.
> 
> Signed-off-by: Pali Rohár <pali at kernel.org>
> Signed-off-by: Marek Behún <marek.behun at nic.cz>

Reviewed-by: Stefan Roese <sr at denx.de>

Thanks,
Stefan

> ---
>   include/linux/byteorder/swab.h | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/include/linux/byteorder/swab.h b/include/linux/byteorder/swab.h
> index 4334fa77e3..5efc252acf 100644
> --- a/include/linux/byteorder/swab.h
> +++ b/include/linux/byteorder/swab.h
> @@ -39,6 +39,10 @@
>   		(__u64)(((__u64)(x) & (__u64)0x00ff000000000000ULL) >> 40) | \
>   		(__u64)(((__u64)(x) & (__u64)0xff00000000000000ULL) >> 56) ))
>   
> +#define ___constant_swab16(x) ___swab16(x)
> +#define ___constant_swab32(x) ___swab32(x)
> +#define ___constant_swab64(x) ___swab64(x)
> +
>   /*
>    * provide defaults when no architecture-specific optimization is detected
>    */
> 

Viele Grüße,
Stefan Roese

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de