OSDN Git Service

[media] xc4000: cleanup dmesg logging
authorDevin Heitmueller <dheitmueller@kernellabs.com>
Sun, 27 Dec 2009 21:15:14 +0000 (18:15 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 27 Jul 2011 20:52:33 +0000 (17:52 -0300)
Remove some printk() calls added during driver development, and demote some
other messages to debug only (to reduce dmesg chatter).

Signed-off-by: Devin Heitmueller <dheitmueller@kernellabs.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/common/tuners/xc4000.c

index 02edd94..bad9c9d 100644 (file)
@@ -639,12 +639,6 @@ static int seek_firmware(struct dvb_frontend *fe, unsigned int type,
        int                 i, best_i = -1, best_nr_matches = 0;
        unsigned int        type_mask = 0;
 
-       printk("%s called, want type=", __func__);
-       if (debug) {
-               dump_firm_type(type);
-               printk("(%x), id %016llx.\n", type, (unsigned long long)*id);
-       }
-
        if (!priv->firm) {
                printk("Error! firmware not loaded\n");
                return -EINVAL;
@@ -715,12 +709,11 @@ found:
        *id = priv->firm[i].id;
 
 ret:
-       printk("%s firmware for type=", (i < 0) ? "Can't find" : "Found");
        if (debug) {
+               printk("%s firmware for type=", (i < 0) ? "Can't find" :
+                      "Found");
                dump_firm_type(type);
                printk("(%x), id %016llx.\n", type, (unsigned long long)*id);
-               if (i < 0)
-                       dump_stack();
        }
        return i;
 }
@@ -732,19 +725,11 @@ static int load_firmware(struct dvb_frontend *fe, unsigned int type,
        int                pos, rc;
        unsigned char      *p;
 
-       printk("%s called\n", __func__);
-
        pos = seek_firmware(fe, type, id);
        if (pos < 0)
                return pos;
 
-       printk("Loading firmware for type=");
-//     dump_firm_type(priv->firm[pos].type);
-       printk("(%x), id %016llx.\n", priv->firm[pos].type,
-              (unsigned long long)*id);
-
        p = priv->firm[pos].ptr;
-       printk("firmware length = %d\n", priv->firm[pos].size);
 
        /* Don't complain when the request fails because of i2c stretching */
        priv->ignore_i2c_write_errors = 1;
@@ -766,8 +751,6 @@ static int xc4000_fwupload(struct dvb_frontend *fe)
        char                  name[33];
        char                  *fname;
 
-       printk("%s called\n", __func__);
-
        fname = XC4000_DEFAULT_FIRMWARE;
 
        printk("Reading firmware %s\n",  fname);
@@ -801,9 +784,9 @@ static int xc4000_fwupload(struct dvb_frontend *fe)
        n_array = get_unaligned_le16(p);
        p += 2;
 
-       printk("Loading %d firmware images from %s, type: %s, ver %d.%d\n",
-                  n_array, fname, name,
-                  priv->firm_version >> 8, priv->firm_version & 0xff);
+       dprintk(1, "Loading %d firmware images from %s, type: %s, ver %d.%d\n",
+               n_array, fname, name,
+               priv->firm_version >> 8, priv->firm_version & 0xff);
 
        priv->firm = kzalloc(sizeof(*priv->firm) * n_array, GFP_KERNEL);
        if (priv->firm == NULL) {
@@ -899,7 +882,7 @@ err:
 done:
        release_firmware(fw);
        if (rc == 0)
-               printk("Firmware files loaded.\n");
+               dprintk(1, "Firmware files loaded.\n");
 
        return rc;
 }
@@ -1060,8 +1043,6 @@ retry:
                goto fail;
        }
 
-       printk("Done with init1\n");
-
 skip_base:
        /*
         * No need to reload standard specific firmware if base firmware