Am 05.01.2014 13:05, schrieb Mauro Carvalho Chehab: > Fix the error handling logic, making it to unregister i2c bus 0, in > case of a failure to register the second bus. > > Reported-by: Frank Schäfer <fschaefer.oss@xxxxxxxxxxxxxx> > > Signed-off-by: Mauro Carvalho Chehab <m.chehab@xxxxxxxxxxx> > --- > drivers/media/usb/em28xx/em28xx-cards.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/media/usb/em28xx/em28xx-cards.c b/drivers/media/usb/em28xx/em28xx-cards.c > index 541de6df127b..dbce4dc421f9 100644 > --- a/drivers/media/usb/em28xx/em28xx-cards.c > +++ b/drivers/media/usb/em28xx/em28xx-cards.c > @@ -2989,6 +2989,9 @@ static int em28xx_init_dev(struct em28xx *dev, struct usb_device *udev, > if (retval < 0) { > em28xx_errdev("%s: em28xx_i2c_register bus 1 - error [%d]!\n", > __func__, retval); > + > + em28xx_i2c_unregister(dev, 0); > + > return retval; > } > } Reviewed-by: Frank Schäfer <fschaefer.oss@xxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html