* [PATCH v7 1/3] dt: bindings: add documentation for zx2967 family thermal sensor @ 2017-02-03 10:10 Baoyou Xie 2017-02-03 10:10 ` [PATCH v7 2/3] MAINTAINERS: add zx2967 thermal drivers to ARM ZTE architecture Baoyou Xie 2017-02-03 10:10 ` [PATCH v7 3/3] thermal: zx2967: add thermal driver for ZTE's zx2967 family Baoyou Xie 0 siblings, 2 replies; 4+ messages in thread From: Baoyou Xie @ 2017-02-03 10:10 UTC (permalink / raw) To: mathieu.poirier, jun.nie, rui.zhang, edubezval, robh+dt, mark.rutland Cc: linux-arm-kernel, linux-pm, devicetree, linux-kernel, shawnguo, baoyou.xie, xie.baoyou, chen.chaokai, wang.qiang01 This patch adds dt-binding documentation for zx2967 family thermal sensor. Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org> Acked-by: Rob Herring <robh@kernel.org> Reviewed-by: Shawn Guo <shawnguo@kernel.org> --- .../devicetree/bindings/thermal/zx2967-thermal.txt | 116 +++++++++++++++++++++ 1 file changed, 116 insertions(+) create mode 100644 Documentation/devicetree/bindings/thermal/zx2967-thermal.txt diff --git a/Documentation/devicetree/bindings/thermal/zx2967-thermal.txt b/Documentation/devicetree/bindings/thermal/zx2967-thermal.txt new file mode 100644 index 0000000..169a699 --- /dev/null +++ b/Documentation/devicetree/bindings/thermal/zx2967-thermal.txt @@ -0,0 +1,116 @@ +* ZTE zx2967 family Thermal + +Required Properties: +- compatible: should be one of the following. + * zte,zx296718-thermal +- reg: physical base address of the controller and length of memory mapped + region. +- clocks : Pairs of phandle and specifier referencing the controller's clocks. +- clock-names: "gate" for the topcrm clock. + "pclk" for the apb clock. +- #thermal-sensor-cells: must be 0. + +Please note: coefficients defined in thermal-zones section need to be +multiplied by 1000. + +Example for tempsensor: + + tempsensor: tempsensor@148a000 { + compatible = "zte,zx296718-thermal"; + reg = <0x0148a000 0x20>; + clocks = <&topcrm TEMPSENSOR_GATE>, <&audiocrm AUDIO_TS_PCLK>; + clock-names = "gate", "pclk"; + #thermal-sensor-cells = <0>; + }; + +Example for cooling device: + + cooling_dev: cooling_dev { + cluster0_cooling_dev: cluster0-cooling-dev { + #cooling-cells = <2>; + cpumask = <0xf>; + capacitance = <1500>; + }; + + cluster1_cooling_dev: cluster1-cooling-dev { + #cooling-cells = <2>; + cpumask = <0x30>; + capacitance = <2000>; + }; + }; + +Example for thermal zones: + + thermal-zones { + zx296718_thermal: zx296718_thermal { + polling-delay-passive = <500>; + polling-delay = <1000>; + sustainable-power = <6500>; + + thermal-sensors = <&tempsensor 0>; + /* + * both slope and offset need to be multiplied by 1000. + */ + coefficients = <1951 (-922)>; + + trips { + trip0: switch_on_temperature { + temperature = <90000>; + hysteresis = <2000>; + type = "passive"; + }; + + trip1: desired_temperature { + temperature = <100000>; + hysteresis = <2000>; + type = "passive"; + }; + + crit: critical_temperature { + temperature = <110000>; + hysteresis = <2000>; + type = "critical"; + }; + }; + + cooling-maps { + map0 { + trip = <&trip0>; + cooling-device = <&gpu 2 5>; + }; + + map1 { + trip = <&trip0>; + cooling-device = <&cluster0_cooling_dev 1 2>; + }; + + map2 { + trip = <&trip1>; + cooling-device = <&cluster0_cooling_dev 1 2>; + }; + + map3 { + trip = <&crit>; + cooling-device = <&cluster0_cooling_dev 1 2>; + }; + + map4 { + trip = <&trip0>; + cooling-device = <&cluster1_cooling_dev 1 2>; + contribution = <9000>; + }; + + map5 { + trip = <&trip1>; + cooling-device = <&cluster1_cooling_dev 1 2>; + contribution = <4096>; + }; + + map6 { + trip = <&crit>; + cooling-device = <&cluster1_cooling_dev 1 2>; + contribution = <4096>; + }; + }; + }; + }; -- 2.7.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v7 2/3] MAINTAINERS: add zx2967 thermal drivers to ARM ZTE architecture 2017-02-03 10:10 [PATCH v7 1/3] dt: bindings: add documentation for zx2967 family thermal sensor Baoyou Xie @ 2017-02-03 10:10 ` Baoyou Xie 2017-02-03 10:10 ` [PATCH v7 3/3] thermal: zx2967: add thermal driver for ZTE's zx2967 family Baoyou Xie 1 sibling, 0 replies; 4+ messages in thread From: Baoyou Xie @ 2017-02-03 10:10 UTC (permalink / raw) To: mathieu.poirier, jun.nie, rui.zhang, edubezval, robh+dt, mark.rutland Cc: linux-arm-kernel, linux-pm, devicetree, linux-kernel, shawnguo, baoyou.xie, xie.baoyou, chen.chaokai, wang.qiang01 Add the zx2967 thermal drivers as maintained by ARM ZTE architecture maintainers, as they're parts of the core IP. Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org> --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 5fb9b62..edfdea3 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1989,10 +1989,12 @@ F: arch/arm/mach-zx/ F: drivers/clk/zte/ F: drivers/reset/reset-zx2967.c F: drivers/soc/zte/ +F: drivers/thermal/zx* F: Documentation/devicetree/bindings/arm/zte.txt F: Documentation/devicetree/bindings/clock/zx296702-clk.txt F: Documentation/devicetree/bindings/reset/zte,zx2967-reset.txt F: Documentation/devicetree/bindings/soc/zte/ +F: Documentation/devicetree/bindings/thermal/zx* F: include/dt-bindings/soc/zx*.h ARM/ZYNQ ARCHITECTURE -- 2.7.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v7 3/3] thermal: zx2967: add thermal driver for ZTE's zx2967 family 2017-02-03 10:10 [PATCH v7 1/3] dt: bindings: add documentation for zx2967 family thermal sensor Baoyou Xie 2017-02-03 10:10 ` [PATCH v7 2/3] MAINTAINERS: add zx2967 thermal drivers to ARM ZTE architecture Baoyou Xie @ 2017-02-03 10:10 ` Baoyou Xie 2017-02-03 16:36 ` Mathieu Poirier 1 sibling, 1 reply; 4+ messages in thread From: Baoyou Xie @ 2017-02-03 10:10 UTC (permalink / raw) To: mathieu.poirier, jun.nie, rui.zhang, edubezval, robh+dt, mark.rutland Cc: linux-arm-kernel, linux-pm, devicetree, linux-kernel, shawnguo, baoyou.xie, xie.baoyou, chen.chaokai, wang.qiang01 This patch adds thermal driver for ZTE's zx2967 family. Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org> --- drivers/thermal/Kconfig | 8 ++ drivers/thermal/Makefile | 1 + drivers/thermal/zx2967_thermal.c | 281 +++++++++++++++++++++++++++++++++++++++ 3 files changed, 290 insertions(+) create mode 100644 drivers/thermal/zx2967_thermal.c diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index 18f2de6..f64bd50 100644 --- a/drivers/thermal/Kconfig +++ b/drivers/thermal/Kconfig @@ -444,4 +444,12 @@ config BCM2835_THERMAL help Support for thermal sensors on Broadcom bcm2835 SoCs. +config ZX2967_THERMAL + tristate "Thermal sensors on zx2967 SoC" + depends on ARCH_ZX || COMPILE_TEST + help + Enable the zx2967 thermal sensors driver, which supports + the primitive temperature sensor embedded in zx2967 SoCs. + This sensor generates the real time die temperature. + endif diff --git a/drivers/thermal/Makefile b/drivers/thermal/Makefile index 677c6d9..c00c05e 100644 --- a/drivers/thermal/Makefile +++ b/drivers/thermal/Makefile @@ -57,3 +57,4 @@ obj-$(CONFIG_HISI_THERMAL) += hisi_thermal.o obj-$(CONFIG_MTK_THERMAL) += mtk_thermal.o obj-$(CONFIG_GENERIC_ADC_THERMAL) += thermal-generic-adc.o obj-$(CONFIG_BCM2835_THERMAL) += bcm2835_thermal.o +obj-$(CONFIG_ZX2967_THERMAL) += zx2967_thermal.o diff --git a/drivers/thermal/zx2967_thermal.c b/drivers/thermal/zx2967_thermal.c new file mode 100644 index 0000000..fba55b1c --- /dev/null +++ b/drivers/thermal/zx2967_thermal.c @@ -0,0 +1,281 @@ +/* + * ZTE's zx2967 family thermal sensor driver + * + * Copyright (C) 2017 ZTE Ltd. + * + * Author: Baoyou Xie <baoyou.xie@linaro.org> + * + * License terms: GNU General Public License (GPL) version 2 + */ + +#include <linux/clk.h> +#include <linux/delay.h> +#include <linux/device.h> +#include <linux/err.h> +#include <linux/iopoll.h> +#include <linux/module.h> +#include <linux/platform_device.h> +#include <linux/thermal.h> + +/* Power Mode: 0->low 1->high */ +#define ZX2967_THERMAL_POWER_MODE 0 +#define ZX2967_POWER_MODE_LOW 0 +#define ZX2967_POWER_MODE_HIGH 1 + +/* DCF Control Register */ +#define ZX2967_THERMAL_DCF 0x4 +#define ZX2967_DCF_EN BIT(1) +#define ZX2967_DCF_FREEZE BIT(0) + +/* Selection Register */ +#define ZX2967_THERMAL_SEL 0x8 + +/* Control Register */ +#define ZX2967_THERMAL_CTRL 0x10 + +#define ZX2967_THERMAL_READY BIT(12) +#define ZX2967_THERMAL_TEMP_MASK 0xfff +#define ZX2967_THERMAL_ID_MASK 0x18 +#define ZX2967_THERMAL_ID0 0x10 +#define ZX2967_THERMAL_ID1 0x8 + +#define ZX2967_GET_TEMP_TIMEOUT_US (100 * 1024) + +struct zx2967_thermal_sensor { + struct zx2967_thermal_priv *priv; + struct thermal_zone_device *tzd; + int id; +}; + +/* + * Now, one sensor remains in system, but in fact, we intend to support + * two sensors. + */ +#define NUM_SENSORS 1 + +struct zx2967_thermal_priv { + struct zx2967_thermal_sensor sensors[NUM_SENSORS]; + /* Prevents reads sensor in parallel */ + struct mutex lock; + struct clk *clk_gate; + struct clk *pclk; + void __iomem *regs; + struct device *dev; +}; + +#ifdef CONFIG_PM_SLEEP +static int zx2967_thermal_suspend(struct device *dev) +{ + struct platform_device *pdev = to_platform_device(dev); + struct zx2967_thermal_priv *priv = platform_get_drvdata(pdev); + + if (priv && priv->pclk) + clk_disable_unprepare(priv->pclk); + + if (priv && priv->clk_gate) + clk_disable_unprepare(priv->clk_gate); + + return 0; +} + +static int zx2967_thermal_resume(struct device *dev) +{ + struct platform_device *pdev = to_platform_device(dev); + struct zx2967_thermal_priv *priv = platform_get_drvdata(pdev); + int error; + + error = clk_prepare_enable(priv->clk_gate); + if (error) + return error; + + error = clk_prepare_enable(priv->pclk); + if (error) { + clk_disable_unprepare(priv->clk_gate); + return error; + } + + return 0; +} +static SIMPLE_DEV_PM_OPS(zx2967_thermal_pm_ops, + zx2967_thermal_suspend, zx2967_thermal_resume); +#endif + +static int zx2967_thermal_get_temp(void *data, int *temp) +{ + void __iomem *regs; + struct zx2967_thermal_sensor *sensor = data; + struct zx2967_thermal_priv *priv = sensor->priv; + u32 val, sel_id; + int ret; + + if (!sensor->tzd) + return -EAGAIN; + + regs = priv->regs; + mutex_lock(&priv->lock); + writel_relaxed(ZX2967_POWER_MODE_LOW, + regs + ZX2967_THERMAL_POWER_MODE); + writel_relaxed(ZX2967_DCF_EN, regs + ZX2967_THERMAL_DCF); + + val = readl_relaxed(regs + ZX2967_THERMAL_SEL); + val &= ~ZX2967_THERMAL_ID_MASK; + sel_id = sensor->id ? ZX2967_THERMAL_ID1 : ZX2967_THERMAL_ID0; + val |= sel_id; + writel_relaxed(val, regs + ZX2967_THERMAL_SEL); + + usleep_range(100, 300); + ret = readx_poll_timeout(readl, regs + ZX2967_THERMAL_CTRL, + val, val & ZX2967_THERMAL_READY, 300, + ZX2967_GET_TEMP_TIMEOUT_US); + if (ret) { + dev_err(priv->dev, "Thermal sensor %d data timeout\n", + sensor->id); + mutex_unlock(&priv->lock); + return ret; + } + + writel_relaxed(ZX2967_DCF_FREEZE | ZX2967_DCF_EN, + regs + ZX2967_THERMAL_DCF); + val = readl_relaxed(regs + ZX2967_THERMAL_CTRL) + & ZX2967_THERMAL_TEMP_MASK; + writel_relaxed(ZX2967_POWER_MODE_HIGH, + regs + ZX2967_THERMAL_POWER_MODE); + + /* + * Calculate temperature + * In dts, coefficients are multiplied by 1000. + */ + *temp = DIV_ROUND_CLOSEST(((s32)val + sensor->tzd->tzp->offset) * 1000, + sensor->tzd->tzp->slope); + + mutex_unlock(&priv->lock); + + return 0; +} + +static struct thermal_zone_of_device_ops zx2967_of_thermal_ops = { + .get_temp = zx2967_thermal_get_temp, +}; + +static int zx2967_thermal_probe(struct platform_device *pdev) +{ + struct zx2967_thermal_priv *priv; + struct resource *res; + int ret, i; + + priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL); + if (!priv) + return -ENOMEM; + + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); + priv->regs = devm_ioremap_resource(&pdev->dev, res); + if (IS_ERR(priv->regs)) + return PTR_ERR(priv->regs); + + priv->clk_gate = devm_clk_get(&pdev->dev, "gate"); + if (IS_ERR(priv->clk_gate)) { + ret = PTR_ERR(priv->clk_gate); + dev_err(&pdev->dev, "failed to get clock gate: %d\n", ret); + return ret; + } + + ret = clk_prepare_enable(priv->clk_gate); + if (ret) { + dev_err(&pdev->dev, "failed to enable converter clock: %d\n", + ret); + return ret; + } + + priv->pclk = devm_clk_get(&pdev->dev, "pclk"); + if (IS_ERR(priv->pclk)) { + ret = PTR_ERR(priv->pclk); + dev_err(&pdev->dev, "failed to get apb clock: %d\n", ret); + goto disable_clk_gate; + } + + ret = clk_prepare_enable(priv->pclk); + if (ret) { + dev_err(&pdev->dev, "failed to enable converter clock: %d\n", + ret); + goto disable_clk_gate; + } + + mutex_init(&priv->lock); + for (i = 0; i < NUM_SENSORS; i++) { + struct zx2967_thermal_sensor *sensor = &priv->sensors[i]; + + sensor->priv = priv; + sensor->id = i; + sensor->tzd = thermal_zone_of_sensor_register(&pdev->dev, + i, sensor, &zx2967_of_thermal_ops); + + if (IS_ERR(sensor->tzd)) { + ret = PTR_ERR(sensor->tzd); + dev_err(&pdev->dev, "failed to register sensor %d: %d\n", + i, ret); + goto remove_ts; + } + + if (sensor->tzd->tzp->slope == 0) { + thermal_zone_of_sensor_unregister(&pdev->dev, + sensor->tzd); + dev_err(&pdev->dev, + "coefficients of sensor-%d is invalid\n", i); + ret = -EINVAL; + goto remove_ts; + } + } + priv->dev = &pdev->dev; + platform_set_drvdata(pdev, priv); + + return 0; + +remove_ts: + for (i--; i >= 0; i--) + thermal_zone_of_sensor_unregister(&pdev->dev, + priv->sensors[i].tzd); + clk_disable_unprepare(priv->pclk); +disable_clk_gate: + clk_disable_unprepare(priv->clk_gate); + + return ret; +} + +static int zx2967_thermal_exit(struct platform_device *pdev) +{ + struct zx2967_thermal_priv *priv = platform_get_drvdata(pdev); + int i; + + for (i = 0; i < NUM_SENSORS; i++) { + struct zx2967_thermal_sensor *sensor = &priv->sensors[i]; + + thermal_zone_of_sensor_unregister(&pdev->dev, sensor->tzd); + } + clk_disable_unprepare(priv->pclk); + clk_disable_unprepare(priv->clk_gate); + + return 0; +} + +static const struct of_device_id zx2967_thermal_id_table[] = { + { .compatible = "zte,zx296718-thermal" }, + {} +}; +MODULE_DEVICE_TABLE(of, zx2967_thermal_id_table); + +static struct platform_driver zx2967_thermal_driver = { + .probe = zx2967_thermal_probe, + .remove = zx2967_thermal_exit, + .driver = { + .name = "zx2967_thermal", + .of_match_table = zx2967_thermal_id_table, +#ifdef CONFIG_PM_SLEEP + .pm = &zx2967_thermal_pm_ops, +#endif + }, +}; +module_platform_driver(zx2967_thermal_driver); + +MODULE_AUTHOR("Baoyou Xie <baoyou.xie@linaro.org>"); +MODULE_DESCRIPTION("ZTE zx2967 thermal driver"); +MODULE_LICENSE("GPL v2"); -- 2.7.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v7 3/3] thermal: zx2967: add thermal driver for ZTE's zx2967 family 2017-02-03 10:10 ` [PATCH v7 3/3] thermal: zx2967: add thermal driver for ZTE's zx2967 family Baoyou Xie @ 2017-02-03 16:36 ` Mathieu Poirier 0 siblings, 0 replies; 4+ messages in thread From: Mathieu Poirier @ 2017-02-03 16:36 UTC (permalink / raw) To: Baoyou Xie Cc: jun.nie, rui.zhang, edubezval, robh+dt, mark.rutland, linux-arm-kernel, linux-pm, devicetree, linux-kernel, shawnguo, xie.baoyou, chen.chaokai, wang.qiang01 On Fri, Feb 03, 2017 at 06:10:55PM +0800, Baoyou Xie wrote: > This patch adds thermal driver for ZTE's zx2967 family. > > Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org> > --- > drivers/thermal/Kconfig | 8 ++ > drivers/thermal/Makefile | 1 + > drivers/thermal/zx2967_thermal.c | 281 +++++++++++++++++++++++++++++++++++++++ > 3 files changed, 290 insertions(+) > create mode 100644 drivers/thermal/zx2967_thermal.c > > diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig > index 18f2de6..f64bd50 100644 > --- a/drivers/thermal/Kconfig > +++ b/drivers/thermal/Kconfig > @@ -444,4 +444,12 @@ config BCM2835_THERMAL > help > Support for thermal sensors on Broadcom bcm2835 SoCs. > > +config ZX2967_THERMAL > + tristate "Thermal sensors on zx2967 SoC" > + depends on ARCH_ZX || COMPILE_TEST > + help > + Enable the zx2967 thermal sensors driver, which supports > + the primitive temperature sensor embedded in zx2967 SoCs. > + This sensor generates the real time die temperature. > + > endif > diff --git a/drivers/thermal/Makefile b/drivers/thermal/Makefile > index 677c6d9..c00c05e 100644 > --- a/drivers/thermal/Makefile > +++ b/drivers/thermal/Makefile > @@ -57,3 +57,4 @@ obj-$(CONFIG_HISI_THERMAL) += hisi_thermal.o > obj-$(CONFIG_MTK_THERMAL) += mtk_thermal.o > obj-$(CONFIG_GENERIC_ADC_THERMAL) += thermal-generic-adc.o > obj-$(CONFIG_BCM2835_THERMAL) += bcm2835_thermal.o > +obj-$(CONFIG_ZX2967_THERMAL) += zx2967_thermal.o > diff --git a/drivers/thermal/zx2967_thermal.c b/drivers/thermal/zx2967_thermal.c > new file mode 100644 > index 0000000..fba55b1c > --- /dev/null > +++ b/drivers/thermal/zx2967_thermal.c > @@ -0,0 +1,281 @@ > +/* > + * ZTE's zx2967 family thermal sensor driver > + * > + * Copyright (C) 2017 ZTE Ltd. > + * > + * Author: Baoyou Xie <baoyou.xie@linaro.org> > + * > + * License terms: GNU General Public License (GPL) version 2 > + */ > + > +#include <linux/clk.h> > +#include <linux/delay.h> > +#include <linux/device.h> > +#include <linux/err.h> > +#include <linux/iopoll.h> > +#include <linux/module.h> > +#include <linux/platform_device.h> > +#include <linux/thermal.h> > + > +/* Power Mode: 0->low 1->high */ > +#define ZX2967_THERMAL_POWER_MODE 0 > +#define ZX2967_POWER_MODE_LOW 0 > +#define ZX2967_POWER_MODE_HIGH 1 > + > +/* DCF Control Register */ > +#define ZX2967_THERMAL_DCF 0x4 > +#define ZX2967_DCF_EN BIT(1) > +#define ZX2967_DCF_FREEZE BIT(0) > + > +/* Selection Register */ > +#define ZX2967_THERMAL_SEL 0x8 > + > +/* Control Register */ > +#define ZX2967_THERMAL_CTRL 0x10 > + > +#define ZX2967_THERMAL_READY BIT(12) > +#define ZX2967_THERMAL_TEMP_MASK 0xfff > +#define ZX2967_THERMAL_ID_MASK 0x18 > +#define ZX2967_THERMAL_ID0 0x10 > +#define ZX2967_THERMAL_ID1 0x8 > + > +#define ZX2967_GET_TEMP_TIMEOUT_US (100 * 1024) > + > +struct zx2967_thermal_sensor { > + struct zx2967_thermal_priv *priv; > + struct thermal_zone_device *tzd; > + int id; > +}; > + > +/* > + * Now, one sensor remains in system, but in fact, we intend to support > + * two sensors. > + */ The general rule of thumb is to write drivers for the current HW and add enhancement when new revision come out but I will let Zhang and Eduardo make the call. > +#define NUM_SENSORS 1 > + > +struct zx2967_thermal_priv { > + struct zx2967_thermal_sensor sensors[NUM_SENSORS]; > + /* Prevents reads sensor in parallel */ > + struct mutex lock; > + struct clk *clk_gate; > + struct clk *pclk; > + void __iomem *regs; > + struct device *dev; > +}; Normally all new structure need to use the proper comment style [1] (as noted in my previous email) but this one is pretty obvious and private to this file. Again I'll let Zhang and Eduardo decide what they want to do here. [1]. http://lxr.free-electrons.com/source/Documentation/CodingStyle > + > +#ifdef CONFIG_PM_SLEEP > +static int zx2967_thermal_suspend(struct device *dev) > +{ > + struct platform_device *pdev = to_platform_device(dev); > + struct zx2967_thermal_priv *priv = platform_get_drvdata(pdev); > + > + if (priv && priv->pclk) > + clk_disable_unprepare(priv->pclk); > + > + if (priv && priv->clk_gate) > + clk_disable_unprepare(priv->clk_gate); > + > + return 0; > +} > + > +static int zx2967_thermal_resume(struct device *dev) > +{ > + struct platform_device *pdev = to_platform_device(dev); > + struct zx2967_thermal_priv *priv = platform_get_drvdata(pdev); > + int error; > + > + error = clk_prepare_enable(priv->clk_gate); > + if (error) > + return error; > + > + error = clk_prepare_enable(priv->pclk); > + if (error) { > + clk_disable_unprepare(priv->clk_gate); > + return error; > + } > + > + return 0; > +} > +static SIMPLE_DEV_PM_OPS(zx2967_thermal_pm_ops, > + zx2967_thermal_suspend, zx2967_thermal_resume); The pm_runtime macros are built in such a way that the call to SIMPLE_DEV_PM_OPS itself doesn't have to be enclosed in the #if/#endif. That way you don't have to add the #if/#endif in the platform driver definition. > +#endif > + > +static int zx2967_thermal_get_temp(void *data, int *temp) > +{ > + void __iomem *regs; > + struct zx2967_thermal_sensor *sensor = data; > + struct zx2967_thermal_priv *priv = sensor->priv; > + u32 val, sel_id; > + int ret; > + > + if (!sensor->tzd) > + return -EAGAIN; > + > + regs = priv->regs; > + mutex_lock(&priv->lock); > + writel_relaxed(ZX2967_POWER_MODE_LOW, > + regs + ZX2967_THERMAL_POWER_MODE); > + writel_relaxed(ZX2967_DCF_EN, regs + ZX2967_THERMAL_DCF); > + > + val = readl_relaxed(regs + ZX2967_THERMAL_SEL); > + val &= ~ZX2967_THERMAL_ID_MASK; > + sel_id = sensor->id ? ZX2967_THERMAL_ID1 : ZX2967_THERMAL_ID0; > + val |= sel_id; > + writel_relaxed(val, regs + ZX2967_THERMAL_SEL); > + > + usleep_range(100, 300); You have not addressed my comment from the previous review, something that affects my motivation to keep reviewing your work and help you get your code upstream. > + ret = readx_poll_timeout(readl, regs + ZX2967_THERMAL_CTRL, > + val, val & ZX2967_THERMAL_READY, 300, > + ZX2967_GET_TEMP_TIMEOUT_US); > + if (ret) { > + dev_err(priv->dev, "Thermal sensor %d data timeout\n", > + sensor->id); > + mutex_unlock(&priv->lock); goto out; > + return ret; > + } > + > + writel_relaxed(ZX2967_DCF_FREEZE | ZX2967_DCF_EN, > + regs + ZX2967_THERMAL_DCF); > + val = readl_relaxed(regs + ZX2967_THERMAL_CTRL) > + & ZX2967_THERMAL_TEMP_MASK; > + writel_relaxed(ZX2967_POWER_MODE_HIGH, > + regs + ZX2967_THERMAL_POWER_MODE); > + > + /* > + * Calculate temperature > + * In dts, coefficients are multiplied by 1000. > + */ > + *temp = DIV_ROUND_CLOSEST(((s32)val + sensor->tzd->tzp->offset) * 1000, > + sensor->tzd->tzp->slope); > out: > + mutex_unlock(&priv->lock); > + > + return 0; return ret; > +} > + > +static struct thermal_zone_of_device_ops zx2967_of_thermal_ops = { > + .get_temp = zx2967_thermal_get_temp, > +}; > + > +static int zx2967_thermal_probe(struct platform_device *pdev) > +{ > + struct zx2967_thermal_priv *priv; > + struct resource *res; > + int ret, i; > + > + priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL); > + if (!priv) > + return -ENOMEM; > + > + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > + priv->regs = devm_ioremap_resource(&pdev->dev, res); > + if (IS_ERR(priv->regs)) > + return PTR_ERR(priv->regs); > + > + priv->clk_gate = devm_clk_get(&pdev->dev, "gate"); > + if (IS_ERR(priv->clk_gate)) { > + ret = PTR_ERR(priv->clk_gate); > + dev_err(&pdev->dev, "failed to get clock gate: %d\n", ret); > + return ret; > + } > + > + ret = clk_prepare_enable(priv->clk_gate); > + if (ret) { > + dev_err(&pdev->dev, "failed to enable converter clock: %d\n", > + ret); > + return ret; > + } > + > + priv->pclk = devm_clk_get(&pdev->dev, "pclk"); > + if (IS_ERR(priv->pclk)) { > + ret = PTR_ERR(priv->pclk); > + dev_err(&pdev->dev, "failed to get apb clock: %d\n", ret); > + goto disable_clk_gate; > + } > + > + ret = clk_prepare_enable(priv->pclk); > + if (ret) { > + dev_err(&pdev->dev, "failed to enable converter clock: %d\n", > + ret); > + goto disable_clk_gate; > + } > + > + mutex_init(&priv->lock); > + for (i = 0; i < NUM_SENSORS; i++) { > + struct zx2967_thermal_sensor *sensor = &priv->sensors[i]; > + > + sensor->priv = priv; > + sensor->id = i; > + sensor->tzd = thermal_zone_of_sensor_register(&pdev->dev, > + i, sensor, &zx2967_of_thermal_ops); > + > + if (IS_ERR(sensor->tzd)) { > + ret = PTR_ERR(sensor->tzd); > + dev_err(&pdev->dev, "failed to register sensor %d: %d\n", > + i, ret); > + goto remove_ts; > + } > + > + if (sensor->tzd->tzp->slope == 0) { > + thermal_zone_of_sensor_unregister(&pdev->dev, > + sensor->tzd); > + dev_err(&pdev->dev, > + "coefficients of sensor-%d is invalid\n", i); > + ret = -EINVAL; > + goto remove_ts; > + } > + } > + priv->dev = &pdev->dev; > + platform_set_drvdata(pdev, priv); > + > + return 0; > + > +remove_ts: > + for (i--; i >= 0; i--) > + thermal_zone_of_sensor_unregister(&pdev->dev, > + priv->sensors[i].tzd); > + clk_disable_unprepare(priv->pclk); > +disable_clk_gate: > + clk_disable_unprepare(priv->clk_gate); > + > + return ret; > +} > + > +static int zx2967_thermal_exit(struct platform_device *pdev) > +{ > + struct zx2967_thermal_priv *priv = platform_get_drvdata(pdev); > + int i; > + > + for (i = 0; i < NUM_SENSORS; i++) { > + struct zx2967_thermal_sensor *sensor = &priv->sensors[i]; > + > + thermal_zone_of_sensor_unregister(&pdev->dev, sensor->tzd); > + } > + clk_disable_unprepare(priv->pclk); > + clk_disable_unprepare(priv->clk_gate); > + > + return 0; > +} > + > +static const struct of_device_id zx2967_thermal_id_table[] = { > + { .compatible = "zte,zx296718-thermal" }, > + {} > +}; > +MODULE_DEVICE_TABLE(of, zx2967_thermal_id_table); > + > +static struct platform_driver zx2967_thermal_driver = { > + .probe = zx2967_thermal_probe, > + .remove = zx2967_thermal_exit, > + .driver = { > + .name = "zx2967_thermal", > + .of_match_table = zx2967_thermal_id_table, > +#ifdef CONFIG_PM_SLEEP > + .pm = &zx2967_thermal_pm_ops, > +#endif > + }, > +}; > +module_platform_driver(zx2967_thermal_driver); > + > +MODULE_AUTHOR("Baoyou Xie <baoyou.xie@linaro.org>"); > +MODULE_DESCRIPTION("ZTE zx2967 thermal driver"); > +MODULE_LICENSE("GPL v2"); > -- > 2.7.4 > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-02-03 16:36 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-02-03 10:10 [PATCH v7 1/3] dt: bindings: add documentation for zx2967 family thermal sensor Baoyou Xie 2017-02-03 10:10 ` [PATCH v7 2/3] MAINTAINERS: add zx2967 thermal drivers to ARM ZTE architecture Baoyou Xie 2017-02-03 10:10 ` [PATCH v7 3/3] thermal: zx2967: add thermal driver for ZTE's zx2967 family Baoyou Xie 2017-02-03 16:36 ` Mathieu Poirier
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).