diff for duplicates of <20160109230709.GA30551@amd> diff --git a/a/1.txt b/N1/1.txt index 7f07f39..6449b6e 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -7,13 +7,13 @@ It used to report battery capacity remaining in percent. Not sure what it reports now, but ain't in percent.... -pavel@n900:/my/tui/ofone$ cat +pavel at n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity 7497 -pavel@n900:/my/tui/ofone$ uname -a +pavel at n900:/my/tui/ofone$ uname -a Linux n900 4.4.0-rc8-omap3-149467-g69aafd8-dirty #114 PREEMPT Sat Jan 9 21:44:00 CET 2016 armv7l GNU/Linux -pavel@n900:/my/tui/ofone$ +pavel at n900:/my/tui/ofone$ And power_supply_class.txt says: @@ -21,26 +21,26 @@ And power_supply_class.txt says: ~ ~ ~ ~ -~ Because both "charge" (µAh) and "energy" (µWh) represents "capacity" +~ Because both "charge" (?Ah) and "energy" (?Wh) represents "capacity" ~ ~ of battery, this class distinguish these terms. Don't mix them! ~ ~ ~ -~ CHARGE_* attributes represents capacity in µAh only. +~ CHARGE_* attributes represents capacity in ?Ah only. ~ -~ ENERGY_* attributes represents capacity in µWh only. +~ ENERGY_* attributes represents capacity in ?Wh only. ~ ~ CAPACITY attribute represents capacity in *percents*, from 0 to 100. ~ --- hmm. So that seems to be a mistake. -pavel@n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity +pavel at n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity 7497 -pavel@n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity +pavel at n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity 16713 -pavel@n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity +pavel at n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity 25162 Converting them to hex gives rather strange values. @@ -125,7 +125,7 @@ index 880233c..e9f26f5 100644 --- a/drivers/power/bq27xxx_battery.c +++ b/drivers/power/bq27xxx_battery.c @@ -483,11 +483,11 @@ static int bq27xxx_battery_read_soc(struct bq27xxx_device_info *di) - * Return a battery charge value in µAh + * Return a battery charge value in ?Ah * Or < 0 if something fails. */ -static int bq27xxx_battery_read_charge(struct bq27xxx_device_info *di, u8 reg) diff --git a/a/content_digest b/N1/content_digest index ef9d29c..a116780 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,20 +1,7 @@ - "From\0Pavel Machek <pavel@ucw.cz>\0" + "From\0pavel@ucw.cz (Pavel Machek)\0" "Subject\0/sys/class/power_supply/bq27200-0/capacity changed meaning between 4.1 and 4.4?\0" "Date\0Sun, 10 Jan 2016 00:07:09 +0100\0" - "To\0pali.rohar@gmail.com" - sre@debian.org - sre@ring0.de - kernel list <linux-kernel@vger.kernel.org> - linux-arm-kernel <linux-arm-kernel@lists.infradead.org> - linux-omap@vger.kernel.org - tony@atomide.com - khilman@kernel.org - aaro.koskinen@iki.fi - ivo.g.dimitrov.75@gmail.com - patrikbachan@gmail.com - " serge@hallyn.com\0" - "Cc\0a.hajda@samsung.com" - " afd@ti.com\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "Hi!\n" @@ -26,13 +13,13 @@ "\n" "Not sure what it reports now, but ain't in percent....\n" "\n" - "pavel@n900:/my/tui/ofone$ cat\n" + "pavel at n900:/my/tui/ofone$ cat\n" "/sys/class/power_supply/bq27200-0/capacity\n" "7497\n" - "pavel@n900:/my/tui/ofone$ uname -a\n" + "pavel at n900:/my/tui/ofone$ uname -a\n" "Linux n900 4.4.0-rc8-omap3-149467-g69aafd8-dirty #114 PREEMPT Sat Jan\n" "9 21:44:00 CET 2016 armv7l GNU/Linux\n" - "pavel@n900:/my/tui/ofone$\n" + "pavel at n900:/my/tui/ofone$\n" "\n" "And power_supply_class.txt says:\n" "\n" @@ -40,26 +27,26 @@ "~ ~\n" "~\n" "~\n" - "~ Because both \"charge\" (\302\265Ah) and \"energy\" (\302\265Wh) represents \"capacity\"\n" + "~ Because both \"charge\" (?Ah) and \"energy\" (?Wh) represents \"capacity\"\n" "~\n" "~ of battery, this class distinguish these terms. Don't mix them!\n" "~\n" "~\n" "~\n" - "~ CHARGE_* attributes represents capacity in \302\265Ah only.\n" + "~ CHARGE_* attributes represents capacity in ?Ah only.\n" "~\n" - "~ ENERGY_* attributes represents capacity in \302\265Wh only.\n" + "~ ENERGY_* attributes represents capacity in ?Wh only.\n" "~\n" "~ CAPACITY attribute represents capacity in *percents*, from 0 to\n" "100. ~\n" "\n" "--- hmm. So that seems to be a mistake.\n" "\n" - "pavel@n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity\n" + "pavel at n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity\n" "7497\n" - "pavel@n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity\n" + "pavel at n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity\n" "16713\n" - "pavel@n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity\n" + "pavel at n900:/my/tui/ofone$ cat /sys/class/power_supply/bq27200-0/capacity\n" "25162\n" "\n" "Converting them to hex gives rather strange values.\n" @@ -144,7 +131,7 @@ "--- a/drivers/power/bq27xxx_battery.c\n" "+++ b/drivers/power/bq27xxx_battery.c\n" "@@ -483,11 +483,11 @@ static int bq27xxx_battery_read_soc(struct bq27xxx_device_info *di)\n" - " * Return a battery charge value in \302\265Ah\n" + " * Return a battery charge value in ?Ah\n" " * Or < 0 if something fails.\n" " */\n" "-static int bq27xxx_battery_read_charge(struct bq27xxx_device_info *di, u8 reg)\n" @@ -234,4 +221,4 @@ "(english) http://www.livejournal.com/~pavelmachek\n" (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -5cb7e380b2cb801b8d2304e6b9042b0a9dee5864ef231db75b6a62040eec2455 +ef6c79fd58a2da2d81a0c3d312df63149ee08f6ab14de8f3a07a2e82302bb342
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.