В очередной раз на моём eeepc сломался индикатор батареи. Апстрим код всё переписывает и переписывает, а нормально написать не может :) В моём случае проблема из-за того, что у меня отсутствуют файлы /sys/class/power_supply/BAT0/energy_* зато есть $ ls /sys/class/power_supply/BAT0/charge_* /sys/class/power_supply/BAT0/charge_full /sys/class/power_supply/BAT0/charge_now /sys/class/power_supply/BAT0/charge_full_design
Created attachment 5322 [details] возможный вариант исправления Данный патч вроде бы не должен ломать индикатор в системах, где energy_now всё-таки присутствует, так как выше по коду есть пересчёт из energy в charge. Но можно ещё и проверку добавить на предмет (b->energy_now == -1 || b->energy_full == -1) Если выполняется, то использовать charge, если нет, то energy
Это после последнего обновления поломалось? Я там вот этот патч применил, который исправил ошибку с заморозкой панели: http://git.altlinux.org/people/radik/packages/lxde-lxpanel.git?p=lxde-lxpanel.git;a=commit;h=0c1fb08a70ae10d4e959988752fa2e4320dd2faa Ваше исправление точно ничего не сломает? Ато я LXDEsktop релизить хочу..
Не должно, но я сегодня попозже напишу патч, с которым при наличии energy_* будет работать точно как в апстриме, а при отсутствии исправляет ошибку.
Created attachment 5324 [details] менее интрузивный вариант Да, сломалось с версией 0.5.8-alt4, но не из-за самого этого фикса, а из-за сопутствующего ему мерджа. Проявляется у меня на eeepc 1000, и, кажется, на эйсере на каком-то, завтра на работе постараюсь убедиться. Больше проверить негде, ноутбуков больше нет. Второй вариант исправления точно ничего не ломает, хотя первый мне кажется ближе к тому, что хотели сделать авторы.
Алексей, а расскажите что значит "сломался индикатор батареи"? Я к тому, что у меня на моем lenovo сейчас все работает. Буду тестить Ваш фикс в том числе на своем ноуте..
(В ответ на комментарий №5) > Алексей, а расскажите что значит "сломался индикатор батареи"? То, что написано в названии бага :) Он всегда показывает 100% независимо от заряда батареи. При этом gkrellm, например, заряд показывает правильно. > Я к тому, что у меня на моем lenovo сейчас все работает. Буду тестить Ваш фикс > в том числе на своем ноуте.. Это означает, что на lenovo есть файлы /sys/sys/class/power_supply/BAT0/energy_now и /class/power_supply/BAT0/energy_full. На eeepc их нет (но есть соответствующие charge_*). На асере не помню, кажется тоже не было, завтра посмотрю. Вероятно у авторов тоже есть эти файлы, раз у них всё работает :)
тьфу, скопировалось криво /sys/class/power_supply/BAT0/energy_now и /sys/class/power_supply/BAT0/energy_full конечно же
вообще, выкинуть бы весь этот мусор и научиться брать из upower...
Выкинуть бы хорошо, только я, например, за это сейчас не возьмусь. Проверил 0.5.8-alt4 на Acer Aspire 2920. Там та же проблема, так что у меня нет ноутбуков, где бы нынешняя сизифная версия нормально работала.
Спасибо.. Соберу lxpanel с Вашим патчем, попробую на себе. Если ничего не отвалится, соберу в сизиф и р6.
lxde-lxpanel-0.5.8-alt5 -> sisyphus: * Thu Feb 23 2012 Radik Usupov <radik@altlinux> 0.5.8-alt5 - Added batt_percentage_fix.patch (by Alexey Borisenkov) (Closes: 26948)
