From: Daniel Lezcano <daniel.lezcano@linaro.org> To: Amit Kucheria <amit.kucheria@verdurent.com>, Vasily Khoruzhick <anarsoul@gmail.com> Cc: "Yangtao Li" <tiny.windzz@gmail.com>, "Zhang Rui" <rui.zhang@intel.com>, "Rob Herring" <robh+dt@kernel.org>, "Mark Rutland" <mark.rutland@arm.com>, "Maxime Ripard" <mripard@kernel.org>, "Chen-Yu Tsai" <wens@csie.org>, "Mauro Carvalho Chehab" <mchehab+samsung@kernel.org>, "David S. Miller" <davem@davemloft.net>, "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>, "Linux PM list" <linux-pm@vger.kernel.org>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, lakml <linux-arm-kernel@lists.infradead.org>, "Ondřej Jirman" <megous@megous.com>, LKML <linux-kernel@vger.kernel.org> Subject: Re: [PATCH v8 0/7] add thermal sensor driver for A64, A83T, H3, H5, H6, R40 Date: Thu, 6 Feb 2020 17:46:36 +0100 [thread overview] Message-ID: <af5383b5-2dd4-92ab-ded2-f1cde48bb21a@linaro.org> (raw) In-Reply-To: <CAHLCerPWEDqEE8LRUiO5GpeP+BfnestocndBQq6oXAxVN=+3ow@mail.gmail.com> Hi Amit, On 06/02/2020 15:13, Amit Kucheria wrote: > Hi Vasily, > > For this entire series, the DTS files don't contain any trip points. > Did I miss some other series? > > At a minimum, you should add some "hot" or "critical" trip points > since then don't require a cooling-map with throttling actions. If you > have "passive" trip points, then you need to provide cooling-maps. Except I'm misunderstanding the bindings, a thermal zone must define these required properties: - polling-delay - polling-delay-passive - thermal-sensors - trips - cooling-maps > Since this series has been merged, could you please follow up with a > fixup series to add the trip points? > > Regards, > Amit > p.s. We should catch all this automatically, I'll send out yaml > bindings for the thermal framework soon that should catch this stuff. +1 There was a small discussion about converting the binding to a schema: https://www.spinics.net/lists/devicetree/msg332424.html > On Thu, Dec 19, 2019 at 10:58 PM Vasily Khoruzhick <anarsoul@gmail.com> wrote: >> >> This patchset adds driver for thermal sensor in A64, A83T, H3, H5, >> H6 and R40 SoCs. >> >> v8: >> - [vasily] Address more Maxime's comments for dt-schema >> - [vasily] Add myself to MAINTAINERS for the driver and schema >> - [vasily] Round calibration data size to word boundary for H6 and A64 >> - [vasily] Change offset for A64 since it reports too low temp otherwise. >> Likely conversion formula in user manual is not correct. >> >> v7: >> - [vasily] Address Maxime's comments for dt-schema >> - [vasily] Move common part of H3 and H5 dts into sunxi-h3-h5.dtsi >> - [vasily] Add Maxime's a-b to the driver patch >> >> v6: >> - [ondrej, vasily] Squash all driver related changes into a >> single patch >> - [ondrej] Rename calib -> calibration >> - [ondrej] Fix thermal zone registration check >> - [ondrej] Lower rate of sensor data interrupts to 4/sec/sensor >> - [ondrej] Rework scale/offset values, H6 calibration >> - [ondrej] Explicitly set mod clock to 24 MHz >> - [ondrej] Set undocumented bits in CTRL0 for H6 >> - [ondrej] Add support for A83T >> - [ondrej] Add dts changes for A83T, H3, H5, H6 >> - [vasily] Add dts changes for A64 >> - [vasily] Address Maxime's comments for YAML scheme >> - [vasily] Make .calc_temp callback mandatory >> - [vasily] Set .max_register in regmap config, so regs can be >> inspected using debugfs >> >> Ondrej Jirman (4): >> ARM: dts: sun8i-a83t: Add thermal sensor and thermal zones >> ARM: dts: sun8i-h3: Add thermal sensor and thermal zones >> arm64: dts: allwinner: h5: Add thermal sensor and thermal zones >> arm64: dts: allwinner: h6: Add thermal sensor and thermal zones >> >> Vasily Khoruzhick (1): >> arm64: dts: allwinner: a64: Add thermal sensors and thermal zones >> >> Yangtao Li (2): >> thermal: sun8i: add thermal driver for H6/H5/H3/A64/A83T/R40 >> dt-bindings: thermal: add YAML schema for sun8i-thermal driver >> bindings >> >> .../thermal/allwinner,sun8i-a83t-ths.yaml | 160 +++++ >> MAINTAINERS | 8 + >> arch/arm/boot/dts/sun8i-a83t.dtsi | 36 + >> arch/arm/boot/dts/sun8i-h3.dtsi | 20 + >> arch/arm/boot/dts/sunxi-h3-h5.dtsi | 6 + >> arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 42 ++ >> arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi | 26 + >> arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 33 + >> drivers/thermal/Kconfig | 14 + >> drivers/thermal/Makefile | 1 + >> drivers/thermal/sun8i_thermal.c | 639 ++++++++++++++++++ >> 11 files changed, 985 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/thermal/allwinner,sun8i-a83t-ths.yaml >> create mode 100644 drivers/thermal/sun8i_thermal.c >> >> -- >> 2.24.1 >> -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Lezcano <daniel.lezcano@linaro.org> To: Amit Kucheria <amit.kucheria@verdurent.com>, Vasily Khoruzhick <anarsoul@gmail.com> Cc: "Mark Rutland" <mark.rutland@arm.com>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, "Ondřej Jirman" <megous@megous.com>, "Linux PM list" <linux-pm@vger.kernel.org>, "Yangtao Li" <tiny.windzz@gmail.com>, "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>, LKML <linux-kernel@vger.kernel.org>, "Maxime Ripard" <mripard@kernel.org>, "Chen-Yu Tsai" <wens@csie.org>, "Rob Herring" <robh+dt@kernel.org>, "Mauro Carvalho Chehab" <mchehab+samsung@kernel.org>, "Zhang Rui" <rui.zhang@intel.com>, "David S. Miller" <davem@davemloft.net>, lakml <linux-arm-kernel@lists.infradead.org> Subject: Re: [PATCH v8 0/7] add thermal sensor driver for A64, A83T, H3, H5, H6, R40 Date: Thu, 6 Feb 2020 17:46:36 +0100 [thread overview] Message-ID: <af5383b5-2dd4-92ab-ded2-f1cde48bb21a@linaro.org> (raw) In-Reply-To: <CAHLCerPWEDqEE8LRUiO5GpeP+BfnestocndBQq6oXAxVN=+3ow@mail.gmail.com> Hi Amit, On 06/02/2020 15:13, Amit Kucheria wrote: > Hi Vasily, > > For this entire series, the DTS files don't contain any trip points. > Did I miss some other series? > > At a minimum, you should add some "hot" or "critical" trip points > since then don't require a cooling-map with throttling actions. If you > have "passive" trip points, then you need to provide cooling-maps. Except I'm misunderstanding the bindings, a thermal zone must define these required properties: - polling-delay - polling-delay-passive - thermal-sensors - trips - cooling-maps > Since this series has been merged, could you please follow up with a > fixup series to add the trip points? > > Regards, > Amit > p.s. We should catch all this automatically, I'll send out yaml > bindings for the thermal framework soon that should catch this stuff. +1 There was a small discussion about converting the binding to a schema: https://www.spinics.net/lists/devicetree/msg332424.html > On Thu, Dec 19, 2019 at 10:58 PM Vasily Khoruzhick <anarsoul@gmail.com> wrote: >> >> This patchset adds driver for thermal sensor in A64, A83T, H3, H5, >> H6 and R40 SoCs. >> >> v8: >> - [vasily] Address more Maxime's comments for dt-schema >> - [vasily] Add myself to MAINTAINERS for the driver and schema >> - [vasily] Round calibration data size to word boundary for H6 and A64 >> - [vasily] Change offset for A64 since it reports too low temp otherwise. >> Likely conversion formula in user manual is not correct. >> >> v7: >> - [vasily] Address Maxime's comments for dt-schema >> - [vasily] Move common part of H3 and H5 dts into sunxi-h3-h5.dtsi >> - [vasily] Add Maxime's a-b to the driver patch >> >> v6: >> - [ondrej, vasily] Squash all driver related changes into a >> single patch >> - [ondrej] Rename calib -> calibration >> - [ondrej] Fix thermal zone registration check >> - [ondrej] Lower rate of sensor data interrupts to 4/sec/sensor >> - [ondrej] Rework scale/offset values, H6 calibration >> - [ondrej] Explicitly set mod clock to 24 MHz >> - [ondrej] Set undocumented bits in CTRL0 for H6 >> - [ondrej] Add support for A83T >> - [ondrej] Add dts changes for A83T, H3, H5, H6 >> - [vasily] Add dts changes for A64 >> - [vasily] Address Maxime's comments for YAML scheme >> - [vasily] Make .calc_temp callback mandatory >> - [vasily] Set .max_register in regmap config, so regs can be >> inspected using debugfs >> >> Ondrej Jirman (4): >> ARM: dts: sun8i-a83t: Add thermal sensor and thermal zones >> ARM: dts: sun8i-h3: Add thermal sensor and thermal zones >> arm64: dts: allwinner: h5: Add thermal sensor and thermal zones >> arm64: dts: allwinner: h6: Add thermal sensor and thermal zones >> >> Vasily Khoruzhick (1): >> arm64: dts: allwinner: a64: Add thermal sensors and thermal zones >> >> Yangtao Li (2): >> thermal: sun8i: add thermal driver for H6/H5/H3/A64/A83T/R40 >> dt-bindings: thermal: add YAML schema for sun8i-thermal driver >> bindings >> >> .../thermal/allwinner,sun8i-a83t-ths.yaml | 160 +++++ >> MAINTAINERS | 8 + >> arch/arm/boot/dts/sun8i-a83t.dtsi | 36 + >> arch/arm/boot/dts/sun8i-h3.dtsi | 20 + >> arch/arm/boot/dts/sunxi-h3-h5.dtsi | 6 + >> arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 42 ++ >> arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi | 26 + >> arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 33 + >> drivers/thermal/Kconfig | 14 + >> drivers/thermal/Makefile | 1 + >> drivers/thermal/sun8i_thermal.c | 639 ++++++++++++++++++ >> 11 files changed, 985 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/thermal/allwinner,sun8i-a83t-ths.yaml >> create mode 100644 drivers/thermal/sun8i_thermal.c >> >> -- >> 2.24.1 >> -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-02-06 16:46 UTC|newest] Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-12-19 17:28 [PATCH v8 0/7] add thermal sensor driver for A64, A83T, H3, H5, H6, R40 Vasily Khoruzhick 2019-12-19 17:28 ` Vasily Khoruzhick 2019-12-19 17:28 ` [PATCH v8 1/7] thermal: sun8i: add thermal driver for H6/H5/H3/A64/A83T/R40 Vasily Khoruzhick 2019-12-19 17:28 ` Vasily Khoruzhick 2019-12-24 13:23 ` Daniel Lezcano 2019-12-24 13:23 ` Daniel Lezcano 2019-12-19 17:28 ` [PATCH v8 2/7] dt-bindings: thermal: add YAML schema for sun8i-thermal driver bindings Vasily Khoruzhick 2019-12-19 17:28 ` Vasily Khoruzhick 2019-12-19 17:32 ` Maxime Ripard 2019-12-19 17:32 ` Maxime Ripard 2019-12-19 17:28 ` [PATCH v8 3/7] ARM: dts: sun8i-a83t: Add thermal sensor and thermal zones Vasily Khoruzhick 2019-12-19 17:28 ` Vasily Khoruzhick 2019-12-19 17:28 ` [PATCH v8 4/7] ARM: dts: sun8i-h3: " Vasily Khoruzhick 2019-12-19 17:28 ` Vasily Khoruzhick 2019-12-19 17:28 ` [PATCH v8 5/7] arm64: dts: allwinner: h5: " Vasily Khoruzhick 2019-12-19 17:28 ` Vasily Khoruzhick 2019-12-19 17:28 ` [PATCH v8 6/7] arm64: dts: allwinner: h6: " Vasily Khoruzhick 2019-12-19 17:28 ` Vasily Khoruzhick 2019-12-19 17:28 ` [PATCH v8 7/7] arm64: dts: allwinner: a64: Add thermal sensors " Vasily Khoruzhick 2019-12-19 17:28 ` Vasily Khoruzhick 2019-12-19 17:33 ` [PATCH v8 0/7] add thermal sensor driver for A64, A83T, H3, H5, H6, R40 Maxime Ripard 2019-12-19 17:33 ` Maxime Ripard 2019-12-24 18:30 ` Daniel Lezcano 2019-12-24 18:30 ` Daniel Lezcano 2019-12-26 9:27 ` Maxime Ripard 2019-12-26 9:27 ` Maxime Ripard 2019-12-26 10:37 ` Ondřej Jirman 2019-12-26 10:37 ` Ondřej Jirman 2019-12-27 15:37 ` Maxime Ripard 2019-12-27 15:37 ` Maxime Ripard 2019-12-26 10:44 ` Corentin Labbe 2019-12-26 10:44 ` Corentin Labbe 2019-12-26 12:47 ` Frank Lee 2019-12-26 12:47 ` Frank Lee 2019-12-26 20:26 ` Corentin Labbe 2019-12-26 20:26 ` Corentin Labbe 2019-12-29 21:18 ` Corentin Labbe 2019-12-29 21:18 ` Corentin Labbe 2019-12-24 13:11 ` Corentin Labbe 2019-12-24 13:11 ` Corentin Labbe 2019-12-24 14:36 ` Frank Lee 2019-12-24 14:36 ` Frank Lee 2020-02-06 14:13 ` Amit Kucheria 2020-02-06 14:13 ` Amit Kucheria 2020-02-06 15:57 ` Ondřej Jirman 2020-02-06 15:57 ` Ondřej Jirman 2020-02-06 19:26 ` Amit Kucheria 2020-02-06 19:26 ` Amit Kucheria 2020-02-06 16:46 ` Daniel Lezcano [this message] 2020-02-06 16:46 ` Daniel Lezcano 2020-02-06 19:23 ` Amit Kucheria 2020-02-06 19:23 ` Amit Kucheria 2020-02-07 9:31 ` Daniel Lezcano 2020-02-07 9:31 ` Daniel Lezcano 2020-02-07 0:49 ` Vasily Khoruzhick 2020-02-07 0:49 ` Vasily Khoruzhick
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=af5383b5-2dd4-92ab-ded2-f1cde48bb21a@linaro.org \ --to=daniel.lezcano@linaro.org \ --cc=amit.kucheria@verdurent.com \ --cc=anarsoul@gmail.com \ --cc=davem@davemloft.net \ --cc=devicetree@vger.kernel.org \ --cc=gregkh@linuxfoundation.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=mchehab+samsung@kernel.org \ --cc=megous@megous.com \ --cc=mripard@kernel.org \ --cc=robh+dt@kernel.org \ --cc=rui.zhang@intel.com \ --cc=tiny.windzz@gmail.com \ --cc=wens@csie.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.