[v2,0/3] Add D1/T113s thermal sensor controller support

Message

Maksim Kiselev June 10, 2023, 8:35 p.m. UTC

This series adds support for Allwinner D1/T113s thermal sensor controller.
THIS controller is similar to the one on H6, but with only one sensor and
uses a different scale and offset values.

v2:
- Fixed SoB tag
- Moved binding patch before driver changes

v1:
- Initial version

Maxim Kiselev (3):
  dt-bindings: thermal: sun8i: Add binding for D1/T113s THS controller
  thermal: sun8i: Add D1/T113s THS controller support
  riscv: dts: allwinner: d1: Add thermal sensor and thermal zone

 .../thermal/allwinner,sun8i-a83t-ths.yaml     | 20 +++++++++++++-
 .../boot/dts/allwinner/sunxi-d1s-t113.dtsi    | 26 +++++++++++++++++++
 drivers/thermal/sun8i_thermal.c               | 13 ++++++++++
 3 files changed, 58 insertions(+), 1 deletion(-)

Comments

Jernej Škrabec June 12, 2023, 6:22 p.m. UTC | #1

Dne sobota, 10. junij 2023 ob 22:35:41 CEST je Maksim Kiselev napisal(a):
> This series adds support for Allwinner D1/T113s thermal sensor controller.
> THIS controller is similar to the one on H6, but with only one sensor and
> uses a different scale and offset values.
> 
> v2:
> - Fixed SoB tag

It doesn't seems you fixed that.

Best regards,
Jernej

> - Moved binding patch before driver changes
> 
> v1:
> - Initial version
> 
> Maxim Kiselev (3):
>   dt-bindings: thermal: sun8i: Add binding for D1/T113s THS controller
>   thermal: sun8i: Add D1/T113s THS controller support
>   riscv: dts: allwinner: d1: Add thermal sensor and thermal zone
> 
>  .../thermal/allwinner,sun8i-a83t-ths.yaml     | 20 +++++++++++++-
>  .../boot/dts/allwinner/sunxi-d1s-t113.dtsi    | 26 +++++++++++++++++++
>  drivers/thermal/sun8i_thermal.c               | 13 ++++++++++
>  3 files changed, 58 insertions(+), 1 deletion(-)
> 
>

Jernej Škrabec June 12, 2023, 6:28 p.m. UTC | #2

Dne ponedeljek, 12. junij 2023 ob 20:22:43 CEST je Jernej Škrabec napisal(a):
> Dne sobota, 10. junij 2023 ob 22:35:41 CEST je Maksim Kiselev napisal(a):
> > This series adds support for Allwinner D1/T113s thermal sensor controller.
> > THIS controller is similar to the one on H6, but with only one sensor and
> > uses a different scale and offset values.
> > 
> > v2:
> > - Fixed SoB tag
> 
> It doesn't seems you fixed that.

Sorry, too many revisions in short time. Wait until discussion is finished.

Best regards,
Jernej
> 
> > - Moved binding patch before driver changes
> > 
> > v1:
> > - Initial version
> > 
> > Maxim Kiselev (3):
> >   dt-bindings: thermal: sun8i: Add binding for D1/T113s THS controller
> >   thermal: sun8i: Add D1/T113s THS controller support
> >   riscv: dts: allwinner: d1: Add thermal sensor and thermal zone
> > 
> >  .../thermal/allwinner,sun8i-a83t-ths.yaml     | 20 +++++++++++++-
> >  .../boot/dts/allwinner/sunxi-d1s-t113.dtsi    | 26 +++++++++++++++++++
> >  drivers/thermal/sun8i_thermal.c               | 13 ++++++++++
> >  3 files changed, 58 insertions(+), 1 deletion(-)
> > 
> > 
> 
> 
> 
> 
>

Maksim Kiselev June 12, 2023, 6:33 p.m. UTC | #3

пн, 12 июн. 2023 г. в 21:22, Jernej Škrabec <[e-mail]>:
>
> Dne sobota, 10. junij 2023 ob 22:35:41 CEST je Maksim Kiselev napisal(a):
> > This series adds support for Allwinner D1/T113s thermal sensor controller.
> > THIS controller is similar to the one on H6, but with only one sensor and
> > uses a different scale and offset values.
> >
> > v2:
> > - Fixed SoB tag
>
> It doesn't seems you fixed that.

Sorry for that. I accidentally sent the wrong version of patches . And
immediately after that I sent v3.