* [PATCH RESEND 2/2] mtd: bcm47: convert to module_platform_driver instead of init/exit
@ 2013-05-30 2:22 ` Libo Chen
0 siblings, 0 replies; 6+ messages in thread
From: Libo Chen @ 2013-05-30 2:22 UTC (permalink / raw)
To: dwmw2, Andy Shevchenko
Cc: Li Zefan, artem.bityutskiy, Rafał Miłecki, LKML,
Bill Pemberton, linux-mtd, hauke
convert to module_platform_driver instead of init/exit
Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Libo chen <libo.chen@huawei.com>
---
drivers/mtd/devices/bcm47xxsflash.c | 20 +-------------------
1 files changed, 1 insertions(+), 19 deletions(-)
diff --git a/drivers/mtd/devices/bcm47xxsflash.c b/drivers/mtd/devices/bcm47xxsflash.c
index 2630d5c..54230cd 100644
--- a/drivers/mtd/devices/bcm47xxsflash.c
+++ b/drivers/mtd/devices/bcm47xxsflash.c
@@ -108,22 +108,4 @@ static struct platform_driver bcma_sflash_driver = {
* Init
**************************************************/
-static int __init bcm47xxsflash_init(void)
-{
- int err;
-
- err = platform_driver_register(&bcma_sflash_driver);
- if (err)
- pr_err("Failed to register BCMA serial flash driver: %d\n",
- err);
-
- return err;
-}
-
-static void __exit bcm47xxsflash_exit(void)
-{
- platform_driver_unregister(&bcma_sflash_driver);
-}
-
-module_init(bcm47xxsflash_init);
-module_exit(bcm47xxsflash_exit);
+module_platform_driver(bcma_sflash_driver);
--
1.7.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH RESEND 2/2] mtd: bcm47: convert to module_platform_driver instead of init/exit
@ 2013-05-30 2:22 ` Libo Chen
0 siblings, 0 replies; 6+ messages in thread
From: Libo Chen @ 2013-05-30 2:22 UTC (permalink / raw)
To: dwmw2, Andy Shevchenko
Cc: Rafał Miłecki, artem.bityutskiy, Bill Pemberton, hauke,
linux-mtd, LKML, Li Zefan
convert to module_platform_driver instead of init/exit
Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Libo chen <libo.chen@huawei.com>
---
drivers/mtd/devices/bcm47xxsflash.c | 20 +-------------------
1 files changed, 1 insertions(+), 19 deletions(-)
diff --git a/drivers/mtd/devices/bcm47xxsflash.c b/drivers/mtd/devices/bcm47xxsflash.c
index 2630d5c..54230cd 100644
--- a/drivers/mtd/devices/bcm47xxsflash.c
+++ b/drivers/mtd/devices/bcm47xxsflash.c
@@ -108,22 +108,4 @@ static struct platform_driver bcma_sflash_driver = {
* Init
**************************************************/
-static int __init bcm47xxsflash_init(void)
-{
- int err;
-
- err = platform_driver_register(&bcma_sflash_driver);
- if (err)
- pr_err("Failed to register BCMA serial flash driver: %d\n",
- err);
-
- return err;
-}
-
-static void __exit bcm47xxsflash_exit(void)
-{
- platform_driver_unregister(&bcma_sflash_driver);
-}
-
-module_init(bcm47xxsflash_init);
-module_exit(bcm47xxsflash_exit);
+module_platform_driver(bcma_sflash_driver);
--
1.7.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH RESEND 2/2] mtd: bcm47: convert to module_platform_driver instead of init/exit
2013-05-30 2:22 ` Libo Chen
@ 2013-05-30 10:30 ` Andy Shevchenko
-1 siblings, 0 replies; 6+ messages in thread
From: Andy Shevchenko @ 2013-05-30 10:30 UTC (permalink / raw)
To: Libo Chen
Cc: Li Zefan, artem.bityutskiy, Rafał Miłecki, LKML,
Bill Pemberton, linux-mtd, hauke, David Woodhouse
On Thu, May 30, 2013 at 5:22 AM, Libo Chen <clbchenlibo.chen@huawei.com> wrote:
>
> convert to module_platform_driver instead of init/exit
I give another thought about it. We have to be sure that the flash
device will be online when it's really needed.
module_init initcall is quite far in the boot process. Is it what we want?
> --- a/drivers/mtd/devices/bcm47xxsflash.c
> +++ b/drivers/mtd/devices/bcm47xxsflash.c
> @@ -108,22 +108,4 @@ static struct platform_driver bcma_sflash_driver = {
> * Init
> **************************************************/
>
> -static int __init bcm47xxsflash_init(void)
> -{
> - int err;
> -
> - err = platform_driver_register(&bcma_sflash_driver);
> - if (err)
> - pr_err("Failed to register BCMA serial flash driver: %d\n",
> - err);
> -
> - return err;
> -}
> -
> -static void __exit bcm47xxsflash_exit(void)
> -{
> - platform_driver_unregister(&bcma_sflash_driver);
> -}
> -
> -module_init(bcm47xxsflash_init);
> -module_exit(bcm47xxsflash_exit);
> +module_platform_driver(bcma_sflash_driver);
> --
> 1.7.1
>
>
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH RESEND 2/2] mtd: bcm47: convert to module_platform_driver instead of init/exit
@ 2013-05-30 10:30 ` Andy Shevchenko
0 siblings, 0 replies; 6+ messages in thread
From: Andy Shevchenko @ 2013-05-30 10:30 UTC (permalink / raw)
To: Libo Chen
Cc: David Woodhouse, Rafał Miłecki, artem.bityutskiy,
Bill Pemberton, hauke, linux-mtd, LKML, Li Zefan
On Thu, May 30, 2013 at 5:22 AM, Libo Chen <clbchenlibo.chen@huawei.com> wrote:
>
> convert to module_platform_driver instead of init/exit
I give another thought about it. We have to be sure that the flash
device will be online when it's really needed.
module_init initcall is quite far in the boot process. Is it what we want?
> --- a/drivers/mtd/devices/bcm47xxsflash.c
> +++ b/drivers/mtd/devices/bcm47xxsflash.c
> @@ -108,22 +108,4 @@ static struct platform_driver bcma_sflash_driver = {
> * Init
> **************************************************/
>
> -static int __init bcm47xxsflash_init(void)
> -{
> - int err;
> -
> - err = platform_driver_register(&bcma_sflash_driver);
> - if (err)
> - pr_err("Failed to register BCMA serial flash driver: %d\n",
> - err);
> -
> - return err;
> -}
> -
> -static void __exit bcm47xxsflash_exit(void)
> -{
> - platform_driver_unregister(&bcma_sflash_driver);
> -}
> -
> -module_init(bcm47xxsflash_init);
> -module_exit(bcm47xxsflash_exit);
> +module_platform_driver(bcma_sflash_driver);
> --
> 1.7.1
>
>
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH RESEND 2/2] mtd: bcm47: convert to module_platform_driver instead of init/exit
2013-05-30 10:30 ` Andy Shevchenko
@ 2013-05-30 10:55 ` Hauke Mehrtens
-1 siblings, 0 replies; 6+ messages in thread
From: Hauke Mehrtens @ 2013-05-30 10:55 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Li Zefan, artem.bityutskiy, Libo Chen, Rafał Miłecki,
LKML, Bill Pemberton, linux-mtd, David Woodhouse
On 05/30/2013 12:30 PM, Andy Shevchenko wrote:
> On Thu, May 30, 2013 at 5:22 AM, Libo Chen <clbchenlibo.chen@huawei.com> wrote:
>>
>> convert to module_platform_driver instead of init/exit
>
> I give another thought about it. We have to be sure that the flash
> device will be online when it's really needed.
> module_init initcall is quite far in the boot process. Is it what we want?
This is no problem, the flash driver is needed when the root file system
is mounted. The arch code reads the SoC configuration values in its own
code in arch/mips/bcm47xx/nvram.c without using the mtd subsystem. These
values are needed very early in the boot process.
Hauke
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH RESEND 2/2] mtd: bcm47: convert to module_platform_driver instead of init/exit
@ 2013-05-30 10:55 ` Hauke Mehrtens
0 siblings, 0 replies; 6+ messages in thread
From: Hauke Mehrtens @ 2013-05-30 10:55 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Libo Chen, David Woodhouse, Rafał Miłecki,
artem.bityutskiy, Bill Pemberton, linux-mtd, LKML, Li Zefan
On 05/30/2013 12:30 PM, Andy Shevchenko wrote:
> On Thu, May 30, 2013 at 5:22 AM, Libo Chen <clbchenlibo.chen@huawei.com> wrote:
>>
>> convert to module_platform_driver instead of init/exit
>
> I give another thought about it. We have to be sure that the flash
> device will be online when it's really needed.
> module_init initcall is quite far in the boot process. Is it what we want?
This is no problem, the flash driver is needed when the root file system
is mounted. The arch code reads the SoC configuration values in its own
code in arch/mips/bcm47xx/nvram.c without using the mtd subsystem. These
values are needed very early in the boot process.
Hauke
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-05-30 10:56 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-30 2:22 [PATCH RESEND 2/2] mtd: bcm47: convert to module_platform_driver instead of init/exit Libo Chen
2013-05-30 2:22 ` Libo Chen
2013-05-30 10:30 ` Andy Shevchenko
2013-05-30 10:30 ` Andy Shevchenko
2013-05-30 10:55 ` Hauke Mehrtens
2013-05-30 10:55 ` Hauke Mehrtens
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.