Re: Error: invalid switch -me200

[Christophe Leroy]

Quoting Michael Ellerman <mpe@xxxxxxxxxxxxxx>:

> Christophe Leroy <christophe.leroy@xxxxxxxxxx> writes:
> > Le 14/11/2020 à 01:20, Segher Boessenkool a écrit :
> > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote:
> > > > > > > Error: invalid switch -me200
> > > > > > > Error: unrecognized option -me200
> > > > > >
> > > > > > 251 cpu-as-$(CONFIG_E200)   += -Wa,-me200
> > > > > >
> > > > > > Are those all broken configs, or is Kconfig messed up such that
> > > > > > randconfig can select these when it should not?
> > > > >
> > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is
> > > > > a thread in 2010 about this that Segher commented on:
> > > > >
> > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@xxxxxxxxxxxxxxxxxxx/
> > > > >
> > > > > Guess this config should be eliminated?
> > >
> > > The help text for this config options says that e200 is used in 55xx,
> > > and there *is* an -me5500 GAS flag (which probably does this same
> > > thing, too).  But is any of this tested, or useful, or wanted?
> > >
> > > Maybe Christophe knows, cc:ed.
> >
> > I don't have much clue on this.
>
> Me either.
>
> > But I see on wikipedia that e5500 is a 64 bits powerpc  
> > (https://en.wikipedia.org/wiki/PowerPC_e5500)
> >
> > What I see is that NXP seems to provide a GCC version that includes  
> > aditionnal cpu (e200z0 e200z2
> > e200z3 e200z4 e200z6 e200z7):
> >
> > valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464  
> > 464fp 476 476fp 505 601 602 603
> > 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860  
> > 970 G3 G4 G5 a2 cell e200z0 e200z2
> > e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500  
> > e6500 ec603e native power3 power4
> > power5 power5+ power6 power6x power7 power8 powerpc powerpc64  
> > powerpc64le rs64 titan "
> >
> > https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049
> >
> > Apparently based on binutils 2.28
> >
> > https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf
> >
> > But that's not exactly -me200 though.
> >
> > Now, I can't see any defconfig that selects CONFIG_E200, so is that  
> > worth keeping it in the kernel
> > at all ?
>
> There was a commit in 2014 that suggests it worked at least to some
> extent then:
>
>   3477e71d5319 ("powerpc/booke: Restrict SPE exception handlers to  
> e200/e500 cores")

Not sure, that patch seems to be focussed on the new e500mc

> Presumably there was a non-upstream toolchain where it was supported?
>
> AFAICS the kernel builds OK with just the cpu-as modification removed:
>
> diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile
> index a4d56f0a41d9..16b8336f91dd 100644
> --- a/arch/powerpc/Makefile
> +++ b/arch/powerpc/Makefile
> @@ -248,7 +248,6 @@ KBUILD_CFLAGS               += $(call  
> cc-option,-mno-string)
>  cpu-as-$(CONFIG_40x)           += -Wa,-m405
>  cpu-as-$(CONFIG_44x)           += -Wa,-m440
>  cpu-as-$(CONFIG_ALTIVEC)       += $(call as-option,-Wa$(comma)-maltivec)
> -cpu-as-$(CONFIG_E200)          += -Wa,-me200
>  cpu-as-$(CONFIG_E500)          += -Wa,-me500
>
>  # When using '-many -mpower4' gas will first try and find a matching power4
>
>
> So that seems like the obvious fix for now.

Or we could do

diff --git a/arch/powerpc/platforms/Kconfig.cputype  
b/arch/powerpc/platforms/Kconfig.cputype
index c194c4ae8bc7..a11cf9431e1e 100644
--- a/arch/powerpc/platforms/Kconfig.cputype
+++ b/arch/powerpc/platforms/Kconfig.cputype
@@ -67,6 +67,7 @@ config 44x
 	select PHYS_64BIT

 config E200
+	depends on $(cc-option,-me200)
 	bool "Freescale e200"

 endchoice
---
Christophe