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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/