On 8/28/19 4:38 AM, Cezary Rojewski wrote:
On 2019-08-23 09:27, Cezary Rojewski wrote:
On 2019-08-22 22:44, Pierre-Louis Bossart wrote:
Please checkout sst-acpi.c file and see declaration of legacy platform descriptors. See the names of PCM devices (platform devices) being declared.
what happens in sst-acpi.c stays in sst-acpi.c I don't get how you retrieve the pdata in the machine driver from *another* driver. Different devices, different platform data.
DAI is tied with platform device called "haswell-pcm-audio" whereas machine board is represented by "broadwell-audio" platform deivce. Which part is still unclear?
Did what you ask and must say, results are not entirely unexpected..
Change:
diff --git a/sound/soc/intel/boards/broadwell.c b/sound/soc/intel/boards/broadwell.c index db7e1e87156d..ee52564437c3 100644 --- a/sound/soc/intel/boards/broadwell.c +++ b/sound/soc/intel/boards/broadwell.c @@ -126,7 +126,8 @@ static const struct snd_soc_ops broadwell_rt286_ops = { static int broadwell_rtd_init(struct snd_soc_pcm_runtime *rtd) { struct snd_soc_component *component = snd_soc_rtdcom_lookup(rtd, DRV_NAME); - struct sst_pdata *pdata = dev_get_platdata(component->dev); + struct snd_soc_acpi_mach *mach = dev_get_platdata(component->dev);
that's not what I had in mind, I was talking about using rtd->card->dev and point to the same argument that is passed during the probe.
Anyways it's not much better so let's forget about it.
+ struct sst_pdata *pdata = mach->pdata; struct sst_hsw *broadwell = pdata->dsp; int ret;
Generates:
[ 24.841747] hsw-acpi INT3438:00: DesignWare DMA Controller, 8 channels [ 24.862260] haswell-pcm-audio haswell-pcm-audio: Direct firmware load for intel/IntcPP01.bin failed with error -2 [ 24.862320] haswell-pcm-audio haswell-pcm-audio: fw image intel/IntcPP01.bin not available(-2) [ 24.862924] haswell-pcm-audio haswell-pcm-audio: FW loaded, mailbox readback FW info: type 01, - version: 00.00, build 77, source commit id: 876ac6906f31a43b6772b23c7c983ce9dcb18a19 [ 24.946651] rt286 i2c-INT343A:00: ASoC: sink widget DMIC1 overwritten [ 24.946882] rt286 i2c-INT343A:00: ASoC: source widget DMIC1 overwritten [ 24.948251] ================================================================== [ 24.948275] BUG: KASAN: user-memory-access in _raw_spin_lock_irqsave+0x7e/0xf0 [ 24.948290] Write of size 4 at addr 0000010400000000 by task systemd-udevd/292
[ 24.948313] CPU: 1 PID: 292 Comm: systemd-udevd Not tainted 5.3.0-rc4+ #111 [ 24.948317] Hardware name: Intel Corporation Broadwell Client platform/Pearl Valley, BIOS BDW-E1R1.86C.0119.R01.1503252201 03/25/2015 [ 24.948319] Call Trace: [ 24.948327] dump_stack+0x71/0xab [ 24.948334] ? _raw_spin_lock_irqsave+0x7e/0xf0 [ 24.948339] ? _raw_spin_lock_irqsave+0x7e/0xf0 [ 24.948346] __kasan_report+0x176/0x192 [ 24.948352] ? _raw_spin_lock_irqsave+0x7e/0xf0 [ 24.948359] kasan_report+0xe/0x20 [ 24.948366] check_memory_region+0x149/0x1a0 [ 24.948372] _raw_spin_lock_irqsave+0x7e/0xf0 [ 24.948378] ? _raw_write_lock_bh+0xe0/0xe0 [ 24.948426] ? snd_soc_dapm_add_route+0x2da/0x4f0 [snd_soc_core] [ 24.948435] ipc_tx_message+0xa8/0x540 [snd_soc_sst_ipc] [ 24.948485] ? snd_soc_dapm_add_path+0x9c0/0x9c0 [snd_soc_core] [ 24.948490] ? 0xffffffffc0bc0000 [ 24.948512] ? snd_ctl_dev_free+0x80/0x80 [snd] [ 24.948522] sst_ipc_tx_message_wait+0x63/0xb0 [snd_soc_sst_ipc] [ 24.948545] sst_hsw_device_set_config+0x13f/0x2d0 [snd_soc_sst_haswell_pcm] [ 24.948552] ? mutex_unlock+0x1d/0x40 [ 24.948572] ? hsw_notification_work+0x2c0/0x2c0 [snd_soc_sst_haswell_pcm] [ 24.948578] ? strcmp+0x30/0x50 [ 24.948584] ? strcmp+0x30/0x50 [ 24.948597] broadwell_rtd_init+0x68/0xa0 [snd_soc_sst_broadwell] [ 24.948642] snd_soc_instantiate_card+0xd81/0x1720 [snd_soc_core] [ 24.948690] ? soc_cleanup_card_resources+0x5a0/0x5a0 [snd_soc_core] [ 24.948697] ? __kasan_kmalloc.constprop.8+0xa0/0xd0 [ 24.948703] ? __kmalloc_node_track_caller+0xf3/0x320 [ 24.948749] snd_soc_register_card+0x25b/0x280 [snd_soc_core] [ 24.948756] ? devres_alloc_node+0x55/0x70 [ 24.948801] devm_snd_soc_register_card+0x3c/0x80 [snd_soc_core] [ 24.948809] platform_drv_probe+0x4d/0xb0 [ 24.948816] really_probe+0x35c/0x5c0 [ 24.948824] driver_probe_device+0x181/0x1b0 [ 24.948831] device_driver_attach+0x8a/0x90 [ 24.948838] ? device_driver_attach+0x90/0x90 [ 24.948843] __driver_attach+0xc1/0x190 [ 24.948849] ? device_driver_attach+0x90/0x90 [ 24.948854] bus_for_each_dev+0xe6/0x140 [ 24.948859] ? _raw_write_trylock+0xe0/0xe0 [ 24.948865] ? subsys_dev_iter_exit+0x10/0x10 [ 24.948871] ? klist_node_init+0x61/0x90 [ 24.948878] bus_add_driver+0x212/0x310 [ 24.948887] driver_register+0xcf/0x1b0 [ 24.948892] ? 0xffffffffc09d8000 [ 24.948900] do_one_initcall+0x8b/0x2b4 [ 24.948907] ? trace_event_raw_event_initcall_finish+0x140/0x140 [ 24.948914] ? kasan_unpoison_shadow+0x30/0x40 [ 24.948921] ? kasan_unpoison_shadow+0x30/0x40 [ 24.948926] ? kasan_unpoison_shadow+0x30/0x40 [ 24.948935] do_init_module+0xe5/0x364 [ 24.948941] load_module+0x4385/0x4b80 [ 24.948960] ? module_frob_arch_sections+0x20/0x20 [ 24.948965] ? ima_read_file+0x10/0x10 [ 24.948971] ? vfs_read+0xc2/0x1a0 [ 24.948978] ? kernel_read+0x95/0xb0 [ 24.948984] ? kernel_read_file+0x14a/0x330 [ 24.948992] ? get_unmapped_area+0x16c/0x1c0 [ 24.949000] ? __do_sys_finit_module+0x193/0x1c0 [ 24.949005] __do_sys_finit_module+0x193/0x1c0 [ 24.949012] ? __ia32_sys_init_module+0x40/0x40 [ 24.949019] ? __do_sys_newfstat+0x7c/0xd0 [ 24.949029] do_syscall_64+0x73/0x1b0 [ 24.949037] entry_SYSCALL_64_after_hwframe+0x44/0xa9 [ 24.949042] RIP: 0033:0x7f03ff6124d9 [ 24.949049] Code: 00 f3 c3 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 40 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 8f 29 2c 00 f7 d8 64 89 01 48 [ 24.949052] RSP: 002b:00007ffd2a5fcce8 EFLAGS: 00000246 ORIG_RAX: 0000000000000139 [ 24.949058] RAX: ffffffffffffffda RBX: 000055c89fafe8e0 RCX: 00007f03ff6124d9 [ 24.949062] RDX: 0000000000000000 RSI: 00007f03ffb08e23 RDI: 0000000000000012 [ 24.949065] RBP: 00007f03ffb08e23 R08: 0000000000000000 R09: 0000000000000000 [ 24.949068] R10: 0000000000000012 R11: 0000000000000246 R12: 0000000000000000 [ 24.949072] R13: 000055c89fb33d50 R14: 0000000000020000 R15: 000000000aba9500 [ 24.949077] ================================================================== [ 24.949092] Disabling lock debugging due to kernel taint [ 24.949100] BUG: unable to handle page fault for address: 0000010400000000 [ 24.949113] #PF: supervisor write access in kernel mode [ 24.949125] #PF: error_code(0x0002) - not-present page [ 24.949135] PGD 0 P4D 0 [ 24.949147] Oops: 0002 [#1] SMP KASAN PTI [ 24.949160] CPU: 1 PID: 292 Comm: systemd-udevd Tainted: G B 5.3.0-rc4+ #111 [ 24.949176] Hardware name: Intel Corporation Broadwell Client platform/Pearl Valley, BIOS BDW-E1R1.86C.0119.R01.1503252201 03/25/2015 [ 24.949198] RIP: 0010:_raw_spin_lock_irqsave+0x96/0xf0 [ 24.949212] Code: be 04 00 00 00 c7 44 24 20 00 00 00 00 e8 82 00 3b ff 48 8d 7c 24 20 be 04 00 00 00 e8 73 00 3b ff ba 01 00 00 00 8b 44 24 20 <f0> 0f b1 13 75 2f 48 b8 00 00 00 00 00 fc ff df 48 c7 44 05 00 00 [ 24.949238] RSP: 0018:ffff888116d9f350 EFLAGS: 00010097 [ 24.949251] RAX: 0000000000000000 RBX: 0000010400000000 RCX: ffffffffad01224d [ 24.949264] RDX: 0000000000000001 RSI: 0000000000000004 RDI: ffff888116d9f370 [ 24.949277] RBP: 1ffff11022db3e6a R08: 0000000000000004 R09: ffffed1022db3e6e [ 24.949291] R10: 0000000000000001 R11: ffffed1022db3e6e R12: 0000000000000246 [ 24.949304] R13: 0000000000000000 R14: 0000000000000000 R15: 0000000000000010 [ 24.949320] FS: 00007f04007898c0(0000) GS:ffff888129480000(0000) knlGS:0000000000000000 [ 24.949335] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 24.949348] CR2: 0000010400000000 CR3: 00000001277cc006 CR4: 00000000003606e0 [ 24.949360] Call Trace: [ 24.949372] ? _raw_write_lock_bh+0xe0/0xe0 [ 24.949426] ? snd_soc_dapm_add_route+0x2da/0x4f0 [snd_soc_core] [ 24.949445] ipc_tx_message+0xa8/0x540 [snd_soc_sst_ipc] [ 24.949501] ? snd_soc_dapm_add_path+0x9c0/0x9c0 [snd_soc_core] [ 24.949515] ? 0xffffffffc0bc0000 [ 24.949543] ? snd_ctl_dev_free+0x80/0x80 [snd] [ 24.949560] sst_ipc_tx_message_wait+0x63/0xb0 [snd_soc_sst_ipc] [ 24.949592] sst_hsw_device_set_config+0x13f/0x2d0 [snd_soc_sst_haswell_pcm] [ 24.949609] ? mutex_unlock+0x1d/0x40 [ 24.949636] ? hsw_notification_work+0x2c0/0x2c0 [snd_soc_sst_haswell_pcm] [ 24.949651] ? strcmp+0x30/0x50 [ 24.949664] ? strcmp+0x30/0x50 [ 24.949682] broadwell_rtd_init+0x68/0xa0 [snd_soc_sst_broadwell] [ 24.949736] snd_soc_instantiate_card+0xd81/0x1720 [snd_soc_core] [ 24.949795] ? soc_cleanup_card_resources+0x5a0/0x5a0 [snd_soc_core] [ 24.949811] ? __kasan_kmalloc.constprop.8+0xa0/0xd0 [ 24.949825] ? __kmalloc_node_track_caller+0xf3/0x320 [ 24.949879] snd_soc_register_card+0x25b/0x280 [snd_soc_core] [ 24.949894] ? devres_alloc_node+0x55/0x70 [ 24.949947] devm_snd_soc_register_card+0x3c/0x80 [snd_soc_core] [ 24.949964] platform_drv_probe+0x4d/0xb0 [ 24.949978] really_probe+0x35c/0x5c0 [ 24.949992] driver_probe_device+0x181/0x1b0 [ 24.950006] device_driver_attach+0x8a/0x90 [ 24.950020] ? device_driver_attach+0x90/0x90 [ 24.950033] __driver_attach+0xc1/0x190 [ 24.950047] ? device_driver_attach+0x90/0x90 [ 24.950059] bus_for_each_dev+0xe6/0x140 [ 24.950071] ? _raw_write_trylock+0xe0/0xe0 [ 24.950084] ? subsys_dev_iter_exit+0x10/0x10 [ 24.950097] ? klist_node_init+0x61/0x90 [ 24.950111] bus_add_driver+0x212/0x310 [ 24.950126] driver_register+0xcf/0x1b0 [ 24.950138] ? 0xffffffffc09d8000 [ 24.950151] do_one_initcall+0x8b/0x2b4 [ 24.950165] ? trace_event_raw_event_initcall_finish+0x140/0x140 [ 24.950181] ? kasan_unpoison_shadow+0x30/0x40 [ 24.950196] ? kasan_unpoison_shadow+0x30/0x40 [ 24.950209] ? kasan_unpoison_shadow+0x30/0x40 [ 24.950224] do_init_module+0xe5/0x364 [ 24.950238] load_module+0x4385/0x4b80 [ 24.950262] ? module_frob_arch_sections+0x20/0x20 [ 24.950275] ? ima_read_file+0x10/0x10 [ 24.950288] ? vfs_read+0xc2/0x1a0 [ 24.950300] ? kernel_read+0x95/0xb0 [ 24.950314] ? kernel_read_file+0x14a/0x330 [ 24.950328] ? get_unmapped_area+0x16c/0x1c0 [ 24.950343] ? __do_sys_finit_module+0x193/0x1c0 [ 24.950356] __do_sys_finit_module+0x193/0x1c0 [ 24.950369] ? __ia32_sys_init_module+0x40/0x40 [ 24.950384] ? __do_sys_newfstat+0x7c/0xd0 [ 24.950401] do_syscall_64+0x73/0x1b0 [ 24.950415] entry_SYSCALL_64_after_hwframe+0x44/0xa9 [ 24.950427] RIP: 0033:0x7f03ff6124d9 [ 24.950440] Code: 00 f3 c3 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 40 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 8f 29 2c 00 f7 d8 64 89 01 48 [ 24.950467] RSP: 002b:00007ffd2a5fcce8 EFLAGS: 00000246 ORIG_RAX: 0000000000000139 [ 24.950484] RAX: ffffffffffffffda RBX: 000055c89fafe8e0 RCX: 00007f03ff6124d9 [ 24.950497] RDX: 0000000000000000 RSI: 00007f03ffb08e23 RDI: 0000000000000012 [ 24.950510] RBP: 00007f03ffb08e23 R08: 0000000000000000 R09: 0000000000000000 [ 24.950523] R10: 0000000000000012 R11: 0000000000000246 R12: 0000000000000000 [ 24.950537] R13: 000055c89fb33d50 R14: 0000000000020000 R15: 000000000aba9500 [ 24.950551] Modules linked in: snd_soc_sst_broadwell(+) intel_rapl_msr snd_soc_sst_haswell_pcm snd_soc_sst_ipc intel_rapl_common x86_pkg_temp_thermal snd_soc_sst_firmware intel_powerclamp coretemp snd_soc_rt298 kvm_intel kvm irqbypass snd_soc_rt286 snd_soc_rl6347a crct10dif_pclmul crc32_pclmul ghash_clmulni_intel snd_soc_core snd_pcm_dmaengine ac97_bus aesni_intel snd_pcm aes_x86_64 crypto_simd input_leds cryptd glue_helper snd_seq_midi serio_raw snd_seq_midi_event intel_pch_thermal snd_rawmidi mei_me mei lpc_ich snd_seq snd_seq_device soc_button_array intel_vbtn snd_timer snd_soc_hsw_acpi snd snd_soc_sst_dsp snd_soc_acpi_intel_match dw_dmac snd_soc_acpi soundcore 8250_dw intel_hid intel_pmc_core sparse_keymap acpi_pad parport_pc ppdev lp parport autofs4 i915 ahci e1000e libahci sdhci_acpi video sdhci [ 24.950717] CR2: 0000010400000000 [ 24.950728] ---[ end trace 7c9279db22368aac ]--- [ 24.950742] RIP: 0010:_raw_spin_lock_irqsave+0x96/0xf0 [ 24.950756] Code: be 04 00 00 00 c7 44 24 20 00 00 00 00 e8 82 00 3b ff 48 8d 7c 24 20 be 04 00 00 00 e8 73 00 3b ff ba 01 00 00 00 8b 44 24 20 <f0> 0f b1 13 75 2f 48 b8 00 00 00 00 00 fc ff df 48 c7 44 05 00 00 [ 24.950783] RSP: 0018:ffff888116d9f350 EFLAGS: 00010097 [ 24.950795] RAX: 0000000000000000 RBX: 0000010400000000 RCX: ffffffffad01224d [ 24.950808] RDX: 0000000000000001 RSI: 0000000000000004 RDI: ffff888116d9f370 [ 24.950822] RBP: 1ffff11022db3e6a R08: 0000000000000004 R09: ffffed1022db3e6e [ 24.950835] R10: 0000000000000001 R11: ffffed1022db3e6e R12: 0000000000000246 [ 24.950848] R13: 0000000000000000 R14: 0000000000000000 R15: 0000000000000010 [ 24.950863] FS: 00007f04007898c0(0000) GS:ffff888129480000(0000) knlGS:0000000000000000 [ 24.950879] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 24.950891] CR2: 0000010400000000 CR3: 00000001277cc006 CR4: 00000000003606e0 _______________________________________________ Alsa-devel mailing list Alsa-devel@alsa-project.org https://mailman.alsa-project.org/mailman/listinfo/alsa-devel