From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756113Ab1KHP5U (ORCPT ); Tue, 8 Nov 2011 10:57:20 -0500 Received: from hqemgate03.nvidia.com ([216.228.121.140]:4787 "EHLO hqemgate03.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755158Ab1KHP5R convert rfc822-to-8bit (ORCPT ); Tue, 8 Nov 2011 10:57:17 -0500 X-PGP-Universal: processed; by hqnvupgp06.nvidia.com on Tue, 08 Nov 2011 07:56:59 -0800 From: Stephen Warren To: Peter De Schrijver CC: Russell King , Colin Cross , Olof Johansson , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linux-tegra@vger.kernel.org" Date: Tue, 8 Nov 2011 07:56:58 -0800 Subject: RE: [PATCH v3 0/8] Add support for tegra30 and cardhu Thread-Topic: [PATCH v3 0/8] Add support for tegra30 and cardhu Thread-Index: AcyeDNDHxUYCtn1eTfqw6gJYybWU2QAIgOrA Message-ID: <74CDBE0F657A3D45AFBB94109FB122FF173F9A5394@HQMAIL01.nvidia.com> References: <1320675617-20751-1-git-send-email-pdeschrijver@nvidia.com> <74CDBE0F657A3D45AFBB94109FB122FF173F9A50F3@HQMAIL01.nvidia.com> <20111108114559.GT29978@tbergstrom-lnx.Nvidia.com> <20111108115153.GU29978@tbergstrom-lnx.Nvidia.com> In-Reply-To: <20111108115153.GU29978@tbergstrom-lnx.Nvidia.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Peter De Schrijver wrote at Tuesday, November 08, 2011 4:52 AM: > On Tue, Nov 08, 2011 at 12:45:59PM +0100, Peter De Schrijver wrote: > > On Mon, Nov 07, 2011 at 08:10:29PM +0100, Stephen Warren wrote: > > > Peter De Schrijver wrote at Monday, November 07, 2011 7:20 AM: > > > > This patchset adds support for the tegra30 SoC and the cardhu development > > > > board. > > > > > > > > Pathset is on top of Will Deacon's ARM reset work. > > > > > > A few minor issues: > > > > > > 1) If I enable just ARCH_TEGRA_3x_SOC, but not ARCH_TEGRA_2x_SOC, the > > > kernel fails to build because arch/arm/mach-tegra/include/mach/irqs.h > > > only defines NR_IRQS if ARCH_TEGRA_2x_SOC is defined. > > > > > > 2) All the Tegra20 boards don't depend on ARCH_TEGRA_2x_SOC, so they can > > > be enabled even if only Tegra30 support is enabled. > > > > > > > Indeed. I also noticed that MACH_VENTANA lacks select MACH_SEABOARD... > > I take that back. It's not needed. > > > Maybe I should submit a seperate patch to fix all this? > > > > I will make a patch to cleanup the ARCH_TEGRA_2x_SOC dependencies. Or do you > want this as part of the tegra30/cardhu series? It should be applied before/with the Tegra30/Cardhu series so that when Tegra30 support is there, all the correct dependencies are. So, since I think you're respinning the Tegra30/Cardhu series, you may as well include it in there somewhere, but no big deal if it's posted separately. -- nvpublic