From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Subject: Re: [PATCH v11 7/7] ARM: tegra: support Trusted Foundations by default Date: Thu, 28 Nov 2013 09:58:33 -0700 Message-ID: <52977639.4000308@wwwdotorg.org> References: <1385274652-16415-1-git-send-email-acourbot@nvidia.com> <1385274652-16415-8-git-send-email-acourbot@nvidia.com> <20131126000608.GA21634@quad.lixom.net> <20131126164730.GE2355@e103592.cambridge.arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Alexandre Courbot , Dave Martin Cc: Olof Johansson , Mark Rutland , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Kevin Hilman , Russell King , Pawel Moll , Tomasz Figa , Ian Campbell , Linux Kernel Mailing List , Rob Herring , Thierry Reding , Alexandre Courbot , "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" List-Id: linux-tegra@vger.kernel.org On 11/27/2013 11:02 PM, Alexandre Courbot wrote: > On Wed, Nov 27, 2013 at 1:47 AM, Dave Martin wrote: >> On Tue, Nov 26, 2013 at 10:35:58AM +0900, Alexandre Courbot wrote: >>> On Tue, Nov 26, 2013 at 9:06 AM, Olof Johansson wrote: >>>> On Sun, Nov 24, 2013 at 03:30:52PM +0900, Alexandre Courbot wrote: >>>>> Support for Trusted Foundations is light and allows the kernel to run on >>>>> a wider range of devices, so enable it by default. >>>>> >>>>> Signed-off-by: Alexandre Courbot >>>>> Reviewed-by: Tomasz Figa >>>>> Reviewed-by: Stephen Warren >>>>> --- >>>>> arch/arm/configs/tegra_defconfig | 1 + >>>>> 1 file changed, 1 insertion(+) >>>> >>>> I think we want this enabled on multi_v7_defconfig too? Send a separate >>>> patch for that once this is merged though. >>> >>> Will do. >> >> Should it just be default y if one of the relevant >> CONFIG_ARCH_TEGRA_*_SOC is selected? >> >> That way, it's automatically included if relevant, and automatically >> excluded if not -- regardless of whether the kernel is multiplatform >> or not. > > So basically, that would mean setting the default to 'y' since the > option is not available unless a supported platform is included? > > I'm fine this way too, if Stephen also agrees. Fine by me. From mboxrd@z Thu Jan 1 00:00:00 1970 From: swarren@wwwdotorg.org (Stephen Warren) Date: Thu, 28 Nov 2013 09:58:33 -0700 Subject: [PATCH v11 7/7] ARM: tegra: support Trusted Foundations by default In-Reply-To: References: <1385274652-16415-1-git-send-email-acourbot@nvidia.com> <1385274652-16415-8-git-send-email-acourbot@nvidia.com> <20131126000608.GA21634@quad.lixom.net> <20131126164730.GE2355@e103592.cambridge.arm.com> Message-ID: <52977639.4000308@wwwdotorg.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 11/27/2013 11:02 PM, Alexandre Courbot wrote: > On Wed, Nov 27, 2013 at 1:47 AM, Dave Martin wrote: >> On Tue, Nov 26, 2013 at 10:35:58AM +0900, Alexandre Courbot wrote: >>> On Tue, Nov 26, 2013 at 9:06 AM, Olof Johansson wrote: >>>> On Sun, Nov 24, 2013 at 03:30:52PM +0900, Alexandre Courbot wrote: >>>>> Support for Trusted Foundations is light and allows the kernel to run on >>>>> a wider range of devices, so enable it by default. >>>>> >>>>> Signed-off-by: Alexandre Courbot >>>>> Reviewed-by: Tomasz Figa >>>>> Reviewed-by: Stephen Warren >>>>> --- >>>>> arch/arm/configs/tegra_defconfig | 1 + >>>>> 1 file changed, 1 insertion(+) >>>> >>>> I think we want this enabled on multi_v7_defconfig too? Send a separate >>>> patch for that once this is merged though. >>> >>> Will do. >> >> Should it just be default y if one of the relevant >> CONFIG_ARCH_TEGRA_*_SOC is selected? >> >> That way, it's automatically included if relevant, and automatically >> excluded if not -- regardless of whether the kernel is multiplatform >> or not. > > So basically, that would mean setting the default to 'y' since the > option is not available unless a supported platform is included? > > I'm fine this way too, if Stephen also agrees. Fine by me. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759292Ab3K1Q6l (ORCPT ); Thu, 28 Nov 2013 11:58:41 -0500 Received: from avon.wwwdotorg.org ([70.85.31.133]:55308 "EHLO avon.wwwdotorg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759065Ab3K1Q6i (ORCPT ); Thu, 28 Nov 2013 11:58:38 -0500 Message-ID: <52977639.4000308@wwwdotorg.org> Date: Thu, 28 Nov 2013 09:58:33 -0700 From: Stephen Warren User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 MIME-Version: 1.0 To: Alexandre Courbot , Dave Martin CC: Olof Johansson , Mark Rutland , "devicetree@vger.kernel.org" , Kevin Hilman , Russell King , Pawel Moll , Tomasz Figa , Ian Campbell , Linux Kernel Mailing List , Rob Herring , Thierry Reding , Alexandre Courbot , "linux-tegra@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH v11 7/7] ARM: tegra: support Trusted Foundations by default References: <1385274652-16415-1-git-send-email-acourbot@nvidia.com> <1385274652-16415-8-git-send-email-acourbot@nvidia.com> <20131126000608.GA21634@quad.lixom.net> <20131126164730.GE2355@e103592.cambridge.arm.com> In-Reply-To: X-Enigmail-Version: 1.5.2 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/27/2013 11:02 PM, Alexandre Courbot wrote: > On Wed, Nov 27, 2013 at 1:47 AM, Dave Martin wrote: >> On Tue, Nov 26, 2013 at 10:35:58AM +0900, Alexandre Courbot wrote: >>> On Tue, Nov 26, 2013 at 9:06 AM, Olof Johansson wrote: >>>> On Sun, Nov 24, 2013 at 03:30:52PM +0900, Alexandre Courbot wrote: >>>>> Support for Trusted Foundations is light and allows the kernel to run on >>>>> a wider range of devices, so enable it by default. >>>>> >>>>> Signed-off-by: Alexandre Courbot >>>>> Reviewed-by: Tomasz Figa >>>>> Reviewed-by: Stephen Warren >>>>> --- >>>>> arch/arm/configs/tegra_defconfig | 1 + >>>>> 1 file changed, 1 insertion(+) >>>> >>>> I think we want this enabled on multi_v7_defconfig too? Send a separate >>>> patch for that once this is merged though. >>> >>> Will do. >> >> Should it just be default y if one of the relevant >> CONFIG_ARCH_TEGRA_*_SOC is selected? >> >> That way, it's automatically included if relevant, and automatically >> excluded if not -- regardless of whether the kernel is multiplatform >> or not. > > So basically, that would mean setting the default to 'y' since the > option is not available unless a supported platform is included? > > I'm fine this way too, if Stephen also agrees. Fine by me.