diff options
| author | Andrew Lunn <andrew@lunn.ch> | 2016-05-10 23:27:22 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-05-12 01:36:28 +0200 |
| commit | 52638f71fcff9386fe64c83a18a129b122333fdf (patch) | |
| tree | 253f15285b3a7bd4c02ed4a7681d0643bea9578c /net/dsa/dsa.c | |
| parent | dsa: Add mdio device support to Marvell switches (diff) | |
| download | linux-52638f71fcff9386fe64c83a18a129b122333fdf.tar.xz linux-52638f71fcff9386fe64c83a18a129b122333fdf.zip | |
dsa: Move gpio reset into switch driver
Resetting the switch is something the driver does, not the framework.
So move the parsing of this property into the driver.
There are no in kernel users of this property, so moving it does not
break anything. There is however a board which will make use of this
property making its way into the kernel.
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dsa/dsa.c')
| -rw-r--r-- | net/dsa/dsa.c | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/net/dsa/dsa.c b/net/dsa/dsa.c index d61ceed912be..df169811f26d 100644 --- a/net/dsa/dsa.c +++ b/net/dsa/dsa.c @@ -659,9 +659,6 @@ static int dsa_of_probe(struct device *dev) const char *port_name; int chip_index, port_index; const unsigned int *sw_addr, *port_reg; - int gpio; - enum of_gpio_flags of_flags; - unsigned long flags; u32 eeprom_len; int ret; @@ -740,19 +737,6 @@ static int dsa_of_probe(struct device *dev) put_device(cd->host_dev); cd->host_dev = &mdio_bus_switch->dev; } - gpio = of_get_named_gpio_flags(child, "reset-gpios", 0, - &of_flags); - if (gpio_is_valid(gpio)) { - flags = (of_flags == OF_GPIO_ACTIVE_LOW ? - GPIOF_ACTIVE_LOW : 0); - ret = devm_gpio_request_one(dev, gpio, flags, - "switch_reset"); - if (ret) - goto out_free_chip; - - cd->reset = gpio_to_desc(gpio); - gpiod_direction_output(cd->reset, 0); - } for_each_available_child_of_node(child, port) { port_reg = of_get_property(port, "reg", NULL); |