[PATCH-next] arm: fix clang build warning in include/asm/memory.h

[Russell King (Oracle)]

On Fri, Mar 15, 2024 at 08:40:28AM +0100, Linus Walleij wrote:
> +1 and I would probably use this as well:
> 
> <linux/limits.h>
> 
> #define IDMAP_INVALID_ADDR    PHYS_ADDR_MAX
> 
> if (IS_ENABLED(CONFIG_PHYS_ADDR_T_64BIT) && addr > PHYS_ADDR_MAX)
>     addr = IDMAP_INVALID_ADDR;
> 
> Because then it is clear what is going on: we are capping to the max physical
> address.
> 
> PHYS_ADDR_MAX is defined to (~(phys_addr_t)0) which on
> ARM is (~(u32)0).

... which is _not_ the same as (u32)~0, so using PHYS_ADDR_MAX is not
appropriate.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!