Re: [PATCH v5 00/18] add thermal driver for h6
Hello Maxime, On Mon, Sep 02, 2019 at 09:27:35AM +0200, Maxime Ripard wrote: > Hi, > > On Sun, Sep 01, 2019 at 11:52:14PM +0200, Ondřej Jirman wrote: > > Hello Yangtao, > > > > On Sat, Aug 10, 2019 at 05:28:11AM +, Yangtao Li wrote: > > > This patchset add support for A64, H3, H5, H6 and R40 thermal sensor. > > > > > > Thx to Icenowy and Vasily. > > > > > > BTY, do a cleanup in thermal makfile. > > > > I've added support for A83T and also some cleanups, according to my > > feedback: > > > > https://megous.com/git/linux/log/?h=ths-5.3 > > > > Feel free to pick up whatever you like from that tree. > > > > For others, there are also DTS patches in that tree for H3, H5, A83T, and > > H6, so > > that shoul make testing of this driver easier. > > I'm not convinced that always expanding the number of SoC supported is > the best strategy to get this merged. Usually, keeping the same > feature set across version, consolidating that, and then once it's in > sending the new SoC support works best. That's fine and all, but I've mostly added DT descriptions for already supported SoCs and fixed bugs in the driver, so that people can actually test the existing driver. I think adding DT changes will actually help get needed exposure for this patch series. A83T support that I added, was actually just a small change to the driver. regards, o. > Maxime > > -- > Maxime Ripard, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com > > ___ > linux-arm-kernel mailing list > linux-arm-ker...@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Re: [PATCH v5 00/18] add thermal driver for h6
Hi, On Sun, Sep 01, 2019 at 11:52:14PM +0200, Ondřej Jirman wrote: > Hello Yangtao, > > On Sat, Aug 10, 2019 at 05:28:11AM +, Yangtao Li wrote: > > This patchset add support for A64, H3, H5, H6 and R40 thermal sensor. > > > > Thx to Icenowy and Vasily. > > > > BTY, do a cleanup in thermal makfile. > > I've added support for A83T and also some cleanups, according to my > feedback: > > https://megous.com/git/linux/log/?h=ths-5.3 > > Feel free to pick up whatever you like from that tree. > > For others, there are also DTS patches in that tree for H3, H5, A83T, and H6, > so > that shoul make testing of this driver easier. I'm not convinced that always expanding the number of SoC supported is the best strategy to get this merged. Usually, keeping the same feature set across version, consolidating that, and then once it's in sending the new SoC support works best. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
Re: [PATCH v5 00/18] add thermal driver for h6
Hello Yangtao, On Sat, Aug 10, 2019 at 05:28:11AM +, Yangtao Li wrote: > This patchset add support for A64, H3, H5, H6 and R40 thermal sensor. > > Thx to Icenowy and Vasily. > > BTY, do a cleanup in thermal makfile. I've added support for A83T and also some cleanups, according to my feedback: https://megous.com/git/linux/log/?h=ths-5.3 Feel free to pick up whatever you like from that tree. For others, there are also DTS patches in that tree for H3, H5, A83T, and H6, so that shoul make testing of this driver easier. regards, Ondrej > Icenowy Zheng (3): > thermal: sun8i: allow to use custom temperature calculation function > thermal: sun8i: add support for Allwinner H5 thermal sensor > thermal: sun8i: add support for Allwinner R40 thermal sensor > > Vasily Khoruzhick (1): > thermal: sun8i: add thermal driver for A64 > > Yangtao Li (14): > thermal: sun8i: add thermal driver for h6 > dt-bindings: thermal: add binding document for h6 thermal controller > thermal: fix indentation in makefile > thermal: sun8i: get ths sensor number from device compatible > thermal: sun8i: rework for sun8i_ths_get_temp() > thermal: sun8i: get ths init func from device compatible > thermal: sun8i: rework for ths irq handler func > thermal: sun8i: support mod clocks > thermal: sun8i: rework for ths calibrate func > dt-bindings: thermal: add binding document for h3 thermal controller > thermal: sun8i: add thermal driver for h3 > dt-bindings: thermal: add binding document for a64 thermal controller > dt-bindings: thermal: add binding document for h5 thermal controller > dt-bindings: thermal: add binding document for r40 thermal controller > > .../bindings/thermal/sun8i-thermal.yaml | 157 + > MAINTAINERS | 7 + > drivers/thermal/Kconfig | 14 + > drivers/thermal/Makefile | 9 +- > drivers/thermal/sun8i_thermal.c | 596 ++ > 5 files changed, 779 insertions(+), 4 deletions(-) > create mode 100644 > Documentation/devicetree/bindings/thermal/sun8i-thermal.yaml > create mode 100644 drivers/thermal/sun8i_thermal.c > > --- > v5: > -add more support > -some trival fix > --- > 2.17.1 > > > ___ > linux-arm-kernel mailing list > linux-arm-ker...@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Re: [PATCH v5 00/18] add thermal driver for h6
On Mon, Aug 12, 2019 at 5:14 AM Clément Péron wrote: > > Hi Yangtao, > > On 10/08/2019 07:28, Yangtao Li wrote: > > This patchset add support for A64, H3, H5, H6 and R40 thermal sensor. > > Could you add the device-tree configuration in the same series? > This will allow user to test it. Ok, it will be added later. Yangtao > > Thanks, > Clément > > > > > Thx to Icenowy and Vasily. > > > > BTY, do a cleanup in thermal makfile. > > > > Icenowy Zheng (3): > >thermal: sun8i: allow to use custom temperature calculation function > >thermal: sun8i: add support for Allwinner H5 thermal sensor > >thermal: sun8i: add support for Allwinner R40 thermal sensor > > > > Vasily Khoruzhick (1): > >thermal: sun8i: add thermal driver for A64 > > > > Yangtao Li (14): > >thermal: sun8i: add thermal driver for h6 > >dt-bindings: thermal: add binding document for h6 thermal controller > >thermal: fix indentation in makefile > >thermal: sun8i: get ths sensor number from device compatible > >thermal: sun8i: rework for sun8i_ths_get_temp() > >thermal: sun8i: get ths init func from device compatible > >thermal: sun8i: rework for ths irq handler func > >thermal: sun8i: support mod clocks > >thermal: sun8i: rework for ths calibrate func > >dt-bindings: thermal: add binding document for h3 thermal controller > >thermal: sun8i: add thermal driver for h3 > >dt-bindings: thermal: add binding document for a64 thermal controller > >dt-bindings: thermal: add binding document for h5 thermal controller > >dt-bindings: thermal: add binding document for r40 thermal controller > > > > .../bindings/thermal/sun8i-thermal.yaml | 157 + > > MAINTAINERS | 7 + > > drivers/thermal/Kconfig | 14 + > > drivers/thermal/Makefile | 9 +- > > drivers/thermal/sun8i_thermal.c | 596 ++ > > 5 files changed, 779 insertions(+), 4 deletions(-) > > create mode 100644 > > Documentation/devicetree/bindings/thermal/sun8i-thermal.yaml > > create mode 100644 drivers/thermal/sun8i_thermal.c > > --- > > v5: > > -add more support > > -some trival fix > > --- > > 2.17.1 > > > >
Re: [PATCH v5 00/18] add thermal driver for h6
Hi Yangtao, On 10/08/2019 07:28, Yangtao Li wrote: This patchset add support for A64, H3, H5, H6 and R40 thermal sensor. Could you add the device-tree configuration in the same series? This will allow user to test it. Thanks, Clément Thx to Icenowy and Vasily. BTY, do a cleanup in thermal makfile. Icenowy Zheng (3): thermal: sun8i: allow to use custom temperature calculation function thermal: sun8i: add support for Allwinner H5 thermal sensor thermal: sun8i: add support for Allwinner R40 thermal sensor Vasily Khoruzhick (1): thermal: sun8i: add thermal driver for A64 Yangtao Li (14): thermal: sun8i: add thermal driver for h6 dt-bindings: thermal: add binding document for h6 thermal controller thermal: fix indentation in makefile thermal: sun8i: get ths sensor number from device compatible thermal: sun8i: rework for sun8i_ths_get_temp() thermal: sun8i: get ths init func from device compatible thermal: sun8i: rework for ths irq handler func thermal: sun8i: support mod clocks thermal: sun8i: rework for ths calibrate func dt-bindings: thermal: add binding document for h3 thermal controller thermal: sun8i: add thermal driver for h3 dt-bindings: thermal: add binding document for a64 thermal controller dt-bindings: thermal: add binding document for h5 thermal controller dt-bindings: thermal: add binding document for r40 thermal controller .../bindings/thermal/sun8i-thermal.yaml | 157 + MAINTAINERS | 7 + drivers/thermal/Kconfig | 14 + drivers/thermal/Makefile | 9 +- drivers/thermal/sun8i_thermal.c | 596 ++ 5 files changed, 779 insertions(+), 4 deletions(-) create mode 100644 Documentation/devicetree/bindings/thermal/sun8i-thermal.yaml create mode 100644 drivers/thermal/sun8i_thermal.c --- v5: -add more support -some trival fix --- 2.17.1
[PATCH v5 00/18] add thermal driver for h6
This patchset add support for A64, H3, H5, H6 and R40 thermal sensor. Thx to Icenowy and Vasily. BTY, do a cleanup in thermal makfile. Icenowy Zheng (3): thermal: sun8i: allow to use custom temperature calculation function thermal: sun8i: add support for Allwinner H5 thermal sensor thermal: sun8i: add support for Allwinner R40 thermal sensor Vasily Khoruzhick (1): thermal: sun8i: add thermal driver for A64 Yangtao Li (14): thermal: sun8i: add thermal driver for h6 dt-bindings: thermal: add binding document for h6 thermal controller thermal: fix indentation in makefile thermal: sun8i: get ths sensor number from device compatible thermal: sun8i: rework for sun8i_ths_get_temp() thermal: sun8i: get ths init func from device compatible thermal: sun8i: rework for ths irq handler func thermal: sun8i: support mod clocks thermal: sun8i: rework for ths calibrate func dt-bindings: thermal: add binding document for h3 thermal controller thermal: sun8i: add thermal driver for h3 dt-bindings: thermal: add binding document for a64 thermal controller dt-bindings: thermal: add binding document for h5 thermal controller dt-bindings: thermal: add binding document for r40 thermal controller .../bindings/thermal/sun8i-thermal.yaml | 157 + MAINTAINERS | 7 + drivers/thermal/Kconfig | 14 + drivers/thermal/Makefile | 9 +- drivers/thermal/sun8i_thermal.c | 596 ++ 5 files changed, 779 insertions(+), 4 deletions(-) create mode 100644 Documentation/devicetree/bindings/thermal/sun8i-thermal.yaml create mode 100644 drivers/thermal/sun8i_thermal.c --- v5: -add more support -some trival fix --- 2.17.1