* Build regressions/improvements in v4.20-rc1 @ 2018-11-05 22:05 Geert Uytterhoeven 2018-11-05 22:12 ` Geert Uytterhoeven 0 siblings, 1 reply; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-05 22:05 UTC (permalink / raw) To: linux-kernel Below is the list of build error/warning regressions/improvements in v4.20-rc1[1] compared to v4.19[2]. Summarized: - build errors: +3/-0 - build warnings: +449/-2712 Happy fixing! ;-) Thanks to the linux-next team for providing the build service. [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) *** ERRORS *** 3 error regressions: + /kisskb/src/arch/s390/kernel/nospec-branch.c: error: macro "memcpy" passed 6 arguments, but takes just 3: => 130:61 + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 + {standard input}: Error: Unrecognized opcode: `slbfee.': => 89, 37 *** WARNINGS *** [Deleted 2862 lines about "warning: -ffunction-sections disabled; it makes profiling impossible [enabled by default]" on powerpc-all{mod,yes}config*] 449 warning regressions: + /kisskb/src/arch/arc/mm/tlb.c: warning: ISO C90 forbids variable length array 'pd0' [-Wvla]: => 914:2 + /kisskb/src/arch/arc/mm/tlb.c: warning: variable length array 'pd0' is used [-Wvla]: => 914:2 + /kisskb/src/arch/m68k/atari/config.c: warning: variable length array 'switches' is used [-Wvla]: => 151:2 + /kisskb/src/arch/m68k/kernel/signal.c: warning: variable length array 'buf' is used [-Wvla]: => 654:3 + /kisskb/src/arch/s390/kernel/machine_kexec.c: warning: 'do_start_kdump' defined but not used [-Wunused-function]: => 145:22 + /kisskb/src/arch/s390/kernel/nospec-branch.c: warning: statement with no effect [-Wunused-value]: => 130:3 + /kisskb/src/arch/um/os-Linux/umid.c: warning: ISO C90 forbids variable length array 'dir' [-Wvla]: => 388:2 + /kisskb/src/arch/um/os-Linux/umid.c: warning: ISO C90 forbids variable length array 'file' [-Wvla]: => 213:2, 138:2 + /kisskb/src/arch/x86/xen/mmu_pv.c: warning: 'i' may be used uninitialized in this function [-Wuninitialized]: => 2787:12 + /kisskb/src/drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c: warning: (near initialization for 'wm.sets') [-Wmissing-braces]: => 94:9 + /kisskb/src/drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer_debug.c: warning: (near initialization for 'wm.sets') [-Wmissing-braces]: => 74:9 + /kisskb/src/drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer_debug.c: warning: missing braces around initializer [-Wmissing-braces]: => 74:9 + /kisskb/src/drivers/gpu/drm/drm_atomic_uapi.c: warning: format '%zu' expects argument of type 'size_t', but argument 5 has type 'unsigned int' [-Wformat=]: => 131:21 + /kisskb/src/drivers/hid/hid-asus.c: warning: 'value' may be used uninitialized in this function [-Wuninitialized]: => 364:2 + /kisskb/src/drivers/lightnvm/core.c: warning: 'ret' may be used uninitialized in this function [-Wuninitialized]: => 977:6 + /kisskb/src/drivers/mtd/maps/sa1100-flash.c: warning: variable length array 'cdev' is used [-Wvla]: => 224:10 + /kisskb/src/drivers/mtd/nand/raw/mpc5121_nfc.c: warning: unused variable 'mtd' [-Wunused-variable]: => 306:19 + /kisskb/src/drivers/net/ethernet/broadcom/bnxt/bnxt.h: warning: "writeq" redefined [enabled by default]: => 1626:0 + /kisskb/src/drivers/net/ethernet/broadcom/bnxt/bnxt.h: warning: "writeq" redefined: => 1626:0, 1626 + /kisskb/src/drivers/net/ethernet/broadcom/bnxt/bnxt.h: warning: "writeq_relaxed" redefined [enabled by default]: => 1634:0 + /kisskb/src/drivers/net/ethernet/broadcom/bnxt/bnxt.h: warning: "writeq_relaxed" redefined: => 1634:0, 1634 + /kisskb/src/drivers/net/ethernet/mellanox/mlx5/core/en_stats.c: warning: the frame size of 1200 bytes is larger than 1024 bytes [-Wframe-larger-than=]: => 216:1 + /kisskb/src/drivers/net/ethernet/mellanox/mlx5/core/en_stats.c: warning: the frame size of 1240 bytes is larger than 1024 bytes [-Wframe-larger-than=]: => 216:1 + /kisskb/src/drivers/net/ethernet/mellanox/mlx5/core/en_stats.c: warning: the frame size of 1256 bytes is larger than 1024 bytes [-Wframe-larger-than=]: => 216:1 + /kisskb/src/drivers/net/ethernet/mellanox/mlx5/core/en_stats.c: warning: the frame size of 1536 bytes is larger than 1024 bytes [-Wframe-larger-than=]: => 216:1 + /kisskb/src/drivers/net/wireless/quantenna/qtnfmac/commands.c: warning: 'resp' may be used uninitialized in this function [-Wmaybe-uninitialized]: => 133:54 + /kisskb/src/drivers/net/wireless/quantenna/qtnfmac/commands.c: warning: 'resp' may be used uninitialized in this function [-Wuninitialized]: => 133:38, 132:11 + /kisskb/src/drivers/scsi/myrs.c: warning: 'sshdr.sense_key' may be used uninitialized in this function [-Wmaybe-uninitialized]: => 818:24 + /kisskb/src/fs/ubifs/replay.c: warning: the frame size of 1160 bytes is larger than 1024 bytes [-Wframe-larger-than=]: => 633:1 + /kisskb/src/include/linux/dynamic_debug.h: warning: 'value' may be used uninitialized in this function [-Wmaybe-uninitialized]: => 135:3 + /kisskb/src/include/linux/spinlock.h: warning: 'rq' may be used uninitialized in this function [-Wmaybe-uninitialized]: => 273:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U 640>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U d20>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U11e0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U1820>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U18c0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U1960>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U1aa0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U1b40>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U1d20>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U2960>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U38c0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U3aa0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U3c80>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U3d20>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U45a0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U58c0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U5b40>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U5be0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U6460>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U6820>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U6aa0>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U6d20>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U78c0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U7aa0>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U8140>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U9460>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33, 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U9500>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U9b40>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U9f00>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ua000>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ua0a0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ua8c0>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uaf00>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ub500>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ubaa0>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32, 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ubb40>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ube60>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uc8c0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uca00>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ucdc0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ud0a0>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ud280>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Udb40>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Udf00>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ue320>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33, 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ue5a0>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ue820>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ue960>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ueaa0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uef00>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uf500>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uf960>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ufb40>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ufc80>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ufdc0>]' may be used uninitialized in this function [-Wuninitialized]: => 133:33 + /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ufe60>]' may be used uninitialized in this function [-Wuninitialized]: => 152:32 + /kisskb/src/kernel/sched/sched.h: warning: 'rq' may be used uninitialized in this function [-Wmaybe-uninitialized]: => 1063:8 + /kisskb/src/kernel/sched/sched.h: warning: 'rq' may be used uninitialized in this function [-Wuninitialized]: => 1063:8 + /kisskb/src/mm/slub.c: warning: 'deactivate_slab.isra.37' uses dynamic stack allocation [enabled by default]: => 2170:1 + /kisskb/src/mm/slub.c: warning: 'get_partial_node.isra.39' uses dynamic stack allocation [enabled by default]: => 1870:1 + /kisskb/src/mm/slub.c: warning: 'unfreeze_partials.isra.38' uses dynamic stack allocation [enabled by default]: => 2238:1 + /kisskb/src/sound/pci/hda/patch_ca0132.c: warning: assignment makes pointer from integer without a cast [enabled by default]: => 8799:18 + /kisskb/src/sound/soc/codecs/wm9712.c: warning: 'regmap' may be used uninitialized in this function [-Wuninitialized]: => 666:31 + warning: vmlinux.o(.text+0x33600): Section mismatch in reference from the function setup_scache() to the function .init.text:loongson3_sc_init(): => N/A + warning: vmlinux.o(.text+0x35c4): Section mismatch in reference from the variable __boot_from_prom to the function .init.text:prom_init(): => N/A + warning: vmlinux.o(.text+0x37ef4): Section mismatch in reference from the function mips_sc_init() to the function .init.text:mips_sc_probe_cm3(): => N/A + warning: vmlinux.o(.text+0x3844): Section mismatch in reference from the variable start_here_multiplatform to the function .init.text:early_setup(): => N/A + warning: vmlinux.o(.text+0x3878): Section mismatch in reference from the variable start_here_common to the function .init.text:start_kernel(): => N/A 2712 warning improvements: - /kisskb/src/arch/s390/crypto/aes_s390.c: warning: 'fallback_blk_dec' uses dynamic stack allocation [enabled by default]: 217:1 => - /kisskb/src/arch/s390/crypto/aes_s390.c: warning: 'fallback_blk_enc' uses dynamic stack allocation [enabled by default]: 234:1 => - /kisskb/src/arch/s390/crypto/aes_s390.c: warning: 'xts_fallback_decrypt' uses dynamic stack allocation [enabled by default]: 486:1 => - /kisskb/src/arch/s390/crypto/aes_s390.c: warning: 'xts_fallback_encrypt' uses dynamic stack allocation [enabled by default]: 505:1 => - /kisskb/src/arch/s390/kernel/perf_cpum_sf.c: warning: 'perf_push_sample' uses dynamic stack allocation [enabled by default]: 1075:1 => - /kisskb/src/arch/sparc/include/asm/cmpxchg_64.h: warning: value computed is not used [-Wunused-value]: 55:22 => - /kisskb/src/arch/um/os-Linux/skas/process.c: warning: control reaches end of non-void function [-Wreturn-type]: 613:1 => - /kisskb/src/arch/x86/xen/mmu.c: warning: 'i' may be used uninitialized in this function [-Wuninitialized]: 156:12 => - /kisskb/src/crypto/algif_aead.c: warning: 'crypto_aead_copy_sgl' uses dynamic stack allocation [enabled by default]: 90:1 => - /kisskb/src/crypto/algif_hash.c: warning: 'hash_accept' uses dynamic stack allocation [enabled by default]: 276:1 => - /kisskb/src/crypto/authenc.c: warning: 'crypto_authenc_copy_assoc' uses dynamic stack allocation [enabled by default]: 197:1 => - /kisskb/src/crypto/authencesn.c: warning: 'crypto_authenc_esn_copy' uses dynamic stack allocation [enabled by default]: 194:1 => - /kisskb/src/crypto/cbc.c: warning: 'crypto_cbc_decrypt' uses dynamic stack allocation [enabled by default]: 79:1 => - /kisskb/src/crypto/ccm.c: warning: 'crypto_ccm_auth' uses dynamic stack allocation [enabled by default]: 235:1 => - /kisskb/src/crypto/cryptd.c: warning: 'cryptd_skcipher_decrypt' uses dynamic stack allocation [enabled by default]: 533:1 => - /kisskb/src/crypto/cryptd.c: warning: 'cryptd_skcipher_encrypt' uses dynamic stack allocation [enabled by default]: 505:1 => - /kisskb/src/crypto/echainiv.c: warning: 'echainiv_encrypt' uses dynamic stack allocation [enabled by default]: 88:1 => - /kisskb/src/crypto/gcm.c: warning: 'crypto_rfc4543_copy_src_to_dst' uses dynamic stack allocation [enabled by default]: 1077:1 => - /kisskb/src/crypto/hmac.c: warning: 'hmac_setkey' uses dynamic stack allocation [enabled by default]: 88:1 => - /kisskb/src/crypto/seqiv.c: warning: 'seqiv_aead_encrypt' uses dynamic stack allocation [enabled by default]: 115:1 => - /kisskb/src/crypto/shash.c: warning: 'shash_final_unaligned' uses dynamic stack allocation [enabled by default]: 141:1 => - /kisskb/src/crypto/shash.c: warning: 'shash_update_unaligned' uses dynamic stack allocation [enabled by default]: 105:1 => - /kisskb/src/crypto/xcbc.c: warning: 'crypto_xcbc_digest_setkey' uses dynamic stack allocation [enabled by default]: 79:1 => - /kisskb/src/drivers/atm/ambassador.c: warning: passing argument 1 of 'virt_to_bus' discards 'volatile' qualifier from pointer target type [enabled by default]: 1762:3 => - /kisskb/src/drivers/block/cryptoloop.c: warning: 'cryptoloop_transfer' uses dynamic stack allocation [enabled by default]: 167:1 => - /kisskb/src/drivers/block/drbd/drbd_receiver.c: warning: 'drbd_do_auth' uses dynamic stack allocation [enabled by default]: 5400:1 => - /kisskb/src/drivers/block/drbd/drbd_worker.c: warning: 'drbd_csum_bio' uses dynamic stack allocation [enabled by default]: 352:1 => - /kisskb/src/drivers/block/drbd/drbd_worker.c: warning: 'drbd_csum_ee' uses dynamic stack allocation [enabled by default]: 325:1 => - /kisskb/src/drivers/crypto/ccree/cc_cipher.c: warning: 'cc_cipher_setkey' uses dynamic stack allocation [enabled by default]: 383:1 => - /kisskb/src/drivers/crypto/chelsio/chcr_algo.c: warning: 'chcr_ahash_setkey' uses dynamic stack allocation [enabled by default]: 2081:1 => - /kisskb/src/drivers/crypto/chelsio/chcr_algo.c: warning: 'chcr_authenc_setkey' uses dynamic stack allocation [enabled by default]: 3496:1 => - /kisskb/src/drivers/crypto/chelsio/chcr_algo.c: warning: 'chcr_cipher_fallback' uses dynamic stack allocation [enabled by default]: 699:1 => - /kisskb/src/drivers/crypto/chelsio/chtls/chtls_cm.c: warning: 'wait_for_states.constprop.22' uses dynamic stack allocation [enabled by default]: 405:1 => - /kisskb/src/drivers/crypto/mediatek/mtk-sha.c: warning: 'mtk_sha_finish_hmac' uses dynamic stack allocation [enabled by default]: 373:1 => - /kisskb/src/drivers/crypto/mediatek/mtk-sha.c: warning: 'mtk_sha_setkey' uses dynamic stack allocation [enabled by default]: 834:1 => - /kisskb/src/drivers/crypto/qce/ablkcipher.c: warning: 'qce_ablkcipher_crypt' uses dynamic stack allocation [enabled by default]: 229:1 => - /kisskb/src/drivers/crypto/s5p-sss.c: warning: 's5p_hash_shash_digest' uses dynamic stack allocation [enabled by default]: 1534:1 => - /kisskb/src/drivers/firmware/google/coreboot_table.c: warning: 'ret' may be used uninitialized in this function [-Wuninitialized]: 102:9 => - /kisskb/src/drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c: warning: (near initialization for 'blnd_cfg.black_color') [-Wmissing-braces]: 1903:9 => - /kisskb/src/drivers/gpu/drm/drm_atomic.c: warning: format '%zu' expects argument of type 'size_t', but argument 5 has type 'unsigned int' [-Wformat=]: 441:21 => - /kisskb/src/drivers/iio/proximity/isl29501.c: warning: 'msb' may be used uninitialized in this function [-Wuninitialized]: 253:7, 253:34 => - /kisskb/src/drivers/infiniband/sw/rxe/rxe_icrc.c: warning: 'rxe_icrc_hdr' uses dynamic stack allocation [enabled by default]: 96:1 => - /kisskb/src/drivers/infiniband/sw/rxe/rxe_mr.c: warning: 'rxe_mem_copy' uses dynamic stack allocation [enabled by default]: 431:1 => - /kisskb/src/drivers/infiniband/sw/rxe/rxe_recv.c: warning: 'rxe_rcv' uses dynamic stack allocation [enabled by default]: 423:1 => - /kisskb/src/drivers/infiniband/sw/rxe/rxe_req.c: warning: 'fill_packet.isra.33' uses dynamic stack allocation [enabled by default]: 506:1 => - /kisskb/src/drivers/md/dm-crypt.c: warning: 'crypt_iv_essiv_init' uses dynamic stack allocation [enabled by default]: 350:1 => - /kisskb/src/drivers/md/dm-crypt.c: warning: 'crypt_iv_lmk_one.isra.26' uses dynamic stack allocation [enabled by default]: 645:1 => - /kisskb/src/drivers/md/dm-crypt.c: warning: 'crypt_iv_tcw_whitening.isra.25' uses dynamic stack allocation [enabled by default]: 792:1 => - /kisskb/src/drivers/md/dm-integrity.c: warning: '__journal_read_write' uses dynamic stack allocation [enabled by default]: 1648:1 => - /kisskb/src/drivers/md/dm-integrity.c: warning: 'do_journal_write' uses dynamic stack allocation [enabled by default]: 2060:1 => - /kisskb/src/drivers/md/dm-integrity.c: warning: 'integrity_metadata' uses dynamic stack allocation [enabled by default]: 1415:1 => - /kisskb/src/drivers/md/dm-integrity.c: warning: 'integrity_sector_checksum' uses dynamic stack allocation [enabled by default]: 1311:1 => - /kisskb/src/drivers/md/dm-integrity.c: warning: 'section_mac' uses dynamic stack allocation [enabled by default]: 574:1 => - /kisskb/src/drivers/md/dm-verity-fec.c: warning: 'fec_read_bufs' uses dynamic stack allocation [enabled by default]: 301:1 => - /kisskb/src/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c: warning: overflow in implicit constant conversion [-Woverflow]: 545:41 => - /kisskb/src/drivers/net/ethernet/neterion/vxge/vxge-config.c: warning: 'vxge_hw_device_hw_info_get' uses dynamic stack allocation [enabled by default]: 1088:1 => - /kisskb/src/drivers/net/ppp/ppp_mppe.c: warning: 'mppe_compress' uses dynamic stack allocation [enabled by default]: 445:1 => - /kisskb/src/drivers/net/ppp/ppp_mppe.c: warning: 'mppe_decompress' uses dynamic stack allocation [enabled by default]: 670:1 => - /kisskb/src/drivers/net/ppp/ppp_mppe.c: warning: 'mppe_rekey' uses dynamic stack allocation [enabled by default]: 186:1 => - /kisskb/src/drivers/net/wan/lmc/lmc_main.c: warning: passing argument 1 of 'virt_to_bus' discards 'volatile' qualifier from pointer target type [enabled by default]: 1874:9, 1863:5, 1876:5, 1861:9, 1852:9, 1877:5 => - /kisskb/src/drivers/nfc/s3fwrn5/firmware.c: warning: 's3fwrn5_fw_download' uses dynamic stack allocation [enabled by default]: 501:1 => - /kisskb/src/drivers/pinctrl/cirrus/pinctrl-madera-core.c: warning: 'reg' may be used uninitialized in this function [-Wuninitialized]: 660:3 => - /kisskb/src/drivers/pinctrl/cirrus/pinctrl-madera-core.c: warning: 'ret' may be used uninitialized in this function [-Wuninitialized]: 659:5 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'ism_add_vlan_id' uses dynamic stack allocation [enabled by default]: 311:1 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'ism_del_vlan_id' uses dynamic stack allocation [enabled by default]: 325:1 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'ism_dev_init' uses dynamic stack allocation [enabled by default]: 487:1 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'ism_query_rgid' uses dynamic stack allocation [enabled by default]: 209:1 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'ism_register_dmb' uses dynamic stack allocation [enabled by default]: 276:1 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'ism_signal_ieq' uses dynamic stack allocation [enabled by default]: 353:1 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'ism_unregister_dmb' uses dynamic stack allocation [enabled by default]: 297:1 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'query_info' uses dynamic stack allocation [enabled by default]: 84:1 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'register_ieq' uses dynamic stack allocation [enabled by default]: 140:1 => - /kisskb/src/drivers/s390/net/ism_drv.c: warning: 'register_sba' uses dynamic stack allocation [enabled by default]: 111:1 => - /kisskb/src/drivers/staging/comedi/drivers/pcl818.c: warning: 'last_chan' may be used uninitialized in this function [-Wuninitialized]: 337:6 => - /kisskb/src/drivers/staging/fsl-dpaa2/ethsw/ethsw.c: warning: 'err' may be used uninitialized in this function [-Wuninitialized]: 927:2, 720:11 => - /kisskb/src/drivers/target/iscsi/cxgbit/cxgbit_target.c: warning: 'cxgbit_tx_datain_iso.isra.32' uses dynamic stack allocation [enabled by default]: 501:1 => - /kisskb/src/drivers/target/iscsi/iscsi_target.c: warning: 'iscsit_send_datain' uses dynamic stack allocation [enabled by default]: 2851:1 => - /kisskb/src/drivers/usb/wusbcore/crypto.c: warning: 'wusb_ccm_mac' uses dynamic stack allocation [enabled by default]: 286:1 => - /kisskb/src/fs/crypto/keyinfo.c: warning: 'derive_essiv_salt' uses dynamic stack allocation [enabled by default]: 250:1 => - /kisskb/src/fs/ecryptfs/crypto.c: warning: 'ecryptfs_hash_digest' uses dynamic stack allocation [enabled by default]: 75:1 => - /kisskb/src/fs/nfsd/nfs4recover.c: warning: 'nfs4_make_rec_clidname' uses dynamic stack allocation [enabled by default]: 147:1 => - /kisskb/src/include/asm-generic/io.h: warning: 'px_cmd' may be used uninitialized in this function [-Wuninitialized]: 232:15 => - /kisskb/src/include/asm-generic/io.h: warning: 'px_is' may be used uninitialized in this function [-Wuninitialized]: 232:15 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U a00>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U aa0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U1000>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U1320>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U2640>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U28c0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U2be0>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U30a0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U3460>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U35a0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U3780>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U40a0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U4280>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U4460>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U4640>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U4820>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U48c0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U4aa0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U5640>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U5dc0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U60a0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U6140>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U6500>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U6640>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U6be0>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U6f00>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U76e0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U7960>]' may be used uninitialized in this function [-Wuninitialized]: 148:32, 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U7dc0>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U8280>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U85a0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U8640>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U8dc0>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U9000>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U91e0>]' may be used uninitialized in this function [-Wuninitialized]: 148:32, 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U95a0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U9a00>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<U9d20>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ua500>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uaa00>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uaaa0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uae60>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ub0a0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ub780>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ubbe0>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uc320>]' may be used uninitialized in this function [-Wuninitialized]: 148:32, 129:33 => 133:33 - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uc3c0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ud140>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ud640>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ud6e0>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ud820>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uda00>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uddc0>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ue3c0>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ueb40>]' may be used uninitialized in this function [-Wuninitialized]: 148:32 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Uf820>]' may be used uninitialized in this function [-Wuninitialized]: 129:33 => - /kisskb/src/kernel/rcu/srcutree.c: warning: 'levelspread[<Ufa00>]' may be used uninitialized in this function [-Wuninitialized]: 148:32, 129:33 => - /kisskb/src/kernel/sched/core.c: warning: unused variable 'steal' [-Wunused-variable]: 139:6 => - /kisskb/src/lib/libcrc32c.c: warning: 'crc32c' uses dynamic stack allocation [enabled by default]: 59:1 => - /kisskb/src/lib/test_ubsan.c: warning: 'test_ubsan_out_of_bounds' uses dynamic stack allocation [enabled by default]: 67:1 => - /kisskb/src/lib/test_ubsan.c: warning: 'test_ubsan_vla_bound_not_positive' uses dynamic stack allocation [enabled by default]: 51:1 => - /kisskb/src/mm/nobootmem.c: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]: 65:8 => - /kisskb/src/mm/slub.c: warning: 'deactivate_slab.isra.50' uses dynamic stack allocation [enabled by default]: 2132:1 => - /kisskb/src/mm/slub.c: warning: 'get_partial_node.isra.52' uses dynamic stack allocation [enabled by default]: 1832:1 => - /kisskb/src/mm/slub.c: warning: 'unfreeze_partials.isra.51' uses dynamic stack allocation [enabled by default]: 2200:1 => - /kisskb/src/mm/swap_state.c: warning: 'error' may be used uninitialized in this function [-Wuninitialized]: 135:5 => - /kisskb/src/net/bridge/netfilter/ebtables.c: warning: 'compat_copy_everything_to_user' uses dynamic stack allocation [enabled by default]: 1906:1 => - /kisskb/src/net/ceph/crypto.c: warning: 'ceph_aes_crypt.isra.4' uses dynamic stack allocation [enabled by default]: 278:1 => - /kisskb/src/net/mac802154/llsec.c: warning: 'llsec_do_decrypt_unauth.isra.3' uses dynamic stack allocation [enabled by default]: 859:1 => - /kisskb/src/net/mac802154/llsec.c: warning: 'llsec_do_encrypt_unauth.isra.4' uses dynamic stack allocation [enabled by default]: 641:1 => - /kisskb/src/net/rxrpc/rxkad.c: warning: 'rxkad_decrypt_response.isra.6' uses dynamic stack allocation [enabled by default]: 1051:1 => - /kisskb/src/net/rxrpc/rxkad.c: warning: 'rxkad_encrypt_response.isra.7' uses dynamic stack allocation [enabled by default]: 772:1 => - /kisskb/src/net/rxrpc/rxkad.c: warning: 'rxkad_prime_packet_security' uses dynamic stack allocation [enabled by default]: 141:1 => - /kisskb/src/net/rxrpc/rxkad.c: warning: 'rxkad_secure_packet' uses dynamic stack allocation [enabled by default]: 313:1 => - /kisskb/src/net/rxrpc/rxkad.c: warning: 'rxkad_verify_packet' uses dynamic stack allocation [enabled by default]: 564:1 => - /kisskb/src/net/sctp/auth.c: warning: 'sctp_auth_calculate_hmac' uses dynamic stack allocation [enabled by default]: 778:1 => - /kisskb/src/net/sctp/sm_make_chunk.c: warning: 'sctp_pack_cookie.isra.9' uses dynamic stack allocation [enabled by default]: 1700:1 => - /kisskb/src/net/sctp/sm_make_chunk.c: warning: 'sctp_unpack_cookie' uses dynamic stack allocation [enabled by default]: 1877:1 => - /kisskb/src/net/sunrpc/auth_gss/gss_krb5_crypto.c: warning: 'gss_decrypt_xdr_buf' uses dynamic stack allocation [enabled by default]: 634:1 => - /kisskb/src/net/sunrpc/auth_gss/gss_krb5_crypto.c: warning: 'gss_encrypt_xdr_buf' uses dynamic stack allocation [enabled by default]: 555:1 => - /kisskb/src/net/sunrpc/auth_gss/gss_krb5_crypto.c: warning: 'gss_krb5_aes_decrypt' uses dynamic stack allocation [enabled by default]: 942:1 => - /kisskb/src/net/sunrpc/auth_gss/gss_krb5_crypto.c: warning: 'gss_krb5_aes_encrypt' uses dynamic stack allocation [enabled by default]: 849:1 => - /kisskb/src/net/sunrpc/auth_gss/gss_krb5_crypto.c: warning: 'gss_krb5_cts_crypt' uses dynamic stack allocation [enabled by default]: 728:1 => - /kisskb/src/net/sunrpc/auth_gss/gss_krb5_crypto.c: warning: 'krb5_decrypt' uses dynamic stack allocation [enabled by default]: 129:1 => - /kisskb/src/net/sunrpc/auth_gss/gss_krb5_crypto.c: warning: 'krb5_encrypt' uses dynamic stack allocation [enabled by default]: 91:1 => - /kisskb/src/net/sunrpc/stats.c: warning: 'rpc_clnt_show_stats' uses dynamic stack allocation [enabled by default]: 268:1 => - /kisskb/src/net/tipc/topsrv.c: warning: 'strncpy' specified bound depends on the length of the source argument [-Wstringop-overflow=]: 660:2 => - /kisskb/src/security/apparmor/crypto.c: warning: 'aa_calc_hash' uses dynamic stack allocation [enabled by default]: 64:1 => - /kisskb/src/security/apparmor/crypto.c: warning: 'aa_calc_profile_hash' uses dynamic stack allocation [enabled by default]: 106:1 => - /kisskb/src/security/integrity/ima/ima_crypto.c: warning: 'calc_buffer_shash_tfm' uses dynamic stack allocation [enabled by default]: 600:1 => - /kisskb/src/security/integrity/ima/ima_crypto.c: warning: 'ima_calc_boot_aggregate_tfm' uses dynamic stack allocation [enabled by default]: 667:1 => - /kisskb/src/security/integrity/ima/ima_crypto.c: warning: 'ima_calc_field_array_hash_tfm.isra.3' uses dynamic stack allocation [enabled by default]: 493:1 => - /kisskb/src/security/integrity/ima/ima_crypto.c: warning: 'ima_calc_file_hash_tfm' uses dynamic stack allocation [enabled by default]: 388:1 => - /kisskb/src/security/keys/encrypted-keys/encrypted.c: warning: 'calc_hash' uses dynamic stack allocation [enabled by default]: 337:1 => - warning: drivers/hwmon/ibmpowernv.o(.text+0xfe8): Section mismatch in reference from the function .create_device_attrs() to the function .init.text:.make_sensor_label(): N/A => - warning: vmlinux.o(.text+0x2f3bf0): Section mismatch in reference from the function .devm_memremap_pages() to the function .meminit.text:.arch_add_memory(): N/A => - warning: vmlinux.o(.text+0x2f4434): Section mismatch in reference from the function .devm_memremap_pages_release() to the function .meminit.text:.arch_remove_memory(): N/A => - warning: vmlinux.o(.text+0x2fa9d0): Section mismatch in reference from the function .devm_memremap_pages() to the function .meminit.text:.arch_add_memory(): N/A => - warning: vmlinux.o(.text+0x2fb214): Section mismatch in reference from the function .devm_memremap_pages_release() to the function .meminit.text:.arch_remove_memory(): N/A => - warning: vmlinux.o(.text+0x336d8): Section mismatch in reference from the function setup_scache() to the function .init.text:loongson3_sc_init(): N/A => - warning: vmlinux.o(.text+0x3679c0): Section mismatch in reference from the function .devm_memremap_pages() to the function .meminit.text:.arch_add_memory(): N/A => - warning: vmlinux.o(.text+0x368204): Section mismatch in reference from the function .devm_memremap_pages_release() to the function .meminit.text:.arch_remove_memory(): N/A => - warning: vmlinux.o(.text+0x36b4): Section mismatch in reference from the variable __boot_from_prom to the function .init.text:prom_init(): N/A => - warning: vmlinux.o(.text+0x380e0): Section mismatch in reference from the function mips_sc_init() to the function .init.text:mips_sc_probe_cm3(): N/A => - warning: vmlinux.o(.text+0x3944): Section mismatch in reference from the variable start_here_multiplatform to the function .init.text:early_setup(): N/A => - warning: vmlinux.o(.text+0x3978): Section mismatch in reference from the variable start_here_common to the function .init.text:start_kernel(): N/A => - warning: vmlinux.o(.text+0x42332c): Section mismatch in reference from the function .hmm_devmem_release() to the function .meminit.text:.arch_remove_memory(): N/A => - warning: vmlinux.o(.text+0x423628): Section mismatch in reference from the function .hmm_devmem_pages_create() to the function .meminit.text:.arch_add_memory(): N/A => - warning: vmlinux.o(.text+0x42aa6c): Section mismatch in reference from the function .hmm_devmem_release() to the function .meminit.text:.arch_remove_memory(): N/A => - warning: vmlinux.o(.text+0x42ad68): Section mismatch in reference from the function .hmm_devmem_pages_create() to the function .meminit.text:.arch_add_memory(): N/A => - warning: vmlinux.o(.text+0x49e40c): Section mismatch in reference from the function .hmm_devmem_release() to the function .meminit.text:.arch_remove_memory(): N/A => - warning: vmlinux.o(.text+0x49e708): Section mismatch in reference from the function .hmm_devmem_pages_create() to the function .meminit.text:.arch_add_memory(): N/A => - warning: vmlinux.o(.text+0x4df549c): Section mismatch in reference from the function .create_device_attrs() to the function .init.text:.make_sensor_label(): N/A => - warning: vmlinux.o(.text.unlikely+0x26a8): Section mismatch in reference from the function .remove_pmd_table() to the function .meminit.text:.split_kernel_mapping(): N/A => - warning: vmlinux.o(.text.unlikely+0x2718): Section mismatch in reference from the function .remove_pmd_table() to the function .meminit.text:.split_kernel_mapping(): N/A => - warning: vmlinux.o(.text.unlikely+0x2820): Section mismatch in reference from the function .remove_pud_table() to the function .meminit.text:.split_kernel_mapping(): N/A => - warning: vmlinux.o(.text.unlikely+0x293c): Section mismatch in reference from the function .remove_pud_table() to the function .meminit.text:.split_kernel_mapping(): N/A => Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: Build regressions/improvements in v4.20-rc1 2018-11-05 22:05 Build regressions/improvements in v4.20-rc1 Geert Uytterhoeven @ 2018-11-05 22:12 ` Geert Uytterhoeven 2018-11-06 1:04 ` Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) Randy Dunlap 0 siblings, 1 reply; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-05 22:12 UTC (permalink / raw) To: Linux Kernel Mailing List Cc: linux-s390, ALSA Development Mailing List, linuxppc-dev On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > Below is the list of build error/warning regressions/improvements in > v4.20-rc1[1] compared to v4.19[2]. > > Summarized: > - build errors: +3/-0 > - build warnings: +449/-2712 > > Happy fixing! ;-) > > Thanks to the linux-next team for providing the build service. > > [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > > > *** ERRORS *** > > 3 error regressions: > + /kisskb/src/arch/s390/kernel/nospec-branch.c: error: macro "memcpy" passed 6 arguments, but takes just 3: => 130:61 s390-all{mod,yes}config > + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 sh4-all{mod,yes}config Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of pci_iomap() on SH") is not sufficient? > + {standard input}: Error: Unrecognized opcode: `slbfee.': => 89, 37 powerpc-all{mod,yes}config Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) 2018-11-05 22:12 ` Geert Uytterhoeven @ 2018-11-06 1:04 ` Randy Dunlap 2018-11-06 8:01 ` Geert Uytterhoeven 2018-11-06 16:18 ` Takashi Iwai 0 siblings, 2 replies; 22+ messages in thread From: Randy Dunlap @ 2018-11-06 1:04 UTC (permalink / raw) To: Geert Uytterhoeven, Linux Kernel Mailing List Cc: linux-s390, ALSA Development Mailing List, linuxppc-dev On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: >> Below is the list of build error/warning regressions/improvements in >> v4.20-rc1[1] compared to v4.19[2]. >> >> Summarized: >> - build errors: +3/-0 >> - build warnings: +449/-2712 >> >> Happy fixing! ;-) >> >> Thanks to the linux-next team for providing the build service. >> >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) >> >> >> *** ERRORS *** >> >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > sh4-all{mod,yes}config > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > pci_iomap() on SH") > is not sufficient? Different problem. This is about "select": config SND_SOC_ALL_CODECS tristate "Build all ASoC CODEC drivers" That enables (sets): select SND_SOC_HDAC_HDA which selects SND_HDA even though CONFIG_PCI is not enabled. After SND_HDA is selected (above), the Kconfig symbols in sound/pci/hda/Kconfig are available for enabling, so SND_HDA_CODEC_CA0132 is enabled but will not build. One simple solution (but possibly too naive) is: --- sound/soc/codecs/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig +++ lnx-420-rc1/sound/soc/codecs/Kconfig @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS select SND_SOC_ES7241 select SND_SOC_GTM601 select SND_SOC_HDAC_HDMI - select SND_SOC_HDAC_HDA + select SND_SOC_HDAC_HDA if PCI select SND_SOC_ICS43432 select SND_SOC_INNO_RK3036 select SND_SOC_ISABELLE if I2C -- ~Randy ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) 2018-11-06 1:04 ` Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) Randy Dunlap 2018-11-06 8:01 ` Geert Uytterhoeven @ 2018-11-06 8:01 ` Geert Uytterhoeven 1 sibling, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-06 8:01 UTC (permalink / raw) To: Randy Dunlap Cc: Linux Kernel Mailing List, linux-s390, ALSA Development Mailing List, linuxppc-dev, Linux-sh list Hi Randy, On Tue, Nov 6, 2018 at 2:06 AM Randy Dunlap <rdunlap@infradead.org> wrote: > On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > >> Below is the list of build error/warning regressions/improvements in > >> v4.20-rc1[1] compared to v4.19[2]. > >> > >> Summarized: > >> - build errors: +3/-0 > >> - build warnings: +449/-2712 > >> > >> Happy fixing! ;-) > >> > >> Thanks to the linux-next team for providing the build service. > >> > >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > >> > >> > >> *** ERRORS *** > >> > >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > > > sh4-all{mod,yes}config > > > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > > pci_iomap() on SH") > > is not sufficient? > > Different problem. This is about "select": > > config SND_SOC_ALL_CODECS > tristate "Build all ASoC CODEC drivers" > > That enables (sets): > select SND_SOC_HDAC_HDA > which selects SND_HDA even though CONFIG_PCI is not enabled. > > After SND_HDA is selected (above), the Kconfig symbols in > sound/pci/hda/Kconfig are available for enabling, so > SND_HDA_CODEC_CA0132 is enabled but will not build. Thanks for looking into this! > One simple solution (but possibly too naive) is: > > --- > sound/soc/codecs/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig > +++ lnx-420-rc1/sound/soc/codecs/Kconfig > @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS > select SND_SOC_ES7241 > select SND_SOC_GTM601 > select SND_SOC_HDAC_HDMI > - select SND_SOC_HDAC_HDA > + select SND_SOC_HDAC_HDA if PCI > select SND_SOC_ICS43432 > select SND_SOC_INNO_RK3036 > select SND_SOC_ISABELLE if I2C I guess that will work. There are already plenty of "select foo if bar" lines. However, looking at what else can enable SND_HDA, I think it should be select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA That still leaves the issue that pci_iomap() on SH should be an empty stub if PCI is not available, like on other architectures. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 8:01 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-06 8:01 UTC (permalink / raw) To: Randy Dunlap Cc: linux-s390, ALSA Development Mailing List, linuxppc-dev, Linux Kernel Mailing List, Linux-sh list Hi Randy, On Tue, Nov 6, 2018 at 2:06 AM Randy Dunlap <rdunlap@infradead.org> wrote: > On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > >> Below is the list of build error/warning regressions/improvements in > >> v4.20-rc1[1] compared to v4.19[2]. > >> > >> Summarized: > >> - build errors: +3/-0 > >> - build warnings: +449/-2712 > >> > >> Happy fixing! ;-) > >> > >> Thanks to the linux-next team for providing the build service. > >> > >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > >> > >> > >> *** ERRORS *** > >> > >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > > > sh4-all{mod,yes}config > > > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > > pci_iomap() on SH") > > is not sufficient? > > Different problem. This is about "select": > > config SND_SOC_ALL_CODECS > tristate "Build all ASoC CODEC drivers" > > That enables (sets): > select SND_SOC_HDAC_HDA > which selects SND_HDA even though CONFIG_PCI is not enabled. > > After SND_HDA is selected (above), the Kconfig symbols in > sound/pci/hda/Kconfig are available for enabling, so > SND_HDA_CODEC_CA0132 is enabled but will not build. Thanks for looking into this! > One simple solution (but possibly too naive) is: > > --- > sound/soc/codecs/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig > +++ lnx-420-rc1/sound/soc/codecs/Kconfig > @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS > select SND_SOC_ES7241 > select SND_SOC_GTM601 > select SND_SOC_HDAC_HDMI > - select SND_SOC_HDAC_HDA > + select SND_SOC_HDAC_HDA if PCI > select SND_SOC_ICS43432 > select SND_SOC_INNO_RK3036 > select SND_SOC_ISABELLE if I2C I guess that will work. There are already plenty of "select foo if bar" lines. However, looking at what else can enable SND_HDA, I think it should be select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA That still leaves the issue that pci_iomap() on SH should be an empty stub if PCI is not available, like on other architectures. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 8:01 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-06 8:01 UTC (permalink / raw) To: Randy Dunlap Cc: Linux Kernel Mailing List, linux-s390, ALSA Development Mailing List, linuxppc-dev, Linux-sh list Hi Randy, On Tue, Nov 6, 2018 at 2:06 AM Randy Dunlap <rdunlap@infradead.org> wrote: > On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > >> Below is the list of build error/warning regressions/improvements in > >> v4.20-rc1[1] compared to v4.19[2]. > >> > >> Summarized: > >> - build errors: +3/-0 > >> - build warnings: +449/-2712 > >> > >> Happy fixing! ;-) > >> > >> Thanks to the linux-next team for providing the build service. > >> > >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > >> > >> > >> *** ERRORS *** > >> > >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > > > sh4-all{mod,yes}config > > > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > > pci_iomap() on SH") > > is not sufficient? > > Different problem. This is about "select": > > config SND_SOC_ALL_CODECS > tristate "Build all ASoC CODEC drivers" > > That enables (sets): > select SND_SOC_HDAC_HDA > which selects SND_HDA even though CONFIG_PCI is not enabled. > > After SND_HDA is selected (above), the Kconfig symbols in > sound/pci/hda/Kconfig are available for enabling, so > SND_HDA_CODEC_CA0132 is enabled but will not build. Thanks for looking into this! > One simple solution (but possibly too naive) is: > > --- > sound/soc/codecs/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig > +++ lnx-420-rc1/sound/soc/codecs/Kconfig > @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS > select SND_SOC_ES7241 > select SND_SOC_GTM601 > select SND_SOC_HDAC_HDMI > - select SND_SOC_HDAC_HDA > + select SND_SOC_HDAC_HDA if PCI > select SND_SOC_ICS43432 > select SND_SOC_INNO_RK3036 > select SND_SOC_ISABELLE if I2C I guess that will work. There are already plenty of "select foo if bar" lines. However, looking at what else can enable SND_HDA, I think it should be select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA That still leaves the issue that pci_iomap() on SH should be an empty stub if PCI is not available, like on other architectures. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) 2018-11-06 8:01 ` Geert Uytterhoeven (?) @ 2018-11-06 14:56 ` Pierre-Louis Bossart -1 siblings, 0 replies; 22+ messages in thread From: Pierre-Louis Bossart @ 2018-11-06 14:56 UTC (permalink / raw) To: Geert Uytterhoeven, Randy Dunlap Cc: linux-s390, ALSA Development Mailing List, linuxppc-dev, Linux Kernel Mailing List, Linux-sh list, Mark Brown >>>> >>>> *** ERRORS *** >>>> >>>> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 >>> sh4-all{mod,yes}config >>> >>> Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of >>> pci_iomap() on SH") >>> is not sufficient? >> Different problem. This is about "select": >> >> config SND_SOC_ALL_CODECS >> tristate "Build all ASoC CODEC drivers" >> >> That enables (sets): >> select SND_SOC_HDAC_HDA >> which selects SND_HDA even though CONFIG_PCI is not enabled. >> >> After SND_HDA is selected (above), the Kconfig symbols in >> sound/pci/hda/Kconfig are available for enabling, so >> SND_HDA_CODEC_CA0132 is enabled but will not build. > Thanks for looking into this! > >> One simple solution (but possibly too naive) is: >> >> --- >> sound/soc/codecs/Kconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig >> +++ lnx-420-rc1/sound/soc/codecs/Kconfig >> @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS >> select SND_SOC_ES7241 >> select SND_SOC_GTM601 >> select SND_SOC_HDAC_HDMI >> - select SND_SOC_HDAC_HDA >> + select SND_SOC_HDAC_HDA if PCI >> select SND_SOC_ICS43432 >> select SND_SOC_INNO_RK3036 >> select SND_SOC_ISABELLE if I2C > I guess that will work. There are already plenty of "select foo if bar" lines. > However, looking at what else can enable SND_HDA, I think it should be > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA This codec can only be used by the Skylake driver (and the upcoming SOF one). For Tegra this module will never be used unless they follow the same path of enabling ASoC to deal with the HDaudio codecs instead of the legacy. Likewise HDAC_HDMI will only work on Intel platforms for now. > > That still leaves the issue that pci_iomap() on SH should be an empty stub if > PCI is not available, like on other architectures. I thought Mark Brown provided a fix to SH maintainers? ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 14:56 ` Pierre-Louis Bossart 0 siblings, 0 replies; 22+ messages in thread From: Pierre-Louis Bossart @ 2018-11-06 14:56 UTC (permalink / raw) To: Geert Uytterhoeven, Randy Dunlap Cc: linux-s390, ALSA Development Mailing List, Linux-sh list, Linux Kernel Mailing List, Mark Brown, linuxppc-dev >>>> >>>> *** ERRORS *** >>>> >>>> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 >>> sh4-all{mod,yes}config >>> >>> Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of >>> pci_iomap() on SH") >>> is not sufficient? >> Different problem. This is about "select": >> >> config SND_SOC_ALL_CODECS >> tristate "Build all ASoC CODEC drivers" >> >> That enables (sets): >> select SND_SOC_HDAC_HDA >> which selects SND_HDA even though CONFIG_PCI is not enabled. >> >> After SND_HDA is selected (above), the Kconfig symbols in >> sound/pci/hda/Kconfig are available for enabling, so >> SND_HDA_CODEC_CA0132 is enabled but will not build. > Thanks for looking into this! > >> One simple solution (but possibly too naive) is: >> >> --- >> sound/soc/codecs/Kconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig >> +++ lnx-420-rc1/sound/soc/codecs/Kconfig >> @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS >> select SND_SOC_ES7241 >> select SND_SOC_GTM601 >> select SND_SOC_HDAC_HDMI >> - select SND_SOC_HDAC_HDA >> + select SND_SOC_HDAC_HDA if PCI >> select SND_SOC_ICS43432 >> select SND_SOC_INNO_RK3036 >> select SND_SOC_ISABELLE if I2C > I guess that will work. There are already plenty of "select foo if bar" lines. > However, looking at what else can enable SND_HDA, I think it should be > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA This codec can only be used by the Skylake driver (and the upcoming SOF one). For Tegra this module will never be used unless they follow the same path of enabling ASoC to deal with the HDaudio codecs instead of the legacy. Likewise HDAC_HDMI will only work on Intel platforms for now. > > That still leaves the issue that pci_iomap() on SH should be an empty stub if > PCI is not available, like on other architectures. I thought Mark Brown provided a fix to SH maintainers? ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 14:56 ` Pierre-Louis Bossart 0 siblings, 0 replies; 22+ messages in thread From: Pierre-Louis Bossart @ 2018-11-06 14:56 UTC (permalink / raw) To: Geert Uytterhoeven, Randy Dunlap Cc: linux-s390, ALSA Development Mailing List, linuxppc-dev, Linux Kernel Mailing List, Linux-sh list, Mark Brown >>>> >>>> *** ERRORS *** >>>> >>>> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 >>> sh4-all{mod,yes}config >>> >>> Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of >>> pci_iomap() on SH") >>> is not sufficient? >> Different problem. This is about "select": >> >> config SND_SOC_ALL_CODECS >> tristate "Build all ASoC CODEC drivers" >> >> That enables (sets): >> select SND_SOC_HDAC_HDA >> which selects SND_HDA even though CONFIG_PCI is not enabled. >> >> After SND_HDA is selected (above), the Kconfig symbols in >> sound/pci/hda/Kconfig are available for enabling, so >> SND_HDA_CODEC_CA0132 is enabled but will not build. > Thanks for looking into this! > >> One simple solution (but possibly too naive) is: >> >> --- >> sound/soc/codecs/Kconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig >> +++ lnx-420-rc1/sound/soc/codecs/Kconfig >> @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS >> select SND_SOC_ES7241 >> select SND_SOC_GTM601 >> select SND_SOC_HDAC_HDMI >> - select SND_SOC_HDAC_HDA >> + select SND_SOC_HDAC_HDA if PCI >> select SND_SOC_ICS43432 >> select SND_SOC_INNO_RK3036 >> select SND_SOC_ISABELLE if I2C > I guess that will work. There are already plenty of "select foo if bar" lines. > However, looking at what else can enable SND_HDA, I think it should be > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA This codec can only be used by the Skylake driver (and the upcoming SOF one). For Tegra this module will never be used unless they follow the same path of enabling ASoC to deal with the HDaudio codecs instead of the legacy. Likewise HDAC_HDMI will only work on Intel platforms for now. > > That still leaves the issue that pci_iomap() on SH should be an empty stub if > PCI is not available, like on other architectures. I thought Mark Brown provided a fix to SH maintainers? ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) 2018-11-06 14:56 ` Pierre-Louis Bossart (?) @ 2018-11-06 15:25 ` Geert Uytterhoeven -1 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-06 15:25 UTC (permalink / raw) To: Pierre-Louis Bossart Cc: Randy Dunlap, linux-s390, ALSA Development Mailing List, linuxppc-dev, Linux Kernel Mailing List, Linux-sh list, Mark Brown, Andrew Morton Hi Pierre, On Tue, Nov 6, 2018 at 3:56 PM Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> wrote: > >>>> > >>>> *** ERRORS *** > >>>> > >>>> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > >>> sh4-all{mod,yes}config > >>> > >>> Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > >>> pci_iomap() on SH") > >>> is not sufficient? > >> Different problem. This is about "select": > >> > >> config SND_SOC_ALL_CODECS > >> tristate "Build all ASoC CODEC drivers" > >> > >> That enables (sets): > >> select SND_SOC_HDAC_HDA > >> which selects SND_HDA even though CONFIG_PCI is not enabled. > >> > >> After SND_HDA is selected (above), the Kconfig symbols in > >> sound/pci/hda/Kconfig are available for enabling, so > >> SND_HDA_CODEC_CA0132 is enabled but will not build. > > Thanks for looking into this! > > > >> One simple solution (but possibly too naive) is: > >> > >> --- > >> sound/soc/codecs/Kconfig | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig > >> +++ lnx-420-rc1/sound/soc/codecs/Kconfig > >> @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS > >> select SND_SOC_ES7241 > >> select SND_SOC_GTM601 > >> select SND_SOC_HDAC_HDMI > >> - select SND_SOC_HDAC_HDA > >> + select SND_SOC_HDAC_HDA if PCI > >> select SND_SOC_ICS43432 > >> select SND_SOC_INNO_RK3036 > >> select SND_SOC_ISABELLE if I2C > > I guess that will work. There are already plenty of "select foo if bar" lines. > > However, looking at what else can enable SND_HDA, I think it should be > > > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA > > This codec can only be used by the Skylake driver (and the upcoming SOF > one). For Tegra this module will never be used unless they follow the > same path of enabling ASoC to deal with the HDaudio codecs instead of > the legacy. > > Likewise HDAC_HDMI will only work on Intel platforms for now. > > > > > That still leaves the issue that pci_iomap() on SH should be an empty stub if > > PCI is not available, like on other architectures. > > I thought Mark Brown provided a fix to SH maintainers? Indeed, https://patchwork.kernel.org/patch/10597409/ As usual, no response from the J^HSH maintainers :-( Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 15:25 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-06 15:25 UTC (permalink / raw) To: Pierre-Louis Bossart Cc: linux-s390, ALSA Development Mailing List, Linux-sh list, Randy Dunlap, Linux Kernel Mailing List, Mark Brown, Andrew Morton, linuxppc-dev Hi Pierre, On Tue, Nov 6, 2018 at 3:56 PM Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> wrote: > >>>> > >>>> *** ERRORS *** > >>>> > >>>> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > >>> sh4-all{mod,yes}config > >>> > >>> Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > >>> pci_iomap() on SH") > >>> is not sufficient? > >> Different problem. This is about "select": > >> > >> config SND_SOC_ALL_CODECS > >> tristate "Build all ASoC CODEC drivers" > >> > >> That enables (sets): > >> select SND_SOC_HDAC_HDA > >> which selects SND_HDA even though CONFIG_PCI is not enabled. > >> > >> After SND_HDA is selected (above), the Kconfig symbols in > >> sound/pci/hda/Kconfig are available for enabling, so > >> SND_HDA_CODEC_CA0132 is enabled but will not build. > > Thanks for looking into this! > > > >> One simple solution (but possibly too naive) is: > >> > >> --- > >> sound/soc/codecs/Kconfig | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig > >> +++ lnx-420-rc1/sound/soc/codecs/Kconfig > >> @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS > >> select SND_SOC_ES7241 > >> select SND_SOC_GTM601 > >> select SND_SOC_HDAC_HDMI > >> - select SND_SOC_HDAC_HDA > >> + select SND_SOC_HDAC_HDA if PCI > >> select SND_SOC_ICS43432 > >> select SND_SOC_INNO_RK3036 > >> select SND_SOC_ISABELLE if I2C > > I guess that will work. There are already plenty of "select foo if bar" lines. > > However, looking at what else can enable SND_HDA, I think it should be > > > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA > > This codec can only be used by the Skylake driver (and the upcoming SOF > one). For Tegra this module will never be used unless they follow the > same path of enabling ASoC to deal with the HDaudio codecs instead of > the legacy. > > Likewise HDAC_HDMI will only work on Intel platforms for now. > > > > > That still leaves the issue that pci_iomap() on SH should be an empty stub if > > PCI is not available, like on other architectures. > > I thought Mark Brown provided a fix to SH maintainers? Indeed, https://patchwork.kernel.org/patch/10597409/ As usual, no response from the J^HSH maintainers :-( Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 15:25 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-06 15:25 UTC (permalink / raw) To: Pierre-Louis Bossart Cc: Randy Dunlap, linux-s390, ALSA Development Mailing List, linuxppc-dev, Linux Kernel Mailing List, Linux-sh list, Mark Brown, Andrew Morton Hi Pierre, On Tue, Nov 6, 2018 at 3:56 PM Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> wrote: > >>>> > >>>> *** ERRORS *** > >>>> > >>>> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > >>> sh4-all{mod,yes}config > >>> > >>> Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > >>> pci_iomap() on SH") > >>> is not sufficient? > >> Different problem. This is about "select": > >> > >> config SND_SOC_ALL_CODECS > >> tristate "Build all ASoC CODEC drivers" > >> > >> That enables (sets): > >> select SND_SOC_HDAC_HDA > >> which selects SND_HDA even though CONFIG_PCI is not enabled. > >> > >> After SND_HDA is selected (above), the Kconfig symbols in > >> sound/pci/hda/Kconfig are available for enabling, so > >> SND_HDA_CODEC_CA0132 is enabled but will not build. > > Thanks for looking into this! > > > >> One simple solution (but possibly too naive) is: > >> > >> --- > >> sound/soc/codecs/Kconfig | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> --- lnx-420-rc1.orig/sound/soc/codecs/Kconfig > >> +++ lnx-420-rc1/sound/soc/codecs/Kconfig > >> @@ -82,7 +82,7 @@ config SND_SOC_ALL_CODECS > >> select SND_SOC_ES7241 > >> select SND_SOC_GTM601 > >> select SND_SOC_HDAC_HDMI > >> - select SND_SOC_HDAC_HDA > >> + select SND_SOC_HDAC_HDA if PCI > >> select SND_SOC_ICS43432 > >> select SND_SOC_INNO_RK3036 > >> select SND_SOC_ISABELLE if I2C > > I guess that will work. There are already plenty of "select foo if bar" lines. > > However, looking at what else can enable SND_HDA, I think it should be > > > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA > > This codec can only be used by the Skylake driver (and the upcoming SOF > one). For Tegra this module will never be used unless they follow the > same path of enabling ASoC to deal with the HDaudio codecs instead of > the legacy. > > Likewise HDAC_HDMI will only work on Intel platforms for now. > > > > > That still leaves the issue that pci_iomap() on SH should be an empty stub if > > PCI is not available, like on other architectures. > > I thought Mark Brown provided a fix to SH maintainers? Indeed, https://patchwork.kernel.org/patch/10597409/ As usual, no response from the J^HSH maintainers :-( Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) 2018-11-06 14:56 ` Pierre-Louis Bossart (?) (?) @ 2018-11-06 15:46 ` Mark Brown -1 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2018-11-06 15:46 UTC (permalink / raw) To: Pierre-Louis Bossart Cc: linux-s390, ALSA Development Mailing List, Linux-sh list, Randy Dunlap, Linux Kernel Mailing List, Geert Uytterhoeven, linuxppc-dev [-- Attachment #1: Type: text/plain, Size: 744 bytes --] On Tue, Nov 06, 2018 at 08:56:39AM -0600, Pierre-Louis Bossart wrote: > > However, looking at what else can enable SND_HDA, I think it should be > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA > This codec can only be used by the Skylake driver (and the upcoming SOF > one). For Tegra this module will never be used unless they follow the same > path of enabling ASoC to deal with the HDaudio codecs instead of the legacy. > Likewise HDAC_HDMI will only work on Intel platforms for now. _ALL_CODECS is a build coverage option, not a useful configuration option. We should be building things in as many different configurations as possible so that it's easier for people to get build coverage when making subsystem wide changes. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 15:46 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2018-11-06 15:46 UTC (permalink / raw) To: Pierre-Louis Bossart Cc: linux-s390, ALSA Development Mailing List, Linux-sh list, Randy Dunlap, Linux Kernel Mailing List, Geert Uytterhoeven, linuxppc-dev [-- Attachment #1: Type: text/plain, Size: 744 bytes --] On Tue, Nov 06, 2018 at 08:56:39AM -0600, Pierre-Louis Bossart wrote: > > However, looking at what else can enable SND_HDA, I think it should be > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA > This codec can only be used by the Skylake driver (and the upcoming SOF > one). For Tegra this module will never be used unless they follow the same > path of enabling ASoC to deal with the HDaudio codecs instead of the legacy. > Likewise HDAC_HDMI will only work on Intel platforms for now. _ALL_CODECS is a build coverage option, not a useful configuration option. We should be building things in as many different configurations as possible so that it's easier for people to get build coverage when making subsystem wide changes. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 15:46 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2018-11-06 15:46 UTC (permalink / raw) To: Pierre-Louis Bossart Cc: linux-s390, ALSA Development Mailing List, Linux-sh list, Randy Dunlap, Linux Kernel Mailing List, Geert Uytterhoeven, linuxppc-dev [-- Attachment #1.1: Type: text/plain, Size: 744 bytes --] On Tue, Nov 06, 2018 at 08:56:39AM -0600, Pierre-Louis Bossart wrote: > > However, looking at what else can enable SND_HDA, I think it should be > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA > This codec can only be used by the Skylake driver (and the upcoming SOF > one). For Tegra this module will never be used unless they follow the same > path of enabling ASoC to deal with the HDaudio codecs instead of the legacy. > Likewise HDAC_HDMI will only work on Intel platforms for now. _ALL_CODECS is a build coverage option, not a useful configuration option. We should be building things in as many different configurations as possible so that it's easier for people to get build coverage when making subsystem wide changes. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] [-- Attachment #2: Type: text/plain, Size: 0 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 15:46 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2018-11-06 15:46 UTC (permalink / raw) To: Pierre-Louis Bossart Cc: Geert Uytterhoeven, Randy Dunlap, linux-s390, ALSA Development Mailing List, linuxppc-dev, Linux Kernel Mailing List, Linux-sh list [-- Attachment #1: Type: text/plain, Size: 744 bytes --] On Tue, Nov 06, 2018 at 08:56:39AM -0600, Pierre-Louis Bossart wrote: > > However, looking at what else can enable SND_HDA, I think it should be > > select SND_SOC_HDAC_HDA if SND_PCI || ARCH_TEGRA > This codec can only be used by the Skylake driver (and the upcoming SOF > one). For Tegra this module will never be used unless they follow the same > path of enabling ASoC to deal with the HDaudio codecs instead of the legacy. > Likewise HDAC_HDMI will only work on Intel platforms for now. _ALL_CODECS is a build coverage option, not a useful configuration option. We should be building things in as many different configurations as possible so that it's easier for people to get build coverage when making subsystem wide changes. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) 2018-11-06 1:04 ` Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) Randy Dunlap @ 2018-11-06 16:18 ` Takashi Iwai 2018-11-06 16:18 ` Takashi Iwai 1 sibling, 0 replies; 22+ messages in thread From: Takashi Iwai @ 2018-11-06 16:18 UTC (permalink / raw) To: Randy Dunlap Cc: Geert Uytterhoeven, Linux Kernel Mailing List, linux-s390, ALSA Development Mailing List, linuxppc-dev On Tue, 06 Nov 2018 02:04:47 +0100, Randy Dunlap wrote: > > On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > >> Below is the list of build error/warning regressions/improvements in > >> v4.20-rc1[1] compared to v4.19[2]. > >> > >> Summarized: > >> - build errors: +3/-0 > >> - build warnings: +449/-2712 > >> > >> Happy fixing! ;-) > >> > >> Thanks to the linux-next team for providing the build service. > >> > >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > >> > >> > >> *** ERRORS *** > >> > >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > > > sh4-all{mod,yes}config > > > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > > pci_iomap() on SH") > > is not sufficient? > > Different problem. This is about "select": > > config SND_SOC_ALL_CODECS > tristate "Build all ASoC CODEC drivers" > > That enables (sets): > select SND_SOC_HDAC_HDA > which selects SND_HDA even though CONFIG_PCI is not enabled. Actually it is OK to enable CONFIG_SND_HDA_CODEC_CA0132 without CONFIG_PCI. IIRC, there was a system like that, too. The commit above should have covered the build failure on SH, but apparently isn't enough for some arch setups, as it seems. The cause is clear now: pci_iomap() is defined in asm-generic/pci_iomap.h only when CONFIG_GENERIC_PCI_IOMAP is defined. Including asm/io.h doesn't help unless CONFIG_PCI is set. Below is a quick fix for this. thanks, Takashi -- 8< -- From: Takashi Iwai <tiwai@suse.de> Subject: [PATCH] ALSA: hda/ca0132 - Yet more fix on build breakage without PCI support The recent change in CA0132 codec driver for supporting more Creative boards includes the pci_iomap() call to get the extra register accesses. This is supposed to work on all archs and setups, by the implicit assumption that every arch would provide a dummy function returning NULL when no PCI is available. But the reality bites, of course; as Geert's regular build test shows, some configs (at least SH4 without CONFIG_PCI) leads to a build error due to the implicit function declaration. So this is another attempt to fix the issue: now we add an ifdef CONFIG_PCI line, so that pci_iomap() won't be called unless PCI is really usable. This should fall back to the standard quirk again with a warning. Fixes: d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of pci_iomap() on SH") Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> Cc: <stable@vger.kernel.org> Signed-off-by: Takashi Iwai <tiwai@suse.de> --- sound/pci/hda/patch_ca0132.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/sound/pci/hda/patch_ca0132.c b/sound/pci/hda/patch_ca0132.c index 0a24037184c3..9ed808b45e75 100644 --- a/sound/pci/hda/patch_ca0132.c +++ b/sound/pci/hda/patch_ca0132.c @@ -8796,7 +8796,13 @@ static int patch_ca0132(struct hda_codec *codec) } if (spec->use_pci_mmio) { + /* + * ifdef below needed due to lack of pci_iomap() decleration + * for some archs when no PCI is defined + */ +#ifdef CONFIG_PCI spec->mem_base = pci_iomap(codec->bus->pci, 2, 0xC20); +#endif if (spec->mem_base == NULL) { codec_warn(codec, "pci_iomap failed! Setting quirk to QUIRK_NONE."); spec->quirk = QUIRK_NONE; -- 2.19.1 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-06 16:18 ` Takashi Iwai 0 siblings, 0 replies; 22+ messages in thread From: Takashi Iwai @ 2018-11-06 16:18 UTC (permalink / raw) To: Randy Dunlap Cc: linux-s390, ALSA Development Mailing List, linuxppc-dev, Geert Uytterhoeven, Linux Kernel Mailing List On Tue, 06 Nov 2018 02:04:47 +0100, Randy Dunlap wrote: > > On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > >> Below is the list of build error/warning regressions/improvements in > >> v4.20-rc1[1] compared to v4.19[2]. > >> > >> Summarized: > >> - build errors: +3/-0 > >> - build warnings: +449/-2712 > >> > >> Happy fixing! ;-) > >> > >> Thanks to the linux-next team for providing the build service. > >> > >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > >> > >> > >> *** ERRORS *** > >> > >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > > > sh4-all{mod,yes}config > > > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > > pci_iomap() on SH") > > is not sufficient? > > Different problem. This is about "select": > > config SND_SOC_ALL_CODECS > tristate "Build all ASoC CODEC drivers" > > That enables (sets): > select SND_SOC_HDAC_HDA > which selects SND_HDA even though CONFIG_PCI is not enabled. Actually it is OK to enable CONFIG_SND_HDA_CODEC_CA0132 without CONFIG_PCI. IIRC, there was a system like that, too. The commit above should have covered the build failure on SH, but apparently isn't enough for some arch setups, as it seems. The cause is clear now: pci_iomap() is defined in asm-generic/pci_iomap.h only when CONFIG_GENERIC_PCI_IOMAP is defined. Including asm/io.h doesn't help unless CONFIG_PCI is set. Below is a quick fix for this. thanks, Takashi -- 8< -- From: Takashi Iwai <tiwai@suse.de> Subject: [PATCH] ALSA: hda/ca0132 - Yet more fix on build breakage without PCI support The recent change in CA0132 codec driver for supporting more Creative boards includes the pci_iomap() call to get the extra register accesses. This is supposed to work on all archs and setups, by the implicit assumption that every arch would provide a dummy function returning NULL when no PCI is available. But the reality bites, of course; as Geert's regular build test shows, some configs (at least SH4 without CONFIG_PCI) leads to a build error due to the implicit function declaration. So this is another attempt to fix the issue: now we add an ifdef CONFIG_PCI line, so that pci_iomap() won't be called unless PCI is really usable. This should fall back to the standard quirk again with a warning. Fixes: d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of pci_iomap() on SH") Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> Cc: <stable@vger.kernel.org> Signed-off-by: Takashi Iwai <tiwai@suse.de> --- sound/pci/hda/patch_ca0132.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/sound/pci/hda/patch_ca0132.c b/sound/pci/hda/patch_ca0132.c index 0a24037184c3..9ed808b45e75 100644 --- a/sound/pci/hda/patch_ca0132.c +++ b/sound/pci/hda/patch_ca0132.c @@ -8796,7 +8796,13 @@ static int patch_ca0132(struct hda_codec *codec) } if (spec->use_pci_mmio) { + /* + * ifdef below needed due to lack of pci_iomap() decleration + * for some archs when no PCI is defined + */ +#ifdef CONFIG_PCI spec->mem_base = pci_iomap(codec->bus->pci, 2, 0xC20); +#endif if (spec->mem_base == NULL) { codec_warn(codec, "pci_iomap failed! Setting quirk to QUIRK_NONE."); spec->quirk = QUIRK_NONE; -- 2.19.1 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) 2018-11-06 16:18 ` Takashi Iwai @ 2018-11-07 8:44 ` Geert Uytterhoeven -1 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-07 8:44 UTC (permalink / raw) To: Takashi Iwai Cc: Randy Dunlap, Linux Kernel Mailing List, linux-s390, ALSA Development Mailing List, linuxppc-dev Hi Iwai-san, On Tue, Nov 6, 2018 at 5:18 PM Takashi Iwai <tiwai@suse.de> wrote: > On Tue, 06 Nov 2018 02:04:47 +0100, > Randy Dunlap wrote: > > > > On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > > > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > >> Below is the list of build error/warning regressions/improvements in > > >> v4.20-rc1[1] compared to v4.19[2]. > > >> > > >> Summarized: > > >> - build errors: +3/-0 > > >> - build warnings: +449/-2712 > > >> > > >> Happy fixing! ;-) > > >> > > >> Thanks to the linux-next team for providing the build service. > > >> > > >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > > >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > > >> > > >> > > >> *** ERRORS *** > > >> > > >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > > > > > sh4-all{mod,yes}config > > > > > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > > > pci_iomap() on SH") > > > is not sufficient? > > > > Different problem. This is about "select": > > > > config SND_SOC_ALL_CODECS > > tristate "Build all ASoC CODEC drivers" > > > > That enables (sets): > > select SND_SOC_HDAC_HDA > > which selects SND_HDA even though CONFIG_PCI is not enabled. > > Actually it is OK to enable CONFIG_SND_HDA_CODEC_CA0132 without > CONFIG_PCI. IIRC, there was a system like that, too. > The commit above should have covered the build failure on SH, but > apparently isn't enough for some arch setups, as it seems. > > The cause is clear now: pci_iomap() is defined in > asm-generic/pci_iomap.h only when CONFIG_GENERIC_PCI_IOMAP is > defined. Including asm/io.h doesn't help unless CONFIG_PCI is set. > > Below is a quick fix for this. > > > thanks, > > Takashi > > -- 8< -- > > From: Takashi Iwai <tiwai@suse.de> > Subject: [PATCH] ALSA: hda/ca0132 - Yet more fix on build breakage without PCI > support > > The recent change in CA0132 codec driver for supporting more > Creative boards includes the pci_iomap() call to get the extra > register accesses. This is supposed to work on all archs and setups, > by the implicit assumption that every arch would provide a dummy > function returning NULL when no PCI is available. But the reality > bites, of course; as Geert's regular build test shows, some configs > (at least SH4 without CONFIG_PCI) leads to a build error due to the > implicit function declaration. > > So this is another attempt to fix the issue: now we add an ifdef > CONFIG_PCI line, so that pci_iomap() won't be called unless PCI is > really usable. This should fall back to the standard quirk again with > a warning. > > Fixes: d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of pci_iomap() on SH") > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > Cc: <stable@vger.kernel.org> > Signed-off-by: Takashi Iwai <tiwai@suse.de> Thanks for your patch! > --- a/sound/pci/hda/patch_ca0132.c > +++ b/sound/pci/hda/patch_ca0132.c > @@ -8796,7 +8796,13 @@ static int patch_ca0132(struct hda_codec *codec) > } > > if (spec->use_pci_mmio) { > + /* > + * ifdef below needed due to lack of pci_iomap() decleration > + * for some archs when no PCI is defined > + */ > +#ifdef CONFIG_PCI > spec->mem_base = pci_iomap(codec->bus->pci, 2, 0xC20); > +#endif I'm sorry, but that is not a proper fix. This should be fixed in the SH-specific code, to behave like other architectures. > if (spec->mem_base == NULL) { > codec_warn(codec, "pci_iomap failed! Setting quirk to QUIRK_NONE."); > spec->quirk = QUIRK_NONE; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-07 8:44 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2018-11-07 8:44 UTC (permalink / raw) To: Takashi Iwai Cc: linux-s390, ALSA Development Mailing List, Randy Dunlap, linuxppc-dev, Linux Kernel Mailing List Hi Iwai-san, On Tue, Nov 6, 2018 at 5:18 PM Takashi Iwai <tiwai@suse.de> wrote: > On Tue, 06 Nov 2018 02:04:47 +0100, > Randy Dunlap wrote: > > > > On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > > > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > >> Below is the list of build error/warning regressions/improvements in > > >> v4.20-rc1[1] compared to v4.19[2]. > > >> > > >> Summarized: > > >> - build errors: +3/-0 > > >> - build warnings: +449/-2712 > > >> > > >> Happy fixing! ;-) > > >> > > >> Thanks to the linux-next team for providing the build service. > > >> > > >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > > >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > > >> > > >> > > >> *** ERRORS *** > > >> > > >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > > > > > sh4-all{mod,yes}config > > > > > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > > > pci_iomap() on SH") > > > is not sufficient? > > > > Different problem. This is about "select": > > > > config SND_SOC_ALL_CODECS > > tristate "Build all ASoC CODEC drivers" > > > > That enables (sets): > > select SND_SOC_HDAC_HDA > > which selects SND_HDA even though CONFIG_PCI is not enabled. > > Actually it is OK to enable CONFIG_SND_HDA_CODEC_CA0132 without > CONFIG_PCI. IIRC, there was a system like that, too. > The commit above should have covered the build failure on SH, but > apparently isn't enough for some arch setups, as it seems. > > The cause is clear now: pci_iomap() is defined in > asm-generic/pci_iomap.h only when CONFIG_GENERIC_PCI_IOMAP is > defined. Including asm/io.h doesn't help unless CONFIG_PCI is set. > > Below is a quick fix for this. > > > thanks, > > Takashi > > -- 8< -- > > From: Takashi Iwai <tiwai@suse.de> > Subject: [PATCH] ALSA: hda/ca0132 - Yet more fix on build breakage without PCI > support > > The recent change in CA0132 codec driver for supporting more > Creative boards includes the pci_iomap() call to get the extra > register accesses. This is supposed to work on all archs and setups, > by the implicit assumption that every arch would provide a dummy > function returning NULL when no PCI is available. But the reality > bites, of course; as Geert's regular build test shows, some configs > (at least SH4 without CONFIG_PCI) leads to a build error due to the > implicit function declaration. > > So this is another attempt to fix the issue: now we add an ifdef > CONFIG_PCI line, so that pci_iomap() won't be called unless PCI is > really usable. This should fall back to the standard quirk again with > a warning. > > Fixes: d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of pci_iomap() on SH") > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > Cc: <stable@vger.kernel.org> > Signed-off-by: Takashi Iwai <tiwai@suse.de> Thanks for your patch! > --- a/sound/pci/hda/patch_ca0132.c > +++ b/sound/pci/hda/patch_ca0132.c > @@ -8796,7 +8796,13 @@ static int patch_ca0132(struct hda_codec *codec) > } > > if (spec->use_pci_mmio) { > + /* > + * ifdef below needed due to lack of pci_iomap() decleration > + * for some archs when no PCI is defined > + */ > +#ifdef CONFIG_PCI > spec->mem_base = pci_iomap(codec->bus->pci, 2, 0xC20); > +#endif I'm sorry, but that is not a proper fix. This should be fixed in the SH-specific code, to behave like other architectures. > if (spec->mem_base == NULL) { > codec_warn(codec, "pci_iomap failed! Setting quirk to QUIRK_NONE."); > spec->quirk = QUIRK_NONE; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) 2018-11-07 8:44 ` Geert Uytterhoeven @ 2018-11-07 9:41 ` Takashi Iwai -1 siblings, 0 replies; 22+ messages in thread From: Takashi Iwai @ 2018-11-07 9:41 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Randy Dunlap, Linux Kernel Mailing List, linux-s390, ALSA Development Mailing List, linuxppc-dev On Wed, 07 Nov 2018 09:44:25 +0100, Geert Uytterhoeven wrote: > > Hi Iwai-san, > > On Tue, Nov 6, 2018 at 5:18 PM Takashi Iwai <tiwai@suse.de> wrote: > > On Tue, 06 Nov 2018 02:04:47 +0100, > > Randy Dunlap wrote: > > > > > > On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > > > > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > > >> Below is the list of build error/warning regressions/improvements in > > > >> v4.20-rc1[1] compared to v4.19[2]. > > > >> > > > >> Summarized: > > > >> - build errors: +3/-0 > > > >> - build warnings: +449/-2712 > > > >> > > > >> Happy fixing! ;-) > > > >> > > > >> Thanks to the linux-next team for providing the build service. > > > >> > > > >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > > > >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > > > >> > > > >> > > > >> *** ERRORS *** > > > >> > > > >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > > > > > > > sh4-all{mod,yes}config > > > > > > > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > > > > pci_iomap() on SH") > > > > is not sufficient? > > > > > > Different problem. This is about "select": > > > > > > config SND_SOC_ALL_CODECS > > > tristate "Build all ASoC CODEC drivers" > > > > > > That enables (sets): > > > select SND_SOC_HDAC_HDA > > > which selects SND_HDA even though CONFIG_PCI is not enabled. > > > > Actually it is OK to enable CONFIG_SND_HDA_CODEC_CA0132 without > > CONFIG_PCI. IIRC, there was a system like that, too. > > The commit above should have covered the build failure on SH, but > > apparently isn't enough for some arch setups, as it seems. > > > > The cause is clear now: pci_iomap() is defined in > > asm-generic/pci_iomap.h only when CONFIG_GENERIC_PCI_IOMAP is > > defined. Including asm/io.h doesn't help unless CONFIG_PCI is set. > > > > Below is a quick fix for this. > > > > > > thanks, > > > > Takashi > > > > -- 8< -- > > > > From: Takashi Iwai <tiwai@suse.de> > > Subject: [PATCH] ALSA: hda/ca0132 - Yet more fix on build breakage without PCI > > support > > > > The recent change in CA0132 codec driver for supporting more > > Creative boards includes the pci_iomap() call to get the extra > > register accesses. This is supposed to work on all archs and setups, > > by the implicit assumption that every arch would provide a dummy > > function returning NULL when no PCI is available. But the reality > > bites, of course; as Geert's regular build test shows, some configs > > (at least SH4 without CONFIG_PCI) leads to a build error due to the > > implicit function declaration. > > > > So this is another attempt to fix the issue: now we add an ifdef > > CONFIG_PCI line, so that pci_iomap() won't be called unless PCI is > > really usable. This should fall back to the standard quirk again with > > a warning. > > > > Fixes: d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of pci_iomap() on SH") > > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > > Cc: <stable@vger.kernel.org> > > Signed-off-by: Takashi Iwai <tiwai@suse.de> > > Thanks for your patch! > > > --- a/sound/pci/hda/patch_ca0132.c > > +++ b/sound/pci/hda/patch_ca0132.c > > @@ -8796,7 +8796,13 @@ static int patch_ca0132(struct hda_codec *codec) > > } > > > > if (spec->use_pci_mmio) { > > + /* > > + * ifdef below needed due to lack of pci_iomap() decleration > > + * for some archs when no PCI is defined > > + */ > > +#ifdef CONFIG_PCI > > spec->mem_base = pci_iomap(codec->bus->pci, 2, 0xC20); > > +#endif > > I'm sorry, but that is not a proper fix. > This should be fixed in the SH-specific code, to behave like other > architectures. OK, this is the thing I didn't want to go deeply, as the problem is messy... The top of iceberg is that asm/pci_iomap.h defines the dummy function of pci_iomap() only when CONFIG_PCI=n && CONFIG_GENERIC_PCI_IOMAP=y. This has been so for long time since the commit 97a29d59fc22 [PARISC] fix compile break caused by iomap: make IOPORT/PCI mapping function s conditional Before that point, the header defined pci_iomap() and pci_ioumap() whenever CONFIG_PCi=n. And, looking at other arch codes, most of them define CONFIG_GENERIC_PCI_IOMAP no matter whether CONFIG_PCI is set or not. Actually this is supposedly OK, as the generic code (lib/pci_iomap.c) contains a large ifdef CONFIG_PCI in it, so essentially it's empty when CONFIG_PCI=n. So, one possible fix would be to convert all fallout archs (alpha, powerpc and sh) to define always CONFIG_GENERIC_PCI_IOMAP. I'm not 100% sure whether this is safe, but we may try. Another fix would be to revert the commit 97a29d59fc22, and let asm/pci_iomap.h providing the dummy functions whenever CONFIG_PCi=n. Along with it, we need to fix the arch side, the duplicated (and useless) definitions in arch/parisc/lib/iomap.c. Through a quick look, arch/sparc also has the pci_iounmap() definition without CONFIG_PCI check, so this needs to be fixed as well. And, what makes the situation messy is that the handling of pci_iounmap() is totally different from pci_iomap(). pci_iomap() is covered by CONFIG_GENERIC_PCI_IOMAP while pci_iounmap() is by CONFIG_GENERIC_IOMAP. And, yet there are multiple dummy function definitions of pci_iounmap(). Once in include/asm-generic/iomap.h, and once in include/asm-generic/io.h. Overall, there are a few rooms of cleanups / fixes, yeah. thanks, Takashi ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [alsa-devel] Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) @ 2018-11-07 9:41 ` Takashi Iwai 0 siblings, 0 replies; 22+ messages in thread From: Takashi Iwai @ 2018-11-07 9:41 UTC (permalink / raw) To: Geert Uytterhoeven Cc: linux-s390, ALSA Development Mailing List, Randy Dunlap, linuxppc-dev, Linux Kernel Mailing List On Wed, 07 Nov 2018 09:44:25 +0100, Geert Uytterhoeven wrote: > > Hi Iwai-san, > > On Tue, Nov 6, 2018 at 5:18 PM Takashi Iwai <tiwai@suse.de> wrote: > > On Tue, 06 Nov 2018 02:04:47 +0100, > > Randy Dunlap wrote: > > > > > > On 11/5/18 2:12 PM, Geert Uytterhoeven wrote: > > > > On Mon, Nov 5, 2018 at 11:07 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > > >> Below is the list of build error/warning regressions/improvements in > > > >> v4.20-rc1[1] compared to v4.19[2]. > > > >> > > > >> Summarized: > > > >> - build errors: +3/-0 > > > >> - build warnings: +449/-2712 > > > >> > > > >> Happy fixing! ;-) > > > >> > > > >> Thanks to the linux-next team for providing the build service. > > > >> > > > >> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/651022382c7f8da46cb4872a545ee1da6d097d2a/ (all 240 configs) > > > >> [2] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d/ (all 240 configs) > > > >> > > > >> > > > >> *** ERRORS *** > > > >> > > > >> + /kisskb/src/sound/pci/hda/patch_ca0132.c: error: implicit declaration of function 'pci_iomap' [-Werror=implicit-function-declaration]: => 8799:3 > > > > > > > > sh4-all{mod,yes}config > > > > > > > > Looks like d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of > > > > pci_iomap() on SH") > > > > is not sufficient? > > > > > > Different problem. This is about "select": > > > > > > config SND_SOC_ALL_CODECS > > > tristate "Build all ASoC CODEC drivers" > > > > > > That enables (sets): > > > select SND_SOC_HDAC_HDA > > > which selects SND_HDA even though CONFIG_PCI is not enabled. > > > > Actually it is OK to enable CONFIG_SND_HDA_CODEC_CA0132 without > > CONFIG_PCI. IIRC, there was a system like that, too. > > The commit above should have covered the build failure on SH, but > > apparently isn't enough for some arch setups, as it seems. > > > > The cause is clear now: pci_iomap() is defined in > > asm-generic/pci_iomap.h only when CONFIG_GENERIC_PCI_IOMAP is > > defined. Including asm/io.h doesn't help unless CONFIG_PCI is set. > > > > Below is a quick fix for this. > > > > > > thanks, > > > > Takashi > > > > -- 8< -- > > > > From: Takashi Iwai <tiwai@suse.de> > > Subject: [PATCH] ALSA: hda/ca0132 - Yet more fix on build breakage without PCI > > support > > > > The recent change in CA0132 codec driver for supporting more > > Creative boards includes the pci_iomap() call to get the extra > > register accesses. This is supposed to work on all archs and setups, > > by the implicit assumption that every arch would provide a dummy > > function returning NULL when no PCI is available. But the reality > > bites, of course; as Geert's regular build test shows, some configs > > (at least SH4 without CONFIG_PCI) leads to a build error due to the > > implicit function declaration. > > > > So this is another attempt to fix the issue: now we add an ifdef > > CONFIG_PCI line, so that pci_iomap() won't be called unless PCI is > > really usable. This should fall back to the standard quirk again with > > a warning. > > > > Fixes: d9b84a15892c0233 ("ALSA: hda: Fix implicit definition of pci_iomap() on SH") > > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > > Cc: <stable@vger.kernel.org> > > Signed-off-by: Takashi Iwai <tiwai@suse.de> > > Thanks for your patch! > > > --- a/sound/pci/hda/patch_ca0132.c > > +++ b/sound/pci/hda/patch_ca0132.c > > @@ -8796,7 +8796,13 @@ static int patch_ca0132(struct hda_codec *codec) > > } > > > > if (spec->use_pci_mmio) { > > + /* > > + * ifdef below needed due to lack of pci_iomap() decleration > > + * for some archs when no PCI is defined > > + */ > > +#ifdef CONFIG_PCI > > spec->mem_base = pci_iomap(codec->bus->pci, 2, 0xC20); > > +#endif > > I'm sorry, but that is not a proper fix. > This should be fixed in the SH-specific code, to behave like other > architectures. OK, this is the thing I didn't want to go deeply, as the problem is messy... The top of iceberg is that asm/pci_iomap.h defines the dummy function of pci_iomap() only when CONFIG_PCI=n && CONFIG_GENERIC_PCI_IOMAP=y. This has been so for long time since the commit 97a29d59fc22 [PARISC] fix compile break caused by iomap: make IOPORT/PCI mapping function s conditional Before that point, the header defined pci_iomap() and pci_ioumap() whenever CONFIG_PCi=n. And, looking at other arch codes, most of them define CONFIG_GENERIC_PCI_IOMAP no matter whether CONFIG_PCI is set or not. Actually this is supposedly OK, as the generic code (lib/pci_iomap.c) contains a large ifdef CONFIG_PCI in it, so essentially it's empty when CONFIG_PCI=n. So, one possible fix would be to convert all fallout archs (alpha, powerpc and sh) to define always CONFIG_GENERIC_PCI_IOMAP. I'm not 100% sure whether this is safe, but we may try. Another fix would be to revert the commit 97a29d59fc22, and let asm/pci_iomap.h providing the dummy functions whenever CONFIG_PCi=n. Along with it, we need to fix the arch side, the duplicated (and useless) definitions in arch/parisc/lib/iomap.c. Through a quick look, arch/sparc also has the pci_iounmap() definition without CONFIG_PCI check, so this needs to be fixed as well. And, what makes the situation messy is that the handling of pci_iounmap() is totally different from pci_iomap(). pci_iomap() is covered by CONFIG_GENERIC_PCI_IOMAP while pci_iounmap() is by CONFIG_GENERIC_IOMAP. And, yet there are multiple dummy function definitions of pci_iounmap(). Once in include/asm-generic/iomap.h, and once in include/asm-generic/io.h. Overall, there are a few rooms of cleanups / fixes, yeah. thanks, Takashi ^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2018-11-07 9:43 UTC | newest] Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-11-05 22:05 Build regressions/improvements in v4.20-rc1 Geert Uytterhoeven 2018-11-05 22:12 ` Geert Uytterhoeven 2018-11-06 1:04 ` Build regressions/improvements in v4.20-rc1 (sound/pci/hda/patch_ca0132.c) Randy Dunlap 2018-11-06 8:01 ` Geert Uytterhoeven 2018-11-06 8:01 ` Geert Uytterhoeven 2018-11-06 8:01 ` Geert Uytterhoeven 2018-11-06 14:56 ` [alsa-devel] " Pierre-Louis Bossart 2018-11-06 14:56 ` Pierre-Louis Bossart 2018-11-06 14:56 ` Pierre-Louis Bossart 2018-11-06 15:25 ` Geert Uytterhoeven 2018-11-06 15:25 ` Geert Uytterhoeven 2018-11-06 15:25 ` Geert Uytterhoeven 2018-11-06 15:46 ` Mark Brown 2018-11-06 15:46 ` Mark Brown 2018-11-06 15:46 ` Mark Brown 2018-11-06 15:46 ` [alsa-devel] " Mark Brown 2018-11-06 16:18 ` Takashi Iwai 2018-11-06 16:18 ` Takashi Iwai 2018-11-07 8:44 ` Geert Uytterhoeven 2018-11-07 8:44 ` Geert Uytterhoeven 2018-11-07 9:41 ` Takashi Iwai 2018-11-07 9:41 ` Takashi Iwai
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.