Messages in this thread | | | Date | Tue, 8 Nov 2022 11:40:28 -0700 | From | Nathan Chancellor <> | Subject | Re: [PATCH v4 2/4] ARM: use .arch directives instead of assembler command line flags |
| |
On Tue, Nov 08, 2022 at 06:37:02PM +0000, Russell King (Oracle) wrote: > 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.
Perfect, thanks a lot for the quick response! I figured it was something like that. I will keep an eye out for any additional breakage and follow up with patches or additional emails as necessary.
Cheers, Nathan
| |