# git rev-parse -q --verify b0b9f351fd616a01eea7c8d44e798db5a122a005^{commit} b0b9f351fd616a01eea7c8d44e798db5a122a005 already have revision, skipping fetch # git checkout -q -f -B kisskb b0b9f351fd616a01eea7c8d44e798db5a122a005 # git clean -qxdf # < git log -1 # commit b0b9f351fd616a01eea7c8d44e798db5a122a005 # Author: Masahiro Yamada # Date: Fri Jul 5 19:01:44 2019 +0900 # # powerpc/boot: pass CONFIG options in a simpler and more robust way # # Commit 5e9dcb6188a4 ("powerpc/boot: Expose Kconfig symbols to wrapper") # was wrong, but commit e41b93a6be57 ("powerpc/boot: Fix build failures # with -j 1") was also wrong. # # The correct dependency is: # # $(obj)/serial.o: $(obj)/autoconf.h # # However, I do not see the reason why we need to copy autoconf.h to # arch/power/boot/. Nor do I see consistency in the way of passing # CONFIG options. # # decompress.c references CONFIG_KERNEL_GZIP and CONFIG_KERNEL_XZ, which # are passed via the command line. # # serial.c includes autoconf.h to reference a couple of CONFIG options, # but this is fragile because we often forget to include "autoconf.h" # from source files. # # In fact, it is already broken. # # ppc_asm.h references CONFIG_PPC_8xx, but utils.S is not given any way # to access CONFIG options. So, CONFIG_PPC_8xx is never defined here. # # Pass $(LINUXINCLUDE) to make sure CONFIG options are accessible from # all .c and .S files in arch/powerpc/boot/. # # I also removed the -traditional flag to make include/linux/kconfig.h # work. This flag makes the preprocessor imitate the behavior of the # pre-standard C compiler, but I do not understand why it is necessary. # # Signed-off-by: Masahiro Yamada # Signed-off-by: Michael Ellerman # Link: https://lore.kernel.org/r/20190705100144.28785-2-yamada.masahiro@socionext.com # < /opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux-ld --version # < git log --format=%s --max-count=1 b0b9f351fd616a01eea7c8d44e798db5a122a005 # < make -s -j 8 ARCH=powerpc O=/kisskb/build/powerpc-next_ps3_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- ps3_defconfig # make -s -j 8 ARCH=powerpc O=/kisskb/build/powerpc-next_ps3_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- In file included from /kisskb/src/include/linux/genhd.h:67:0, from /kisskb/src/include/linux/blkdev.h:11, from /kisskb/src/drivers/block/ps3vram.c:12: /kisskb/src/drivers/block/ps3vram.c: In function 'ps3vram_probe': /kisskb/src/drivers/block/ps3vram.c:770:23: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'sector_t {aka long long unsigned int}' [-Wformat=] dev_info(&dev->core, "%s: Using %lu MiB of GPU memory\n", ^ /kisskb/src/include/linux/device.h:1400:22: note: in definition of macro 'dev_fmt' #define dev_fmt(fmt) fmt ^ /kisskb/src/drivers/block/ps3vram.c:770:2: note: in expansion of macro 'dev_info' dev_info(&dev->core, "%s: Using %lu MiB of GPU memory\n", ^ Completed OK # rm -rf /kisskb/build/powerpc-next_ps3_defconfig_powerpc-gcc5 # Build took: 0:02:01.794209