On 14:04 Wed 15 Feb , Sascha Hauer wrote: > Hi Jean-Christophe, > > On Fri, Feb 10, 2017 at 10:56:55AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote: > > Today the EFI is build as an other ARCH when in fact it's just a boot mode > > > > so move it back to arch/x86 for the spicific x86 part and the common > > to common and driver > > > > The following changes since commit 55d29525146dcd280987dfd565bfd34f08858fb3: > > > > ata: ide-sff: don't call free for ide_port in error path (2017-01-18 12:01:38 +0100) > > > > are available in the git repository at: > > > > git://git.jcrosoft.org/barebox.git > > > > for you to fetch changes up to 3e07e7f43bcfb5160cc6104f06535f147222570b: > > > > efi: bus: add firmware vendor and resision and tables info (2017-02-09 17:13:41 +0800) > > > > ---------------------------------------------------------------- > > Jean-Christophe PLAGNIOL-VILLARD (9): > > efi: move block io driver to driver/block > > efi: move clocksource out of arch > > efi: move bus driver to driver/efi > > efi: move debug_ll.h to include/efi > > efi: move startup and payload to common/efi > > x86: move bios bootup code to arch/x86/bios > > efi: move x86 efi boot support to x86 arch > > ARCH: efi: Finally drop it as now we can build efi bootup from x86 > > efi: bus: add firmware vendor and resision and tables info > > Some problems found with this series: > > - bisectability breaks in 3/9: > > drivers/built-in.o: In function `efi_driver_register': > /home/sha/dude/barebox/barebox-test/include/efi/efi-device.h:38: undefined reference to `efi_bus' > fs/built-in.o: In function `efi_driver_register': > /home/sha/dude/barebox/barebox-test/include/efi/efi-device.h:38: undefined reference to `efi_bus' > arch/efi/efi/built-in.o: In function `efi_execute_image': > /home/sha/dude/barebox/barebox-test/arch/efi/efi/efi-image.c:162: undefined reference to `efi_connect_all' > /home/sha/dude/barebox/barebox-test/arch/efi/efi/efi-image.c:163: undefined reference to `efi_register_devices' > Makefile:756: recipe for target 'barebox' failed > make: *** [barebox] Error 1 > > - The end result won't compile: > > arch/x86/Makefile:88: *** recipe commences before first target. Stop. > > With this one fixed we get: > > In file included from common/efi/env-efi.bbenv.S:1:0: > include/asm-generic/barebox.lds.h:12:30: fatal error: mach/barebox.lds.h: No such file or directory > #include <mach/barebox.lds.h> > > I think you can squash most of the patches together, maybe leaving 6/9 > and 9/9 as separate patches. The rest is probably hard to keep separated > while maintaining bisectability I prefer to keep it seperated As this allow to compare arch/efi and the new arch/x86 with efi boot before droping it. I fix the bisecting issue in the v2 and add few patch that help to debug the efi. Best Regards, J. _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
