Bard Liao

Results 204 comments of Bard Liao

> @bardliao did you get any warning about a non-registered source as discussed earlier? Yes, ``` [ 169.988327] rt712-sdca sdw:0:0:025d:0712:01: sdca_interrupt_register_handler: interrupt 0x49, registered sources 0x101, unregistered sources will be...

> Humm, this is confusing @bardliao, the 'registered_sources' are not supposed to change. Maybe the log is incorrect? There are 2 interrupt sources sdca_int_0 and sdca_int_8. So, the 'registered_sources' are...

> There's still no way we can have registered sources 0x101 by using BIT(0) and BIT(8). We must do something wrong. Sorry, I didn't get the point. BIT(0) | BIT(8)...

> sorry, I confused binary and hex haha. I added a 0xff mask to only look at the 8 bits per register, that should make things clearer in the log...

Let's continue with https://github.com/thesofproject/linux/pull/5199 and focus on SDCA functions and version lookup first.

The issue only happens on `ba-lnlm-rvp-sdw-01`, but not `jf-lnlm-rvp-sdw-1`. And I checked the dmesg found that [ 8.481076] snd_soc_sof_sdw:sof_card_dai_links_create: sof_sdw sof_sdw: sdw 5, ssp 0, **dmic 2**, hdmi 0, bt:...

I added `options snd_sof_intel_hda_generic dmic_num=0` as a temporary solution.

> Please check if following PR could solve this issue and sorry for the regression. https://github.com/thesofproject/linux/pull/5125 Thanks @brentlu I confirmed https://github.com/thesofproject/linux/pull/5125 works. > So @bardliao you're saying it is actually...

> Another possible locking issue: > > a) the bank switch starts and the bus takes the msg->lock b) an interrupt happens and the workqueue schedules a read from IntStat0...

Same test as https://github.com/thesofproject/linux/issues/5052#issuecomment-2160611644 with https://github.com/thesofproject/linux/pull/5047 ``` [ 43.664000] sdw_acquire_bus_lock: acquiring locks for stream subdevice #0-Playback [ 43.664002] sdw_acquire_bus_lock: acquired locks for stream subdevice #0-Playback [ 43.664006] soundwire sdw-master-0-0: sdw_transfer:...