coreboot icon indicating copy to clipboard operation
coreboot copied to clipboard

mb/clevo/adl-p: Add HDA verbs from vendor BIOS

Open miczyg1 opened this issue 1 year ago • 6 comments

miczyg1 avatar Oct 24 '23 13:10 miczyg1

@mkopec extracted from vendor BIOS.

I don't think it will help for headphones vs headset detection (which our customer reported for recent release), but may help with other things (like some weird sounds from speakers?).

UNTESTED

miczyg1 avatar Oct 24 '23 13:10 miczyg1

Yeah headset detection is a problem with the hardware, not sure if we can do anything from FW side. Will test tomorrow.

But maybe it will help with this: https://github.com/Dasharo/dasharo-issues/issues/224

miczyg1 avatar Oct 24 '23 15:10 miczyg1

AIUI that issue was caused by coil whine, I once tried enabling Acoustic Noise Optimization to mitigate this, but I didn't know what the correct setting should be there, so for now it's disabled. But it might help, I'll test.

mkopec avatar Oct 25 '23 09:10 mkopec

@mkopec any updates on testing it?

I also noticed that the subvendor differed between NS5X and NS7X

miczyg1 avatar Nov 14 '23 10:11 miczyg1

AIUI that issue was caused by coil whine, I once tried enabling Acoustic Noise Optimization to mitigate this, but I didn't know what the correct setting should be there, so for now it's disabled. But it might help, I'll test.

It is not coil whine. Instead, it is really coming from the internal speakers. If we remove those internal speakers, the problem is no longer there. Also, this only happens on NV4x 11th Gen, not on NV4x 12th Gen.

Please let us know if these HDA verbs are included in the latest test binary we have received. If not, can it be?

wessel-novacustom avatar Nov 24 '23 07:11 wessel-novacustom

@mkopec still no news?

miczyg1 avatar Jan 09 '24 10:01 miczyg1