On Tue, Aug 4, 2020 at 12:07 PM Joel Stanley <j...@jms.id.au> wrote: > > Messy: > > $ git grep "define DBG(" arch/powerpc/ |grep -v print > arch/powerpc/kernel/crash_dump.c:#define DBG(fmt...) > arch/powerpc/kernel/iommu.c:#define DBG(...) > arch/powerpc/kernel/legacy_serial.c:#define DBG(fmt...) do { } while(0) > arch/powerpc/kernel/prom.c:#define DBG(fmt...) > arch/powerpc/kernel/setup-common.c:#define DBG(fmt...) > arch/powerpc/kernel/setup_32.c:#define DBG(fmt...) > arch/powerpc/kernel/smp.c:#define DBG(fmt...) > arch/powerpc/kernel/vdso.c:#define DBG(fmt...) > arch/powerpc/kvm/book3s_hv_rm_xive.c:#define DBG(fmt...) do { } while(0) > arch/powerpc/mm/book3s64/hash_utils.c:#define DBG(fmt...) > arch/powerpc/platforms/83xx/mpc832x_mds.c:#define DBG(fmt...) > arch/powerpc/platforms/83xx/mpc832x_rdb.c:#define DBG(fmt...) > arch/powerpc/platforms/83xx/mpc836x_mds.c:#define DBG(fmt...) > arch/powerpc/platforms/85xx/mpc85xx_ds.c:#define DBG(fmt, args...) > arch/powerpc/platforms/85xx/mpc85xx_mds.c:#define DBG(fmt...) > arch/powerpc/platforms/85xx/mpc85xx_rdb.c:#define DBG(fmt, args...) > arch/powerpc/platforms/86xx/mpc86xx_hpcn.c:#define DBG(fmt...) do { } while(0) > arch/powerpc/platforms/cell/setup.c:#define DBG(fmt...) > arch/powerpc/platforms/cell/smp.c:#define DBG(fmt...) > arch/powerpc/platforms/embedded6xx/mpc7448_hpc2.c:#define DBG(fmt...) > do { } while(0) > arch/powerpc/platforms/maple/pci.c:#define DBG(x...) > arch/powerpc/platforms/maple/setup.c:#define DBG(fmt...) > arch/powerpc/platforms/maple/time.c:#define DBG(x...) > arch/powerpc/platforms/powermac/bootx_init.c:#define DBG(fmt...) do { } > while(0) > arch/powerpc/platforms/powermac/feature.c:#define DBG(fmt...) > arch/powerpc/platforms/powermac/low_i2c.c:#define DBG(x...) do {\ > arch/powerpc/platforms/powermac/low_i2c.c:#define DBG(x...) > arch/powerpc/platforms/powermac/nvram.c:#define DBG(x...) > arch/powerpc/platforms/powermac/pci.c:#define DBG(x...) > arch/powerpc/platforms/powermac/pfunc_base.c:#define DBG(fmt...) > arch/powerpc/platforms/powermac/pfunc_core.c:#define DBG(fmt...) > arch/powerpc/platforms/powermac/smp.c:#define DBG(fmt...) > arch/powerpc/platforms/powermac/time.c:#define DBG(x...) > arch/powerpc/platforms/powernv/smp.c:#define DBG(fmt...) > arch/powerpc/sysdev/dart_iommu.c:#define DBG(...) > arch/powerpc/sysdev/ge/ge_pic.c:#define DBG(fmt...) do { } while (0) > arch/powerpc/sysdev/mpic.c:#define DBG(fmt...) > arch/powerpc/sysdev/tsi108_dev.c:#define DBG(fmt...) do { } while(0) > arch/powerpc/sysdev/tsi108_pci.c:#define DBG(x...)
I started off writing a patch that fixed all these too. When I went to test it I discovered there's a giant pile of other W=1 warnings from other parts of arch/powerpc/ so I figured I'd start with something less ambitious.