docg3: Fix miuse of seq_printf return value
authorJoe Perches <joe@perches.com>
Mon, 29 Sep 2014 23:08:27 +0000 (16:08 -0700)
committerBrian Norris <computersforpeace@gmail.com>
Wed, 22 Oct 2014 05:28:57 +0000 (22:28 -0700)
seq_printf doesn't return a useful value, so remove
these misuses.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
drivers/mtd/devices/docg3.c

index 21cc4b66feaabdd24d24557df73b60b3ae07f8e0..68ff83cc6660a06e37c675bfbf58339ad69d9da9 100644 (file)
@@ -1655,22 +1655,21 @@ static int dbg_flashctrl_show(struct seq_file *s, void *p)
 {
        struct docg3 *docg3 = (struct docg3 *)s->private;
 
-       int pos = 0;
        u8 fctrl;
 
        mutex_lock(&docg3->cascade->lock);
        fctrl = doc_register_readb(docg3, DOC_FLASHCONTROL);
        mutex_unlock(&docg3->cascade->lock);
 
-       pos += seq_printf(s,
-                "FlashControl : 0x%02x (%s,CE# %s,%s,%s,flash %s)\n",
-                fctrl,
-                fctrl & DOC_CTRL_VIOLATION ? "protocol violation" : "-",
-                fctrl & DOC_CTRL_CE ? "active" : "inactive",
-                fctrl & DOC_CTRL_PROTECTION_ERROR ? "protection error" : "-",
-                fctrl & DOC_CTRL_SEQUENCE_ERROR ? "sequence error" : "-",
-                fctrl & DOC_CTRL_FLASHREADY ? "ready" : "not ready");
-       return pos;
+       seq_printf(s, "FlashControl : 0x%02x (%s,CE# %s,%s,%s,flash %s)\n",
+                  fctrl,
+                  fctrl & DOC_CTRL_VIOLATION ? "protocol violation" : "-",
+                  fctrl & DOC_CTRL_CE ? "active" : "inactive",
+                  fctrl & DOC_CTRL_PROTECTION_ERROR ? "protection error" : "-",
+                  fctrl & DOC_CTRL_SEQUENCE_ERROR ? "sequence error" : "-",
+                  fctrl & DOC_CTRL_FLASHREADY ? "ready" : "not ready");
+
+       return 0;
 }
 DEBUGFS_RO_ATTR(flashcontrol, dbg_flashctrl_show);
 
