Dnia 2012-02-04, sob o godzinie 10:52 -0600, Chris Bagwell pisze:
Chris, 
irst of all - thanks for your comments!
> I've been monitoring battery topics lately (no pun intended :-) )
> because I'm adding adding similar logic to wacom_wac for wireless.  I
> know that the function power_supply_powers() that is in stack trace is
> brand new (not in official kernel) and associates a power supply to a
> specific device (nice so GUI's don't think our wacom's are powering
> our machines).

It is in the mainline now [1]

> So I would suspect that feature first.  Can you try deleting the two
> calls to that function and see if crash goes away?

I'm trying to figure out if theat's correct that Jeremy added twice the
same call: power_supply_powers(&wdata->battery, &hdev->dev); instead of:
        power_supply_powers(&wdata->battery, &hdev->dev);
 and
        power_supply_powers(&wdata->ac, &hdev->dev);
It might be the reason - compiling now...

> I peeked in jikos's hid tree and the function looks like it registers
> a sysfs directory called "powers".  I suspect its meant to be under
> the "wacom_battery" directory and thus the call needs to be moved
> *after* registering the battery device.

I'll test that as well.

> This looks needed even for graphire's to work.

Anyone with Graphire to test it?

[1] http://git.kernel.org/?p=linux%2Fkernel%2Fgit%2Ftorvalds%
2Flinux.git&a=search&h=HEAD&st=grep&s=power_supply_powers

-- 
Przemo Firszt <[email protected]>


------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Linuxwacom-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to