Nicholas Piggin <npiggin@xxxxxxxxx> writes: > The powerpc toolchain can compile combinations of 32/64 bit and > big/little endian, so it's convenient to consider, e.g., > > `CC -m64 -mbig-endian` > > To be the C compiler for the purpose of invoking it to build target > artifacts. So overriding the the CC variable to include thse flags > works for this purpose. > > Unfortunately that is not compatible with the way the proposed new > Kconfig macro language will work. > > After previous patches in this series, these flags can be carefully > passed in using flags instead. > > Signed-off-by: Nicholas Piggin <npiggin@xxxxxxxxx> > --- > arch/powerpc/Makefile | 16 +++++++++------- > scripts/recordmcount.pl | 8 ++++++++ We should probably at least Cc Rostedt on the recordmcount.pl change. > diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl > index 191eb949d52c..919711dd9d6c 100755 > --- a/scripts/recordmcount.pl > +++ b/scripts/recordmcount.pl > @@ -274,6 +274,14 @@ if ($arch eq "x86_64") { Despite what the above says you're patching the powerpc entry AFAICS, so I guess that's pretty harmless. > if ($bits == 64) { > $type = ".quad"; > } > + $ld .= " -EB "; > + if ($endian eq "big") { > + $cc .= " -mbig-endian "; > + $ld .= " -EB "; Don't we end up with two "-EB"s for the big endian case? Or do I understand perl even less than I thought I did? > + } else { > + $cc .= " -mlittle-endian "; > + $ld .= " -EL "; > + } > > } elsif ($arch eq "arm") { > $alignment = 2; cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html