pinctrl: pinconf: separate config parameters with commas for debugfs
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Wed, 25 May 2016 06:37:27 +0000 (15:37 +0900)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 31 May 2016 08:29:06 +0000 (10:29 +0200)
To improve debugfs readability, use commas instead of whitespaces
for separating configuration parameters.

For example, the "pinconf-pins" dump on my board will change as follows:

Without this commit:

 # head -5 pinconf-pins
 Pin config settings per pin
 Format: pin (name): configs
 pin 0 (ED0): input bias pull down output drive strength (8 mA) input enabled
 pin 1 (ED1): input bias pull down output drive strength (8 mA) input enabled
 pin 2 (ED2): input bias pull down output drive strength (8 mA) input enabled

With this commit:

 # head -5 pinconf-pins
 Pin config settings per pin
 Format: pin (name): configs
 pin 0 (ED0): input bias pull down, output drive strength (8 mA), input enabled
 pin 1 (ED1): input bias pull down, output drive strength (8 mA), input enabled
 pin 2 (ED2): input bias pull down, output drive strength (8 mA), input enabled

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinconf-generic.c
drivers/pinctrl/pinconf.c

index d5bf9fae2ddd425cf32570b668a2c0a5b3aec59f..34b601b06764bdc1e0664878ff973b6eabf16497 100644 (file)
@@ -53,7 +53,7 @@ static void pinconf_generic_dump_one(struct pinctrl_dev *pctldev,
                                     struct seq_file *s, const char *gname,
                                     unsigned pin,
                                     const struct pin_config_item *items,
-                                    int nitems)
+                                    int nitems, int *print_sep)
 {
        int i;
 
@@ -75,8 +75,10 @@ static void pinconf_generic_dump_one(struct pinctrl_dev *pctldev,
                        seq_printf(s, "ERROR READING CONFIG SETTING %d ", i);
                        continue;
                }
-               /* Space between multiple configs */
-               seq_puts(s, " ");
+               /* comma between multiple configs */
+               if (*print_sep)
+                       seq_puts(s, ", ");
+               *print_sep = 1;
                seq_puts(s, items[i].display);
                /* Print unit if available */
                if (items[i].has_arg) {
@@ -105,19 +107,21 @@ void pinconf_generic_dump_pins(struct pinctrl_dev *pctldev, struct seq_file *s,
                               const char *gname, unsigned pin)
 {
        const struct pinconf_ops *ops = pctldev->desc->confops;
+       int print_sep = 0;
 
        if (!ops->is_generic)
                return;
 
        /* generic parameters */
        pinconf_generic_dump_one(pctldev, s, gname, pin, conf_items,
-                                ARRAY_SIZE(conf_items));
+                                ARRAY_SIZE(conf_items), &print_sep);
        /* driver-specific parameters */
        if (pctldev->desc->num_custom_params &&
            pctldev->desc->custom_conf_items)
                pinconf_generic_dump_one(pctldev, s, gname, pin,
                                         pctldev->desc->custom_conf_items,
-                                        pctldev->desc->num_custom_params);
+                                        pctldev->desc->num_custom_params,
+                                        &print_sep);
 }
 
 void pinconf_generic_dump_config(struct pinctrl_dev *pctldev,
index 3f1b6f03abc9dd9aece08c124e38eee0ab16c134..799048f3c8d4ec271c5c8849d3a8f17c8ac4e1ff 100644 (file)
@@ -310,7 +310,7 @@ static int pinconf_pins_show(struct seq_file *s, void *what)
                if (desc == NULL)
                        continue;
 
-               seq_printf(s, "pin %d (%s):", pin, desc->name);
+               seq_printf(s, "pin %d (%s): ", pin, desc->name);
 
                pinconf_dump_pin(pctldev, s, pin);
 
@@ -347,7 +347,7 @@ static int pinconf_groups_show(struct seq_file *s, void *what)
        while (selector < ngroups) {
                const char *gname = pctlops->get_group_name(pctldev, selector);
 
-               seq_printf(s, "%u (%s):", selector, gname);
+               seq_printf(s, "%u (%s): ", selector, gname);
                pinconf_dump_group(pctldev, s, selector, gname);
                seq_printf(s, "\n");