ata: palmld: Convert to GPIO descriptors
authorLinus Walleij <linus.walleij@linaro.org>
Tue, 13 Nov 2018 13:01:02 +0000 (14:01 +0100)
committerJens Axboe <axboe@kernel.dk>
Wed, 5 Dec 2018 00:15:26 +0000 (17:15 -0700)
Instead of passing GPIO numbers directly to the PalmLD
ATA driver, pass GPIO descriptors from the board file and
handle these in the driver.

Cc: Marek Vasut <marek.vasut@gmail.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
arch/arm/mach-pxa/palm27x.h
arch/arm/mach-pxa/palmld.c
drivers/ata/pata_palmld.c

index d4eac3d6ffb5ecd04df4abeb0ed45b32a2882705..3316ed2016f3c3650e68996976ad798dc481a9de 100644 (file)
@@ -12,6 +12,8 @@
 #ifndef        __INCLUDE_MACH_PALM27X__
 #define        __INCLUDE_MACH_PALM27X__
 
+#include <linux/gpio/machine.h>
+
 #if defined(CONFIG_MMC_PXA) || defined(CONFIG_MMC_PXA_MODULE)
 extern void __init palm27x_mmc_init(int detect, int ro, int power,
                                        int power_inverted);
index 980f2847f5b5251d2199f3821d144c61c6226e48..a37ceec2290361b65a788eff9c385a6567afde01 100644 (file)
@@ -288,8 +288,20 @@ static struct platform_device palmld_ide_device = {
        .id     = -1,
 };
 
+static struct gpiod_lookup_table palmld_ide_gpio_table = {
+       .dev_id = "pata_palmld",
+       .table = {
+               GPIO_LOOKUP("gpio-pxa", GPIO_NR_PALMLD_IDE_PWEN,
+                           "power", GPIO_ACTIVE_HIGH),
+               GPIO_LOOKUP("gpio-pxa", GPIO_NR_PALMLD_IDE_RESET,
+                           "reset", GPIO_ACTIVE_LOW),
+               { },
+       },
+};
+
 static void __init palmld_ide_init(void)
 {
+       gpiod_add_lookup_table(&palmld_ide_gpio_table);
        platform_device_register(&palmld_ide_device);
 }
 #else
index d071ab6864a8a82cafd2befcfefbfaba93952dd8..e2933c324d415781a70aee0322d1c9e6be8d1bd1 100644 (file)
 #include <linux/irq.h>
 #include <linux/platform_device.h>
 #include <linux/delay.h>
-#include <linux/gpio.h>
+#include <linux/gpio/consumer.h>
 
 #include <scsi/scsi_host.h>
 #include <mach/palmld.h>
 
 #define DRV_NAME "pata_palmld"
 
-static struct gpio palmld_hdd_gpios[] = {
-       { GPIO_NR_PALMLD_IDE_PWEN,      GPIOF_INIT_HIGH,        "HDD Power" },
-       { GPIO_NR_PALMLD_IDE_RESET,     GPIOF_INIT_LOW,         "HDD Reset" },
-};
+static struct gpio_desc *palmld_pata_power;
 
 static struct scsi_host_template palmld_sht = {
        ATA_PIO_SHT(DRV_NAME),
@@ -53,32 +50,34 @@ static int palmld_pata_probe(struct platform_device *pdev)
        struct ata_host *host;
        struct ata_port *ap;
        void __iomem *mem;
+       struct device *dev = &pdev->dev;
+       struct gpio_desc *reset;
        int ret;
 
        /* allocate host */
-       host = ata_host_alloc(&pdev->dev, 1);
-       if (!host) {
-               ret = -ENOMEM;
-               goto err1;
-       }
+       host = ata_host_alloc(dev, 1);
+       if (!host)
+               return -ENOMEM;
 
        /* remap drive's physical memory address */
-       mem = devm_ioremap(&pdev->dev, PALMLD_IDE_PHYS, 0x1000);
-       if (!mem) {
-               ret = -ENOMEM;
-               goto err1;
+       mem = devm_ioremap(dev, PALMLD_IDE_PHYS, 0x1000);
+       if (!mem)
+               return -ENOMEM;
+
+       /* request and activate power and reset GPIOs */
+       palmld_pata_power = devm_gpiod_get(dev, "power", GPIOD_OUT_HIGH);
+       if (IS_ERR(palmld_pata_power))
+               return PTR_ERR(palmld_pata_power);
+       reset = devm_gpiod_get(dev, "reset", GPIOD_OUT_HIGH);
+       if (IS_ERR(reset)) {
+               gpiod_set_value(palmld_pata_power, 0);
+               return PTR_ERR(reset);
        }
 
-       /* request and activate power GPIO, IRQ GPIO */
-       ret = gpio_request_array(palmld_hdd_gpios,
-                               ARRAY_SIZE(palmld_hdd_gpios));
-       if (ret)
-               goto err1;
-
-       /* reset the drive */
-       gpio_set_value(GPIO_NR_PALMLD_IDE_RESET, 0);
+       /* Assert reset to reset the drive */
+       gpiod_set_value(reset, 1);
        msleep(30);
-       gpio_set_value(GPIO_NR_PALMLD_IDE_RESET, 1);
+       gpiod_set_value(reset, 0);
        msleep(30);
 
        /* setup the ata port */
@@ -98,14 +97,9 @@ static int palmld_pata_probe(struct platform_device *pdev)
        /* activate host */
        ret = ata_host_activate(host, 0, NULL, IRQF_TRIGGER_RISING,
                                        &palmld_sht);
+       /* power down on failure */
        if (ret)
-               goto err2;
-
-       return ret;
-
-err2:
-       gpio_free_array(palmld_hdd_gpios, ARRAY_SIZE(palmld_hdd_gpios));
-err1:
+               gpiod_set_value(palmld_pata_power, 0);
        return ret;
 }
 
@@ -114,9 +108,7 @@ static int palmld_pata_remove(struct platform_device *dev)
        ata_platform_remove_one(dev);
 
        /* power down the HDD */
-       gpio_set_value(GPIO_NR_PALMLD_IDE_PWEN, 0);
-
-       gpio_free_array(palmld_hdd_gpios, ARRAY_SIZE(palmld_hdd_gpios));
+       gpiod_set_value(palmld_pata_power, 0);
 
        return 0;
 }