2019-06-23 16:45:04

by Frank Lee

[permalink] [raw]
Subject: [PATCH v4 00/11] add thermal driver for h6

This patchset add support for H3 and H6 thermal sensor.

BTY, do a cleanup in thermal makfile.

Yangtao Li (11):
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 ahb 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

.../bindings/thermal/sun8i-thermal.yaml | 94 +++
MAINTAINERS | 7 +
drivers/thermal/Kconfig | 14 +
drivers/thermal/Makefile | 9 +-
drivers/thermal/sun8i_thermal.c | 534 ++++++++++++++++++
5 files changed, 654 insertions(+), 4 deletions(-)
create mode 100644 Documentation/devicetree/bindings/thermal/sun8i-thermal.yaml
create mode 100644 drivers/thermal/sun8i_thermal.c

---
v4:
-add h3 support
-fix yaml file
---
2.17.1


2019-07-10 23:11:32

by Vasily Khoruzhick

[permalink] [raw]
Subject: Re: [PATCH v4 00/11] add thermal driver for h6

On Sun, Jun 23, 2019 at 9:42 AM Yangtao Li <[email protected]> wrote:
>
> This patchset add support for H3 and H6 thermal sensor.
>
> BTY, do a cleanup in thermal makfile.
>
> Yangtao Li (11):
> 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 ahb 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

It would be nice to add dts changes to this series. It's unlikely that
you'll get any "Tested-by" otherwise.


> .../bindings/thermal/sun8i-thermal.yaml | 94 +++
> MAINTAINERS | 7 +
> drivers/thermal/Kconfig | 14 +
> drivers/thermal/Makefile | 9 +-
> drivers/thermal/sun8i_thermal.c | 534 ++++++++++++++++++
> 5 files changed, 654 insertions(+), 4 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/thermal/sun8i-thermal.yaml
> create mode 100644 drivers/thermal/sun8i_thermal.c
>
> ---
> v4:
> -add h3 support
> -fix yaml file
> ---
> 2.17.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

2019-07-13 18:03:21

by Vasily Khoruzhick

[permalink] [raw]
Subject: Re: [PATCH v4 00/11] add thermal driver for h6

On Wed, Jul 10, 2019 at 4:09 PM Vasily Khoruzhick <[email protected]> wrote:
>
> On Sun, Jun 23, 2019 at 9:42 AM Yangtao Li <[email protected]> wrote:
> >
> > This patchset add support for H3 and H6 thermal sensor.
> >
> > BTY, do a cleanup in thermal makfile.
> >
> > Yangtao Li (11):
> > 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 ahb 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
>
> It would be nice to add dts changes to this series. It's unlikely that
> you'll get any "Tested-by" otherwise.

I added A64 support on top of this series, see
https://github.com/anarsoul/linux-2.6/tree/v5.2-thermal

Branch also contains patches to enable DVFS on A64, feel free to
cherry pick only those related to thermal driver if you want to
include A64 support into v5 series.

>
> > .../bindings/thermal/sun8i-thermal.yaml | 94 +++
> > MAINTAINERS | 7 +
> > drivers/thermal/Kconfig | 14 +
> > drivers/thermal/Makefile | 9 +-
> > drivers/thermal/sun8i_thermal.c | 534 ++++++++++++++++++
> > 5 files changed, 654 insertions(+), 4 deletions(-)
> > create mode 100644 Documentation/devicetree/bindings/thermal/sun8i-thermal.yaml
> > create mode 100644 drivers/thermal/sun8i_thermal.c
> >
> > ---
> > v4:
> > -add h3 support
> > -fix yaml file
> > ---
> > 2.17.1
> >
> >
> > _______________________________________________
> > linux-arm-kernel mailing list
> > [email protected]
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

2019-07-26 21:49:38

by Vasily Khoruzhick

[permalink] [raw]
Subject: Re: [PATCH v4 00/11] add thermal driver for h6

Hey Yangtao,

Are you planning to send v5 anytime soon?

