projects / linux.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: d0918a8)
gpio-f7188x: use unique labels for banks/chips
authorHenning Schild <henning.schild@siemens.com>
Thu, 25 Aug 2022 10:44:19 +0000 (12:44 +0200)
committerHans de Goede <hdegoede@redhat.com>
Thu, 1 Sep 2022 14:14:35 +0000 (16:14 +0200)
So that drivers building on top can find those pins with GPIO_LOOKUP
helpers.

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Acked-by: Simon Guinot <simon.guinot@sequanux.org>
Signed-off-by: Henning Schild <henning.schild@siemens.com>
Acked-by: Bartosz Golaszewski <brgl@bgdev.pl>
Link: https://lore.kernel.org/r/20220825104422.14156-5-henning.schild@siemens.com
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/gpio/gpio-f7188x.c patch | blob | history

diff --git a/drivers/gpio/gpio-f7188x.c b/drivers/gpio/gpio-f7188x.c
index 29c73ea96466df7ad460a5bf6facbccc47d5ec87..9effa7769bef5bcdea5865b964f78f261da40dfe 100644 (file)
--- a/drivers/gpio/gpio-f7188x.c
+++ b/drivers/gpio/gpio-f7188x.c
@@ -163,10 +163,10 @@ static void f7188x_gpio_set(struct gpio_chip *chip, unsigned offset, int value);
 static int f7188x_gpio_set_config(struct gpio_chip *chip, unsigned offset,
                                  unsigned long config);
 
