* [PATCH v2] thermal: rcar_gen3_thermal: fix initialization sequence for H3 ES2.0
@ 2017-10-16 14:52 Niklas Söderlund
2017-10-16 15:39 ` Niklas Söderlund
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Niklas Söderlund @ 2017-10-16 14:52 UTC (permalink / raw)
To: linux-pm, Wolfram Sang
Cc: linux-renesas-soc, Zhang Rui, Eduardo Valentin,
Geert Uytterhoeven, Niklas Söderlund
The initialization sequence for H3 (r8a7795) ES1.x and ES2.0 is
different. H3 ES2.0 and later uses the same sequence as M3 (r8a7796)
ES1.0. Fix this by not looking at compatible strings and instead
defaulting to the r8a7796 initialization sequence and use
soc_device_match() to check for H3 ES1.x.
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
drivers/thermal/rcar_gen3_thermal.c | 34 +++++++++++++++-------------------
1 file changed, 15 insertions(+), 19 deletions(-)
diff --git a/drivers/thermal/rcar_gen3_thermal.c b/drivers/thermal/rcar_gen3_thermal.c
index 203aca44a2bb4bdf..288104440226d90f 100644
--- a/drivers/thermal/rcar_gen3_thermal.c
+++ b/drivers/thermal/rcar_gen3_thermal.c
@@ -24,6 +24,7 @@
#include <linux/platform_device.h>
#include <linux/pm_runtime.h>
#include <linux/spinlock.h>
+#include <linux/sys_soc.h>
#include <linux/thermal.h>
#include "thermal_core.h"
@@ -90,10 +91,6 @@ struct rcar_gen3_thermal_priv {
struct rcar_gen3_thermal_tsc *tscs[TSC_MAX_NUM];
unsigned int num_tscs;
spinlock_t lock; /* Protect interrupts on and off */
- const struct rcar_gen3_thermal_data *data;
-};
-
-struct rcar_gen3_thermal_data {
void (*thermal_init)(struct rcar_gen3_thermal_tsc *tsc);
};
@@ -278,7 +275,12 @@ static irqreturn_t rcar_gen3_thermal_irq_thread(int irq, void *data)
return IRQ_HANDLED;
}
-static void r8a7795_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
+static const struct soc_device_attribute r8a7795es1[] = {
+ { .soc_id = "r8a7795", .revision = "ES1.*" },
+ { /* sentinel */ }
+};
+
+static void rcar_gen3_thermal_init_r8a7795es1(struct rcar_gen3_thermal_tsc *tsc)
{
rcar_gen3_thermal_write(tsc, REG_GEN3_CTSR, CTSR_THBGR);
rcar_gen3_thermal_write(tsc, REG_GEN3_CTSR, 0x0);
@@ -303,7 +305,7 @@ static void r8a7795_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
usleep_range(1000, 2000);
}
-static void r8a7796_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
+static void rcar_gen3_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
{
u32 reg_val;
@@ -324,17 +326,9 @@ static void r8a7796_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
usleep_range(1000, 2000);
}
-static const struct rcar_gen3_thermal_data r8a7795_data = {
- .thermal_init = r8a7795_thermal_init,
-};
-
-static const struct rcar_gen3_thermal_data r8a7796_data = {
- .thermal_init = r8a7796_thermal_init,
-};
-
static const struct of_device_id rcar_gen3_thermal_dt_ids[] = {
- { .compatible = "renesas,r8a7795-thermal", .data = &r8a7795_data},
- { .compatible = "renesas,r8a7796-thermal", .data = &r8a7796_data},
+ { .compatible = "renesas,r8a7795-thermal", },
+ { .compatible = "renesas,r8a7796-thermal", },
{},
};
MODULE_DEVICE_TABLE(of, rcar_gen3_thermal_dt_ids);
@@ -371,7 +365,9 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev)
if (!priv)
return -ENOMEM;
- priv->data = of_device_get_match_data(dev);
+ priv->thermal_init = &rcar_gen3_thermal_init;
+ if (soc_device_match(r8a7795es1))
+ priv->thermal_init = &rcar_gen3_thermal_init_r8a7795es1;
spin_lock_init(&priv->lock);
@@ -423,7 +419,7 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev)
priv->tscs[i] = tsc;
- priv->data->thermal_init(tsc);
+ priv->thermal_init(tsc);
rcar_gen3_thermal_calc_coefs(&tsc->coef, ptat, thcode[i]);
zone = devm_thermal_zone_of_sensor_register(dev, i, tsc,
@@ -476,7 +472,7 @@ static int __maybe_unused rcar_gen3_thermal_resume(struct device *dev)
for (i = 0; i < priv->num_tscs; i++) {
struct rcar_gen3_thermal_tsc *tsc = priv->tscs[i];
- priv->data->thermal_init(tsc);
+ priv->thermal_init(tsc);
rcar_gen3_thermal_set_trips(tsc, tsc->low, tsc->high);
}
--
2.14.2
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH v2] thermal: rcar_gen3_thermal: fix initialization sequence for H3 ES2.0
2017-10-16 14:52 [PATCH v2] thermal: rcar_gen3_thermal: fix initialization sequence for H3 ES2.0 Niklas Söderlund
@ 2017-10-16 15:39 ` Niklas Söderlund
2017-10-16 16:35 ` Geert Uytterhoeven
2017-10-16 22:07 ` Wolfram Sang
2 siblings, 0 replies; 5+ messages in thread
From: Niklas Söderlund @ 2017-10-16 15:39 UTC (permalink / raw)
To: linux-pm, Wolfram Sang
Cc: linux-renesas-soc, Zhang Rui, Eduardo Valentin, Geert Uytterhoeven
Hi,
I forgot the changes from v1, sorry about that.
On 2017-10-16 16:52:59 +0200, Niklas S�derlund wrote:
> The initialization sequence for H3 (r8a7795) ES1.x and ES2.0 is
> different. H3 ES2.0 and later uses the same sequence as M3 (r8a7796)
> ES1.0. Fix this by not looking at compatible strings and instead
> defaulting to the r8a7796 initialization sequence and use
> soc_device_match() to check for H3 ES1.x.
>
> Signed-off-by: Niklas S�derlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> drivers/thermal/rcar_gen3_thermal.c | 34 +++++++++++++++-------------------
> 1 file changed, 15 insertions(+), 19 deletions(-)
* Changes since v1
- Use positive test to check for pre-production SoCs as suggested by
Geert.
- Renamed init functions as currently the only known odd init case is H3
ES1.x, other Gen3 SoC uses same init process.
- Remove one layer of indirection by removing the struct
rcar_gen3_thermal_data and store the thermal_init function pointer
directly in struct rcar_gen3_thermal_priv.
- Fix spelling mistakes.
>
> diff --git a/drivers/thermal/rcar_gen3_thermal.c b/drivers/thermal/rcar_gen3_thermal.c
> index 203aca44a2bb4bdf..288104440226d90f 100644
> --- a/drivers/thermal/rcar_gen3_thermal.c
> +++ b/drivers/thermal/rcar_gen3_thermal.c
> @@ -24,6 +24,7 @@
> #include <linux/platform_device.h>
> #include <linux/pm_runtime.h>
> #include <linux/spinlock.h>
> +#include <linux/sys_soc.h>
> #include <linux/thermal.h>
>
> #include "thermal_core.h"
> @@ -90,10 +91,6 @@ struct rcar_gen3_thermal_priv {
> struct rcar_gen3_thermal_tsc *tscs[TSC_MAX_NUM];
> unsigned int num_tscs;
> spinlock_t lock; /* Protect interrupts on and off */
> - const struct rcar_gen3_thermal_data *data;
> -};
> -
> -struct rcar_gen3_thermal_data {
> void (*thermal_init)(struct rcar_gen3_thermal_tsc *tsc);
> };
>
> @@ -278,7 +275,12 @@ static irqreturn_t rcar_gen3_thermal_irq_thread(int irq, void *data)
> return IRQ_HANDLED;
> }
>
> -static void r8a7795_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> +static const struct soc_device_attribute r8a7795es1[] = {
> + { .soc_id = "r8a7795", .revision = "ES1.*" },
> + { /* sentinel */ }
> +};
> +
> +static void rcar_gen3_thermal_init_r8a7795es1(struct rcar_gen3_thermal_tsc *tsc)
> {
> rcar_gen3_thermal_write(tsc, REG_GEN3_CTSR, CTSR_THBGR);
> rcar_gen3_thermal_write(tsc, REG_GEN3_CTSR, 0x0);
> @@ -303,7 +305,7 @@ static void r8a7795_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> usleep_range(1000, 2000);
> }
>
> -static void r8a7796_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> +static void rcar_gen3_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> {
> u32 reg_val;
>
> @@ -324,17 +326,9 @@ static void r8a7796_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> usleep_range(1000, 2000);
> }
>
> -static const struct rcar_gen3_thermal_data r8a7795_data = {
> - .thermal_init = r8a7795_thermal_init,
> -};
> -
> -static const struct rcar_gen3_thermal_data r8a7796_data = {
> - .thermal_init = r8a7796_thermal_init,
> -};
> -
> static const struct of_device_id rcar_gen3_thermal_dt_ids[] = {
> - { .compatible = "renesas,r8a7795-thermal", .data = &r8a7795_data},
> - { .compatible = "renesas,r8a7796-thermal", .data = &r8a7796_data},
> + { .compatible = "renesas,r8a7795-thermal", },
> + { .compatible = "renesas,r8a7796-thermal", },
> {},
> };
> MODULE_DEVICE_TABLE(of, rcar_gen3_thermal_dt_ids);
> @@ -371,7 +365,9 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev)
> if (!priv)
> return -ENOMEM;
>
> - priv->data = of_device_get_match_data(dev);
> + priv->thermal_init = &rcar_gen3_thermal_init;
> + if (soc_device_match(r8a7795es1))
> + priv->thermal_init = &rcar_gen3_thermal_init_r8a7795es1;
>
> spin_lock_init(&priv->lock);
>
> @@ -423,7 +419,7 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev)
>
> priv->tscs[i] = tsc;
>
> - priv->data->thermal_init(tsc);
> + priv->thermal_init(tsc);
> rcar_gen3_thermal_calc_coefs(&tsc->coef, ptat, thcode[i]);
>
> zone = devm_thermal_zone_of_sensor_register(dev, i, tsc,
> @@ -476,7 +472,7 @@ static int __maybe_unused rcar_gen3_thermal_resume(struct device *dev)
> for (i = 0; i < priv->num_tscs; i++) {
> struct rcar_gen3_thermal_tsc *tsc = priv->tscs[i];
>
> - priv->data->thermal_init(tsc);
> + priv->thermal_init(tsc);
> rcar_gen3_thermal_set_trips(tsc, tsc->low, tsc->high);
> }
>
> --
> 2.14.2
>
--
Regards,
Niklas S�derlund
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] thermal: rcar_gen3_thermal: fix initialization sequence for H3 ES2.0
@ 2017-10-16 15:39 ` Niklas Söderlund
0 siblings, 0 replies; 5+ messages in thread
From: Niklas Söderlund @ 2017-10-16 15:39 UTC (permalink / raw)
To: linux-pm, Wolfram Sang
Cc: linux-renesas-soc, Zhang Rui, Eduardo Valentin, Geert Uytterhoeven
Hi,
I forgot the changes from v1, sorry about that.
On 2017-10-16 16:52:59 +0200, Niklas Söderlund wrote:
> The initialization sequence for H3 (r8a7795) ES1.x and ES2.0 is
> different. H3 ES2.0 and later uses the same sequence as M3 (r8a7796)
> ES1.0. Fix this by not looking at compatible strings and instead
> defaulting to the r8a7796 initialization sequence and use
> soc_device_match() to check for H3 ES1.x.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> drivers/thermal/rcar_gen3_thermal.c | 34 +++++++++++++++-------------------
> 1 file changed, 15 insertions(+), 19 deletions(-)
* Changes since v1
- Use positive test to check for pre-production SoCs as suggested by
Geert.
- Renamed init functions as currently the only known odd init case is H3
ES1.x, other Gen3 SoC uses same init process.
- Remove one layer of indirection by removing the struct
rcar_gen3_thermal_data and store the thermal_init function pointer
directly in struct rcar_gen3_thermal_priv.
- Fix spelling mistakes.
>
> diff --git a/drivers/thermal/rcar_gen3_thermal.c b/drivers/thermal/rcar_gen3_thermal.c
> index 203aca44a2bb4bdf..288104440226d90f 100644
> --- a/drivers/thermal/rcar_gen3_thermal.c
> +++ b/drivers/thermal/rcar_gen3_thermal.c
> @@ -24,6 +24,7 @@
> #include <linux/platform_device.h>
> #include <linux/pm_runtime.h>
> #include <linux/spinlock.h>
> +#include <linux/sys_soc.h>
> #include <linux/thermal.h>
>
> #include "thermal_core.h"
> @@ -90,10 +91,6 @@ struct rcar_gen3_thermal_priv {
> struct rcar_gen3_thermal_tsc *tscs[TSC_MAX_NUM];
> unsigned int num_tscs;
> spinlock_t lock; /* Protect interrupts on and off */
> - const struct rcar_gen3_thermal_data *data;
> -};
> -
> -struct rcar_gen3_thermal_data {
> void (*thermal_init)(struct rcar_gen3_thermal_tsc *tsc);
> };
>
> @@ -278,7 +275,12 @@ static irqreturn_t rcar_gen3_thermal_irq_thread(int irq, void *data)
> return IRQ_HANDLED;
> }
>
> -static void r8a7795_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> +static const struct soc_device_attribute r8a7795es1[] = {
> + { .soc_id = "r8a7795", .revision = "ES1.*" },
> + { /* sentinel */ }
> +};
> +
> +static void rcar_gen3_thermal_init_r8a7795es1(struct rcar_gen3_thermal_tsc *tsc)
> {
> rcar_gen3_thermal_write(tsc, REG_GEN3_CTSR, CTSR_THBGR);
> rcar_gen3_thermal_write(tsc, REG_GEN3_CTSR, 0x0);
> @@ -303,7 +305,7 @@ static void r8a7795_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> usleep_range(1000, 2000);
> }
>
> -static void r8a7796_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> +static void rcar_gen3_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> {
> u32 reg_val;
>
> @@ -324,17 +326,9 @@ static void r8a7796_thermal_init(struct rcar_gen3_thermal_tsc *tsc)
> usleep_range(1000, 2000);
> }
>
> -static const struct rcar_gen3_thermal_data r8a7795_data = {
> - .thermal_init = r8a7795_thermal_init,
> -};
> -
> -static const struct rcar_gen3_thermal_data r8a7796_data = {
> - .thermal_init = r8a7796_thermal_init,
> -};
> -
> static const struct of_device_id rcar_gen3_thermal_dt_ids[] = {
> - { .compatible = "renesas,r8a7795-thermal", .data = &r8a7795_data},
> - { .compatible = "renesas,r8a7796-thermal", .data = &r8a7796_data},
> + { .compatible = "renesas,r8a7795-thermal", },
> + { .compatible = "renesas,r8a7796-thermal", },
> {},
> };
> MODULE_DEVICE_TABLE(of, rcar_gen3_thermal_dt_ids);
> @@ -371,7 +365,9 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev)
> if (!priv)
> return -ENOMEM;
>
> - priv->data = of_device_get_match_data(dev);
> + priv->thermal_init = &rcar_gen3_thermal_init;
> + if (soc_device_match(r8a7795es1))
> + priv->thermal_init = &rcar_gen3_thermal_init_r8a7795es1;
>
> spin_lock_init(&priv->lock);
>
> @@ -423,7 +419,7 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev)
>
> priv->tscs[i] = tsc;
>
> - priv->data->thermal_init(tsc);
> + priv->thermal_init(tsc);
> rcar_gen3_thermal_calc_coefs(&tsc->coef, ptat, thcode[i]);
>
> zone = devm_thermal_zone_of_sensor_register(dev, i, tsc,
> @@ -476,7 +472,7 @@ static int __maybe_unused rcar_gen3_thermal_resume(struct device *dev)
> for (i = 0; i < priv->num_tscs; i++) {
> struct rcar_gen3_thermal_tsc *tsc = priv->tscs[i];
>
> - priv->data->thermal_init(tsc);
> + priv->thermal_init(tsc);
> rcar_gen3_thermal_set_trips(tsc, tsc->low, tsc->high);
> }
>
> --
> 2.14.2
>
--
Regards,
Niklas Söderlund
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] thermal: rcar_gen3_thermal: fix initialization sequence for H3 ES2.0
2017-10-16 14:52 [PATCH v2] thermal: rcar_gen3_thermal: fix initialization sequence for H3 ES2.0 Niklas Söderlund
2017-10-16 15:39 ` Niklas Söderlund
@ 2017-10-16 16:35 ` Geert Uytterhoeven
2017-10-16 22:07 ` Wolfram Sang
2 siblings, 0 replies; 5+ messages in thread
From: Geert Uytterhoeven @ 2017-10-16 16:35 UTC (permalink / raw)
To: Niklas Söderlund
Cc: Linux PM list, Wolfram Sang, Linux-Renesas, Zhang Rui, Eduardo Valentin
Hi Niklas,
On Mon, Oct 16, 2017 at 4:52 PM, Niklas Söderlund
<niklas.soderlund+renesas@ragnatech.se> wrote:
> The initialization sequence for H3 (r8a7795) ES1.x and ES2.0 is
> different. H3 ES2.0 and later uses the same sequence as M3 (r8a7796)
> ES1.0. Fix this by not looking at compatible strings and instead
> defaulting to the r8a7796 initialization sequence and use
> soc_device_match() to check for H3 ES1.x.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> --- a/drivers/thermal/rcar_gen3_thermal.c
> +++ b/drivers/thermal/rcar_gen3_thermal.c
> @@ -371,7 +365,9 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev)
> if (!priv)
> return -ENOMEM;
>
> - priv->data = of_device_get_match_data(dev);
> + priv->thermal_init = &rcar_gen3_thermal_init;
You can drop the "&".
> + if (soc_device_match(r8a7795es1))
> + priv->thermal_init = &rcar_gen3_thermal_init_r8a7795es1;
Likewise.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] thermal: rcar_gen3_thermal: fix initialization sequence for H3 ES2.0
2017-10-16 14:52 [PATCH v2] thermal: rcar_gen3_thermal: fix initialization sequence for H3 ES2.0 Niklas Söderlund
2017-10-16 15:39 ` Niklas Söderlund
2017-10-16 16:35 ` Geert Uytterhoeven
@ 2017-10-16 22:07 ` Wolfram Sang
2 siblings, 0 replies; 5+ messages in thread
From: Wolfram Sang @ 2017-10-16 22:07 UTC (permalink / raw)
To: Niklas Söderlund
Cc: linux-pm, Wolfram Sang, linux-renesas-soc, Zhang Rui,
Eduardo Valentin, Geert Uytterhoeven
[-- Attachment #1: Type: text/plain, Size: 568 bytes --]
On Mon, Oct 16, 2017 at 04:52:59PM +0200, Niklas Söderlund wrote:
> The initialization sequence for H3 (r8a7795) ES1.x and ES2.0 is
> different. H3 ES2.0 and later uses the same sequence as M3 (r8a7796)
> ES1.0. Fix this by not looking at compatible strings and instead
> defaulting to the r8a7796 initialization sequence and use
> soc_device_match() to check for H3 ES1.x.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Looks good to me, and what Geert said :)
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] 5+ messages in thread
end of thread, other threads:[~2017-10-16 22:07 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-16 14:52 [PATCH v2] thermal: rcar_gen3_thermal: fix initialization sequence for H3 ES2.0 Niklas Söderlund
2017-10-16 15:39 ` Niklas Söderlund
2017-10-16 15:39 ` Niklas Söderlund
2017-10-16 16:35 ` Geert Uytterhoeven
2017-10-16 22:07 ` Wolfram Sang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.