mbox series

[V3,0/4] thermal/drivers/hisi: Add hi3660 thermal driver support

Message ID 1508662475-6763-1-git-send-email-daniel.lezcano@linaro.org
Headers show
Series thermal/drivers/hisi: Add hi3660 thermal driver support | expand

Message

Daniel Lezcano Oct. 22, 2017, 8:54 a.m. UTC
This series adds the support for the hikey960 thermal driver. The tsensor
is almost the same than the hi6220 but with a memory shared updated by a
Cortex-M4.

In order to handle both, the hi6220 and the hi3660, the current code should
be prefixed with the platform name as we will add functions with the same
purpose but slightly different. In addition, the core driver code is
maintained untouched by using a set of ops filled at boot time.

Changelog:

 - V3 :
	- Fixed sparse warning about void const * mismatch assignment

 - V2 : 
        - Fixed extra carriage return, description length reported
          by checkpatch
        - Removed the multi-threshold interrupt
        - Removed DT bindings documentation and DT change as it is
          merge in the arm-soc tree now
        - Tested on hi6220

 - V1 : initial post

Kevin Wangtao (4):
  thermal/drivers/hisi: Put platform code together
  thermal/drivers/hisi: Add platform prefix to function name
  thermal/drivers/hisi: Prepare to add support for other hisi platforms
  thermal/drivers/hisi: Add support for hi3660 SoC

 drivers/thermal/hisi_thermal.c | 452 +++++++++++++++++++++++++++++------------
 1 file changed, 319 insertions(+), 133 deletions(-)

-- 
2.7.4

Comments

Eduardo Valentin Oct. 23, 2017, 4:21 p.m. UTC | #1
On Sun, Oct 22, 2017 at 10:54:31AM +0200, Daniel Lezcano wrote:
> This series adds the support for the hikey960 thermal driver. The tsensor

> is almost the same than the hi6220 but with a memory shared updated by a

> Cortex-M4.

> 

> In order to handle both, the hi6220 and the hi3660, the current code should

> be prefixed with the platform name as we will add functions with the same

> purpose but slightly different. In addition, the core driver code is

> maintained untouched by using a set of ops filled at boot time.

> 

> Changelog:

> 

>  - V3 :

> 	- Fixed sparse warning about void const * mismatch assignment


Thanks for fixing this, the series is now applied.

Please have a look on my -linus branch and let me know if I missed any
patches from you guys. Feel free to [RESEND] if I missed anything.