* [PATCH/RFT] thermal: rcar_thermal: update calculation formula for E3
@ 2019-02-25 20:45 Yoshihiro Kaneko
2019-02-26 11:12 ` Simon Horman
2019-03-05 11:02 ` Wolfram Sang
0 siblings, 2 replies; 3+ messages in thread
From: Yoshihiro Kaneko @ 2019-02-25 20:45 UTC (permalink / raw)
To: linux-pm
Cc: Zhang Rui, Eduardo Valentin, Rob Herring, Simon Horman,
Magnus Damm, linux-renesas-soc
HW manual changes temperature calculation formula for E3:
- When CTEMP is less than 24
T = CTEMP[5:0] * 5.5 - 72
- When CTEMP is equal to/greater than 24
T = CTEMP[5:0] * 5 - 60
This was inspired by a patch in the BSP by Van Do <van.do.xw@renesas.com>
Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
---
This patch is based on the master branch of Linus Torvalds's linux tree.
drivers/thermal/rcar_thermal.c | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/thermal/rcar_thermal.c b/drivers/thermal/rcar_thermal.c
index 97462e9..11df0cc 100644
--- a/drivers/thermal/rcar_thermal.c
+++ b/drivers/thermal/rcar_thermal.c
@@ -52,6 +52,7 @@ struct rcar_thermal_chip {
unsigned int irq_per_ch : 1;
unsigned int needs_suspend_resume : 1;
unsigned int nirqs;
+ unsigned int ctemp_bands;
};
static const struct rcar_thermal_chip rcar_thermal = {
@@ -60,6 +61,7 @@ struct rcar_thermal_chip {
.irq_per_ch = 0,
.needs_suspend_resume = 0,
.nirqs = 1,
+ .ctemp_bands = 1,
};
static const struct rcar_thermal_chip rcar_gen2_thermal = {
@@ -68,6 +70,7 @@ struct rcar_thermal_chip {
.irq_per_ch = 0,
.needs_suspend_resume = 0,
.nirqs = 1,
+ .ctemp_bands = 1,
};
static const struct rcar_thermal_chip rcar_gen3_thermal = {
@@ -80,6 +83,7 @@ struct rcar_thermal_chip {
* interrupts to detect a temperature change, rise or fall.
*/
.nirqs = 2,
+ .ctemp_bands = 2,
};
struct rcar_thermal_priv {
@@ -263,7 +267,12 @@ static int rcar_thermal_get_current_temp(struct rcar_thermal_priv *priv,
return ret;
mutex_lock(&priv->lock);
- tmp = MCELSIUS((priv->ctemp * 5) - 65);
+ if (priv->chip->ctemp_bands == 1)
+ tmp = MCELSIUS((priv->ctemp * 5) - 65);
+ else if (priv->ctemp < 24)
+ tmp = MCELSIUS(((priv->ctemp * 55) - 720) / 10);
+ else
+ tmp = MCELSIUS((priv->ctemp * 5) - 60);
mutex_unlock(&priv->lock);
if ((tmp < MCELSIUS(-45)) || (tmp > MCELSIUS(125))) {
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH/RFT] thermal: rcar_thermal: update calculation formula for E3
2019-02-25 20:45 [PATCH/RFT] thermal: rcar_thermal: update calculation formula for E3 Yoshihiro Kaneko
@ 2019-02-26 11:12 ` Simon Horman
2019-03-05 11:02 ` Wolfram Sang
1 sibling, 0 replies; 3+ messages in thread
From: Simon Horman @ 2019-02-26 11:12 UTC (permalink / raw)
To: Yoshihiro Kaneko
Cc: linux-pm, Zhang Rui, Eduardo Valentin, Rob Herring, Magnus Damm,
linux-renesas-soc
On Tue, Feb 26, 2019 at 05:45:58AM +0900, Yoshihiro Kaneko wrote:
> HW manual changes temperature calculation formula for E3:
Thanks Kaneko-san,
I think it would be good do describe this patch not just in terms of E3.
Of the IP versions supported by the driver I believe this effects:
R-Car V3M (r8a77970)
R-Car E3 (r8a77990)
R-Car D3 (r8a77995)
RZ/G2E (r8a774c0)
> - When CTEMP is less than 24
> T = CTEMP[5:0] * 5.5 - 72
> - When CTEMP is equal to/greater than 24
> T = CTEMP[5:0] * 5 - 60
>
> This was inspired by a patch in the BSP by Van Do <van.do.xw@renesas.com>
>
> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
> ---
>
> This patch is based on the master branch of Linus Torvalds's linux tree.
>
> drivers/thermal/rcar_thermal.c | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
Using renesas-devel-20190225-v5.0-rc8 on an E3 based Ebisu board, I see:
# cat /sys/devices/virtual/thermal/thermal_zone0/temp
25000
# N=1; while [ $N -lt 1000000 ]; do N=$(($N + 1)); done & \
N=1; while [ $N -lt 1000000 ]; do N=$(($N + 1)); done & \
wait; \
cat /sys/devices/virtual/thermal/thermal_zone0/temp
30000
And with this patch applied on top of renesas-devel-20190225-v5.0-rc8 I see:
# cat /sys/devices/virtual/thermal/thermal_zone0/temp
27000
# N=1; while [ $N -lt 1000000 ]; do N=$(($N + 1)); done & \
N=1; while [ $N -lt 1000000 ]; do N=$(($N + 1)); done & \
wait; \
cat /sys/devices/virtual/thermal/thermal_zone0/temp
32000
I see the same, 27000/32000, result when running the test on
BSP v3.9.3-rc1, which includes Van Do's work.
The E3 has 2 cores
Tested-by: Simon Horman <horms+renesas@verge.net.au>
> diff --git a/drivers/thermal/rcar_thermal.c b/drivers/thermal/rcar_thermal.c
> index 97462e9..11df0cc 100644
> --- a/drivers/thermal/rcar_thermal.c
> +++ b/drivers/thermal/rcar_thermal.c
> @@ -52,6 +52,7 @@ struct rcar_thermal_chip {
> unsigned int irq_per_ch : 1;
> unsigned int needs_suspend_resume : 1;
> unsigned int nirqs;
> + unsigned int ctemp_bands;
> };
>
> static const struct rcar_thermal_chip rcar_thermal = {
> @@ -60,6 +61,7 @@ struct rcar_thermal_chip {
> .irq_per_ch = 0,
> .needs_suspend_resume = 0,
> .nirqs = 1,
> + .ctemp_bands = 1,
> };
>
> static const struct rcar_thermal_chip rcar_gen2_thermal = {
> @@ -68,6 +70,7 @@ struct rcar_thermal_chip {
> .irq_per_ch = 0,
> .needs_suspend_resume = 0,
> .nirqs = 1,
> + .ctemp_bands = 1,
> };
>
> static const struct rcar_thermal_chip rcar_gen3_thermal = {
> @@ -80,6 +83,7 @@ struct rcar_thermal_chip {
> * interrupts to detect a temperature change, rise or fall.
> */
> .nirqs = 2,
> + .ctemp_bands = 2,
> };
>
> struct rcar_thermal_priv {
> @@ -263,7 +267,12 @@ static int rcar_thermal_get_current_temp(struct rcar_thermal_priv *priv,
> return ret;
>
> mutex_lock(&priv->lock);
> - tmp = MCELSIUS((priv->ctemp * 5) - 65);
> + if (priv->chip->ctemp_bands == 1)
> + tmp = MCELSIUS((priv->ctemp * 5) - 65);
> + else if (priv->ctemp < 24)
> + tmp = MCELSIUS(((priv->ctemp * 55) - 720) / 10);
> + else
> + tmp = MCELSIUS((priv->ctemp * 5) - 60);
> mutex_unlock(&priv->lock);
>
> if ((tmp < MCELSIUS(-45)) || (tmp > MCELSIUS(125))) {
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH/RFT] thermal: rcar_thermal: update calculation formula for E3
2019-02-25 20:45 [PATCH/RFT] thermal: rcar_thermal: update calculation formula for E3 Yoshihiro Kaneko
2019-02-26 11:12 ` Simon Horman
@ 2019-03-05 11:02 ` Wolfram Sang
1 sibling, 0 replies; 3+ messages in thread
From: Wolfram Sang @ 2019-03-05 11:02 UTC (permalink / raw)
To: Yoshihiro Kaneko
Cc: linux-pm, Zhang Rui, Eduardo Valentin, Rob Herring, Simon Horman,
Magnus Damm, linux-renesas-soc
[-- Attachment #1: Type: text/plain, Size: 473 bytes --]
On Tue, Feb 26, 2019 at 05:45:58AM +0900, Yoshihiro Kaneko wrote:
> HW manual changes temperature calculation formula for E3:
> - When CTEMP is less than 24
> T = CTEMP[5:0] * 5.5 - 72
> - When CTEMP is equal to/greater than 24
> T = CTEMP[5:0] * 5 - 60
>
> This was inspired by a patch in the BSP by Van Do <van.do.xw@renesas.com>
>
> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
Acked-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-03-05 11:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-25 20:45 [PATCH/RFT] thermal: rcar_thermal: update calculation formula for E3 Yoshihiro Kaneko
2019-02-26 11:12 ` Simon Horman
2019-03-05 11:02 ` Wolfram Sang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).