From mboxrd@z Thu Jan 1 00:00:00 1970 From: linux@roeck-us.net (Guenter Roeck) Date: Wed, 30 Nov 2016 02:59:29 -0800 Subject: [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS In-Reply-To: References: <1476906582-14241-1-git-send-email-javier@osg.samsung.com> <20161020222833.GA27421@roeck-us.net> Message-ID: <66487a0e-1d20-519f-6cbf-645f8bfd33da@roeck-us.net> To: linus-amlogic@lists.infradead.org List-Id: linus-amlogic.lists.infradead.org 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 >> >> Reviewed-by: Guenter Roeck >> > > 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