maniacx / Battery-Health-Charging

GNU General Public License v3.0
157 stars 14 forks source link

Apple Intel Not working #78

Closed maniacx closed 4 months ago

maniacx commented 4 months ago

Hello gealum.

Can you post the output of these two commands in terminal / console

ls -l /sys/class/power_supply/BAT0/charge_control_end_threshold
ls -l  /sys/module/applesmc
kaveht commented 4 months ago

Hello maniacx, here are the outputs: $ ls -l /sys/module/applesmc total 0 -r--r--r-- 1 root root 4096 Feb 19 14:35 coresize drwxr-xr-x 2 root root 0 Feb 19 12:17 drivers drwxr-xr-x 2 root root 0 Feb 19 12:17 holders -r--r--r-- 1 root root 4096 Feb 19 14:35 initsize -r--r--r-- 1 root root 4096 Feb 18 18:37 initstate drwxr-xr-x 2 root root 0 Feb 19 12:17 notes -r--r--r-- 1 root root 4096 Feb 19 14:35 refcnt drwxr-xr-x 2 root root 0 Feb 19 12:17 sections -r--r--r-- 1 root root 4096 Feb 19 14:35 srcversion -r--r--r-- 1 root root 4096 Feb 19 14:35 taint --w------- 1 root root 4096 Feb 18 18:37 uevent -r--r--r-- 1 root root 4096 Feb 19 14:35 version ls -l /sys/class/power_supply/BAT0/charge_control_end_threshold -rw-r--r-- 1 root root 4096 Feb 18 18:37 /sys/class/power_supply/BAT0/charge_control_end_threshold

maniacx commented 4 months ago

You seem to have the correct path. I need to get more details. I have added some logs to test.zip. Follow the instructions

1 Please remove/uninstall this extension.

2 Run command the command to reset gsetting for the extension

gsettings --schemadir /home/$USER/.local/share/gnome-shell/extensions/Battery-Health-Charging@maniacx.github.com/schemas reset-recursively org.gnome.shell.extensions.Battery-Health-Charging
  1. Install gettext. (Require to install extension manually)

    sudo apt install gettext
  2. Install the test.zip test.zip Download. In the zip right-click on Install.sh and run as program

  3. log out and relogin

  4. Monitor and read log. Open terminal journalctl -f -o cat /usr/bin/gnome-shell Now this terminal window will display realtime logs

  5. Enable the extension.

Check and post here here any logs that starts with or related to Battery Health Charging

Also give me the output of cat /sys/devices/virtual/dmi/id/sys_vendor I need this to fix the issue of pointing to wrong readme file which displayed link for apple-m-series insteaa of intel

kaveht commented 4 months ago

Here is the output from journalctl Battery Health Extension: Checking apple intel Battery Health Extension: -- found apple path check 1 -- Battery Health Extension: -- found apple path check 2 -- `$ cat /sys/devices/virtual/dmi/id/sys_vendor

Apple Inc. `

maniacx commented 4 months ago

So what exactly is the issue? is it still giving notification saying missing dependencies?

kaveht commented 4 months ago

This time it didn't. I believe it is working and I am able to change the thresholds. Thanks for your help.

maniacx commented 4 months ago

Strange... I havent changed anything. Only added logs.

Anyway thanks for point out about the wrong dependencies link. Seems like both m-series and intel uses same Vendor DMI Apple Inc. So I will just link to device-compatiblity page instead of specific device page.

kaveht commented 4 months ago

Since I am new to the linux world, I had asked a friend to reinstall pop os and install the extension; I wonder if that is why it is working. In any case, it is great having the option to change the thresholds. Thanks again.

maniacx commented 4 months ago

No problem. I will close this issue. If any problem feel free to re-open this issue.