Re: [patch 1/3] powerpc: delete -mstring option

[Akinobu Mita <mita@fixstars.com>]

On Tue, Apr 10, 2007 at 01:42:15PM -0500, Kumar Gala wrote:
> >This is the right solution, thank you for the patch.
> >But please fold these two together so that you're not
> >introducing a regression halfway the patch series.
> 
> I agree this looks good and merge patches 1/3 and 2/3 into a single  
> patch.

Thansk, this is updated patch.

From: Akinobu Mita <mita@fixstars.com>
Subject: powerpc: delete -mstring option

There is no reason to force -mstring option.

This patch removes -mstring option, and adds -mno-string option
for FLS_BOOKE (Because GCC is now putting string instructions
even if you don't specify -mstring).

Signed-off-by: Akinobu Mita <mita@fixstars.com>

---
 arch/powerpc/Makefile |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Index: 2.6-git-ps3/arch/powerpc/Makefile
===================================================================
--- 2.6-git-ps3.orig/arch/powerpc/Makefile
+++ 2.6-git-ps3/arch/powerpc/Makefile
@@ -102,8 +102,8 @@ CFLAGS += $(call cc-option,-mno-altivec)
 # kernel considerably.
 CFLAGS += $(call cc-option,-funit-at-a-time)
 
-ifndef CONFIG_FSL_BOOKE
-CFLAGS		+= -mstring
+ifeq ($(CONFIG_FSL_BOOKE),y)
+CFLAGS         += -mno-string
 endif
 
 ifeq ($(CONFIG_6xx),y)