From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752202AbcBOHT7 (ORCPT ); Mon, 15 Feb 2016 02:19:59 -0500 Received: from hqemgate14.nvidia.com ([216.228.121.143]:6231 "EHLO hqemgate14.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751551AbcBOHT5 convert rfc822-to-8bit (ORCPT ); Mon, 15 Feb 2016 02:19:57 -0500 X-PGP-Universal: processed; by hqnvupgp08.nvidia.com on Sun, 14 Feb 2016 23:20:08 -0800 Message-ID: <56C17BFF.3040104@nvidia.com> Date: Mon, 15 Feb 2016 15:19:27 +0800 From: Wei Ni User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Sascha Hauer CC: Rob Herring , Thierry Reding , Zhang Rui , , Stephen Warren , Mikko Perttunen , "linux-tegra@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" Subject: Re: [PATCH V5 07/11] of: Add bindings of hw-trip-points for soctherm References: <1454057260-16412-1-git-send-email-wni@nvidia.com> <56C17AFD.8050203@nvidia.com> In-Reply-To: <56C17AFD.8050203@nvidia.com> X-Originating-IP: [10.19.224.146] X-ClientProxiedBy: HKMAIL101.nvidia.com (10.18.16.10) To HKMAIL101.nvidia.com (10.18.16.10) Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2016年02月15日 15:15, Wei Ni wrote: > Hi, Rob > > On 2016年02月12日 22:42, Rob Herring wrote: >> On Fri, Jan 29, 2016 at 2:47 AM, Wei Ni wrote: >>> Add hw-trips sub-node for soctherm, which is >>> used to describe the hardware trip points for >>> each soctherm sensors. >> >> I still don't understand why you are doing something custom here. >> What's wrong with what is defined in: >> >> Documentation/devicetree/bindings/thermal/thermal.txt > > In current thermal framework, it support to set trip points for the thermal > zones, but this trip point is handled by SW, doesn't have callbacks to program > the hardware. > There had patches: > [PATCH 12/16] thermal: thermal: Add support for hardware-tracked trip points, > http://lists.infradead.org/pipermail/linux-mediatek/2015-May/000866.html, > which can support to program hardware to set trip points, but it seems this > series didn't be taken. > So I add this hw-trip-points sub-node for soctherm. > > Hi, Sascha Hauer Hi, Sasha Sorry, I forgot to type messages in previous mail. I noticed that you submitted "[PATCH 12/16] thermal: thermal: Add support for hardware-tracked trip points" long time ago, I think it's very useful, do you know why it didn't be taken? > > Thanks. > Wei. > >> >> Rob >>