Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752419AbaL2UiU (ORCPT ); Mon, 29 Dec 2014 15:38:20 -0500 Received: from devils.ext.ti.com ([198.47.26.153]:51137 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752094AbaL2UiS (ORCPT ); Mon, 29 Dec 2014 15:38:18 -0500 Message-ID: <54A1BB2B.9060204@ti.com> Date: Mon, 29 Dec 2014 14:35:55 -0600 From: Nishanth Menon User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: Pavel Machek CC: Grazvydas Ignotas , Sebastian Reichel , Mark Rutland , dt list , Pawel Moll , Ian Campbell , Tony Lindgren , Kumar Gala , "linux-kernel@vger.kernel.org" , , Rob Herring , Jean Delvare , =?windows-1252?Q?Beno=EEt_Cousson?= , Pali Rohar , "linux-omap@vger.kernel.org" , Guenter Roeck , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH 2/3] hwmon: Driver for OMAP3 temperature sensor References: <20141226102933.GA28778@amd> <1419597294-21487-1-git-send-email-sre@kernel.org> <1419597294-21487-3-git-send-email-sre@kernel.org> <20141229181556.GA26926@amd> In-Reply-To: <20141229181556.GA26926@amd> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/29/2014 12:15 PM, Pavel Machek wrote: > On Mon 2014-12-29 12:01:03, Nishanth Menon wrote: >> On Mon, Dec 29, 2014 at 11:52 AM, Grazvydas Ignotas wrote: >>> On Fri, Dec 26, 2014 at 2:34 PM, Sebastian Reichel wrote: >>>> OMAP34xx and OMAP36xx processors contain a register in the syscon area, >>>> which can be used to determine the SoCs temperature. This patch provides >>>> a DT based driver for the temperature sensor based on an older driver >>>> written by Peter De Schrijver for the Nokia N900 and N9. >>> >>> The sensor looks like an earlier iteration of sensors used in newer >>> OMAPs, which are already supported by maybe >>> drivers/thermal/ti-soc-thermal/ , maybe it would make sense to update >>> that driver instead? >> >> Just to be clear - OMAP4 is the first time that the sensors were >> reliable enough to be used. > > When testing initial version of the patch, they seem to work very well > in the omap3 case. Just be careful when you try to make thermal policy like decisions based on this sensor. Placement of the sensor w.r.t the actual logic generating heat has to be a factor as well. If you are just looking for an approximation temperature (thermometerish kind), you might be ok with this. I am not sure we'd find any TI data around this.. just a heads up. Also notice http://www.ti.com/lit/er/sprz278f/sprz278f.pdf "Advisory 3.1.1.186 MMC OCP Clock Not Gated When Thermal Sensor Is Used" I think there were accuracy issues at certain values etc.. So remember to do a off mode type PM tests as well before you consider requesting these to be merged. -- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/