Re: [PATCH] HID: hid-input: default HID_BATTERY_STRENGTH to no

[Jeremy Fitzhardinge <jeremy@goop.org>]

On 04/18/2012 07:24 AM, Matthew Garrett wrote:
> On Wed, Apr 18, 2012 at 10:11:31AM -0400, Josh Boyer wrote:
>> On Wed, Apr 18, 2012 at 03:08:54PM +0100, Bastien Nocera wrote:
>>> Don't. The only bit of user-space that's broken is upower, and it's
>>> getting a fixed release shortly.
>> Matthew suggested that additional kernel patches were required to add a
>> scope parameter so upower could understand that the battery is powering
>> a device and not the system.
>>
>> Also, when did it become OK to force-enable a new feature that is
>> _known_ to not work properly on recent userspace?  That makes no sense
>> to me.
> I agree. It's a great feature, but it shouldn't be default y if 
> userspace isn't ready for it yet.

Dammit, I should have pushed harder to get the scope into the same
release.  Oh well.

Does upower trigger a shutdown/hibernation if *any* power supply is low,
or just if all of them are?  Are the failing cases ones where the BT
keyboard is the only visible power supply?

In any case, no objections to disabling it if its the only way to
prevent regressions for now.

    J