Messages in this thread | | | Date | Tue, 11 Nov 2014 09:30:24 +0900 | From | jonghwa3.lee@samsung ... | Subject | Re: [PATCH 2/3] power: core: Add variables related temperature to power_supply_info. |
| |
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 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 of temperature zone tables present > struct batt_temp_mon_table temp_mon_tbl[MAX_TEMP_MON_TABLE]; //temperature zone table array > > struct batt_temp_mon_table { > short int temp_max; > short int cc; > short int cv; > short int vbat_vchk_drop_uv; > short int temp_min; > }; >
IMO, throttling cc/cv according the temperature can be done via thermal fw interface. However voltage drop and charging termination current can be added here.
Jonghwa
> -Jenny > >
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |