2018-03-20 07:48:49

by <Hariprasath Elango>

[permalink] [raw]

Subject: [PATCH] staging: mt7621-gpio: remove redundant owner assignments of drivers

---

From: HariPrasath Elango <[email protected]>

Remove the reduntant owner initialization from this platform driver as
the platform_driver_register() takes care of it.

Signed-off-by: HariPrasath Elango <[email protected]>
---
drivers/staging/mt7621-gpio/gpio-mt7621.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/staging/mt7621-gpio/gpio-mt7621.c b/drivers/staging/mt7621-gpio/gpio-mt7621.c
index 5c57abe..830d429 100644
--- a/drivers/staging/mt7621-gpio/gpio-mt7621.c
+++ b/drivers/staging/mt7621-gpio/gpio-mt7621.c
@@ -339,7 +339,6 @@ static struct platform_driver mediatek_gpio_driver = {
.probe = mediatek_gpio_probe,
.driver = {
.name = "mt7621_gpio",
- .owner = THIS_MODULE,
.of_match_table = mediatek_gpio_match,
},
};
--
2.10.0.GIT

2018-03-20 22:28:00

by NeilBrown

[permalink] [raw]

Subject: Re: [PATCH] staging: mt7621-gpio: remove redundant owner assignments of drivers

---

On Tue, Mar 20 2018, [email protected] wrote:

> From: HariPrasath Elango <[email protected]>
>
> Remove the reduntant owner initialization from this platform driver as
> the platform_driver_register() takes care of it.
>
> Signed-off-by: HariPrasath Elango <[email protected]>
> ---
> drivers/staging/mt7621-gpio/gpio-mt7621.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/staging/mt7621-gpio/gpio-mt7621.c b/drivers/staging/mt7621-gpio/gpio-mt7621.c
> index 5c57abe..830d429 100644
> --- a/drivers/staging/mt7621-gpio/gpio-mt7621.c
> +++ b/drivers/staging/mt7621-gpio/gpio-mt7621.c
> @@ -339,7 +339,6 @@ static struct platform_driver mediatek_gpio_driver = {
> .probe = mediatek_gpio_probe,
> .driver = {
> .name = "mt7621_gpio",
> - .owner = THIS_MODULE,
> .of_match_table = mediatek_gpio_match,
> },
> };
> --
> 2.10.0.GIT

Reviewed-by: NeilBrown <[email protected]>

Thanks,
NeilBrown

---

Attachments:

signature.asc (847.00 B)