Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support

From	Sascha Hauer <s.hauer@pengutronix.de>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support
Date	2016-02-19 08:30 +0100
Message-ID	<r3NFM-1zJ-3@gated-at.bofh.it> (permalink)
References	(5 earlier) <r2gVz-2Vi-3@gated-at.bofh.it> <r2gVz-2Vi-1@gated-at.bofh.it> <r3dLX-10I-1@gated-at.bofh.it> <r3utt-4hK-29@gated-at.bofh.it> <r3yx5-7uQ-25@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

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 |

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support Daniel Kurtz <djkurtz@chromium.org> - 2016-02-15 03:20 +0100
  Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support Matthias Brugger <matthias.bgg@gmail.com> - 2016-02-17 18:10 +0100
    Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support Sascha Hauer <s.hauer@pengutronix.de> - 2016-02-18 12:00 +0100
      Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support Javi Merino <javi.merino@arm.com> - 2016-02-18 15:30 +0100
      Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support Eduardo Valentin <edubezval@gmail.com> - 2016-02-18 16:20 +0100
        Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support Sascha Hauer <s.hauer@pengutronix.de> - 2016-02-19 08:30 +0100

csiph-web