1) In your patch, please add the "Origin" and "Bug-Ubuntu" dep3 tags.
2) Please change your patch filename to begin with "lp2146747" instead of 0001.

Done by re-upload debdiff

3) In the Where problems could occur section, you only mention that
CS42L43B is changed, and that CS42L43 is changed as well (typo).

Update the description in "Where problems could occur" section

4) In your testcase, is it really necessary to delete the amixer state?
Regular users certainly won't do this on regular upgrade.

yes, because it has change for the codec init, so the alsa state need to
be cleared

5) In the testcase, please also test sound recording and playback to ensure 
microphone works.
6) In the Impact section, please make it clear that audio input will not work 
until enablement is done.

The description is updated, please check if it is more applicable.

7) Does CS42L43B need kernel enablement?

The kernel patch is included in the SRU.
https://bugs.launchpad.net/ubuntu/+source/linux-oem-6.17/+bug/2143301

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2146747

Title:
  [SRU] Add support for CS42L43B variant

To manage notifications about this bug go to:
https://bugs.launchpad.net/oem-priority/+bug/2146747/+subscriptions


-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to