On 2014-01-07 21:43, Bernd Porr wrote:
Comedi core now reports that a device has been attached so that
the driver itself won't need to do it any longer. The driver now
just outputs the offset of the ADC converter which is a soft indicator
of the health of the board and also the user can grep this value
from the kernel log easier for debugging purposes.

Signed-off-by: Bernd Porr <m...@berndporr.me.uk>
---
  drivers/staging/comedi/drivers/usbduxsigma.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/drivers/usbduxsigma.c 
b/drivers/staging/comedi/drivers/usbduxsigma.c
index ff521b3..3beeb12 100644
--- a/drivers/staging/comedi/drivers/usbduxsigma.c
+++ b/drivers/staging/comedi/drivers/usbduxsigma.c
@@ -1661,7 +1661,7 @@ static int usbduxsigma_auto_attach(struct comedi_device 
*dev,
                return offset;
        }

-       dev_info(dev->class_dev, "attached, ADC_zero = %x\n", offset);
+       dev_info(dev->class_dev, "ADC_zero = %x\n", offset);

        return 0;
  }


Reviewed-by: Ian Abbott <abbo...@mev.co.uk>

(I guess a few other drivers will also have unnecessary "attached" dev_info messages now as well.)

--
-=( Ian Abbott @ MEV Ltd.    E-mail: <abbo...@mev.co.uk>        )=-
-=( Tel: +44 (0)161 477 1898   FAX: +44 (0)161 718 3587         )=-
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to