-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug][BYT] unload /reload audio module failure. #1059
Comments
This issue also can be reproduced on WHL/I2S mode. |
This issue still can be reproduced on daily test. |
WHL RVP (I2S ALC5682 codec) has different failure, tracked by #1110 |
@bardliao I don't have rt5651 anymore but I get little bit similar stuff with rt5682: [ 76.455294] Workqueue: events_power_efficient rt5682_jack_detect_handler [snd_soc_rt5682] So some jack_detection functions are not happy when removing modules... Any idea? |
This issue also can be reproduced on rt5682 platform:
Environment |
@wenqingfu @Jiangxinx rt5682 module loading issue should be now solved in #1147. As said I don't have working rt5651 setup, so I'm not sure I can try to fix rt5651 issue with remote device. At least with my mb-rt5682 setup the kernel oopses cause system wide instability and I have to power off/on the device... |
@Jiangxinx can someone with rt5651 device test if this helps: #1149 |
@juimonen I have tried #1149 with rt5651, found it can not boot sof.
|
@Jiangxinx oh, had a really bad copy/paste error... probe was running remove also. Can you please still try again with updated version... |
@juimonen OK,no problem. |
@Jiangxinx can you confirm #1147 fixes the issue for rt5682. I'd like a check before merging, thank you. Also recheck #1149, it'd be good to submit all machine driver changes upstream in one shot. |
Add today's check rerult:
|
@Jiangxinx can you try the rt5651 case with updated version of £1149 I just pushed. Sorry for the trouble, I just don't have the platform to test myself... |
i meant #1149 :) |
@juimonen #1149 still can not work:
|
@Jiangxinx ok thanks, I'll have to find a better way to test this.... |
I was able to render @Jiangxinx device remotely unusable, so if someone with rt5651 can try the latest update of #1149 that would be nice. I disassembled the crash to rt5651_support_button_press but not sure my fix will make things better... |
There is a feature in CI to offline a DUT from pool for dedicated debugging. I sent you and @xiulipan an email. |
@Jiangxinx can you still do 1 more round with the latest #1149? I was about to test that when things went wrong... I'll leave you alone after this :) |
@juimonen With the latest #1149 ,this issue still exist on rt5651:
|
Passed with the new #1149 on rt5651 platform. |
Closed as #1149 is merged. |
Describe the bug
Can not Removing snd_soc_sst_bytcr_rt5651 while doing load/reload test on byt-rt5651 platform.
To Reproduce
1.sudo ./sof_bootone.sh
Actual result
dmesg
bisect result
Caused by topic/sof-dev (#1018).
Environment
Firmware: 25bbc9f https://github.com/thesofproject/sof/commits/master
Kernel: 0087d4a https://github.com/thesofproject/linux/commits/topic/sof-dev
Topology: sof-byt-rt5651.tplg
error log
dmesg.log
The text was updated successfully, but these errors were encountered: