regulator: dbx500: Remove use of seq_puts/seq_printf return value

The seq_puts/seq_printf return value, because it's frequently misused,
will eventually be converted to void.

See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
     seq_has_overflowed() and make public")

Miscellanea:

o Remove unnecessary dev_err("seq_<foo> overflow\n") messages

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Mark Brown <broonie@kernel.org>

1 files changed, 9 insertions, 23 deletions

diff --git a/drivers/regulator/dbx500-prcmu.c b/drivers/regulator/dbx500-prcmu.c
index 2d16b9f16de7..3963dfad766c 100644
--- a/drivers/regulator/dbx500-prcmu.c
+++ b/drivers/regulator/dbx500-prcmu.c
@@ -95,14 +95,9 @@ void ux500_regulator_resume_debug(void)
 
 static int ux500_regulator_power_state_cnt_print(struct seq_file *s, void *p)
 {
-        struct device *dev = s->private;
-        int err;
-
         /* print power state count */
-        err = seq_printf(s, "ux500-regulator power state count: %i\n",
-                power_state_active_get());
-        if (err < 0)
-                dev_err(dev, "seq_printf overflow\n");
+        seq_printf(s, "ux500-regulator power state count: %i\n",
+                   power_state_active_get());
 
         return 0;
 }
@@ -124,19 +119,11 @@ static const struct file_operations ux500_regulator_power_state_cnt_fops = {
 
 static int ux500_regulator_status_print(struct seq_file *s, void *p)
 {
-        struct device *dev = s->private;
-        int err;
         int i;
 
         /* print dump header */
-        err = seq_puts(s, "ux500-regulator status:\n");
-        if (err < 0)
-                dev_err(dev, "seq_puts overflow\n");
-
-        err = seq_printf(s, "%31s : %8s : %8s\n", "current",
-                "before", "after");
-        if (err < 0)
-                dev_err(dev, "seq_printf overflow\n");
+        seq_puts(s, "ux500-regulator status:\n");
+        seq_printf(s, "%31s : %8s : %8s\n", "current", "before", "after");
 
         for (i = 0; i < rdebug.num_regulators; i++) {
                 struct dbx500_regulator_info *info;
@@ -144,12 +131,11 @@ static int ux500_regulator_status_print(struct seq_file *s, void *p)
                 info = &rdebug.regulator_array[i];
 
                 /* print status */
-                err = seq_printf(s, "%20s : %8s : %8s : %8s\n", info->desc.name,
-                        info->is_enabled ? "enabled" : "disabled",
-                        rdebug.state_before_suspend[i] ? "enabled" : "disabled",
-                        rdebug.state_after_suspend[i] ? "enabled" : "disabled");
-                if (err < 0)
-                        dev_err(dev, "seq_printf overflow\n");
+                seq_printf(s, "%20s : %8s : %8s : %8s\n",
+                           info->desc.name,
+                           info->is_enabled ? "enabled" : "disabled",
+                           rdebug.state_before_suspend[i] ? "enabled" : "disabled",
+                           rdebug.state_after_suspend[i] ? "enabled" : "disabled");
         }
 
         return 0;