Re: [PATCH 3/4] ACPI / battery: Add handling for devices which wrongly report discharging state

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On April 12, 2018 10:14:01 AM GMT+02:00, Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
>Hi,
>
>On 11-04-18 19:47, Daniel Drake wrote:
>> On Wed, Apr 11, 2018 at 11:42 AM, Hans de Goede <hdegoede@xxxxxxxxxx>
>wrote:
>>> Either way userspace regressions are a big no-no and there are
>enough
>>> people running new kernels with an old(ish) userspace for this to be
>>> a real problem, so we really need to go with FULL for now.
>> 
>> Would it be a regression though? According to your analysis userspace
>> is already reporting an inaccurate state.
>
>Hmm, you are right, so if we do the right thing and resort
>NOT_CHARGING then upower will end up reporting UNKNOWN to
>the desktop environment instead of the current DISCHARGING,
>which is a small improvement already, although some may see
>this as a regression.

My patch was reverted because Rafael treated that as regression. A lot of people see that as regression.

>
>So I've just checked what GNOME3 does with upower translating
>the "not charging" or in upower terms "pending-charge" for the
>BATC device to "unknown" for the display device which is what
>most DE-s use. This results in a battery icon with a ? inside
>it being shown, which is actually an improvement over the
>discharging icon shown before.
>
>And you are right, that since currently we are reporting discharging
>in this case, which certainly is wrong, this is not a regression.
>
>So I will do a v2 of this patch-set switching to NOT_CHARGING
>so that we fix this correctly in one go
>
>Regards,
>
>Hans

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux