On Thursday 17 March 2016 10:10:40 H Hartley Sweeten wrote:
> Arnd Bergmann pointed out that gcc-6 warns about passing negative signed
> integer into swab16() due to the macro expansion of 'outw'.
> 
> It appears that the register map constants are causing the warnings.
> Actually, it might just be the (1 << 15) ones...
> 
> Convert all the constants as suggested by checkpatch.pl:
> CHECK: Prefer using the BIT macro
> 
> The BIT() macro will make all the constants explicitly 'unsigned', which
> helps to avoid the warning.
> 
> Fix the, unsused, DT2821_CHANCSR_PRESLA() macro. The "Present List
> Address" (PRESLA) bits in the CHANCSR register are read only. This
> define was meant to extract the bits from the read value.
> 
> Signed-off-by: H Hartley Sweeten <[email protected]>
> Reported-by: Arnd Bergmann <[email protected]>
> Cc: Ian Abbott <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> 

Tested-by: Arnd Bergmann <[email protected]>
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to