Re: [U-Boot] [RFC PATCH] bitops: Fix GENMASK definition for Sandbox

2019-01-21 Thread Simon Glass
On Fri, 11 Jan 2019 at 01:56, Simon Glass wrote: > > On Mon, 17 Dec 2018 at 06:31, Vignesh R wrote: > > > > In arch/sandbox/include/asm/types.h we have > > Therefore for 32 bit Sandbox build BITS_PER_LONG turns out to be 32 as > > CONFIG_PHYS64 is not set > > > > This messes up the current logic

Re: [U-Boot] [RFC PATCH] bitops: Fix GENMASK definition for Sandbox

2019-01-10 Thread Simon Glass
On Mon, 17 Dec 2018 at 06:31, Vignesh R wrote: > > In arch/sandbox/include/asm/types.h we have > Therefore for 32 bit Sandbox build BITS_PER_LONG turns out to be 32 as > CONFIG_PHYS64 is not set > > This messes up the current logic of GENMASK macro due to mismatch b/w > size of unsigned long (64 b

Re: [U-Boot] [RFC PATCH] bitops: Fix GENMASK definition for Sandbox

2019-01-09 Thread Vignesh R
On 17/12/18 7:02 PM, Vignesh R wrote: > In arch/sandbox/include/asm/types.h we have > Therefore for 32 bit Sandbox build BITS_PER_LONG turns out to be 32 as > CONFIG_PHYS64 is not set > > This messes up the current logic of GENMASK macro due to mismatch b/w > size of unsigned long (64 bit) and t

[U-Boot] [RFC PATCH] bitops: Fix GENMASK definition for Sandbox

2018-12-17 Thread Vignesh R
In arch/sandbox/include/asm/types.h we have Therefore for 32 bit Sandbox build BITS_PER_LONG turns out to be 32 as CONFIG_PHYS64 is not set This messes up the current logic of GENMASK macro due to mismatch b/w size of unsigned long (64 bit) and that of BITS_PER_LONG. Fix this by using CONFIG_SANDB