From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Brown Subject: Re: [PATCHv2 2/2] ARM: ARM_PATCH_PHYS_VIRT_16BIT no longer depends on MSM Date: Tue, 26 Jul 2011 14:39:10 -0700 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 Return-path: 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 Content-Disposition: inline In-Reply-To: Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org 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 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. From mboxrd@z Thu Jan 1 00:00:00 1970 From: davidb@codeaurora.org (David Brown) Date: Tue, 26 Jul 2011 14:39:10 -0700 Subject: [PATCHv2 2/2] ARM: ARM_PATCH_PHYS_VIRT_16BIT no longer depends on MSM In-Reply-To: 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> Message-ID: <20110726213910.GA11358@huya.qualcomm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.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.