[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <81ff6195-19c3-95d6-6d37-58f7719a9265@kernel.org>
Date: Fri, 2 Jun 2023 08:50:22 +0900
From: Damien Le Moal <dlemoal@...nel.org>
To: Nikita Shubin <nikita.shubin@...uefel.me>,
Alexander Sverdlin <alexander.sverdlin@...il.com>,
Arnd Bergmann <arnd@...db.de>,
Linus Walleij <linus.walleij@...aro.org>,
Sergey Shtylyov <s.shtylyov@....ru>
Cc: Michael Peters <mpeters@...eddedTS.com>,
Kris Bahnsen <kris@...eddedTS.com>, linux-ide@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 38/43] pata: cirrus: drop legacy pinctrl
On 6/1/23 14:45, Nikita Shubin wrote:
> Drop legacy acquire/release since we are using
> pinctrl for this now.
Please use full 72 chars lines.
>
> Signed-off-by: Nikita Shubin <nikita.shubin@...uefel.me>
Please change the commit title to:
ata: pata_ep93xx: remove legacy pinctrl use
Also, are the functions ep93xx_ide_acquire_gpio() & ep93xx_ide_release_gpio()
used somewhere else ? If not, this patch should remove these functions as well.
> ---
> drivers/ata/pata_ep93xx.c | 25 ++++++-------------------
> 1 file changed, 6 insertions(+), 19 deletions(-)
>
> diff --git a/drivers/ata/pata_ep93xx.c b/drivers/ata/pata_ep93xx.c
> index 8d363bc71342..88cf31000000 100644
> --- a/drivers/ata/pata_ep93xx.c
> +++ b/drivers/ata/pata_ep93xx.c
> @@ -921,28 +921,18 @@ static int ep93xx_pata_probe(struct platform_device *pdev)
> void __iomem *ide_base;
> int err;
>
> - err = ep93xx_ide_acquire_gpio(pdev);
> - if (err)
> - return err;
> -
> /* INT[3] (IRQ_EP93XX_EXT3) line connected as pull down */
> irq = platform_get_irq(pdev, 0);
> - if (irq < 0) {
> - err = irq;
> - goto err_rel_gpio;
> - }
> + if (irq < 0)
> + return irq;
>
> ide_base = devm_platform_get_and_ioremap_resource(pdev, 0, &mem_res);
> - if (IS_ERR(ide_base)) {
> - err = PTR_ERR(ide_base);
> - goto err_rel_gpio;
> - }
> + if (IS_ERR(ide_base))
> + return PTR_ERR(ide_base);
>
> drv_data = devm_kzalloc(&pdev->dev, sizeof(*drv_data), GFP_KERNEL);
> - if (!drv_data) {
> - err = -ENXIO;
> - goto err_rel_gpio;
> - }
> + if (!drv_data)
> + return -ENXIO;
>
> drv_data->pdev = pdev;
> drv_data->ide_base = ide_base;
> @@ -1000,8 +990,6 @@ static int ep93xx_pata_probe(struct platform_device *pdev)
>
> err_rel_dma:
> ep93xx_pata_release_dma(drv_data);
> -err_rel_gpio:
> - ep93xx_ide_release_gpio(pdev);
> return err;
> }
>
> @@ -1013,7 +1001,6 @@ static int ep93xx_pata_remove(struct platform_device *pdev)
> ata_host_detach(host);
> ep93xx_pata_release_dma(drv_data);
> ep93xx_pata_clear_regs(drv_data->ide_base);
> - ep93xx_ide_release_gpio(pdev);
> return 0;
> }
>
--
Damien Le Moal
Western Digital Research
Powered by blists - more mailing lists