* [PATCH] mailbox/omap: Use module_platform_driver
@ 2014-07-22 12:13 Tobias Klauser
2014-07-22 15:00 ` Suman Anna
2014-07-23 0:52 ` Jingoo Han
0 siblings, 2 replies; 6+ messages in thread
From: Tobias Klauser @ 2014-07-22 12:13 UTC (permalink / raw)
To: Suman Anna, Tony Lindgren; +Cc: Jingoo Han, linux-kernel
Use the module_platform_driver to omit module init/exit boilerplate code.
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
---
drivers/mailbox/mailbox-omap1.c | 13 +------------
drivers/mailbox/mailbox-omap2.c | 13 +------------
2 files changed, 2 insertions(+), 24 deletions(-)
diff --git a/drivers/mailbox/mailbox-omap1.c b/drivers/mailbox/mailbox-omap1.c
index 9001b76..735aeef 100644
--- a/drivers/mailbox/mailbox-omap1.c
+++ b/drivers/mailbox/mailbox-omap1.c
@@ -184,18 +184,7 @@ static struct platform_driver omap1_mbox_driver = {
},
};
-static int __init omap1_mbox_init(void)
-{
- return platform_driver_register(&omap1_mbox_driver);
-}
-
-static void __exit omap1_mbox_exit(void)
-{
- platform_driver_unregister(&omap1_mbox_driver);
-}
-
-module_init(omap1_mbox_init);
-module_exit(omap1_mbox_exit);
+module_platform_driver(omap1_mbox_driver);
MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("omap mailbox: omap1 architecture specific functions");
diff --git a/drivers/mailbox/mailbox-omap2.c b/drivers/mailbox/mailbox-omap2.c
index 42d2b89..a9dedf0 100644
--- a/drivers/mailbox/mailbox-omap2.c
+++ b/drivers/mailbox/mailbox-omap2.c
@@ -337,18 +337,7 @@ static struct platform_driver omap2_mbox_driver = {
},
};
-static int __init omap2_mbox_init(void)
-{
- return platform_driver_register(&omap2_mbox_driver);
-}
-
-static void __exit omap2_mbox_exit(void)
-{
- platform_driver_unregister(&omap2_mbox_driver);
-}
-
-module_init(omap2_mbox_init);
-module_exit(omap2_mbox_exit);
+module_platform_driver(omap2_mbox_driver);
MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("omap mailbox: omap2/3/4 architecture specific functions");
--
2.0.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] mailbox/omap: Use module_platform_driver
2014-07-22 12:13 [PATCH] mailbox/omap: Use module_platform_driver Tobias Klauser
@ 2014-07-22 15:00 ` Suman Anna
2014-07-23 0:57 ` Jingoo Han
2014-07-23 7:43 ` Tobias Klauser
2014-07-23 0:52 ` Jingoo Han
1 sibling, 2 replies; 6+ messages in thread
From: Suman Anna @ 2014-07-22 15:00 UTC (permalink / raw)
To: Tobias Klauser, Tony Lindgren; +Cc: Jingoo Han, linux-kernel
Hi Tobias,
On 07/22/2014 07:13 AM, Tobias Klauser wrote:
> Use the module_platform_driver to omit module init/exit boilerplate code.
>
> Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
> ---
> drivers/mailbox/mailbox-omap1.c | 13 +------------
> drivers/mailbox/mailbox-omap2.c | 13 +------------
> 2 files changed, 2 insertions(+), 24 deletions(-)
I have submitted a series that cleans up/consolidates the OMAP mailbox
driver [1], and this patch won't be necessary as the same lines get deleted.
regards
Suman
[1] http://www.spinics.net/lists/linux-omap/msg108574.html
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mailbox/omap: Use module_platform_driver
2014-07-22 15:00 ` Suman Anna
@ 2014-07-23 0:57 ` Jingoo Han
2014-07-23 1:24 ` Suman Anna
2014-07-23 7:43 ` Tobias Klauser
1 sibling, 1 reply; 6+ messages in thread
From: Jingoo Han @ 2014-07-23 0:57 UTC (permalink / raw)
To: 'Suman Anna'
Cc: linux-kernel, 'Tobias Klauser', 'Tony Lindgren',
'Jingoo Han'
On Wednesday, July 23, 2014 12:01 AM, Suman Anna wrote:
> On 07/22/2014 07:13 AM, Tobias Klauser wrote:
> > Use the module_platform_driver to omit module init/exit boilerplate code.
> >
> > Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
> > ---
> > drivers/mailbox/mailbox-omap1.c | 13 +------------
> > drivers/mailbox/mailbox-omap2.c | 13 +------------
> > 2 files changed, 2 insertions(+), 24 deletions(-)
>
> I have submitted a series that cleans up/consolidates the OMAP mailbox
> driver [1], and this patch won't be necessary as the same lines get deleted.
Oh, I see. So, this patch is unnecessary. I hope that Jassi's mailbox framework
is submitted soon. Thank you.
Best regards,
Jingoo Han
>
> regards
> Suman
>
> [1] http://www.spinics.net/lists/linux-omap/msg108574.html
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mailbox/omap: Use module_platform_driver
2014-07-23 0:57 ` Jingoo Han
@ 2014-07-23 1:24 ` Suman Anna
0 siblings, 0 replies; 6+ messages in thread
From: Suman Anna @ 2014-07-23 1:24 UTC (permalink / raw)
To: Jingoo Han
Cc: linux-kernel, 'Tobias Klauser', 'Tony Lindgren'
On 07/22/2014 07:57 PM, Jingoo Han wrote:
> On Wednesday, July 23, 2014 12:01 AM, Suman Anna wrote:
>> On 07/22/2014 07:13 AM, Tobias Klauser wrote:
>>> Use the module_platform_driver to omit module init/exit boilerplate code.
>>>
>>> Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
>>> ---
>>> drivers/mailbox/mailbox-omap1.c | 13 +------------
>>> drivers/mailbox/mailbox-omap2.c | 13 +------------
>>> 2 files changed, 2 insertions(+), 24 deletions(-)
>>
>> I have submitted a series that cleans up/consolidates the OMAP mailbox
>> driver [1], and this patch won't be necessary as the same lines get deleted.
>
> Oh, I see. So, this patch is unnecessary. I hope that Jassi's mailbox framework
> is submitted soon. Thank you.
Yep, its very close now, there is a v9 version posted today. In any
case, the cleanup series is independent of the framework.
regards
Suman
>>
>> [1] http://www.spinics.net/lists/linux-omap/msg108574.html
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mailbox/omap: Use module_platform_driver
2014-07-22 15:00 ` Suman Anna
2014-07-23 0:57 ` Jingoo Han
@ 2014-07-23 7:43 ` Tobias Klauser
1 sibling, 0 replies; 6+ messages in thread
From: Tobias Klauser @ 2014-07-23 7:43 UTC (permalink / raw)
To: Suman Anna; +Cc: Tony Lindgren, Jingoo Han, linux-kernel
On 2014-07-22 at 17:00:44 +0200, Suman Anna <s-anna@ti.com> wrote:
> Hi Tobias,
>
> On 07/22/2014 07:13 AM, Tobias Klauser wrote:
> > Use the module_platform_driver to omit module init/exit boilerplate code.
> >
> > Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
> > ---
> > drivers/mailbox/mailbox-omap1.c | 13 +------------
> > drivers/mailbox/mailbox-omap2.c | 13 +------------
> > 2 files changed, 2 insertions(+), 24 deletions(-)
>
> I have submitted a series that cleans up/consolidates the OMAP mailbox
> driver [1], and this patch won't be necessary as the same lines get deleted.
Ah, I wasn't aware of this series. Should do my homework better next
time ;) Sorry for the noise.
Thanks
Tobias
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mailbox/omap: Use module_platform_driver
2014-07-22 12:13 [PATCH] mailbox/omap: Use module_platform_driver Tobias Klauser
2014-07-22 15:00 ` Suman Anna
@ 2014-07-23 0:52 ` Jingoo Han
1 sibling, 0 replies; 6+ messages in thread
From: Jingoo Han @ 2014-07-23 0:52 UTC (permalink / raw)
To: 'Tobias Klauser'
Cc: linux-kernel, 'Suman Anna', 'Tony Lindgren',
'Jingoo Han'
On Tuesday, July 22, 2014 9:14 PM, Tobias Klauser wrote:
>
> Use the module_platform_driver to omit module init/exit boilerplate code.
>
> Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Reviewed-by: Jingoo Han <jg1.han@samsung.com>
Best regards,
Jingoo Han
> ---
> drivers/mailbox/mailbox-omap1.c | 13 +------------
> drivers/mailbox/mailbox-omap2.c | 13 +------------
> 2 files changed, 2 insertions(+), 24 deletions(-)
>
> diff --git a/drivers/mailbox/mailbox-omap1.c b/drivers/mailbox/mailbox-omap1.c
> index 9001b76..735aeef 100644
> --- a/drivers/mailbox/mailbox-omap1.c
> +++ b/drivers/mailbox/mailbox-omap1.c
> @@ -184,18 +184,7 @@ static struct platform_driver omap1_mbox_driver = {
> },
> };
>
> -static int __init omap1_mbox_init(void)
> -{
> - return platform_driver_register(&omap1_mbox_driver);
> -}
> -
> -static void __exit omap1_mbox_exit(void)
> -{
> - platform_driver_unregister(&omap1_mbox_driver);
> -}
> -
> -module_init(omap1_mbox_init);
> -module_exit(omap1_mbox_exit);
> +module_platform_driver(omap1_mbox_driver);
>
> MODULE_LICENSE("GPL v2");
> MODULE_DESCRIPTION("omap mailbox: omap1 architecture specific functions");
> diff --git a/drivers/mailbox/mailbox-omap2.c b/drivers/mailbox/mailbox-omap2.c
> index 42d2b89..a9dedf0 100644
> --- a/drivers/mailbox/mailbox-omap2.c
> +++ b/drivers/mailbox/mailbox-omap2.c
> @@ -337,18 +337,7 @@ static struct platform_driver omap2_mbox_driver = {
> },
> };
>
> -static int __init omap2_mbox_init(void)
> -{
> - return platform_driver_register(&omap2_mbox_driver);
> -}
> -
> -static void __exit omap2_mbox_exit(void)
> -{
> - platform_driver_unregister(&omap2_mbox_driver);
> -}
> -
> -module_init(omap2_mbox_init);
> -module_exit(omap2_mbox_exit);
> +module_platform_driver(omap2_mbox_driver);
>
> MODULE_LICENSE("GPL v2");
> MODULE_DESCRIPTION("omap mailbox: omap2/3/4 architecture specific functions");
> --
> 2.0.1
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-23 7:43 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-22 12:13 [PATCH] mailbox/omap: Use module_platform_driver Tobias Klauser
2014-07-22 15:00 ` Suman Anna
2014-07-23 0:57 ` Jingoo Han
2014-07-23 1:24 ` Suman Anna
2014-07-23 7:43 ` Tobias Klauser
2014-07-23 0:52 ` Jingoo Han
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox