On Sun, Feb 5, 2012 at 1:15 PM, Przemo Firszt <prz...@firszt.eu> wrote:
> Dnia 2012-02-04, sob o godzinie 10:52 -0600, Chris Bagwell pisze:
>> 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).
>>
>> So I would suspect that feature first.  Can you try deleting the two
>> calls to that function and see if crash goes away?
> Hi Chris,
> Yes, it does :-)
>>
>> 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.
>>
>> This looks needed even for graphire's to work.
>
> Works!
> Looks like there were 2 issues: a wrong (battery->ac) call & calls in
> wrong place.
>
> Patch is on the way. Let me know if I can I add your signed-by to the
> patch.
>
> Thanks!

Glad to hear its working.  Yes, you can add my sign off if you like.

Chris

------------------------------------------------------------------------------
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
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to