Lee Jones <lee.jo...@linaro.org> wrote: >At the moment the driver is silent in some error cases and if >successful. >Prior to this patch there was no clear way to know if the driver >succeeded >or not without looking deep into sysfs. > >Signed-off-by: Lee Jones <lee.jo...@linaro.org> >--- > drivers/iio/pressure/st_pressure_core.c | 3 +++ > 1 file changed, 3 insertions(+) > >diff --git a/drivers/iio/pressure/st_pressure_core.c >b/drivers/iio/pressure/st_pressure_core.c >index 6ffd949..34b3fb1 100644 >--- a/drivers/iio/pressure/st_pressure_core.c >+++ b/drivers/iio/pressure/st_pressure_core.c >@@ -280,6 +280,9 @@ int st_press_common_probe(struct iio_dev >*indio_dev, > if (err && irq > 0) > goto st_press_device_register_error; > >+ if (!err) >+ dev_info(&indio_dev->dev, "Successfully registered\n"); >+ Not keen. That to my mind is pointless noise. If this made sense then it would be in the driver core not here. > return err; > > st_press_device_register_error:
-- Sent from my Android phone with K-9 Mail. Please excuse my brevity. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
