something wrong in drivers/power
Pavel Machek
pavel at ucw.cz
Sun Sep 27 13:35:16 EDT 2009
On Sat 2009-09-26 18:55:36, Dmitry Eremin-Solenikov wrote:
> On Sat, Sep 26, 2009 at 11:55:55AM +0200, Pavel Machek wrote:
> >
> > Tosa battery driver seems to be confused about its own identity.
>
> NAK. Please see the drivers/input/touchscree/wm97xx-core.c
Could you elaborate?
I took a look at touchscreen file, and it does not explain
anything. Having tosa_battery calling itself wm97xx-battery is at
least confusing, and there's no explaining comment. Given that we have
different wm97xx_battery.c driver calling itself "wm97xx-battery"... I
believe that's a bug.
Pavel
> > Signed-off-by: Pavel Machek <pavel at ucw.cz>
> >
> > diff -ur power.ofic/tosa_battery.c power/tosa_battery.c
> > --- linux/drivers/power.ofic/tosa_battery.c 2009-09-06 08:56:36.000000000 +0200
> > +++ linux/drivers/power/tosa_battery.c 2009-09-15 15:14:08.000000000 +0200
> > @@ -459,7 +459,7 @@
> > }
> >
> > static struct platform_driver tosa_bat_driver = {
> > - .driver.name = "wm97xx-battery",
> > + .driver.name = "tosa-battery",
> > .driver.owner = THIS_MODULE,
> > .probe = tosa_bat_probe,
> > .remove = __devexit_p(tosa_bat_remove),
> > @@ -483,4 +483,4 @@
> > MODULE_LICENSE("GPL");
> > MODULE_AUTHOR("Dmitry Baryshkov");
> > MODULE_DESCRIPTION("Tosa battery driver");
> > -MODULE_ALIAS("platform:wm97xx-battery");
> > +MODULE_ALIAS("platform:tosa-battery");
> >
> >
> > --
> > (english) http://www.livejournal.com/~pavelmachek
> > (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
>
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
More information about the linux-arm-kernel
mailing list