On 02-11-18, Hans de Goede wrote:
Hi,
On 01-11-18 16:50, Dean Wallace wrote:
On 01-11-18, Hans de Goede wrote:
Hi,
On 01-11-18 15:28, Dean Wallace wrote:
On 01-11-18, Hans de Goede wrote:
Hi,
On 01-11-18 11:37, Dean Wallace wrote:
On 31-10-18, Pierre-Louis Bossart wrote: > > > Just thought it worth mentioning, this new patch that fixes sound > > again, seems to have ressurected an old issue with PLL unlock. I'm > > seeing journal entries after fresh boot ...... > > > > ``` > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > picard systemd[462]: Started Sound Service. > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > picard kernel: max98090_pll_work: 141 callbacks suppressed > > picard kernel: max98090 i2c-193C9890:00: PLL unlocked > > ``` > > > > sound is ok, but sometimes plugging in headphones spams journal with > > those PLL messages, and sound turns into "daleks", and I have to > > remove/insert headphones few times or stop/start audio to fix it. > > It's a very old issue, maybe you'd know more about it. > > I noticed this error on my Orco device used for tests many moons ago, but I > could never find out what led to this error case, it wasn't deterministic > and didn't impact the audio quality. All I could do is rate_limit it... If > we have an A vs. B situation it'd be really helpful to diagnose further. > > Is there really a causality between the changes from Hans and this PLL > unlock error? Are you 100% sure this was not present in the previous install > you used (4.18.14 as mentioned earlier in the thread)? > > Thanks > > -Pierre > Well, numerous boots, kernels, headphone inserting - no PLL or 'Daleks'. My laptop must have been haunted that day (halloween). I'll put it to bed.
So you can no longer reproduce. Bummer. Note this might be caused by the temperature of the laptop when you were running the tests...
Anyways if you hit this again and you can reproduce it, please give adding a msleep(10) after code mucking with the clk a try.
Regards,
Hans
Right then, I can make it unlock and 'daleks' by going into pavucontrol and switching the Profile back and forth from Stereo Output to Stereo Output+Analog Mono Input, which is actually something I've done to make it correct itself as well. I don't use the mic or anything so I've had it set to Stereo Ouput only which I 'think' has somehow made it more stable for me. With all my playing around, one of the things I did was clean out my .config/pulse folder which meant by default the 'Profile' in pavucontrol was set to Output+Input, which seems to help trigger the PLL issue when inserting headphones.
So what would you like me to do, as I can trigger it on demand it seems.
Please give the attached patch a try (on top of my patch for the clk quirk) and let us know if that fixes these errors.
Regards,
Hans
Sorry, it's not being consistent with me Now, fresh boot again, no errors in journal, no errors while plugging/unplugging earphones. I think it's definitely more stable with the profile in pavucontrol set to Output only (no + Input), but I can still trigger it by switching it, and it never corrects itself until I switch a few more times.
Ok, so there are 2 issues here as I see it:
- It does not always reproduce
- It does still reproduce sometimes with the patch, so the patch does not
fix it.
So I think we just have to live with this for now.
Regards,
Hans
Yeah, I've never known it act any different so I'm used to it. I do know a few people in the past who have just bought a usb sound device because they want to use a mic as well, and that just didn't work quite right for them, even if sound output worked they had issues as soon as they tried using mic. I forget which, 4.1 or 4.4, 1 guy mentioned still using it just because output/input worked flawlessly. ¯_(ツ)_/¯
-Dean