Em Sun, 19 Jan 2014 22:48:37 +0100 Frank Schäfer <fschaefer....@googlemail.com> escreveu:
> Signed-off-by: Frank Schäfer <fschaefer....@googlemail.com> > --- > drivers/media/usb/em28xx/em28xx-i2c.c | 11 +++-------- > 1 Datei geändert, 3 Zeilen hinzugefügt(+), 8 Zeilen entfernt(-) > > diff --git a/drivers/media/usb/em28xx/em28xx-i2c.c > b/drivers/media/usb/em28xx/em28xx-i2c.c > index a26d7d4..1a514ca 100644 > --- a/drivers/media/usb/em28xx/em28xx-i2c.c > +++ b/drivers/media/usb/em28xx/em28xx-i2c.c > @@ -535,14 +535,9 @@ static int em28xx_i2c_xfer(struct i2c_adapter *i2c_adap, > * This code is only called during device probe. > */ > rc = i2c_check_for_device(i2c_bus, addr); > - if (rc < 0) { > - if (rc == -ENXIO) { > - if (i2c_debug > 1) > - printk(KERN_CONT " no > device\n"); > - } else { > - if (i2c_debug > 1) > - printk(KERN_CONT " ERROR: > %i\n", rc); > - } > + if (rc == -ENXIO) { > + if (i2c_debug > 1) > + printk(KERN_CONT " no device\n"); Even if the previous patch were accepted, this one is wrong, as -ENXIO doesn't always mean that there's no device. Also, other return codes may happen here (like -EIO). > rt_mutex_unlock(&dev->i2c_bus_lock); > return rc; > } -- Cheers, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html