Re: Mess in the /sysfs: class/power_supply/bq2415x

[Pali RohÃr]

On Wednesday 25 February 2015 22:44:42 Sebastian Reichel wrote:
> Hi Pavel,
> 
> On Sat, Jan 31, 2015 at 10:39:19PM +0100, Pavel Machek wrote:
> > So we have multiple values in one sysfs file:
> > 
> > root@n900:/sys/class/power_supply# cat bq24150a-0/registers
> > 0x00=0x50
> > 0x01=0xc8
> > 0x02=0x8e
> > 0x03=0x4b
> > 0x04=0x99
> > 
> > (should this go to debugfs somewhere?)
> 
> I guess this should be dropped in favour of debugfs. A
> standardized debugfs node will be exported automatically if
> the driver is converted to regmap.
> 
> > Unobvious and undocumented entries:
> > 
> > root@n900:/sys/class/power_supply# cat
> > bq24150a-0/boost_status 0
> > pavel@duo:/data/l/linux-n900$ grep -ri boost_status
> > Documentation/ pavel@duo:/data/l/linux-n900$ grep -ri
> > termination_current Documentation/
> > pavel@duo:/data/l/linux-n900$ grep -ri current_now
> > Documentation/ pavel@duo:/data/l/linux-n900$ grep -ri
> > time_to_empty_avg Documentation/
> > pavel@duo:/data/l/linux-n900$
> 
> Anton seems to have missed making Pali to write a
> Documentation patch. Pali: Can you do so, please?
> 

Some of properties are standard power_supply properties and those 
are documented in Documentation/power/power_supply_class.txt

bq2415x specific are documented only in bq2415x_charger.c source 
code. So probably we should move documentation into 
Documentation/power/bq2415x_charer.txt

But I think there are other stuff with higher priority for N900 
as moving comments/documentation...

> > And different units in different parts of tree:
> > 
> > (these are mA)
> > 
> > root@n900:/sys/class/power_supply# cat
> > bq24150a-0/current_limit 1800
> > root@n900:/sys/class/power_supply# cat
> > bq24150a-0/termination_current 100
> > 

Those are non standard power-supply properties and so unit is not 
defined.

> > (and this is in uA)
> > 
> > root@n900:/sys/class/power_supply# cat bq27200-0/current_now
> > 12138
> > 

This is power supply.

> > (I believe uA is right, so it is bq24150a that should be
> > fixed....?)
> > 

Units
~~~~~
Quoting include/linux/power_supply.h:

  All voltages, currents, charges, energies, time and 
temperatures in ÂV,
  ÂA, ÂAh, ÂWh, seconds and tenths of degree Celsius unless 
otherwise
  stated. It's driver's job to convert its raw values to units in 
which
  this class operates.

> > Ok, of course questions are: do we need to do something with
> > it? If so, what?
> 
> For consistency it should be uA, but I guess this would
> introduce a regression for Maemo people. Pali?
> 
> -- Sebastian

It looks like that in Maemo & CSSU we are using only:

/sys/class/power_supply/bq24150a-0/mode
/sys/class/power_supply/bq24150a-0/stat_pin_enable

But it does not mean that some other people do not use other 
bq2415x sysfs nodes.

-- 
Pali RohÃr
pali.rohar@xxxxxxxxx

Attachment: signature.asc
Description: This is a digitally signed message part.