* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS
@ 2016-10-19 19:49 ` Javier Martinez Canillas
0 siblings, 0 replies; 18+ messages in thread
From: Javier Martinez Canillas @ 2016-10-19 19:49 UTC (permalink / raw)
To: linus-amlogic
The Amlogic Meson is a DT-only platform, which means the devices are
registered via OF and not using the legacy platform devices support.
So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt")
since the reported uevent MODALIAS to user-space will be the OF one.
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---
drivers/watchdog/meson_gxbb_wdt.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/watchdog/meson_gxbb_wdt.c b/drivers/watchdog/meson_gxbb_wdt.c
index 44d180a2c5e5..45d47664a00a 100644
--- a/drivers/watchdog/meson_gxbb_wdt.c
+++ b/drivers/watchdog/meson_gxbb_wdt.c
@@ -264,7 +264,6 @@ static struct platform_driver meson_gxbb_wdt_driver = {
module_platform_driver(meson_gxbb_wdt_driver);
-MODULE_ALIAS("platform:meson-gxbb-wdt");
MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>");
MODULE_DESCRIPTION("Amlogic Meson GXBB Watchdog timer driver");
MODULE_LICENSE("Dual BSD/GPL");
--
2.7.4
^ permalink raw reply related [flat|nested] 18+ messages in thread* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-10-19 19:49 ` Javier Martinez Canillas 0 siblings, 0 replies; 18+ messages in thread From: Javier Martinez Canillas @ 2016-10-19 19:49 UTC (permalink / raw) To: linux-arm-kernel The Amlogic Meson is a DT-only platform, which means the devices are registered via OF and not using the legacy platform devices support. So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") since the reported uevent MODALIAS to user-space will be the OF one. Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> --- drivers/watchdog/meson_gxbb_wdt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/watchdog/meson_gxbb_wdt.c b/drivers/watchdog/meson_gxbb_wdt.c index 44d180a2c5e5..45d47664a00a 100644 --- a/drivers/watchdog/meson_gxbb_wdt.c +++ b/drivers/watchdog/meson_gxbb_wdt.c @@ -264,7 +264,6 @@ static struct platform_driver meson_gxbb_wdt_driver = { module_platform_driver(meson_gxbb_wdt_driver); -MODULE_ALIAS("platform:meson-gxbb-wdt"); MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>"); MODULE_DESCRIPTION("Amlogic Meson GXBB Watchdog timer driver"); MODULE_LICENSE("Dual BSD/GPL"); -- 2.7.4 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-10-19 19:49 ` Javier Martinez Canillas 0 siblings, 0 replies; 18+ messages in thread From: Javier Martinez Canillas @ 2016-10-19 19:49 UTC (permalink / raw) To: linux-kernel Cc: Javier Martinez Canillas, Guenter Roeck, linux-watchdog, Kevin Hilman, Wim Van Sebroeck, Carlo Caione, linux-amlogic, linux-arm-kernel The Amlogic Meson is a DT-only platform, which means the devices are registered via OF and not using the legacy platform devices support. So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") since the reported uevent MODALIAS to user-space will be the OF one. Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> --- drivers/watchdog/meson_gxbb_wdt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/watchdog/meson_gxbb_wdt.c b/drivers/watchdog/meson_gxbb_wdt.c index 44d180a2c5e5..45d47664a00a 100644 --- a/drivers/watchdog/meson_gxbb_wdt.c +++ b/drivers/watchdog/meson_gxbb_wdt.c @@ -264,7 +264,6 @@ static struct platform_driver meson_gxbb_wdt_driver = { module_platform_driver(meson_gxbb_wdt_driver); -MODULE_ALIAS("platform:meson-gxbb-wdt"); MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>"); MODULE_DESCRIPTION("Amlogic Meson GXBB Watchdog timer driver"); MODULE_LICENSE("Dual BSD/GPL"); -- 2.7.4 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS 2016-10-19 19:49 ` Javier Martinez Canillas (?) @ 2016-10-20 16:23 ` Kevin Hilman -1 siblings, 0 replies; 18+ messages in thread From: Kevin Hilman @ 2016-10-20 16:23 UTC (permalink / raw) To: linus-amlogic Javier Martinez Canillas <javier@osg.samsung.com> writes: > The Amlogic Meson is a DT-only platform, which means the devices are > registered via OF and not using the legacy platform devices support. > > So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") > since the reported uevent MODALIAS to user-space will be the OF one. > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> Acked-by: Kevin Hilman <khilman@baylibre.com> > --- > > drivers/watchdog/meson_gxbb_wdt.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/watchdog/meson_gxbb_wdt.c b/drivers/watchdog/meson_gxbb_wdt.c > index 44d180a2c5e5..45d47664a00a 100644 > --- a/drivers/watchdog/meson_gxbb_wdt.c > +++ b/drivers/watchdog/meson_gxbb_wdt.c > @@ -264,7 +264,6 @@ static struct platform_driver meson_gxbb_wdt_driver = { > > module_platform_driver(meson_gxbb_wdt_driver); > > -MODULE_ALIAS("platform:meson-gxbb-wdt"); > MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>"); > MODULE_DESCRIPTION("Amlogic Meson GXBB Watchdog timer driver"); > MODULE_LICENSE("Dual BSD/GPL"); ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-10-20 16:23 ` Kevin Hilman 0 siblings, 0 replies; 18+ messages in thread From: Kevin Hilman @ 2016-10-20 16:23 UTC (permalink / raw) To: linux-arm-kernel Javier Martinez Canillas <javier@osg.samsung.com> writes: > The Amlogic Meson is a DT-only platform, which means the devices are > registered via OF and not using the legacy platform devices support. > > So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") > since the reported uevent MODALIAS to user-space will be the OF one. > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> Acked-by: Kevin Hilman <khilman@baylibre.com> > --- > > drivers/watchdog/meson_gxbb_wdt.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/watchdog/meson_gxbb_wdt.c b/drivers/watchdog/meson_gxbb_wdt.c > index 44d180a2c5e5..45d47664a00a 100644 > --- a/drivers/watchdog/meson_gxbb_wdt.c > +++ b/drivers/watchdog/meson_gxbb_wdt.c > @@ -264,7 +264,6 @@ static struct platform_driver meson_gxbb_wdt_driver = { > > module_platform_driver(meson_gxbb_wdt_driver); > > -MODULE_ALIAS("platform:meson-gxbb-wdt"); > MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>"); > MODULE_DESCRIPTION("Amlogic Meson GXBB Watchdog timer driver"); > MODULE_LICENSE("Dual BSD/GPL"); ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-10-20 16:23 ` Kevin Hilman 0 siblings, 0 replies; 18+ messages in thread From: Kevin Hilman @ 2016-10-20 16:23 UTC (permalink / raw) To: Javier Martinez Canillas Cc: linux-kernel, Guenter Roeck, linux-watchdog, Wim Van Sebroeck, Carlo Caione, linux-amlogic, linux-arm-kernel Javier Martinez Canillas <javier@osg.samsung.com> writes: > The Amlogic Meson is a DT-only platform, which means the devices are > registered via OF and not using the legacy platform devices support. > > So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") > since the reported uevent MODALIAS to user-space will be the OF one. > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> Acked-by: Kevin Hilman <khilman@baylibre.com> > --- > > drivers/watchdog/meson_gxbb_wdt.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/watchdog/meson_gxbb_wdt.c b/drivers/watchdog/meson_gxbb_wdt.c > index 44d180a2c5e5..45d47664a00a 100644 > --- a/drivers/watchdog/meson_gxbb_wdt.c > +++ b/drivers/watchdog/meson_gxbb_wdt.c > @@ -264,7 +264,6 @@ static struct platform_driver meson_gxbb_wdt_driver = { > > module_platform_driver(meson_gxbb_wdt_driver); > > -MODULE_ALIAS("platform:meson-gxbb-wdt"); > MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>"); > MODULE_DESCRIPTION("Amlogic Meson GXBB Watchdog timer driver"); > MODULE_LICENSE("Dual BSD/GPL"); ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS 2016-10-19 19:49 ` Javier Martinez Canillas (?) @ 2016-10-20 22:28 ` Guenter Roeck -1 siblings, 0 replies; 18+ messages in thread From: Guenter Roeck @ 2016-10-20 22:28 UTC (permalink / raw) To: linus-amlogic On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: > The Amlogic Meson is a DT-only platform, which means the devices are > registered via OF and not using the legacy platform devices support. > > So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") > since the reported uevent MODALIAS to user-space will be the OF one. > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> Reviewed-by: Guenter Roeck <linux@roeck-us.net> ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-10-20 22:28 ` Guenter Roeck 0 siblings, 0 replies; 18+ messages in thread From: Guenter Roeck @ 2016-10-20 22:28 UTC (permalink / raw) To: linux-arm-kernel On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: > The Amlogic Meson is a DT-only platform, which means the devices are > registered via OF and not using the legacy platform devices support. > > So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") > since the reported uevent MODALIAS to user-space will be the OF one. > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> Reviewed-by: Guenter Roeck <linux@roeck-us.net> ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-10-20 22:28 ` Guenter Roeck 0 siblings, 0 replies; 18+ messages in thread From: Guenter Roeck @ 2016-10-20 22:28 UTC (permalink / raw) To: Javier Martinez Canillas Cc: linux-kernel, linux-watchdog, Kevin Hilman, Wim Van Sebroeck, Carlo Caione, linux-amlogic, linux-arm-kernel On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: > The Amlogic Meson is a DT-only platform, which means the devices are > registered via OF and not using the legacy platform devices support. > > So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") > since the reported uevent MODALIAS to user-space will be the OF one. > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> Reviewed-by: Guenter Roeck <linux@roeck-us.net> ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS 2016-10-20 22:28 ` Guenter Roeck (?) @ 2016-11-29 14:57 ` Javier Martinez Canillas -1 siblings, 0 replies; 18+ messages in thread From: Javier Martinez Canillas @ 2016-11-29 14:57 UTC (permalink / raw) To: linus-amlogic Hello Wim, On 10/20/2016 07:28 PM, Guenter Roeck wrote: > On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: >> The Amlogic Meson is a DT-only platform, which means the devices are >> registered via OF and not using the legacy platform devices support. >> >> So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") >> since the reported uevent MODALIAS to user-space will be the OF one. >> >> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> > > Reviewed-by: Guenter Roeck <linux@roeck-us.net> > Any comments about this patch? There are other two similar fixes for watchdog drivers that were also reviewed by Guenter but never picked: https://lkml.org/lkml/2016/10/14/412 https://lkml.org/lkml/2016/10/14/413 Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-11-29 14:57 ` Javier Martinez Canillas 0 siblings, 0 replies; 18+ messages in thread From: Javier Martinez Canillas @ 2016-11-29 14:57 UTC (permalink / raw) To: linux-arm-kernel Hello Wim, On 10/20/2016 07:28 PM, Guenter Roeck wrote: > On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: >> The Amlogic Meson is a DT-only platform, which means the devices are >> registered via OF and not using the legacy platform devices support. >> >> So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") >> since the reported uevent MODALIAS to user-space will be the OF one. >> >> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> > > Reviewed-by: Guenter Roeck <linux@roeck-us.net> > Any comments about this patch? There are other two similar fixes for watchdog drivers that were also reviewed by Guenter but never picked: https://lkml.org/lkml/2016/10/14/412 https://lkml.org/lkml/2016/10/14/413 Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-11-29 14:57 ` Javier Martinez Canillas 0 siblings, 0 replies; 18+ messages in thread From: Javier Martinez Canillas @ 2016-11-29 14:57 UTC (permalink / raw) To: Wim Van Sebroeck Cc: Guenter Roeck, linux-kernel, linux-watchdog, Kevin Hilman, Carlo Caione, linux-amlogic, linux-arm-kernel Hello Wim, On 10/20/2016 07:28 PM, Guenter Roeck wrote: > On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: >> The Amlogic Meson is a DT-only platform, which means the devices are >> registered via OF and not using the legacy platform devices support. >> >> So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") >> since the reported uevent MODALIAS to user-space will be the OF one. >> >> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> > > Reviewed-by: Guenter Roeck <linux@roeck-us.net> > Any comments about this patch? There are other two similar fixes for watchdog drivers that were also reviewed by Guenter but never picked: https://lkml.org/lkml/2016/10/14/412 https://lkml.org/lkml/2016/10/14/413 Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS 2016-11-29 14:57 ` Javier Martinez Canillas (?) @ 2016-11-30 10:59 ` Guenter Roeck -1 siblings, 0 replies; 18+ messages in thread From: Guenter Roeck @ 2016-11-30 10:59 UTC (permalink / raw) To: linus-amlogic On 11/29/2016 06:57 AM, Javier Martinez Canillas wrote: > Hello Wim, > > On 10/20/2016 07:28 PM, Guenter Roeck wrote: >> On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: >>> The Amlogic Meson is a DT-only platform, which means the devices are >>> registered via OF and not using the legacy platform devices support. >>> >>> So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") >>> since the reported uevent MODALIAS to user-space will be the OF one. >>> >>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >> >> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >> > > Any comments about this patch? There are other two similar fixes for > watchdog drivers that were also reviewed by Guenter but never picked: > > https://lkml.org/lkml/2016/10/14/412 > https://lkml.org/lkml/2016/10/14/413 > > Best regards, > The patches are all in my watchdog-next branch, so Wim will hopefully pick them up for the next release. Guenter ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-11-30 10:59 ` Guenter Roeck 0 siblings, 0 replies; 18+ messages in thread From: Guenter Roeck @ 2016-11-30 10:59 UTC (permalink / raw) To: linux-arm-kernel On 11/29/2016 06:57 AM, Javier Martinez Canillas wrote: > Hello Wim, > > On 10/20/2016 07:28 PM, Guenter Roeck wrote: >> On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: >>> The Amlogic Meson is a DT-only platform, which means the devices are >>> registered via OF and not using the legacy platform devices support. >>> >>> So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") >>> since the reported uevent MODALIAS to user-space will be the OF one. >>> >>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >> >> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >> > > Any comments about this patch? There are other two similar fixes for > watchdog drivers that were also reviewed by Guenter but never picked: > > https://lkml.org/lkml/2016/10/14/412 > https://lkml.org/lkml/2016/10/14/413 > > Best regards, > The patches are all in my watchdog-next branch, so Wim will hopefully pick them up for the next release. Guenter ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-11-30 10:59 ` Guenter Roeck 0 siblings, 0 replies; 18+ messages in thread From: Guenter Roeck @ 2016-11-30 10:59 UTC (permalink / raw) To: Javier Martinez Canillas, Wim Van Sebroeck Cc: linux-kernel, linux-watchdog, Kevin Hilman, Carlo Caione, linux-amlogic, linux-arm-kernel On 11/29/2016 06:57 AM, Javier Martinez Canillas wrote: > Hello Wim, > > On 10/20/2016 07:28 PM, Guenter Roeck wrote: >> On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: >>> The Amlogic Meson is a DT-only platform, which means the devices are >>> registered via OF and not using the legacy platform devices support. >>> >>> So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") >>> since the reported uevent MODALIAS to user-space will be the OF one. >>> >>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >> >> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >> > > Any comments about this patch? There are other two similar fixes for > watchdog drivers that were also reviewed by Guenter but never picked: > > https://lkml.org/lkml/2016/10/14/412 > https://lkml.org/lkml/2016/10/14/413 > > Best regards, > The patches are all in my watchdog-next branch, so Wim will hopefully pick them up for the next release. Guenter ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS 2016-11-30 10:59 ` Guenter Roeck (?) @ 2016-11-30 12:41 ` Javier Martinez Canillas -1 siblings, 0 replies; 18+ messages in thread From: Javier Martinez Canillas @ 2016-11-30 12:41 UTC (permalink / raw) To: linus-amlogic Hello Guenter, On 11/30/2016 07:59 AM, Guenter Roeck wrote: > On 11/29/2016 06:57 AM, Javier Martinez Canillas wrote: >> Hello Wim, >> >> On 10/20/2016 07:28 PM, Guenter Roeck wrote: >>> On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: >>>> The Amlogic Meson is a DT-only platform, which means the devices are >>>> registered via OF and not using the legacy platform devices support. >>>> >>>> So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") >>>> since the reported uevent MODALIAS to user-space will be the OF one. >>>> >>>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >>> >>> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >>> >> >> Any comments about this patch? There are other two similar fixes for >> watchdog drivers that were also reviewed by Guenter but never picked: >> >> https://lkml.org/lkml/2016/10/14/412 >> https://lkml.org/lkml/2016/10/14/413 >> >> Best regards, >> > The patches are all in my watchdog-next branch, so Wim will hopefully > pick them up for the next release. > Great, thanks a lot for the info. > Guenter > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-11-30 12:41 ` Javier Martinez Canillas 0 siblings, 0 replies; 18+ messages in thread From: Javier Martinez Canillas @ 2016-11-30 12:41 UTC (permalink / raw) To: linux-arm-kernel Hello Guenter, On 11/30/2016 07:59 AM, Guenter Roeck wrote: > On 11/29/2016 06:57 AM, Javier Martinez Canillas wrote: >> Hello Wim, >> >> On 10/20/2016 07:28 PM, Guenter Roeck wrote: >>> On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: >>>> The Amlogic Meson is a DT-only platform, which means the devices are >>>> registered via OF and not using the legacy platform devices support. >>>> >>>> So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") >>>> since the reported uevent MODALIAS to user-space will be the OF one. >>>> >>>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >>> >>> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >>> >> >> Any comments about this patch? There are other two similar fixes for >> watchdog drivers that were also reviewed by Guenter but never picked: >> >> https://lkml.org/lkml/2016/10/14/412 >> https://lkml.org/lkml/2016/10/14/413 >> >> Best regards, >> > The patches are all in my watchdog-next branch, so Wim will hopefully > pick them up for the next release. > Great, thanks a lot for the info. > Guenter > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS @ 2016-11-30 12:41 ` Javier Martinez Canillas 0 siblings, 0 replies; 18+ messages in thread From: Javier Martinez Canillas @ 2016-11-30 12:41 UTC (permalink / raw) To: Guenter Roeck, Wim Van Sebroeck Cc: linux-kernel, linux-watchdog, Kevin Hilman, Carlo Caione, linux-amlogic, linux-arm-kernel Hello Guenter, On 11/30/2016 07:59 AM, Guenter Roeck wrote: > On 11/29/2016 06:57 AM, Javier Martinez Canillas wrote: >> Hello Wim, >> >> On 10/20/2016 07:28 PM, Guenter Roeck wrote: >>> On Wed, Oct 19, 2016 at 04:49:42PM -0300, Javier Martinez Canillas wrote: >>>> The Amlogic Meson is a DT-only platform, which means the devices are >>>> registered via OF and not using the legacy platform devices support. >>>> >>>> So there's no need to have a MODULE_ALIAS("platform:meson-gxbb-wdt") >>>> since the reported uevent MODALIAS to user-space will be the OF one. >>>> >>>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >>> >>> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >>> >> >> Any comments about this patch? There are other two similar fixes for >> watchdog drivers that were also reviewed by Guenter but never picked: >> >> https://lkml.org/lkml/2016/10/14/412 >> https://lkml.org/lkml/2016/10/14/413 >> >> Best regards, >> > The patches are all in my watchdog-next branch, so Wim will hopefully > pick them up for the next release. > Great, thanks a lot for the info. > Guenter > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2016-11-30 12:41 UTC | newest] Thread overview: 18+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-10-19 19:49 [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS Javier Martinez Canillas 2016-10-19 19:49 ` Javier Martinez Canillas 2016-10-19 19:49 ` Javier Martinez Canillas 2016-10-20 16:23 ` Kevin Hilman 2016-10-20 16:23 ` Kevin Hilman 2016-10-20 16:23 ` Kevin Hilman 2016-10-20 22:28 ` Guenter Roeck 2016-10-20 22:28 ` Guenter Roeck 2016-10-20 22:28 ` Guenter Roeck 2016-11-29 14:57 ` Javier Martinez Canillas 2016-11-29 14:57 ` Javier Martinez Canillas 2016-11-29 14:57 ` Javier Martinez Canillas 2016-11-30 10:59 ` Guenter Roeck 2016-11-30 10:59 ` Guenter Roeck 2016-11-30 10:59 ` Guenter Roeck 2016-11-30 12:41 ` Javier Martinez Canillas 2016-11-30 12:41 ` Javier Martinez Canillas 2016-11-30 12:41 ` Javier Martinez Canillas
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.