-#define F7188X_GPIO_BANK(_base, _ngpio, _regbase)                      \
+#define F7188X_GPIO_BANK(_base, _ngpio, _regbase, _label)                      \
        {                                                               \
                .chip = {                                               \
-                       .label            = DRVNAME,                    \
+                       .label            = _label,                     \
                        .owner            = THIS_MODULE,                \
                        .get_direction    = f7188x_gpio_get_direction,  \
                        .direction_input  = f7188x_gpio_direction_in,   \
@@ -191,98 +191,98 @@ static int f7188x_gpio_set_config(struct gpio_chip *chip, unsigned offset,
 #define f7188x_gpio_data_single(type)  ((type) == nct6116d)
 
 static struct f7188x_gpio_bank f71869_gpio_bank[] = {
-       F7188X_GPIO_BANK(0, 6, 0xF0),
-       F7188X_GPIO_BANK(10, 8, 0xE0),
-       F7188X_GPIO_BANK(20, 8, 0xD0),
-       F7188X_GPIO_BANK(30, 8, 0xC0),
-       F7188X_GPIO_BANK(40, 8, 0xB0),
-       F7188X_GPIO_BANK(50, 5, 0xA0),
-       F7188X_GPIO_BANK(60, 6, 0x90),
+       F7188X_GPIO_BANK(0, 6, 0xF0, DRVNAME "-0"),
+       F7188X_GPIO_BANK(10, 8, 0xE0, DRVNAME "-1"),
+       F7188X_GPIO_BANK(20, 8, 0xD0, DRVNAME "-2"),
+       F7188X_GPIO_BANK(30, 8, 0xC0, DRVNAME "-3"),
+       F7188X_GPIO_BANK(40, 8, 0xB0, DRVNAME "-4"),
+       F7188X_GPIO_BANK(50, 5, 0xA0, DRVNAME "-5"),
+       F7188X_GPIO_BANK(60, 6, 0x90, DRVNAME "-6"),
 };
 
 static struct f7188x_gpio_bank f71869a_gpio_bank[] = {
-       F7188X_GPIO_BANK(0, 6, 0xF0),
-       F7188X_GPIO_BANK(10, 8, 0xE0),
-       F7188X_GPIO_BANK(20, 8, 0xD0),
-       F7188X_GPIO_BANK(30, 8, 0xC0),
-       F7188X_GPIO_BANK(40, 8, 0xB0),
-       F7188X_GPIO_BANK(50, 5, 0xA0),
-       F7188X_GPIO_BANK(60, 8, 0x90),
-       F7188X_GPIO_BANK(70, 8, 0x80),
+       F7188X_GPIO_BANK(0, 6, 0xF0, DRVNAME "-0"),
+       F7188X_GPIO_BANK(10, 8, 0xE0, DRVNAME "-1"),
+       F7188X_GPIO_BANK(20, 8, 0xD0, DRVNAME "-2"),
+       F7188X_GPIO_BANK(30, 8, 0xC0, DRVNAME "-3"),
+       F7188X_GPIO_BANK(40, 8, 0xB0, DRVNAME "-4"),
+       F7188X_GPIO_BANK(50, 5, 0xA0, DRVNAME "-5"),
+       F7188X_GPIO_BANK(60, 8, 0x90, DRVNAME "-6"),
+       F7188X_GPIO_BANK(70, 8, 0x80, DRVNAME "-7"),
 };
 
 static struct f7188x_gpio_bank f71882_gpio_bank[] = {
-       F7188X_GPIO_BANK(0, 8, 0xF0),
-       F7188X_GPIO_BANK(10, 8, 0xE0),
-       F7188X_GPIO_BANK(20, 8, 0xD0),
-       F7188X_GPIO_BANK(30, 4, 0xC0),
-       F7188X_GPIO_BANK(40, 4, 0xB0),
+       F7188X_GPIO_BANK(0, 8, 0xF0, DRVNAME "-0"),
+       F7188X_GPIO_BANK(10, 8, 0xE0, DRVNAME "-1"),
+       F7188X_GPIO_BANK(20, 8, 0xD0, DRVNAME "-2"),
+       F7188X_GPIO_BANK(30, 4, 0xC0, DRVNAME "-3"),
+       F7188X_GPIO_BANK(40, 4, 0xB0, DRVNAME "-4"),
 };
 
 static struct f7188x_gpio_bank f71889a_gpio_bank[] = {
-       F7188X_GPIO_BANK(0, 7, 0xF0),
-       F7188X_GPIO_BANK(10, 7, 0xE0),
-       F7188X_GPIO_BANK(20, 8, 0xD0),
-       F7188X_GPIO_BANK(30, 8, 0xC0),
-       F7188X_GPIO_BANK(40, 8, 0xB0),
-       F7188X_GPIO_BANK(50, 5, 0xA0),
-       F7188X_GPIO_BANK(60, 8, 0x90),
-       F7188X_GPIO_BANK(70, 8, 0x80),
+       F7188X_GPIO_BANK(0, 7, 0xF0, DRVNAME "-0"),
+       F7188X_GPIO_BANK(10, 7, 0xE0, DRVNAME "-1"),
+       F7188X_GPIO_BANK(20, 8, 0xD0, DRVNAME "-2"),
+       F7188X_GPIO_BANK(30, 8, 0xC0, DRVNAME "-3"),
+       F7188X_GPIO_BANK(40, 8, 0xB0, DRVNAME "-4"),
+       F7188X_GPIO_BANK(50, 5, 0xA0, DRVNAME "-5"),
+       F7188X_GPIO_BANK(60, 8, 0x90, DRVNAME "-6"),
+       F7188X_GPIO_BANK(70, 8, 0x80, DRVNAME "-7"),
 };
 
 static struct f7188x_gpio_bank f71889_gpio_bank[] = {
-       F7188X_GPIO_BANK(0, 7, 0xF0),
-       F7188X_GPIO_BANK(10, 7, 0xE0),
-       F7188X_GPIO_BANK(20, 8, 0xD0),
-       F7188X_GPIO_BANK(30, 8, 0xC0),
-       F7188X_GPIO_BANK(40, 8, 0xB0),
-       F7188X_GPIO_BANK(50, 5, 0xA0),
-       F7188X_GPIO_BANK(60, 8, 0x90),
-       F7188X_GPIO_BANK(70, 8, 0x80),
+       F7188X_GPIO_BANK(0, 7, 0xF0, DRVNAME "-0"),
+       F7188X_GPIO_BANK(10, 7, 0xE0, DRVNAME "-1"),
+       F7188X_GPIO_BANK(20, 8, 0xD0, DRVNAME "-2"),
+       F7188X_GPIO_BANK(30, 8, 0xC0, DRVNAME "-3"),
+       F7188X_GPIO_BANK(40, 8, 0xB0, DRVNAME "-4"),
+       F7188X_GPIO_BANK(50, 5, 0xA0, DRVNAME "-5"),
+       F7188X_GPIO_BANK(60, 8, 0x90, DRVNAME "-6"),
+       F7188X_GPIO_BANK(70, 8, 0x80, DRVNAME "-7"),
 };
 
 static struct f7188x_gpio_bank f81866_gpio_bank[] = {
-       F7188X_GPIO_BANK(0, 8, 0xF0),
-       F7188X_GPIO_BANK(10, 8, 0xE0),
-       F7188X_GPIO_BANK(20, 8, 0xD0),
-       F7188X_GPIO_BANK(30, 8, 0xC0),
-       F7188X_GPIO_BANK(40, 8, 0xB0),
-       F7188X_GPIO_BANK(50, 8, 0xA0),
-       F7188X_GPIO_BANK(60, 8, 0x90),
-       F7188X_GPIO_BANK(70, 8, 0x80),
-       F7188X_GPIO_BANK(80, 8, 0x88),
+       F7188X_GPIO_BANK(0, 8, 0xF0, DRVNAME "-0"),
+       F7188X_GPIO_BANK(10, 8, 0xE0, DRVNAME "-1"),
+       F7188X_GPIO_BANK(20, 8, 0xD0, DRVNAME "-2"),
+       F7188X_GPIO_BANK(30, 8, 0xC0, DRVNAME "-3"),
+       F7188X_GPIO_BANK(40, 8, 0xB0, DRVNAME "-4"),
+       F7188X_GPIO_BANK(50, 8, 0xA0, DRVNAME "-5"),
+       F7188X_GPIO_BANK(60, 8, 0x90, DRVNAME "-6"),
+       F7188X_GPIO_BANK(70, 8, 0x80, DRVNAME "-7"),
+       F7188X_GPIO_BANK(80, 8, 0x88, DRVNAME "-8"),
 };
 
 
 static struct f7188x_gpio_bank f81804_gpio_bank[] = {
-       F7188X_GPIO_BANK(0, 8, 0xF0),
-       F7188X_GPIO_BANK(10, 8, 0xE0),
-       F7188X_GPIO_BANK(20, 8, 0xD0),
-       F7188X_GPIO_BANK(50, 8, 0xA0),
-       F7188X_GPIO_BANK(60, 8, 0x90),
-       F7188X_GPIO_BANK(70, 8, 0x80),
-       F7188X_GPIO_BANK(90, 8, 0x98),
+       F7188X_GPIO_BANK(0, 8, 0xF0, DRVNAME "-0"),
+       F7188X_GPIO_BANK(10, 8, 0xE0, DRVNAME "-1"),
+       F7188X_GPIO_BANK(20, 8, 0xD0, DRVNAME "-2"),
+       F7188X_GPIO_BANK(50, 8, 0xA0, DRVNAME "-3"),
+       F7188X_GPIO_BANK(60, 8, 0x90, DRVNAME "-4"),
+       F7188X_GPIO_BANK(70, 8, 0x80, DRVNAME "-5"),
+       F7188X_GPIO_BANK(90, 8, 0x98, DRVNAME "-6"),
 };
 
 static struct f7188x_gpio_bank f81865_gpio_bank[] = {
-       F7188X_GPIO_BANK(0, 8, 0xF0),
-       F7188X_GPIO_BANK(10, 8, 0xE0),
-       F7188X_GPIO_BANK(20, 8, 0xD0),
-       F7188X_GPIO_BANK(30, 8, 0xC0),
-       F7188X_GPIO_BANK(40, 8, 0xB0),
-       F7188X_GPIO_BANK(50, 8, 0xA0),
-       F7188X_GPIO_BANK(60, 5, 0x90),
+       F7188X_GPIO_BANK(0, 8, 0xF0, DRVNAME "-0"),
+       F7188X_GPIO_BANK(10, 8, 0xE0, DRVNAME "-1"),
+       F7188X_GPIO_BANK(20, 8, 0xD0, DRVNAME "-2"),
+       F7188X_GPIO_BANK(30, 8, 0xC0, DRVNAME "-3"),
+       F7188X_GPIO_BANK(40, 8, 0xB0, DRVNAME "-4"),
+       F7188X_GPIO_BANK(50, 8, 0xA0, DRVNAME "-5"),
+       F7188X_GPIO_BANK(60, 5, 0x90, DRVNAME "-6"),
 };
 
 static struct f7188x_gpio_bank nct6116d_gpio_bank[] = {
-       F7188X_GPIO_BANK(0, 8, 0xE0),
-       F7188X_GPIO_BANK(10, 8, 0xE4),
-       F7188X_GPIO_BANK(20, 8, 0xE8),
-       F7188X_GPIO_BANK(30, 8, 0xEC),
-       F7188X_GPIO_BANK(40, 8, 0xF0),
-       F7188X_GPIO_BANK(50, 8, 0xF4),
-       F7188X_GPIO_BANK(60, 8, 0xF8),
-       F7188X_GPIO_BANK(70, 1, 0xFC),
+       F7188X_GPIO_BANK(0, 8, 0xE0, DRVNAME "-0"),
+       F7188X_GPIO_BANK(10, 8, 0xE4, DRVNAME "-1"),
+       F7188X_GPIO_BANK(20, 8, 0xE8, DRVNAME "-2"),
+       F7188X_GPIO_BANK(30, 8, 0xEC, DRVNAME "-3"),
+       F7188X_GPIO_BANK(40, 8, 0xF0, DRVNAME "-4"),
+       F7188X_GPIO_BANK(50, 8, 0xF4, DRVNAME "-5"),
+       F7188X_GPIO_BANK(60, 8, 0xF8, DRVNAME "-6"),
+       F7188X_GPIO_BANK(70, 1, 0xFC, DRVNAME "-7"),
 };
 
 static int f7188x_gpio_get_direction(struct gpio_chip *chip, unsigned offset)
ep93xx device tree rework repo: git://git.maquefel.me/linux.git