On Wed, Jun 26, 2019 at 10:44:03AM +0200, Linus Walleij wrote: > gpiochip_remove() was called on the errorpath if > gpiochip_add() failed: this is wrong, if the chip failed > to add it is not there so it should not be removed. > > Cc: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Acked-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> Does this warrant a Fixes: be8c8facc707 ("gpio: new driver to work with a 8x12 siox") ? Thanks Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ |