Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-30 Thread Arnd Bergmann
On Tuesday, November 22, 2016 7:47:52 PM CET Vladimir Zapolskiy wrote: > > On 11/22/2016 04:30 PM, Arnd Bergmann wrote: > > A recent patch added a new function that is now unused whenever > > CONFIG_OF is disabled: > > > > drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but not >

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-30 Thread Arnd Bergmann
On Tuesday, November 22, 2016 7:47:52 PM CET Vladimir Zapolskiy wrote: > > On 11/22/2016 04:30 PM, Arnd Bergmann wrote: > > A recent patch added a new function that is now unused whenever > > CONFIG_OF is disabled: > > > > drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but not >

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-29 Thread Alexandre Belloni
On 29/11/2016 at 21:01:53 +0100, Greg Kroah-Hartman wrote : > On Tue, Nov 29, 2016 at 08:54:00PM +0100, Alexandre Belloni wrote: > > On 29/11/2016 at 20:41:54 +0100, Greg Kroah-Hartman wrote : > > > On Tue, Nov 22, 2016 at 03:30:54PM +0100, Arnd Bergmann wrote: > > > > A recent patch added a new

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-29 Thread Alexandre Belloni
On 29/11/2016 at 21:01:53 +0100, Greg Kroah-Hartman wrote : > On Tue, Nov 29, 2016 at 08:54:00PM +0100, Alexandre Belloni wrote: > > On 29/11/2016 at 20:41:54 +0100, Greg Kroah-Hartman wrote : > > > On Tue, Nov 22, 2016 at 03:30:54PM +0100, Arnd Bergmann wrote: > > > > A recent patch added a new

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-29 Thread Greg Kroah-Hartman
On Tue, Nov 29, 2016 at 08:54:00PM +0100, Alexandre Belloni wrote: > On 29/11/2016 at 20:41:54 +0100, Greg Kroah-Hartman wrote : > > On Tue, Nov 22, 2016 at 03:30:54PM +0100, Arnd Bergmann wrote: > > > A recent patch added a new function that is now unused whenever > > > CONFIG_OF is disabled: > >

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-29 Thread Greg Kroah-Hartman
On Tue, Nov 29, 2016 at 08:54:00PM +0100, Alexandre Belloni wrote: > On 29/11/2016 at 20:41:54 +0100, Greg Kroah-Hartman wrote : > > On Tue, Nov 22, 2016 at 03:30:54PM +0100, Arnd Bergmann wrote: > > > A recent patch added a new function that is now unused whenever > > > CONFIG_OF is disabled: > >

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-29 Thread Alexandre Belloni
On 29/11/2016 at 20:41:54 +0100, Greg Kroah-Hartman wrote : > On Tue, Nov 22, 2016 at 03:30:54PM +0100, Arnd Bergmann wrote: > > A recent patch added a new function that is now unused whenever > > CONFIG_OF is disabled: > > > > drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-29 Thread Alexandre Belloni
On 29/11/2016 at 20:41:54 +0100, Greg Kroah-Hartman wrote : > On Tue, Nov 22, 2016 at 03:30:54PM +0100, Arnd Bergmann wrote: > > A recent patch added a new function that is now unused whenever > > CONFIG_OF is disabled: > > > > drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-29 Thread Greg Kroah-Hartman
On Tue, Nov 22, 2016 at 03:30:54PM +0100, Arnd Bergmann wrote: > A recent patch added a new function that is now unused whenever > CONFIG_OF is disabled: > > drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but not used > [-Werror=unused-function] > > There is actually no reason

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-29 Thread Greg Kroah-Hartman
On Tue, Nov 22, 2016 at 03:30:54PM +0100, Arnd Bergmann wrote: > A recent patch added a new function that is now unused whenever > CONFIG_OF is disabled: > > drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but not used > [-Werror=unused-function] > > There is actually no reason

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-22 Thread Alexandre Belloni
Hi, On 22/11/2016 at 19:47:52 +0200, Vladimir Zapolskiy wrote : > Hi Arnd, > > On 11/22/2016 04:30 PM, Arnd Bergmann wrote: > > A recent patch added a new function that is now unused whenever > > CONFIG_OF is disabled: > > > > drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-22 Thread Alexandre Belloni
Hi, On 22/11/2016 at 19:47:52 +0200, Vladimir Zapolskiy wrote : > Hi Arnd, > > On 11/22/2016 04:30 PM, Arnd Bergmann wrote: > > A recent patch added a new function that is now unused whenever > > CONFIG_OF is disabled: > > > > drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-22 Thread Vladimir Zapolskiy
Hi Arnd, On 11/22/2016 04:30 PM, Arnd Bergmann wrote: A recent patch added a new function that is now unused whenever CONFIG_OF is disabled: drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but not used [-Werror=unused-function] There is actually no reason for the #ifdef,

Re: [PATCH] misc: sram: remove useless #ifdef

2016-11-22 Thread Vladimir Zapolskiy
Hi Arnd, On 11/22/2016 04:30 PM, Arnd Bergmann wrote: A recent patch added a new function that is now unused whenever CONFIG_OF is disabled: drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but not used [-Werror=unused-function] There is actually no reason for the #ifdef,

[PATCH] misc: sram: remove useless #ifdef

2016-11-22 Thread Arnd Bergmann
A recent patch added a new function that is now unused whenever CONFIG_OF is disabled: drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but not used [-Werror=unused-function] There is actually no reason for the #ifdef, because the driver currently cannot be used in a meaningful

[PATCH] misc: sram: remove useless #ifdef

2016-11-22 Thread Arnd Bergmann
A recent patch added a new function that is now unused whenever CONFIG_OF is disabled: drivers/misc/sram.c:342:12: error: 'atmel_securam_wait' defined but not used [-Werror=unused-function] There is actually no reason for the #ifdef, because the driver currently cannot be used in a meaningful