pinctrl: mxs: avoid defines prefixed with CONFIG
authorLukas Bulwahn <lukas.bulwahn@gmail.com>
Thu, 15 Dec 2022 09:21:28 +0000 (10:21 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 29 Dec 2022 01:05:16 +0000 (02:05 +0100)
Defines prefixed with "CONFIG" should be limited to proper Kconfig options,
that are introduced in a Kconfig file.

Here, expressions to convert pin configurations to booleans for pull-up,
voltage and mA are macro definitions that begin with "CONFIG".

To avoid defines prefixed with "CONFIG", rename these defines to begin with
"PIN_CONFIG" instead.

No functional change.

Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Link: https://lore.kernel.org/r/20221215092128.3954-1-lukas.bulwahn@gmail.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/freescale/pinctrl-mxs.c
drivers/pinctrl/freescale/pinctrl-mxs.h

index 9f78c9b..cf3f4d2 100644 (file)
@@ -269,9 +269,9 @@ static int mxs_pinconf_group_set(struct pinctrl_dev *pctldev,
        for (n = 0; n < num_configs; n++) {
                config = configs[n];
 
-               ma = CONFIG_TO_MA(config);
-               vol = CONFIG_TO_VOL(config);
-               pull = CONFIG_TO_PULL(config);
+               ma = PIN_CONFIG_TO_MA(config);
+               vol = PIN_CONFIG_TO_VOL(config);
+               pull = PIN_CONFIG_TO_PULL(config);
 
                for (i = 0; i < g->npins; i++) {
                        bank = PINID_TO_BANK(g->pins[i]);
index ab9f834..5b26511 100644 (file)
@@ -44,9 +44,9 @@
 #define VOL_SHIFT              3
 #define MA_PRESENT             (1 << 2)
 #define MA_SHIFT               0
-#define CONFIG_TO_PULL(c)      ((c) >> PULL_SHIFT & 0x1)
-#define CONFIG_TO_VOL(c)       ((c) >> VOL_SHIFT & 0x1)
-#define CONFIG_TO_MA(c)                ((c) >> MA_SHIFT & 0x3)
+#define PIN_CONFIG_TO_PULL(c)  ((c) >> PULL_SHIFT & 0x1)
+#define PIN_CONFIG_TO_VOL(c)   ((c) >> VOL_SHIFT & 0x1)
+#define PIN_CONFIG_TO_MA(c)    ((c) >> MA_SHIFT & 0x3)
 
 struct mxs_function {
        const char *name;