RE: [PATCH 2/3] power: core: Add variables related temperature to power_supply_info.

2014-11-11 Thread Tc, Jenny
> > The CC/CV for each battery temperature zone is defined as part of battery > > spec. > This is > > as per the JEITA/PSE standards. So IMO, this is a battery charging > > information > > (charging object) rather than a thermal throttling information. > > > > Also the battery information may not

Re: [PATCH 2/3] power: core: Add variables related temperature to power_supply_info.

2014-11-11 Thread Pavel Machek
On Tue 2014-11-11 04:56:31, Tc, Jenny wrote: > > > The CC,CV and restart threshold would vary based on the battery > > > temperature > > > So I would suggest to have temperature zone table as part of battery info > > > along with other attributes. > > > > > > int iterm; //charge termination curre

RE: [PATCH 2/3] power: core: Add variables related temperature to power_supply_info.

2014-11-10 Thread Tc, Jenny
> > The CC,CV and restart threshold would vary based on the battery temperature > > So I would suggest to have temperature zone table as part of battery info > > along with other attributes. > > > > int iterm; //charge termination current (used to stop charging) > > int temp_zone_count; // number

Re: [PATCH 2/3] power: core: Add variables related temperature to power_supply_info.

2014-11-10 Thread jonghwa3 . lee
On 2014년 11월 10일 20:16, Tc, Jenny wrote: >> @@ -241,6 +241,8 @@ struct power_supply_info { >> int charge_empty_design; >> int energy_full_design; >> int energy_empty_design; >> +int temperature_max; >> +int temperature_min; >> int use_for_apm; >> }; > > > The CC,CV a

RE: [PATCH 2/3] power: core: Add variables related temperature to power_supply_info.

2014-11-10 Thread Tc, Jenny
> @@ -241,6 +241,8 @@ struct power_supply_info { > int charge_empty_design; > int energy_full_design; > int energy_empty_design; > + int temperature_max; > + int temperature_min; > int use_for_apm; > }; The CC,CV and restart threshold would vary based on the batte