| Message ID | 20190810052829.6032-1-tiny.windzz@gmail.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | add thermal driver for h6 | expand |
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 > >
On Mon, Aug 12, 2019 at 5:14 AM Clément Péron <peron.clem@gmail.com> 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 > > > >
Hello Yangtao, On Sat, Aug 10, 2019 at 05:28:11AM +0000, 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-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
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 +0000, 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
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 +0000, 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-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
On Mon, Sep 2, 2019 at 3:58 AM Ondřej Jirman <megous@megous.com> wrote: > > 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 +0000, 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. Hey Yangtao, Are there any plans for v6? Regards, Vasily > > > 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-kernel@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel