Re: [PATCH] watchdog: meson: Remove unneeded platform MODULE_ALIAS

From: Guenter Roeck
Date: Wed Nov 30 2016 - 05:59:54 EST


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@xxxxxxxxxxxxxxx>

Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx>


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