Re: [PATCH v4 1/2] power: act8945a: add charger driver for ACT8945A

From: Javier Martinez Canillas
Date: Wed Jan 20 2016 - 07:18:21 EST


Hello Wenyou,

On Wed, Jan 20, 2016 at 4:24 AM, Yang, Wenyou <Wenyou.Yang@xxxxxxxxx> wrote:

[snip]

>> > +}
>> > +
>> > +static struct platform_driver act8945a_charger_driver = {
>> > + .driver = {
>> > + .name = "act8945a-charger",
>> > + },
>> > + .probe = act8945a_charger_probe,
>> > +};
>> > +module_platform_driver(act8945a_charger_driver);
>>
>> Is your driver auto-loaded properly when built as a module? By auto-loading I
>> mean auto-loaded by udev, not by putting to some rc-script or other modprobe-
>> load scripts.
>>
>
> I failed to configure auto-loaded in rootfs generated by buildroot. I need time to dig it.
>
> Anyway I add OF match table.
>

If you don't have udev/kmod working on your system, you can check if
the modalias reported in the driver's uevent sysfs entry is a valid
alias in the kernel module, i.e:

$ grep MODALIAS $(find -name uevent | grep act8945a-charger)
$ modinfo act8945a_charger | grep alias

>
> Best Regards,
> Wenyou Yang

Best regards,
Javier