staging: mt7621-gpio: use custom xlate function
authorSergio Paracuellos <sergio.paracuellos@gmail.com>
Thu, 5 Jul 2018 06:00:27 +0000 (08:00 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 6 Jul 2018 15:32:24 +0000 (17:32 +0200)
The default gpio.of_xlate function assumes there is one gpio chip
for each devicetree node. Device tree had changed to only use one node,
which corresponds to 3 different gpio chips now. For that approach
to work we need a custom xlate function.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Reviewed-by: NeilBrown <neil@brown.name>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/mt7621-gpio/gpio-mt7621.c

index 06024a3f649c1cb3cff5e8e73e2c7442f0fbcaf0..ccf2aa85d665c6f82bb9a1bbbc1f98826cb67604 100644 (file)
@@ -205,6 +205,22 @@ static inline const char * const mediatek_gpio_bank_name(int bank)
        return bank_names[bank];
 }
 
+static int
+mediatek_gpio_xlate(struct gpio_chip *chip,
+                   const struct of_phandle_args *spec, u32 *flags)
+{
+       int gpio = spec->args[0];
+       struct mtk_gc *rg = to_mediatek_gpio(chip);
+
+       if (rg->bank != gpio / MTK_BANK_WIDTH)
+               return -EINVAL;
+
+       if (flags)
+               *flags = spec->args[1];
+
+       return gpio % MTK_BANK_WIDTH;
+}
+
 static int
 mediatek_gpio_bank_probe(struct platform_device *pdev,
                         struct device_node *node, int bank)
@@ -220,6 +236,8 @@ mediatek_gpio_bank_probe(struct platform_device *pdev,
        spin_lock_init(&rg->lock);
        rg->chip.of_node = node;
        rg->bank = bank;
+       rg->chip.of_gpio_n_cells = 2;
+       rg->chip.of_xlate = mediatek_gpio_xlate;
        rg->chip.label = mediatek_gpio_bank_name(rg->bank);
 
        dat = gpio->gpio_membase + GPIO_REG_DATA + (rg->bank * GPIO_BANK_WIDE);