Hello Guenter, On 11/11/2016 04:47 PM, Guenter Roeck wrote: > Hi Javier, > > On Fri, Nov 11, 2016 at 04:43:37PM -0300, Javier Martinez Canillas wrote: >> Hello Guenter, >> >> On 10/14/2016 01:55 PM, Guenter Roeck wrote: >>> On Fri, Oct 14, 2016 at 12:23:50PM -0300, Javier Martinez Canillas wrote: >>>> If the driver is built as a module, autoload won't work because the module >>>> alias information is not filled. So user-space can't match the registered >>>> device with the corresponding module. >>>> >>>> Export the module alias information using the MODULE_DEVICE_TABLE() macro. >>>> >>>> Before this patch: >>>> >>>> $ modinfo drivers/watchdog/bcm7038_wdt.ko | grep alias >>>> $ >>>> >>>> After this patch: >>>> >>>> $ modinfo drivers/watchdog/bcm7038_wdt.ko | grep alias >>>> alias: of:N*T*Cbrcm,bcm7038-wdtC* >>>> alias: of:N*T*Cbrcm,bcm7038-wdt >>>> >>>> Signed-off-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> >>> >>> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> >>> >> >> Are you going to pick this (and patch 2/2)? >> >> I'm asking because you provided your r-b tag but the patch >> was never merged, so I wonder who should I ping about it. >> > Wim would be the one to pick it. > > Maybe it got lost. I hope I'll find some time during the weekend > to update my tracking branch. > Ok, thanks a lot for the info. > Guenter > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
