On 06.09.2021 18:13, Anthony PERARD wrote: > I hope this is useful: > > On Tue, Aug 24, 2021 at 11:49:47AM +0100, Anthony PERARD wrote: >> Anthony PERARD (51): >> build: introduce cpp_flags macro >> build: use if_changed on built_in.o >> build: use if_changed_rules with %.o:%.c targets > > all 3 ready to commit > >> build: factorise generation of the linker scripts > > may need a ack from arm maintainers, does it? Otherwise, ready.
Ah yes. Will need to wait. Which is the first place pointing out that you answer my question only partly: Does "ready to commit" include "independent of earlier patches in this series"? Same e.g. ... >> x86/mm: avoid building multiple .o from a single .c file >> build,include: rework compat-build-source.py >> build,include: rework compat-build-header.py >> build: fix clean targets when subdir-y is used >> build: use subdir-y in test/Makefile > > all 5 ready to commit ... here: In my reply to this cover letter I did specifically that I've skipped patch 5. As said in a reply just sent there, when writing that I didn't realize I had already given R-b for it. I intend to put in some of this series in a couple of minutes; I'll try to judge what is okay to commit independent of earlier skipped patches. Thanks for the summary nevertheless, Jan >> build,arm: move LDFLAGS change to arch.mk > need edit commit description, but otherwise ready > not needed before [PATCH 21/51] build: set ALL_OBJS to main Makefile; > move prelink.o to main Makefile > >> build: move make option changes check earlier >> build: avoid building arm/arm/*/head.o twice > both ready > > So up to here (12/51) all patch seems to be ready to commit. > >> build: convert binfile use to if_changed >> xen: move include/asm-* to arch/*/include/asm > both need a ack > >> build,riscv: tell the build system about riscv64/head.S > patch ready > but depends on "[PATCH 14/51] xen: move include/asm-* to arch/*/include/asm" > for the rune written in the commit message to work. > > >> build: generate "include/xen/compile.h" with if_changed >> build: set XEN_BUILD_EFI earlier >> build: fix $(TARGET).efi creation in arch/arm > patches needs review > >> build: fix arch/x86/note.o rule > ready, standalone > > > The rest of the series needs review. > > Cheers, >
