Hi,

-----Original Message-----
From: Bartosz Golaszewski <redacted>
Sent: Friday, April 16, 2021 11:58 PM
To: Srinivas Neeli <redacted>
Cc: Andy Shevchenko <redacted>;
linus.walleij@linaro.org; Michal Simek [off-list ref]; Shubhrajyoti
Datta [off-list ref]; Srinivas Goud [off-list ref]; linux-
gpio@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
kernel@vger.kernel.org; git [off-list ref]
Subject: Re: [PATCH 1/3] gpio: zynq: use module_platform_driver to simplify
the code

On Wed, Apr 14, 2021 at 4:45 PM Srinivas Neeli [off-list ref] wrote:

quoted

HI baratosz and Andy,

It's Bartosz. You literally just need to copy & paste the name from my email...

quoted

quoted

-----Original Message-----
From: Bartosz Golaszewski <redacted>
Sent: Tuesday, April 13, 2021 4:14 PM
To: Andy Shevchenko <redacted>
Cc: Srinivas Neeli <redacted>; linus.walleij@linaro.org;
Michal Simek [off-list ref]; Shubhrajyoti Datta
[off-list ref]; Srinivas Goud [off-list ref];
linux-gpio@vger.kernel.org; linux-arm- kernel@lists.infradead.org;
linux-kernel@vger.kernel.org; git [off-list ref]
Subject: Re: [PATCH 1/3] gpio: zynq: use module_platform_driver to
simplify the code

On Sat, Apr 10, 2021 at 12:08 AM Andy Shevchenko
[off-list ref] wrote:

quoted

On Friday, April 9, 2021, Srinivas Neeli [off-list ref]

wrote:

quoted

quoted

quoted

quoted

module_platform_driver() makes the code simpler by eliminating
boilerplate code.

Signed-off-by: Srinivas Neeli <redacted>
---
 drivers/gpio/gpio-zynq.c | 17 +----------------
 1 file changed, 1 insertion(+), 16 deletions(-)

diff --git a/drivers/gpio/gpio-zynq.c b/drivers/gpio/gpio-zynq.c
index 3521c1dc3ac0..bb1ac0c5cf26 100644
--- a/drivers/gpio/gpio-zynq.c
+++ b/drivers/gpio/gpio-zynq.c

@@ -1020,22 +1020,7 @@ static struct platform_driver

zynq_gpio_driver

= {

quoted

quoted

        .remove = zynq_gpio_remove,  };

-/**
- * zynq_gpio_init - Initial driver registration call
- *
- * Return: value from platform_driver_register
- */
-static int __init zynq_gpio_init(void) -{
-       return platform_driver_register(&zynq_gpio_driver);
-}
-postcore_initcall(zynq_gpio_init);

It’s not an equivalent. Have you tested on actual hardware? If no,
there is

no go for this change.

quoted

Yep, this has been like this since the initial introduction of this driver.
Unfortunately there's no documented reason so unless we can test it,
it has to stay this way.

I tested driver, functionality wise everything working fine.
Based on below conversation, I moved driver to module driver.
https://lore.kernel.org/patchwork/patch/818202/

Andy: How about we give it a try then? If anyone yells, we'll just revert it.

Could you please apply this series to gpio for-next branch if there are no issues .

quoted

Thanks
Srinivas Neeli

quoted

Bartosz

Bartosz

Thanks
Srinivas Neeli