Resurrection of Indicator Upower

[Eric H]

@paulcarroty
According to https://android.stackexchange.com/questions/200032/difference-between-system-files-charging-enabled-vs-battery-charging-enabled battery-charging-enabled has a different behavior than charging-enabled.
So I will just make a MR to add the above udev rule in surya and karna.
Thanks for your work

[mihael]

On my N100 I don't have /sys/class/power_supply/battery/charging_enabled but instead I have /sys/class/power_supply/battery/battery_charging_enabled

[Eric H]

@paulcarroty
A merge request has been proposed on surya, but another (global) solution is suggested in the comments :
https://gitlab.com/ubports/porting/community-ports/android10/xiaomi-poco-x3/xiaomi-surya/-/merge_requests/20

[paulcarroty]

Ahoy, there's battery_charging_enabled support in the latest builds and I need beta testers to check the limiting battery's charge.

You need to apply new udev rule from README and install the latest build from Github Actions.

[Eric H]

@paulcarroty said in Resurrection of Indicator Upower:

Ahoy, there's battery_charging_enabled support in the latest builds and I need beta testers to check the limiting battery's charge.

You need to apply new udev rule from README and install the latest build from Github Actions.

Thanks !
I installed latest build from Github Actions.

Notice that, on surya, default permissions are :

• -rwxrwxrwx 1 system system 4096 févr. 17 11:15 battery_charging_enabled
• -rw-r--r-- 1 system system 4096 févr. 17 11:15 charging_enabled

I leave them untouched for the first tests
Let me know if you need tests with different permissions

[Eric H]

@paulcarroty said in Resurrection of Indicator Upower:

I need beta testers to check the limiting battery's charge.

Test 1:

• when charge is less than 85%, device charges :

  tail /sys/class/power_supply/battery/{battery_,}charging_enabled
  ==> /sys/class/power_supply/battery/battery_charging_enabled <==
  1
  
  ==> /sys/class/power_supply/battery/charging_enabled <==
  1
  

• when charge is more than 85%, device doesn't charge (and doesn't seem to discharge until unplugged)

  tail /sys/class/power_supply/battery/{battery_,}charging_enabled
  ==> /sys/class/power_supply/battery/battery_charging_enabled <==
  0
  
  ==> /sys/class/power_supply/battery/charging_enabled <==
  1
  

  (when unplugging, the values in the two files return to 1)

[Eric H]

@paulcarroty

Test 2 (with udev rules) : same results as test 1

I applied udev rules from README :

phablet@ubuntu-phablet:~$ ls -al /sys/class/power_supply/battery/{battery_,}charging_enabled
-rw-rw-r-- 1 phablet system 4096 févr. 17 17:28 /sys/class/power_supply/battery/battery_charging_enabled
-rw-rw-r-- 1 phablet system 4096 févr. 17 17:06 /sys/class/power_supply/battery/charging_enabled

• when charge is less than 85%, device charges :

  tail /sys/class/power_supply/battery/{battery_,}charging_enabled
  ==> /sys/class/power_supply/battery/battery_charging_enabled <==
  1
  
  ==> /sys/class/power_supply/battery/charging_enabled <==
  1
  

• when charge is more than 85%, device doesn't charge (and doesn't seem to discharge until unplugged) :

  tail /sys/class/power_supply/battery/{battery_,}charging_enabled
  ==> /sys/class/power_supply/battery/battery_charging_enabled <==
  0
  
  ==> /sys/class/power_supply/battery/charging_enabled <==
  1
  

(when unplugging, the values in the two files return to 1)

[paulcarroty]

@Eric-H great job.

[paulcarroty]

New release with many fixes.

[paulcarroty]

Yo, power monitoring and fast charging detection are now supported and I need beta testers (again!) to check how well it works.

You need to install the latest build from Github Actions and start charging your device. If something is wrong - submit your power.txt.

[Eric H]

@paulcarroty
I installed latest build, and the app no longer starts.

https://dpaste.com/6CTBE25DB

[paulcarroty]

@Eric-H fixed.

[Eric H]

@paulcarroty
Reinstalled.
Does not work on surya.

https://dpaste.com/G66MBRASC

[Eric H]

About this error :
ValueError: could not convert string to float: '3,65353'

Maybe it is a localization error (french) : there is a comma instead of a point

[paulcarroty]

@Eric-H yeah, upower depends on current locale. Fixed by enforcing

[Eric H]

@paulcarroty
Reinstalled. It seems to work fine.
Do you need specific tests ?

[paulcarroty]

@Eric-H test fast charging if you have the hardware for.

[Eric H]

@paulcarroty
Sorry, I couldn't do the test before today...

So I used the Xiaomi charger supplied with the phone (which fully charges in less than half an hour) with the latest version (0.7) of uPower, but it seems that fast charging is not detected and there's nothing related in the logs.

[paulcarroty]

Eric-H Attach the power.txt file when the battery is being charged and has less than 50% capacity.

[Eric H]

@paulcarroty
I tried to generate the power.txt file but there is an error on stderr :

cat: '/sys/devices/platform/soc/c440000.qcom,spmi/spmi-0/spmi0-00/c440000.qcom,spmi:qcom,pm6150@0:qcom,qpnp-smb5/power_supply/main/health': Invalid argument

The health files exists and is world readable :

phablet@ubuntu-phablet:~/tmp$ ls -al /sys/devices/platform/soc/c440000.qcom\,spmi/spmi-0/spmi0-00/c440000.qcom\,spmi\:qcom\,pm6150@0\:qcom\,qpnp-smb5/power_supply/main/health 
-r--r--r-- 1 system system 4096 mai    6 06:21 /sys/devices/platform/soc/c440000.qcom,spmi/spmi-0/spmi0-00/c440000.qcom,spmi:qcom,pm6150@0:qcom,qpnp-smb5/power_supply/main/health

The error persists even if I try to

• add double quotes,
• add simple quotes
• replace cat with tail