[Powerdevil] [Bug 449997] Battery charge limit is not supported on laptops that have charge_control_end_threshold, but not charge_control_start_threshold

2022-03-26 Thread Buovjaga
https://bugs.kde.org/show_bug.cgi?id=449997

--- Comment #6 from Buovjaga  ---
(In reply to Buovjaga from comment #3)
> I repro this with a KDE Slimbook and Plasma 5.24. Looking forward to the
> merged fix :)

I discovered that Slimbook and other TongFang-based laptops lack kernel support
for charging limits:
https://slimbook.es/en/forum/tag/amd/question/5384-pro-x-amd-set-max-battery-charge-percentage

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 449997] Battery charge limit is not supported on laptops that have charge_control_end_threshold, but not charge_control_start_threshold

2022-02-14 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=449997

Nate Graham  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
   |ma/powerdevil/commit/f7b1f2 |ma/powerdevil/commit/29154c
   |f182ab97aab1590ad47804c1dd7 |13a186f5d13191db4bca1984ba2
   |f04ef1f |a21b839

--- Comment #5 from Nate Graham  ---
Git commit 29154c13a186f5d13191db4bca1984ba2a21b839 by Nate Graham, on behalf
of Méven Car.
Committed on 14/02/2022 at 21:18.
Pushed by ngraham into branch 'Plasma/5.24'.

Support hardware with only one charging threshold, not both

getthreshold will return -1 for unsupported thresholds.

setthreshold will tollerate receving no value or -1 to ignore setting a
threshold.
FIXED-IN: 5.24.1


(cherry picked from commit f7b1f2f182ab97aab1590ad47804c1dd7f04ef1f)

M  +24   -15   daemon/chargethresholdhelper.cpp
M  +2-2daemon/powerdevilcore.cpp
M  +11   -10   kcmodule/global/GeneralPage.cpp

https://invent.kde.org/plasma/powerdevil/commit/29154c13a186f5d13191db4bca1984ba2a21b839

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 449997] Battery charge limit is not supported on laptops that have charge_control_end_threshold, but not charge_control_start_threshold

2022-02-14 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=449997

Nate Graham  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED
  Latest Commit||https://invent.kde.org/plas
   ||ma/powerdevil/commit/f7b1f2
   ||f182ab97aab1590ad47804c1dd7
   ||f04ef1f
   Version Fixed In||5.24.1

--- Comment #4 from Nate Graham  ---
Git commit f7b1f2f182ab97aab1590ad47804c1dd7f04ef1f by Nate Graham, on behalf
of Méven Car.
Committed on 14/02/2022 at 21:15.
Pushed by ngraham into branch 'master'.

Support hardware with only one charging threshold, not both

getthreshold will return -1 for unsupported thresholds.

setthreshold will tollerate receving no value or -1 to ignore setting a
threshold.
FIXED-IN: 5.24.1

M  +24   -15   daemon/chargethresholdhelper.cpp
M  +2-2daemon/powerdevilcore.cpp
M  +11   -10   kcmodule/global/GeneralPage.cpp

https://invent.kde.org/plasma/powerdevil/commit/f7b1f2f182ab97aab1590ad47804c1dd7f04ef1f

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 449997] Battery charge limit is not supported on laptops that have charge_control_end_threshold, but not charge_control_start_threshold

2022-02-13 Thread Buovjaga
https://bugs.kde.org/show_bug.cgi?id=449997

Buovjaga  changed:

   What|Removed |Added

 CC||ilmari.lauhakangas@libreoff
   ||ice.org

--- Comment #3 from Buovjaga  ---
(In reply to aeneid from comment #0)
> STEPS TO REPRODUCE
> 1. In the terminal run "systemsettings"

Correction: "systemsettings5"

> 2. Navigate to Power Management > Advanced Power Settings
> 
> OBSERVED RESULT
> The charged limit settings are not shown.
> 
> The terminal logs:
> Warning from helper: Failed to open
> "/sys/class/power_supply/BAT0/charge_control_start_threshold" for reading
> Warning from helper: Failed to open
> "/sys/class/power_supply/BAT0/charge_start_threshold" for reading
> Warning from helper: Failed to open
> "/sys/class/power_supply/BAT0/charge_stop_threshold" for reading
> org.kde.powerdevil.chargethresholdhelper.getthreshold failed "Charge
> thresholds not supported"

I repro this with a KDE Slimbook and Plasma 5.24. Looking forward to the merged
fix :)

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 449997] Battery charge limit is not supported on laptops that have charge_control_end_threshold, but not charge_control_start_threshold

2022-02-12 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=449997

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #2 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/plasma/powerdevil/-/merge_requests/77

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 449997] Battery charge limit is not supported on laptops that have charge_control_end_threshold, but not charge_control_start_threshold

2022-02-11 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=449997

Nate Graham  changed:

   What|Removed |Added

Summary|Battery charge limit is not |Battery charge limit is not
   |supported on Asus laptops   |supported on laptops that
   ||have
   ||charge_control_end_threshol
   ||d, but not
   ||charge_control_start_thresh
   ||old
 Status|REPORTED|CONFIRMED
 Ever confirmed|0   |1

-- 
You are receiving this mail because:
You are watching all bug changes.