From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1426389AbcBRK4Q (ORCPT ); Thu, 18 Feb 2016 05:56:16 -0500 Received: from metis.ext.4.pengutronix.de ([92.198.50.35]:57840 "EHLO metis.ext.4.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1426346AbcBRK4N (ORCPT ); Thu, 18 Feb 2016 05:56:13 -0500 Date: Thu, 18 Feb 2016 11:56:03 +0100 From: Sascha Hauer To: Matthias Brugger Cc: Daniel Kurtz , Eddie Huang , Eduardo Valentin , "linux-pm@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-mediatek@lists.infradead.org" , "kernel@pengutronix.de" , Zhang Rui , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH 2/3] thermal: Add Mediatek thermal controller support Message-ID: <20160218105603.GE3939@pengutronix.de> References: <1448883753-19068-1-git-send-email-s.hauer@pengutronix.de> <1448883753-19068-3-git-send-email-s.hauer@pengutronix.de> <20151217193332.GB7999@localhost.localdomain> <20160104141940.GI13058@pengutronix.de> <20160119072917.GA13237@pengutronix.de> <1454295246.22151.6.camel@mtksdaap41> <56C4A875.6080908@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <56C4A875.6080908@gmail.com> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-IRC: #ptxdist @freenode X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-Uptime: 08:14:16 up 18:58, 22 users, load average: 0.00, 0.03, 0.05 User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: 2001:67c:670:100:1d::c0 X-SA-Exim-Mail-From: sha@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 17, 2016 at 06:05:57PM +0100, Matthias Brugger wrote: > > > On 15/02/16 03:14, Daniel Kurtz wrote: > >On Mon, Feb 15, 2016 at 10:11 AM, Daniel Kurtz wrote: > >>Hi Eduardo, Sascha, > >> > >>>>Any input on this? I really like to get this driver upstream as it is > >>>>currently blocking other Mediatek drivers. > >>>> > >>> > >>>Hi Eduardo, > >>> > >>>Do you have any comment about Sascha's response ? We really hope get > >>>your comment since Mediatek thermal driver already reviewed in public > >>>over half years, and we have other patches [0] [1] depend on thermal > >>>driver. > >>> > >>>[0]: > >>>http://lists.infradead.org/pipermail/linux-arm-kernel/2015-December/394084.html > >>>[1]: > >>>http://lists.infradead.org/pipermail/linux-arm-kernel/2016-January/401055.html > >> > >>Friendly ping on the Mediatek thermal driver. > >>The "EFUSE" dependency has now landed in v4.5-rc4. > > > >Actually, it landed in char-misc-next, not v4.5-rc4. > > > >>So, AFAICT, the only thing left that may be blocking landing Mediatek > >>thermal driver is resolution of this discussion about thermal zones. > >>Can we kindly resolve this soon so we have a chance to land it in v4.6. > >> > > > 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. 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 |