Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1373991 > unrolled thread

[PATCH] mmc: sdhci-pic32: remove owner assignment

Started byMasahiro Yamada <yamada.masahiro@socionext.com>
First post2016-04-08 07:30 +0200
Last post2016-04-11 15:40 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] mmc: sdhci-pic32: remove owner assignment Masahiro Yamada <yamada.masahiro@socionext.com> - 2016-04-08 07:30 +0200
    Re: [PATCH] mmc: sdhci-pic32: remove owner assignment Adrian Hunter <adrian.hunter@intel.com> - 2016-04-08 09:10 +0200
    Re: [PATCH] mmc: sdhci-pic32: remove owner assignment Ulf Hansson <ulf.hansson@linaro.org> - 2016-04-11 15:40 +0200

#1373991 — [PATCH] mmc: sdhci-pic32: remove owner assignment

FromMasahiro Yamada <yamada.masahiro@socionext.com>
Date2016-04-08 07:30 +0200
Subject[PATCH] mmc: sdhci-pic32: remove owner assignment
Message-ID<rlx9w-7sb-15@gated-at.bofh.it>
A platform_driver does not need to set an owner, it will be populated
by the driver core.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 drivers/mmc/host/sdhci-pic32.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/mmc/host/sdhci-pic32.c b/drivers/mmc/host/sdhci-pic32.c
index 059df70..72c13b6 100644
--- a/drivers/mmc/host/sdhci-pic32.c
+++ b/drivers/mmc/host/sdhci-pic32.c
@@ -243,7 +243,6 @@ MODULE_DEVICE_TABLE(of, pic32_sdhci_id_table);
 static struct platform_driver pic32_sdhci_driver = {
 	.driver = {
 		.name	= "pic32-sdhci",
-		.owner	= THIS_MODULE,
 		.of_match_table = of_match_ptr(pic32_sdhci_id_table),
 	},
 	.probe		= pic32_sdhci_probe,
-- 
1.9.1

[toc] | [next] | [standalone]


#1374025

FromAdrian Hunter <adrian.hunter@intel.com>
Date2016-04-08 09:10 +0200
Message-ID<rlyIh-bh-5@gated-at.bofh.it>
In reply to#1373991
On 08/04/16 08:22, Masahiro Yamada wrote:
> A platform_driver does not need to set an owner, it will be populated
> by the driver core.
> 
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
> 
>  drivers/mmc/host/sdhci-pic32.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/mmc/host/sdhci-pic32.c b/drivers/mmc/host/sdhci-pic32.c
> index 059df70..72c13b6 100644
> --- a/drivers/mmc/host/sdhci-pic32.c
> +++ b/drivers/mmc/host/sdhci-pic32.c
> @@ -243,7 +243,6 @@ MODULE_DEVICE_TABLE(of, pic32_sdhci_id_table);
>  static struct platform_driver pic32_sdhci_driver = {
>  	.driver = {
>  		.name	= "pic32-sdhci",
> -		.owner	= THIS_MODULE,
>  		.of_match_table = of_match_ptr(pic32_sdhci_id_table),
>  	},
>  	.probe		= pic32_sdhci_probe,
> 

Acked-by: Adrian Hunter <adrian.hunter@intel.com>

[toc] | [prev] | [next] | [standalone]


#1375894

FromUlf Hansson <ulf.hansson@linaro.org>
Date2016-04-11 15:40 +0200
Message-ID<rmKep-6GA-89@gated-at.bofh.it>
In reply to#1373991
On 8 April 2016 at 07:22, Masahiro Yamada <yamada.masahiro@socionext.com> wrote:
> A platform_driver does not need to set an owner, it will be populated
> by the driver core.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

Thanks, applied for next!

Kind regards
Uffe

> ---
>
>  drivers/mmc/host/sdhci-pic32.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/mmc/host/sdhci-pic32.c b/drivers/mmc/host/sdhci-pic32.c
> index 059df70..72c13b6 100644
> --- a/drivers/mmc/host/sdhci-pic32.c
> +++ b/drivers/mmc/host/sdhci-pic32.c
> @@ -243,7 +243,6 @@ MODULE_DEVICE_TABLE(of, pic32_sdhci_id_table);
>  static struct platform_driver pic32_sdhci_driver = {
>         .driver = {
>                 .name   = "pic32-sdhci",
> -               .owner  = THIS_MODULE,
>                 .of_match_table = of_match_ptr(pic32_sdhci_id_table),
>         },
>         .probe          = pic32_sdhci_probe,
> --
> 1.9.1
>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web