Re: [PATCH v4 2/4] ARM: use .arch directives instead of assembler command line flags

Russell King (Oracle)
2022-11-08T18:38:20Z

On Tue, Nov 08, 2022 at 11:30:39AM -0700, Nathan Chancellor wrote:
> I noticed that this series appears to be partially applied in
> next-20221108:
>
> 59247fe730db ARM: 9265/1: pass -march= only to compiler
> bc21212c5564 ARM: 9264/1: only use -mtp=cp15 for the compiler
> 5aa4860eb50f ARM: 9262/1: remove lazy evaluation in Makefile
>
> This patch (submitted as 9263/1) is needed by 9265/1; without it, builds
> are going to break:
>
> https://builds.tuxbuild.com/2HGSpQMAKxJuaMAz2ZZc1UgUvvu/build.log
>
> Was there a problem with applying this patch or just an oversight? It
> would be great to get this whole series applied so that we can turn
> -Wunused-command-line-argument into a hard error.

Patching 9263/1...
git apply --whitespace=fix -p1 --index --check > /tmp/pdb.726086 2>&1
exited with non-zero status: 256
error: patch failed: arch/arm/mach-mvebu/Makefile:1
error: arch/arm/mach-mvebu/Makefile: patch does not apply

The problem appears to be:

84fc86360623 ARM: make ARCH_MULTIPLATFORM user-visible

which is not in my misc branch. I've applied your patch using GNU
patch with a fuzz of 2, and the resulting diff looks sensible. I'll
push it out later this evening.

Thanks for the heads-up.

--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!