From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1035183AbdD0XKU (ORCPT ); Thu, 27 Apr 2017 19:10:20 -0400 Received: from mail-qk0-f172.google.com ([209.85.220.172]:35966 "EHLO mail-qk0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1034362AbdD0XKH (ORCPT ); Thu, 27 Apr 2017 19:10:07 -0400 Subject: Re: [PATCH v2 1/3] ARM: BCM: Enable thermal support for iProc SoCs To: Jon Mason , Florian Fainelli , Zhang Rui , Eduardo Valentin , Rob Herring , Mark Rutland References: <1493328194-1766-1-git-send-email-jon.mason@broadcom.com> <1493328194-1766-2-git-send-email-jon.mason@broadcom.com> Cc: bcm-kernel-feedback-list@broadcom.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, devicetree@vger.kernel.org From: Scott Branden Message-ID: Date: Thu, 27 Apr 2017 16:10:02 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <1493328194-1766-2-git-send-email-jon.mason@broadcom.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 17-04-27 02:23 PM, Jon Mason wrote: > Change the iProc Kconfig to select THERMAL and THERMAL_OF, which allows > the ns-thermal driver to be selected via menuconfig. > > Signed-off-by: Jon Mason > --- > arch/arm/mach-bcm/Kconfig | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig > index a0e66d8..da2bfeb 100644 > --- a/arch/arm/mach-bcm/Kconfig > +++ b/arch/arm/mach-bcm/Kconfig > @@ -19,6 +19,8 @@ config ARCH_BCM_IPROC > select GPIOLIB > select ARM_AMBA > select PINCTRL > + select THERMAL > + select THERMAL_OF This is NSP specific at this point. Also, If it increases code size in any way it shouldn't be selected for all IPROC SoCS. I'd rather this was just selected via defconfig > help > This enables support for systems based on Broadcom IPROC architected SoCs. > The IPROC complex contains one or more ARM CPUs along with common >