@@ -1678,58 +1677,56 @@ static int dbg_asicmode_show(struct seq_file *s, void *p)
 {
        struct docg3 *docg3 = (struct docg3 *)s->private;
 
-       int pos = 0, pctrl, mode;
+       int pctrl, mode;
 
        mutex_lock(&docg3->cascade->lock);
        pctrl = doc_register_readb(docg3, DOC_ASICMODE);
        mode = pctrl & 0x03;
        mutex_unlock(&docg3->cascade->lock);
 
-       pos += seq_printf(s,
-                        "%04x : RAM_WE=%d,RSTIN_RESET=%d,BDETCT_RESET=%d,WRITE_ENABLE=%d,POWERDOWN=%d,MODE=%d%d (",
-                        pctrl,
-                        pctrl & DOC_ASICMODE_RAM_WE ? 1 : 0,
-                        pctrl & DOC_ASICMODE_RSTIN_RESET ? 1 : 0,
-                        pctrl & DOC_ASICMODE_BDETCT_RESET ? 1 : 0,
-                        pctrl & DOC_ASICMODE_MDWREN ? 1 : 0,
-                        pctrl & DOC_ASICMODE_POWERDOWN ? 1 : 0,
-                        mode >> 1, mode & 0x1);
+       seq_printf(s,
+                  "%04x : RAM_WE=%d,RSTIN_RESET=%d,BDETCT_RESET=%d,WRITE_ENABLE=%d,POWERDOWN=%d,MODE=%d%d (",
+                  pctrl,
+                  pctrl & DOC_ASICMODE_RAM_WE ? 1 : 0,
+                  pctrl & DOC_ASICMODE_RSTIN_RESET ? 1 : 0,
+                  pctrl & DOC_ASICMODE_BDETCT_RESET ? 1 : 0,
+                  pctrl & DOC_ASICMODE_MDWREN ? 1 : 0,
+                  pctrl & DOC_ASICMODE_POWERDOWN ? 1 : 0,
+                  mode >> 1, mode & 0x1);
 
        switch (mode) {
        case DOC_ASICMODE_RESET:
-               pos += seq_puts(s, "reset");
+               seq_puts(s, "reset");
                break;
        case DOC_ASICMODE_NORMAL:
-               pos += seq_puts(s, "normal");
+               seq_puts(s, "normal");
                break;
        case DOC_ASICMODE_POWERDOWN:
-               pos += seq_puts(s, "powerdown");
+               seq_puts(s, "powerdown");
                break;
        }
-       pos += seq_puts(s, ")\n");
-       return pos;
+       seq_puts(s, ")\n");
+       return 0;
 }
 DEBUGFS_RO_ATTR(asic_mode, dbg_asicmode_show);
 
 static int dbg_device_id_show(struct seq_file *s, void *p)
 {
        struct docg3 *docg3 = (struct docg3 *)s->private;
-       int pos = 0;
        int id;
 
        mutex_lock(&docg3->cascade->lock);
        id = doc_register_readb(docg3, DOC_DEVICESELECT);
        mutex_unlock(&docg3->cascade->lock);
 
-       pos += seq_printf(s, "DeviceId = %d\n", id);
-       return pos;
+       seq_printf(s, "DeviceId = %d\n", id);
+       return 0;
 }
 DEBUGFS_RO_ATTR(device_id, dbg_device_id_show);
 
 static int dbg_protection_show(struct seq_file *s, void *p)
 {
        struct docg3 *docg3 = (struct docg3 *)s->private;
-       int pos = 0;
        int protect, dps0, dps0_low, dps0_high, dps1, dps1_low, dps1_high;
 
        mutex_lock(&docg3->cascade->lock);
@@ -1742,45 +1739,40 @@ static int dbg_protection_show(struct seq_file *s, void *p)
        dps1_high = doc_register_readw(docg3, DOC_DPS1_ADDRHIGH);
        mutex_unlock(&docg3->cascade->lock);
 
-       pos += seq_printf(s, "Protection = 0x%02x (",
-                        protect);
+       seq_printf(s, "Protection = 0x%02x (", protect);
        if (protect & DOC_PROTECT_FOUNDRY_OTP_LOCK)
-               pos += seq_puts(s, "FOUNDRY_OTP_LOCK,");
+               seq_puts(s, "FOUNDRY_OTP_LOCK,");
        if (protect & DOC_PROTECT_CUSTOMER_OTP_LOCK)
-               pos += seq_puts(s, "CUSTOMER_OTP_LOCK,");
+               seq_puts(s, "CUSTOMER_OTP_LOCK,");
        if (protect & DOC_PROTECT_LOCK_INPUT)
-               pos += seq_puts(s, "LOCK_INPUT,");
+               seq_puts(s, "LOCK_INPUT,");
        if (protect & DOC_PROTECT_STICKY_LOCK)
-               pos += seq_puts(s, "STICKY_LOCK,");
+               seq_puts(s, "STICKY_LOCK,");
        if (protect & DOC_PROTECT_PROTECTION_ENABLED)
-               pos += seq_puts(s, "PROTECTION ON,");
+               seq_puts(s, "PROTECTION ON,");
        if (protect & DOC_PROTECT_IPL_DOWNLOAD_LOCK)
-               pos += seq_puts(s, "IPL_DOWNLOAD_LOCK,");
+               seq_puts(s, "IPL_DOWNLOAD_LOCK,");
        if (protect & DOC_PROTECT_PROTECTION_ERROR)
-               pos += seq_puts(s, "PROTECT_ERR,");
+               seq_puts(s, "PROTECT_ERR,");
        else
-               pos += seq_puts(s, "NO_PROTECT_ERR");
-       pos += seq_puts(s, ")\n");
-
-       pos += seq_printf(s, "DPS0 = 0x%02x : "
-                        "Protected area [0x%x - 0x%x] : OTP=%d, READ=%d, "
-                        "WRITE=%d, HW_LOCK=%d, KEY_OK=%d\n",
-                        dps0, dps0_low, dps0_high,
-                        !!(dps0 & DOC_DPS_OTP_PROTECTED),
-                        !!(dps0 & DOC_DPS_READ_PROTECTED),
-                        !!(dps0 & DOC_DPS_WRITE_PROTECTED),
-                        !!(dps0 & DOC_DPS_HW_LOCK_ENABLED),
-                        !!(dps0 & DOC_DPS_KEY_OK));
-       pos += seq_printf(s, "DPS1 = 0x%02x : "
-                        "Protected area [0x%x - 0x%x] : OTP=%d, READ=%d, "
-                        "WRITE=%d, HW_LOCK=%d, KEY_OK=%d\n",
-                        dps1, dps1_low, dps1_high,
-                        !!(dps1 & DOC_DPS_OTP_PROTECTED),
-                        !!(dps1 & DOC_DPS_READ_PROTECTED),
-                        !!(dps1 & DOC_DPS_WRITE_PROTECTED),
-                        !!(dps1 & DOC_DPS_HW_LOCK_ENABLED),
-                        !!(dps1 & DOC_DPS_KEY_OK));
-       return pos;
+               seq_puts(s, "NO_PROTECT_ERR");
+       seq_puts(s, ")\n");
+
+       seq_printf(s, "DPS0 = 0x%02x : Protected area [0x%x - 0x%x] : OTP=%d, READ=%d, WRITE=%d, HW_LOCK=%d, KEY_OK=%d\n",
+                  dps0, dps0_low, dps0_high,
+                  !!(dps0 & DOC_DPS_OTP_PROTECTED),
+                  !!(dps0 & DOC_DPS_READ_PROTECTED),
+                  !!(dps0 & DOC_DPS_WRITE_PROTECTED),
+                  !!(dps0 & DOC_DPS_HW_LOCK_ENABLED),
+                  !!(dps0 & DOC_DPS_KEY_OK));
+       seq_printf(s, "DPS1 = 0x%02x : Protected area [0x%x - 0x%x] : OTP=%d, READ=%d, WRITE=%d, HW_LOCK=%d, KEY_OK=%d\n",
+                  dps1, dps1_low, dps1_high,
+                  !!(dps1 & DOC_DPS_OTP_PROTECTED),
+                  !!(dps1 & DOC_DPS_READ_PROTECTED),
+                  !!(dps1 & DOC_DPS_WRITE_PROTECTED),
+                  !!(dps1 & DOC_DPS_HW_LOCK_ENABLED),
+                  !!(dps1 & DOC_DPS_KEY_OK));
+       return 0;
 }
 DEBUGFS_RO_ATTR(protection, dbg_protection_show);