Re: [PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-29 Thread Will Deacon
On Thu, Nov 29, 2018 at 09:17:38AM -0700, Nathan Chancellor wrote: > On Thu, Nov 29, 2018 at 04:13:37PM +, Will Deacon wrote: > > On Thu, Nov 29, 2018 at 09:10:39AM -0700, Nathan Chancellor wrote: > > > This doesn't appear to work, I get this error: > > > > > > In file included from arch/arm64

Re: [PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-29 Thread Nathan Chancellor
On Thu, Nov 29, 2018 at 04:13:37PM +, Will Deacon wrote: > On Thu, Nov 29, 2018 at 09:10:39AM -0700, Nathan Chancellor wrote: > > On Thu, Nov 29, 2018 at 10:49:03AM +, Will Deacon wrote: > > > On Thu, Nov 29, 2018 at 09:03:54AM +, Julien Thierry wrote: > > > > On 29/11/18 04:19, Nick De

Re: [PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-29 Thread Russell King - ARM Linux
On Thu, Nov 29, 2018 at 09:10:39AM -0700, Nathan Chancellor wrote: > On Thu, Nov 29, 2018 at 10:49:03AM +, Will Deacon wrote: > > On Thu, Nov 29, 2018 at 09:03:54AM +, Julien Thierry wrote: > > > > > > > > > On 29/11/18 04:19, Nick Desaulniers wrote: > > > > Fixes the warning produced from

Re: [PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-29 Thread Will Deacon
On Thu, Nov 29, 2018 at 09:10:39AM -0700, Nathan Chancellor wrote: > On Thu, Nov 29, 2018 at 10:49:03AM +, Will Deacon wrote: > > On Thu, Nov 29, 2018 at 09:03:54AM +, Julien Thierry wrote: > > > On 29/11/18 04:19, Nick Desaulniers wrote: > > > > Fixes the warning produced from Clang: > > >

Re: [PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-29 Thread Nathan Chancellor
On Thu, Nov 29, 2018 at 10:49:03AM +, Will Deacon wrote: > On Thu, Nov 29, 2018 at 09:03:54AM +, Julien Thierry wrote: > > > > > > On 29/11/18 04:19, Nick Desaulniers wrote: > > > Fixes the warning produced from Clang: > > > ./include/asm-generic/io.h:711:9: warning: value size does not mat

Re: [PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-29 Thread Will Deacon
On Thu, Nov 29, 2018 at 09:03:54AM +, Julien Thierry wrote: > > > On 29/11/18 04:19, Nick Desaulniers wrote: > > Fixes the warning produced from Clang: > > ./include/asm-generic/io.h:711:9: warning: value size does not match > > register size specified by the constraint and modifier > > [-Was

Re: [PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-29 Thread Julien Thierry
On 29/11/18 04:19, Nick Desaulniers wrote: > Fixes the warning produced from Clang: > ./include/asm-generic/io.h:711:9: warning: value size does not match > register size specified by the constraint and modifier > [-Wasm-operand-widths] > return readl(addr); >^ > ./arch/ar

[PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-28 Thread Nick Desaulniers
Fixes the warning produced from Clang: ./include/asm-generic/io.h:711:9: warning: value size does not match register size specified by the constraint and modifier [-Wasm-operand-widths] return readl(addr); ^ ./arch/arm64/include/asm/io.h:149:58: note: expanded from macro 'rea