Messages in this thread | | | Date | Fri, 19 Feb 2016 08:21:32 +0100 | From | Sascha Hauer <> | Subject | Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support |
| |
On Thu, Feb 18, 2016 at 07:15:54AM -0800, Eduardo Valentin wrote: > Folks, > > > > I think the problem is, that Eduardo wants to see the hierachical thermal > > > zones being used. But there is still a discussion ongoing [1]. > > > > It seems the original Author lost interest in the hierarchical thermal > > zones. I am not convinced that we need hierarchical thermal zones for > > the Mediatek driver since from the five sensors we only need the maximum > > temperature (If this ever changes we could still rework it). > > > > Given the current speed of communication I am not willing to add > > another, possibly controversal, dependency to an otherwise simple > > driver. I am even less willing when concerns like these come after *v12* > > of this series. > > > > Eduardo, it would really help to get a word from you. > > Apologize for the long delays here. In fact I want the hierarchical > support on this driver. But given that it is not really a strong > dependency and the hierarchical support is still an ongoing development, > I don't see why we should not merge this driver. > > I also have had the chance to try it out in a board, and seams to work > for me. I am adding to my tree.
Thanks for applying :)
Sascha
-- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
| |