Re: [PATCH 0/2] staging -> thermal: move ti-soc-thermal to drivers/thermal

[Eduardo Valentin <eduardo.valentin@ti.com>]

[-- Attachment #1: Type: text/plain, Size: 4279 bytes --]

On 27-05-2013 22:41, Zhang Rui wrote:
> On Wed, 2013-05-15 at 11:45 -0400, Eduardo Valentin wrote:
>> Hello Greg and Rui,
>>
>> I am proposing to move the ti-soc-thermal driver from drivers/stating/
>> to drivers/thermal. It has been some time that we have been fixing and
>> improving the code under staging tree.
>>
>> I believe it is time to move on. The driver now has a better code
>> organization (split between code and data), DT support, better coding
>> style, it supports TI OMAP chips 4430, 4460, 4470 and 5430, and it is
>> ready for easy addition of other SoC chips containing the bandgap IP.
>> Besides, this driver required some major changes in the thermal framework,
>> before it could provide basic functionality using the device capability.
>> These changes have been already merged in previous merge windows,
>> so there is no pending dependencies.
>>
>> The remaining items in the TODO list are actually not a blocker for this
>> move, IMO. The PM support is not fully completed on all supported OMAP chips.
>> The locking so far has not caused any issues, and I believe that most of
>> the interesting data is protected by the thermal framework.
>>
>> I am sending this patch for your appreciation and also to get wider review.
>>
> 
> patch applied to thermal -next tree and targeted for 3.11.


Thanks Rui,

I am going to send the usual ti soc thermal fixes patch set to you now.
I ve been sending it anyway to linux-pm. Only thing is that it should go
via your tree now.

Next set should contain some fixes identified during internal testing.

> 
> thanks,
> rui
>> All best,
>>
>> Eduardo Valentin (2):
>>   thermal: introduce TI SoC thermal driver
>>   MAINTAINERS: update TI SoC thermal driver entry
>>
>>  .../devicetree/bindings/thermal}/ti_soc_thermal.txt                   | 0
>>  MAINTAINERS                                                           | 4 ++--
>>  drivers/staging/Kconfig                                               | 2 --
>>  drivers/staging/Makefile                                              | 1 -
>>  drivers/thermal/Kconfig                                               | 3 +++
>>  drivers/thermal/Makefile                                              | 2 +-
>>  drivers/{staging => thermal}/ti-soc-thermal/Kconfig                   | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/Makefile                  | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/TODO                      | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/omap4-thermal-data.c      | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/omap4xxx-bandgap.h        | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/omap5-thermal-data.c      | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/omap5xxx-bandgap.h        | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/ti-bandgap.c              | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/ti-bandgap.h              | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/ti-thermal-common.c       | 0
>>  drivers/{staging => thermal}/ti-soc-thermal/ti-thermal.h              | 0
>>  17 files changed, 6 insertions(+), 6 deletions(-)
>>  rename {drivers/staging/ti-soc-thermal => Documentation/devicetree/bindings/thermal}/ti_soc_thermal.txt (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/Kconfig (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/Makefile (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/TODO (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/omap4-thermal-data.c (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/omap4xxx-bandgap.h (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/omap5-thermal-data.c (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/omap5xxx-bandgap.h (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/ti-bandgap.c (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/ti-bandgap.h (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/ti-thermal-common.c (100%)
>>  rename drivers/{staging => thermal}/ti-soc-thermal/ti-thermal.h (100%)
>>
> 
> 
> 
> 


-- 
You have got to be excited about what you are doing. (L. Lamport)

Eduardo Valentin


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 295 bytes --]