Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752829AbbBXT4f (ORCPT ); Tue, 24 Feb 2015 14:56:35 -0500 Received: from mail-oi0-f50.google.com ([209.85.218.50]:65054 "EHLO mail-oi0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752592AbbBXT4d (ORCPT ); Tue, 24 Feb 2015 14:56:33 -0500 MIME-Version: 1.0 In-Reply-To: <20150224183616.GD3448@developer.amazonguestwifi.org> References: <1423608615-6575-1-git-send-email-tylerwhall@gmail.com> <20150224183616.GD3448@developer.amazonguestwifi.org> From: Tyler Hall Date: Tue, 24 Feb 2015 14:56:12 -0500 Message-ID: Subject: Re: [PATCH] thermal: armada: read stable temp on Armada XP To: Eduardo Valentin Cc: Ezequiel Garcia , linux-pm@vger.kernel.org, Linux Kernel Mailing List , "devicetree@vger.kernel.org" , Zhang Rui Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1251 Lines: 29 Eduardo, On Tue, Feb 24, 2015 at 1:36 PM, Eduardo Valentin wrote: > The fix seams reasonable. Although, it remains the question what is > applicability to other Armada chips? Besides, shouldn't we simply use it > by default? Also, do you plan to send updates in the DTS files? As far as I can tell, Armada 370 is already using the equivalent of this register I'd like to use in Armada XP. I'm not sure about the other mvebu platforms. I couldn't just change the device tree for XP to instantiate the 370 sensor, however, as they have different initialization routines. Possibly Eziquiel can comment on the significance of the differences between armadaxp_init_sensor() and armada370_init_sensor(). I would like to change the default going forward, but I don't think it can be changed on platforms using an older DTB. I had planned to submit the dts change separately. It's not clear to me how that's supposed to be handled if they might go through different trees. Thanks, Tyler -- 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/