Commit bdc4480a authored by Yoshihiro Kaneko's avatar Yoshihiro Kaneko Committed by Eduardo Valentin

thermal: rcar_gen3_thermal: Update calculation formula of IRQTEMP

Update the formula to calculate CTEMP:
Currently, the CTEMP is average of val1 (is calculated by
formula 1) and val2 (is calculated by formula 2). But,
as description in HWM (chapter 10A.3.1.1 Setting of Normal Mode)

If (STEMP < Tj_T) CTEMP value should be val1.
If (STEMP > Tj_T) CTEMP value should be val2.
Signed-off-by: default avatarYoshihiro Kaneko <ykaneko0929@gmail.com>
Signed-off-by: default avatarEduardo Valentin <edubezval@gmail.com>
parent 4eb39f79
...@@ -76,6 +76,7 @@ struct rcar_gen3_thermal_tsc { ...@@ -76,6 +76,7 @@ struct rcar_gen3_thermal_tsc {
struct equation_coefs coef; struct equation_coefs coef;
int low; int low;
int high; int high;
int tj_t;
}; };
struct rcar_gen3_thermal_priv { struct rcar_gen3_thermal_priv {
...@@ -124,28 +125,26 @@ static inline void rcar_gen3_thermal_write(struct rcar_gen3_thermal_tsc *tsc, ...@@ -124,28 +125,26 @@ static inline void rcar_gen3_thermal_write(struct rcar_gen3_thermal_tsc *tsc,
/* no idea where these constants come from */ /* no idea where these constants come from */
#define TJ_3 -41 #define TJ_3 -41
static void rcar_gen3_thermal_calc_coefs(struct equation_coefs *coef, static void rcar_gen3_thermal_calc_coefs(struct rcar_gen3_thermal_tsc *tsc,
int *ptat, int *thcode, int *ptat, int *thcode,
int ths_tj_1) int ths_tj_1)
{ {
int tj_2;
/* TODO: Find documentation and document constant calculation formula */ /* TODO: Find documentation and document constant calculation formula */
/* /*
* Division is not scaled in BSP and if scaled it might overflow * Division is not scaled in BSP and if scaled it might overflow
* the dividend (4095 * 4095 << 14 > INT_MAX) so keep it unscaled * the dividend (4095 * 4095 << 14 > INT_MAX) so keep it unscaled
*/ */
tj_2 = (FIXPT_INT((ptat[1] - ptat[2]) * 157) tsc->tj_t = (FIXPT_INT((ptat[1] - ptat[2]) * 157)
/ (ptat[0] - ptat[2])) + FIXPT_INT(TJ_3); / (ptat[0] - ptat[2])) + FIXPT_INT(TJ_3);
coef->a1 = FIXPT_DIV(FIXPT_INT(thcode[1] - thcode[2]), tsc->coef.a1 = FIXPT_DIV(FIXPT_INT(thcode[1] - thcode[2]),
tj_2 - FIXPT_INT(TJ_3)); tsc->tj_t - FIXPT_INT(TJ_3));
coef->b1 = FIXPT_INT(thcode[2]) - coef->a1 * TJ_3; tsc->coef.b1 = FIXPT_INT(thcode[2]) - tsc->coef.a1 * TJ_3;
coef->a2 = FIXPT_DIV(FIXPT_INT(thcode[1] - thcode[0]), tsc->coef.a2 = FIXPT_DIV(FIXPT_INT(thcode[1] - thcode[0]),
tj_2 - FIXPT_INT(ths_tj_1)); tsc->tj_t - FIXPT_INT(ths_tj_1));
coef->b2 = FIXPT_INT(thcode[0]) - coef->a2 * ths_tj_1; tsc->coef.b2 = FIXPT_INT(thcode[0]) - tsc->coef.a2 * ths_tj_1;
} }
static int rcar_gen3_thermal_round(int temp) static int rcar_gen3_thermal_round(int temp)
...@@ -184,13 +183,15 @@ static int rcar_gen3_thermal_get_temp(void *devdata, int *temp) ...@@ -184,13 +183,15 @@ static int rcar_gen3_thermal_get_temp(void *devdata, int *temp)
static int rcar_gen3_thermal_mcelsius_to_temp(struct rcar_gen3_thermal_tsc *tsc, static int rcar_gen3_thermal_mcelsius_to_temp(struct rcar_gen3_thermal_tsc *tsc,
int mcelsius) int mcelsius)
{ {
int celsius, val1, val2; int celsius, val;
celsius = DIV_ROUND_CLOSEST(mcelsius, 1000); celsius = DIV_ROUND_CLOSEST(mcelsius, 1000);
val1 = celsius * tsc->coef.a1 + tsc->coef.b1; if (celsius <= INT_FIXPT(tsc->tj_t))
val2 = celsius * tsc->coef.a2 + tsc->coef.b2; val = celsius * tsc->coef.a1 + tsc->coef.b1;
else
val = celsius * tsc->coef.a2 + tsc->coef.b2;
return INT_FIXPT((val1 + val2) / 2); return INT_FIXPT(val);
} }
static int rcar_gen3_thermal_set_trips(void *devdata, int low, int high) static int rcar_gen3_thermal_set_trips(void *devdata, int low, int high)
...@@ -417,7 +418,7 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev) ...@@ -417,7 +418,7 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev)
priv->tscs[i] = tsc; priv->tscs[i] = tsc;
priv->thermal_init(tsc); priv->thermal_init(tsc);
rcar_gen3_thermal_calc_coefs(&tsc->coef, ptat, thcode[i], rcar_gen3_thermal_calc_coefs(tsc, ptat, thcode[i],
*rcar_gen3_ths_tj_1); *rcar_gen3_ths_tj_1);
zone = devm_thermal_zone_of_sensor_register(dev, i, tsc, zone = devm_thermal_zone_of_sensor_register(dev, i, tsc,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment