Messages in this thread | | | Date | Wed, 4 Mar 2015 16:08:15 +0100 | From | Alexandre Belloni <> | Subject | Re: [PATCH v3 3/9] watchdog: at91rm9200: use the system timer syscon |
| |
On 04/03/2015 at 06:51:51 -0800, Guenter Roeck wrote : > >@@ -267,7 +276,7 @@ static struct platform_driver at91wdt_driver = { > > .suspend = at91wdt_suspend, > > .resume = at91wdt_resume, > > .driver = { > >- .name = "at91_wdt", > >+ .name = "atmel_st_watchdog", > > If you change the driver name you also need to change the MODULE_ALIAS > a few lines down. > > But why change it in the first place ? Maybe that was discussed somewhere, > but not here where it belongs, and not in patch 0/9 either. >
That's because it conflicts with drivers/watchdog/at91sam9_wdt.c
I'm not sure that is relevant anymore as they will only be probed using DT now.
-- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
| |