pinctrl: sirf/atlas7: stop poking around in GPIO internals
authorLinus Walleij <linus.walleij@linaro.org>
Fri, 12 Feb 2016 08:22:11 +0000 (09:22 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 15 Feb 2016 23:19:54 +0000 (00:19 +0100)
This code is poking around in the gpio_chip:s internal structures
to achieve some kind of pin to GPIO mappings.

- It is wrong to poke around in these structs and the pinctrl
  maintainer was stupid to let it pass unnoticed, mea culpa.

- The right interface to use is gpiochip_add_pin_range()

- The code appears unused: the pin control part of the driver
  is not adding any ranges, so we're iterating over an empty
  list. Maybe it is poking around in some other pin controllers
  GPIO ranges, and that's just totally wrong, again use
  gpiochip_add_pin_range() and specify the right pin
  controller.

Cc: Barry Song <baohua@kernel.org>
Cc: Guoying Zhang <Guoying.Zhang@csr.com>
Cc: Wei Chen <Wei.Chen@csr.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/sirf/pinctrl-atlas7.c

index beb024c31a5d8b5ffb115fa94283dc7909219f7c..3d233fc3448ad4675a08a52b7ca4c44a11a4737b 100644 (file)
@@ -338,7 +338,6 @@ struct atlas7_pinctrl_data {
 #define ATLAS7_GPIO_CTL_DATAIN_MASK            BIT(7)
 
 struct atlas7_gpio_bank {
-       struct pinctrl_dev *pctldev;
        int id;
        int irq;
        void __iomem *base;
@@ -6070,7 +6069,6 @@ static int atlas7_gpio_probe(struct platform_device *pdev)
        }
 
        for (idx = 0; idx < nbank; idx++) {
-               struct gpio_pin_range *pin_range;
                struct atlas7_gpio_bank *bank;
 
                bank = &a7gc->banks[idx];
@@ -6088,22 +6086,6 @@ static int atlas7_gpio_probe(struct platform_device *pdev)
 
                gpiochip_set_chained_irqchip(chip, &atlas7_gpio_irq_chip,
                                        bank->irq, atlas7_gpio_handle_irq);
-
-               /* Records gpio_pin_range to a7gc */
-               list_for_each_entry(pin_range, &chip->pin_ranges, node) {
-                       struct pinctrl_gpio_range *range;
-
-                       range = &pin_range->range;
-                       if (range->id == NGPIO_OF_BANK * idx) {
-                               bank->gpio_offset = range->id;
-                               bank->ngpio = range->npins;
-                               bank->gpio_pins = range->pins;
-                               bank->pctldev = pin_range->pctldev;
-                               break;
-                       }
-               }
-
-               BUG_ON(!bank->pctldev);
        }
 
        platform_set_drvdata(pdev, a7gc);