On Sat, Jul 13, 2019 at 11:01 AM Vasily Khoruzhick <[email protected]> wrote:
>
> On Wed, Jul 10, 2019 at 4:09 PM Vasily Khoruzhick <[email protected]> wrote:
> >
> > On Sun, Jun 23, 2019 at 9:42 AM Yangtao Li <[email protected]> wrote:
> > >
> > > This patchset add support for H3 and H6 thermal sensor.
> > >
> > > BTY, do a cleanup in thermal makfile.
> > >
> > > Yangtao Li (11):
> > > 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 ahb 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
> >
> > It would be nice to add dts changes to this series. It's unlikely that
> > you'll get any "Tested-by" otherwise.
>
> I added A64 support on top of this series, see
> https://github.com/anarsoul/linux-2.6/tree/v5.2-thermal
>
> Branch also contains patches to enable DVFS on A64, feel free to
> cherry pick only those related to thermal driver if you want to
> include A64 support into v5 series.
>
> >
> > > .../bindings/thermal/sun8i-thermal.yaml | 94 +++
> > > MAINTAINERS | 7 +
> > > drivers/thermal/Kconfig | 14 +
> > > drivers/thermal/Makefile | 9 +-
> > > drivers/thermal/sun8i_thermal.c | 534 ++++++++++++++++++
> > > 5 files changed, 654 insertions(+), 4 deletions(-)
> > > create mode 100644 Documentation/devicetree/bindings/thermal/sun8i-thermal.yaml
> > > create mode 100644 drivers/thermal/sun8i_thermal.c
> > >
> > > ---
> > > v4:
> > > -add h3 support
> > > -fix yaml file
> > > ---
> > > 2.17.1
> > >
> > >
> > > _______________________________________________
> > > linux-arm-kernel mailing list
> > > [email protected]
> > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

2019-07-28 12:12:27

by Frank Lee

[permalink] [raw]
Subject: Re: [PATCH v4 00/11] add thermal driver for h6

HI Vasily,

On Sat, Jul 27, 2019 at 1:46 AM Vasily Khoruzhick <[email protected]> wrote:
>
> Hey Yangtao,
>
> Are you planning to send v5 anytime soon?

Yeah, Thank you for your contribution.

This month is a bit busy, I will probably start the V5 version soon.
The next version will add your A64 support, and possibly H5 support
from icenowy.

Yours,
Yangtao

>
> On Sat, Jul 13, 2019 at 11:01 AM Vasily Khoruzhick <[email protected]> wrote:
> >
> > On Wed, Jul 10, 2019 at 4:09 PM Vasily Khoruzhick <[email protected]> wrote:
> > >
> > > On Sun, Jun 23, 2019 at 9:42 AM Yangtao Li <[email protected]> wrote:
> > > >
> > > > This patchset add support for H3 and H6 thermal sensor.
> > > >
> > > > BTY, do a cleanup in thermal makfile.
> > > >
> > > > Yangtao Li (11):
> > > > 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 ahb 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
> > >
> > > It would be nice to add dts changes to this series. It's unlikely that
> > > you'll get any "Tested-by" otherwise.
> >
> > I added A64 support on top of this series, see
> > https://github.com/anarsoul/linux-2.6/tree/v5.2-thermal
> >
> > Branch also contains patches to enable DVFS on A64, feel free to
> > cherry pick only those related to thermal driver if you want to
> > include A64 support into v5 series.
> >
> > >
> > > > .../bindings/thermal/sun8i-thermal.yaml | 94 +++
> > > > MAINTAINERS | 7 +
> > > > drivers/thermal/Kconfig | 14 +
> > > > drivers/thermal/Makefile | 9 +-
> > > > drivers/thermal/sun8i_thermal.c | 534 ++++++++++++++++++
> > > > 5 files changed, 654 insertions(+), 4 deletions(-)
> > > > create mode 100644 Documentation/devicetree/bindings/thermal/sun8i-thermal.yaml
> > > > create mode 100644 drivers/thermal/sun8i_thermal.c
> > > >
> > > > ---
> > > > v4:
> > > > -add h3 support
> > > > -fix yaml file
> > > > ---
> > > > 2.17.1
> > > >
> > > >
> > > > _______________________________________________
> > > > linux-arm-kernel mailing list
> > > > [email protected]
> > > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel