2012-08-20 21:31:20

by Hartley Sweeten

[permalink] [raw]
Subject: [PATCH 4/4] staging: comedi: comedi_parport: remove the printk noise

Remove the function trace message in parport_intr_cancel() and the
"bogus irq..." message is parport_interrupt(). They are just added
noise.

Signed-off-by: H Hartley Sweeten <[email protected]>
Cc: Ian Abbott <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
---
drivers/staging/comedi/drivers/comedi_parport.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/staging/comedi/drivers/comedi_parport.c b/drivers/staging/comedi/drivers/comedi_parport.c
index 48d529e..2dce9df 100644
--- a/drivers/staging/comedi/drivers/comedi_parport.c
+++ b/drivers/staging/comedi/drivers/comedi_parport.c
@@ -265,8 +265,6 @@ static int parport_intr_cancel(struct comedi_device *dev,
{
struct parport_private *devpriv = dev->private;

- printk(KERN_DEBUG "parport_intr_cancel()\n");
-
devpriv->c_data &= ~0x10;
outb(devpriv->c_data, dev->iobase + PARPORT_C);

@@ -281,10 +279,8 @@ static irqreturn_t parport_interrupt(int irq, void *d)
struct parport_private *devpriv = dev->private;
struct comedi_subdevice *s = dev->subdevices + 3;

- if (!devpriv->enable_irq) {
- printk(KERN_ERR "comedi_parport: bogus irq, ignored\n");
+ if (!devpriv->enable_irq)
return IRQ_NONE;
- }

comedi_buf_put(s->async, 0);
s->async->events |= COMEDI_CB_BLOCK | COMEDI_CB_EOS;
--
1.7.11