Hi all, 0day reported a build error in arch/powerpc/platforms/embedded6xx/mvme5100.c when building with clang [1]. This is not a clang specific issue since it also happens with gcc:
$ curl -LSs https://lore.kernel.org/lkml/202004131704.6mh1jcq3%25...@intel.com/2-a.bin | gzip -d > .config $ make -j$(nproc) -s ARCH=powerpc CROSS_COMPILE=powerpc-linux- olddefconfig arch/powerpc/platforms/embedded6xx/mvme5100.o arch/powerpc/platforms/embedded6xx/mvme5100.c: In function 'mvme5100_add_bridge': arch/powerpc/platforms/embedded6xx/mvme5100.c:135:58: error: passing argument 5 of 'early_read_config_dword' from incompatible pointer type [-Werror=incompatible-pointer-types] 135 | early_read_config_dword(hose, 0, 0, PCI_BASE_ADDRESS_1, &pci_membase); | ^~~~~~~~~~~~ | | | phys_addr_t * {aka long long unsigned int *} In file included from arch/powerpc/platforms/embedded6xx/mvme5100.c:18: ./arch/powerpc/include/asm/pci-bridge.h:139:32: note: expected 'u32 *' {aka 'unsigned int *'} but argument is of type 'phys_addr_t *' {aka 'long long unsigned int *'} 139 | int dev_fn, int where, u32 *val); | ~~~~~^~~ In file included from ./include/linux/printk.h:7, from ./include/linux/kernel.h:15, from ./include/linux/list.h:9, from ./include/linux/rculist.h:10, from ./include/linux/pid.h:5, from ./include/linux/sched.h:14, from ./include/linux/ratelimit.h:6, from ./include/linux/dev_printk.h:16, from ./include/linux/device.h:15, from ./include/linux/of_platform.h:9, from arch/powerpc/platforms/embedded6xx/mvme5100.c:15: ./include/linux/kern_levels.h:5:18: error: format '%x' expects argument of type 'unsigned int', but argument 2 has type 'phys_addr_t' {aka 'long long unsigned int'} [-Werror=format=] 5 | #define KERN_SOH "\001" /* ASCII Start Of Header */ | ^~~~~~ ./include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH' 14 | #define KERN_INFO KERN_SOH "6" /* informational */ | ^~~~~~~~ ./include/linux/printk.h:305:9: note: in expansion of macro 'KERN_INFO' 305 | printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__) | ^~~~~~~~~ arch/powerpc/platforms/embedded6xx/mvme5100.c:142:2: note: in expansion of macro 'pr_info' 142 | pr_info("mvme5100_pic_init: pci_membase: %x\n", pci_membase); | ^~~~~~~ arch/powerpc/platforms/embedded6xx/mvme5100.c:142:44: note: format string is defined here 142 | pr_info("mvme5100_pic_init: pci_membase: %x\n", pci_membase); | ~^ | | | unsigned int | %llx cc1: all warnings being treated as errors make[4]: *** [scripts/Makefile.build:267: arch/powerpc/platforms/embedded6xx/mvme5100.o] Error 1 make[3]: *** [scripts/Makefile.build:488: arch/powerpc/platforms/embedded6xx] Error 2 make[2]: *** [scripts/Makefile.build:488: arch/powerpc/platforms] Error 2 make[1]: *** [Makefile:1722: arch/powerpc] Error 2 make: *** [Makefile:328: __build_one_by_one] Error 2 I am not sure how exactly this should be fixed. Should this driver just not be selectable when CONFIG_PHYS_ADDR_T_64BIT is selected or is there something else that I am missing? [1]: https://lore.kernel.org/lkml/202004131704.6mh1jcq3%25...@intel.com/ Cheers, Nathan
