Re: [PATCH v12 4/4] Watchdog: introduce ARM SBSA watchdog driver

From: Guenter Roeck
Date: Tue Feb 16 2016 - 19:09:51 EST


On Wed, Feb 17, 2016 at 12:33:24AM +0800, Fu Wei wrote:
[ ... ]
> >>> +MODULE_AUTHOR("Fu Wei <fu.wei@xxxxxxxxxx>");
> >>> +MODULE_AUTHOR("Suravee Suthikulpanit <Suravee.Suthikulpanit@xxxxxxx>");
> >>> +MODULE_AUTHOR("Al Stone <al.stone@xxxxxxxxxx>");
> >>> +MODULE_AUTHOR("Timur Tabi <timur@xxxxxxxxxxxxxx>");
> >>> +MODULE_LICENSE("GPL v2");
> >>>
> >> Do you need a MODULE_ALIAS ?
> >
> > For now, I thinks we don't need it, Hope I didn't miss something :-)
> > Because this module can be mounted automatically with dtb or ACPI(if
> > apply my GTDT patch).
> > Do you have any suggestion or concern? :-)
>
> re-think about that , will add
> MODULE_ALIAS("platform:sbsa-gwdt");
>
> Do you agree ? :-)
>
Sure.

Guenter