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

[Greg KH <gregkh@linuxfoundation.org>]

On Sun, Nov 06, 2016 at 02:15:07PM +0100, Ralf Ramsauer wrote:
> commit 6a676fb69dcbf3310b9e462c1db66c8e7f6ead38 upstream.
> 
> Instantiated I2C device nodes are marked with OF_POPULATE. This was
> introduced in 4f001fd30145a6. On unloading, loaded device nodes will of
> course be unmarked. The problem are nodes that fail during
> initialisation: If a node fails, it won't be unloaded and hence not be
> unmarked.
> 
> If a I2C driver module is unloaded and reloaded, it will skip nodes that
> failed before.
> 
> Skip device nodes that are already populated and mark them only in case
> of success.
> 
> Fixes: 4f001fd30145a6 ("i2c: Mark instantiated device nodes with OF_POPULATE")
> Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Pantelis Antoniou <pantelis.antoniou@konsulko.com>
> [wsa: use 14-digit commit sha]
> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
> Cc: stable@kernel.org
> ---
> Hi Greg,
> 
> this one is the backport for 4.8.y.  Only hunks and surrounding context lines
> shifted slightly, no functional changes compared to its upstream commit
> 6a676fb69dcbf.

Thanks for the backport, now applied.

greg k-h