From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753899Ab1GZVjP (ORCPT ); Tue, 26 Jul 2011 17:39:15 -0400 Received: from wolverine01.qualcomm.com ([199.106.114.254]:16386 "EHLO wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753608Ab1GZVjM (ORCPT ); Tue, 26 Jul 2011 17:39:12 -0400 X-IronPort-AV: E=McAfee;i="5400,1158,6419"; a="106089736" Date: Tue, 26 Jul 2011 14:39:10 -0700 From: David Brown To: Nicolas Pitre Cc: Stephen Boyd , Russell King - ARM Linux , lkml , linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, David Brown Subject: Re: [PATCHv2 2/2] ARM: ARM_PATCH_PHYS_VIRT_16BIT no longer depends on MSM Message-ID: <20110726213910.GA11358@huya.qualcomm.com> References: <1311614362-9011-1-git-send-email-sboyd@codeaurora.org> <1311614362-9011-2-git-send-email-sboyd@codeaurora.org> <20110725203114.GF14955@n2100.arm.linux.org.uk> <4E2EFC0E.8020805@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 26, 2011 at 02:24:26PM -0400, Nicolas Pitre wrote: > On Tue, 26 Jul 2011, Stephen Boyd wrote: > > > On 07/25/2011 01:31 PM, Russell King - ARM Linux wrote: > > > On Mon, Jul 25, 2011 at 04:25:26PM -0400, Nicolas Pitre wrote: > > >> On Mon, 25 Jul 2011, Stephen Boyd wrote: > > >> > > >>> MSM no longer requires the 16bit version of dynamic P2V. Drop the > > >>> dependency. > > >>> > > >>> Signed-off-by: Stephen Boyd > > >>> Cc: Nicolas Pitre > > >>> Cc: David Brown > > >> Acked-by: Nicolas Pitre > > > As it's now unused, we can kill the additional code which makes things > > > needlessly more complex... > > > > Ok. I'll remove the extra code in v3. > > Something like this (untested): > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > diff --git a/arch/arm/kernel/head.S b/arch/arm/kernel/head.S I've tested this on MSM8660. Works with CONFIG_ARM_PATCH_PHYS_VIRT enabled (but an offset of zero). I'd like to make this up into a patch, Nicolas, do you want to be the author (and do you sign-off)? Thanks, David -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.