Re: [PATCH 10/15] ARM: orion5x: cleanup cppcheck shifting errors

From: Andrew Lunn
Date: Sun Jun 23 2019 - 11:24:27 EST


On Sun, Jun 23, 2019 at 10:13:08PM +0700, Phong Tran wrote:
> [arch/arm/mach-orion5x/pci.c:281]: (error) Shifting signed 32-bit value
> by 31 bits is undefined behaviour
> [arch/arm/mach-orion5x/pci.c:305]: (error) Shifting signed 32-bit value
> by 31 bits is undefined behaviour
>
> Signed-off-by: Phong Tran <tranmanphong@xxxxxxxxx>

Maybe using the BIT() macro would be better, but this is O.K.

Reviewed-by: Andrew Lunn <andrew@xxxxxxx>

Andrew