Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-17 Thread Wolfram Sang
> > Thanks for this. This is a very rare case that’s easy to slip through. > > It is good to be consistent :) > > I read this as acked-by for the series? Ah, I mixed up. You acked V2 :) signature.asc Description: PGP signature

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-17 Thread Wolfram Sang
> > Thanks for this. This is a very rare case that’s easy to slip through. > > It is good to be consistent :) > > I read this as acked-by for the series? Ah, I mixed up. You acked V2 :) signature.asc Description: PGP signature

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-17 Thread Wolfram Sang
On Mon, Oct 17, 2016 at 10:20:47PM +0300, Pantelis Antoniou wrote: > Hi Ralf, > > > On Oct 16, 2016, at 12:55 , Ralf Ramsauer > > wrote: > > > > Hi Geert, > > > > On 10/16/2016 10:49 AM, Geert Uytterhoeven wrote: > >> Hi Ralf, > >> > >> (Cc i2c) > >> > >> On

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-17 Thread Wolfram Sang
On Mon, Oct 17, 2016 at 10:20:47PM +0300, Pantelis Antoniou wrote: > Hi Ralf, > > > On Oct 16, 2016, at 12:55 , Ralf Ramsauer > > wrote: > > > > Hi Geert, > > > > On 10/16/2016 10:49 AM, Geert Uytterhoeven wrote: > >> Hi Ralf, > >> > >> (Cc i2c) > >> > >> On Fri, Oct 14, 2016 at 9:31 PM,

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-17 Thread Pantelis Antoniou
Hi Ralf, > On Oct 16, 2016, at 12:55 , Ralf Ramsauer wrote: > > Hi Geert, > > On 10/16/2016 10:49 AM, Geert Uytterhoeven wrote: >> Hi Ralf, >> >> (Cc i2c) >> >> On Fri, Oct 14, 2016 at 9:31 PM, Ralf Ramsauer >> wrote: >>>

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-17 Thread Pantelis Antoniou
Hi Ralf, > On Oct 16, 2016, at 12:55 , Ralf Ramsauer wrote: > > Hi Geert, > > On 10/16/2016 10:49 AM, Geert Uytterhoeven wrote: >> Hi Ralf, >> >> (Cc i2c) >> >> On Fri, Oct 14, 2016 at 9:31 PM, Ralf Ramsauer >> wrote: >>> Instantiated SPI device nodes are marked with OF_POPULATE. This was

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-16 Thread Ralf Ramsauer
Hi Geert, On 10/16/2016 10:49 AM, Geert Uytterhoeven wrote: > Hi Ralf, > > (Cc i2c) > > On Fri, Oct 14, 2016 at 9:31 PM, Ralf Ramsauer > wrote: >> Instantiated SPI device nodes are marked with OF_POPULATE. This was >> introduced in bd6c164. On unloading, loaded

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-16 Thread Ralf Ramsauer
Hi Geert, On 10/16/2016 10:49 AM, Geert Uytterhoeven wrote: > Hi Ralf, > > (Cc i2c) > > On Fri, Oct 14, 2016 at 9:31 PM, Ralf Ramsauer > wrote: >> Instantiated SPI device nodes are marked with OF_POPULATE. This was >> introduced in bd6c164. On unloading, loaded device nodes will of course >>

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-16 Thread Wolfram Sang
On Sun, Oct 16, 2016 at 10:49:11AM +0200, Geert Uytterhoeven wrote: > Hi Ralf, > > (Cc i2c) Thanks for letting me know! Adding Pantelis to CC, as he is the original author of OF_DYNAMIC. Please keep me in the loop. > > On Fri, Oct 14, 2016 at 9:31 PM, Ralf Ramsauer >

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-16 Thread Wolfram Sang
On Sun, Oct 16, 2016 at 10:49:11AM +0200, Geert Uytterhoeven wrote: > Hi Ralf, > > (Cc i2c) Thanks for letting me know! Adding Pantelis to CC, as he is the original author of OF_DYNAMIC. Please keep me in the loop. > > On Fri, Oct 14, 2016 at 9:31 PM, Ralf Ramsauer > wrote: > > Instantiated

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-16 Thread Geert Uytterhoeven
Hi Ralf, (Cc i2c) On Fri, Oct 14, 2016 at 9:31 PM, Ralf Ramsauer wrote: > Instantiated SPI device nodes are marked with OF_POPULATE. This was > introduced in bd6c164. On unloading, loaded device nodes will of course > be unmarked. The problem are nodes the fail

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-16 Thread Geert Uytterhoeven
Hi Ralf, (Cc i2c) On Fri, Oct 14, 2016 at 9:31 PM, Ralf Ramsauer wrote: > Instantiated SPI device nodes are marked with OF_POPULATE. This was > introduced in bd6c164. On unloading, loaded device nodes will of course > be unmarked. The problem are nodes the fail during initialisation: If a >

[PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-14 Thread Ralf Ramsauer
Instantiated SPI device nodes are marked with OF_POPULATE. This was introduced in bd6c164. On unloading, loaded device nodes will of course be unmarked. The problem are nodes the fail during initialisation: If a node failed during registration, it won't be unloaded and hence never be unmarked

[PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-14 Thread Ralf Ramsauer
Instantiated SPI device nodes are marked with OF_POPULATE. This was introduced in bd6c164. On unloading, loaded device nodes will of course be unmarked. The problem are nodes the fail during initialisation: If a node failed during registration, it won't be unloaded and hence never be unmarked