2012-06-22 23:37:07

by Hartley Sweeten

[permalink] [raw]
Subject: [PATCH 27/31] staging: comedi: ni_daq_700: cleanup pcmcia debug output messages

These messages should probably just be removed. For now just clean
then up.

Remove a couple redundant KERN_INFO messages in the pcmcia support
code and leave the dev_dbg() ones.

Change the dev_dgb() messages to use __func__ instead of the open
coded string.

Signed-off-by: H Hartley Sweeten <[email protected]>
Cc: Ian Abbott <[email protected]>
Cc: Frank Mori Hess <[email protected]>
Cc: Greg Kroah-hartman <[email protected]>
---
drivers/staging/comedi/drivers/ni_daq_700.c | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/drivers/staging/comedi/drivers/ni_daq_700.c b/drivers/staging/comedi/drivers/ni_daq_700.c
index cc46948..bb857ac4 100644
--- a/drivers/staging/comedi/drivers/ni_daq_700.c
+++ b/drivers/staging/comedi/drivers/ni_daq_700.c
@@ -166,7 +166,7 @@ static struct comedi_driver driver_dio700 = {

static void dio700_release(struct pcmcia_device *link)
{
- dev_dbg(&link->dev, "dio700_release\n");
+ dev_dbg(&link->dev, "%s\n", __func__);

pcmcia_disable_device(link);
}
@@ -184,9 +184,7 @@ static void dio700_config(struct pcmcia_device *link)
{
int ret;

- printk(KERN_INFO "ni_daq_700: cs-config\n");
-
- dev_dbg(&link->dev, "dio700_config\n");
+ dev_dbg(&link->dev, "%s\n", __func__);

link->config_flags |= CONF_ENABLE_IRQ | CONF_AUTO_AUDIO |
CONF_AUTO_SET_IO;
@@ -207,7 +205,7 @@ static void dio700_config(struct pcmcia_device *link)
return;

failed:
- printk(KERN_INFO "ni_daq_700 cs failed");
+ dev_dbg(&link->dev, "%s failed\n", __func__);
dio700_release(link);

}
@@ -222,9 +220,7 @@ static int dio700_cs_attach(struct pcmcia_device *link)
{
struct local_info_t *local;

- printk(KERN_INFO "ni_daq_700: cs-attach\n");
-
- dev_dbg(&link->dev, "dio700_cs_attach()\n");
+ dev_dbg(&link->dev, "%s\n", __func__);

/* Allocate space for private device-specific data */
local = kzalloc(sizeof(struct local_info_t), GFP_KERNEL);
--
1.7.11