| Message ID | 20200413145957.22459-1-youling257@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | bytcht-es8316: fix Speaker or Headphone no sound if enable Mic or Headset | expand |
Hi, On 4/13/20 4:59 PM, youling257 wrote: > alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Mic, Speaker will no sound. > alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Headset, Headphone will no sound. I see the problem, so the idea of the global EnableSequence is to set defaults and then the EnableSequence of currently active devices run on top. If we drop the default off of both switched and then shutdown the machine with no headphones inserted, so the speakers being used, then the 'Speaker Switch' on state will be saved on shutdown and restored on boot. If we then boot with headphones inserted then AFAIK pulseaudio will run the global EnableSequence + the HeadPhones EnableSequence, leaving the 'Speaker Switch' on and now we get audio on both. The problem seems to be that alsaucm not only runs the EnableSequence for the selected device but also the global one, which pulseaudio normally only does once at bootup. I might be wrong here, maybe pulse will run the global EnableSeq + EnableSeq of active devices + DisableSeq of inactive devices when it starts. In that case the running of the DisableSeq of inactive devices will take care of turning off the speaker-switch when booting with the headphones inserted. In which case the default off of all switches can indeed be removed. Note in that case we should also remove the default off for the Headset + Internal Mic Switch, so a nack to this patch regardless. Jaroslav, can you give us some guidance on how this is supposed to work ? Also you say you use pulseaudio for your testing, AFAIK pulse does not suffer from the problem the alsaucm commands you mention do, so as long as you only use pulse things should work. Regards, Hans > > Signed-off-by: youling257 <youling257@gmail.com> > --- > ucm2/codecs/es8316/EnableSeq.conf | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/ucm2/codecs/es8316/EnableSeq.conf b/ucm2/codecs/es8316/EnableSeq.conf > index 551b79b..53bb115 100644 > --- a/ucm2/codecs/es8316/EnableSeq.conf > +++ b/ucm2/codecs/es8316/EnableSeq.conf > @@ -1,6 +1,4 @@ > # Disable all inputs / outputs > -cset "name='Speaker Switch' off" > -cset "name='Headphone Switch' off" > cset "name='Headset Mic Switch' off" > cset "name='Internal Mic Switch' off" > >
Dne 13. 04. 20 v 17:46 Hans de Goede napsal(a): > Hi, > > On 4/13/20 4:59 PM, youling257 wrote: >> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Mic, Speaker will no sound. >> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Headset, Headphone will no sound. > > I see the problem, so the idea of the global EnableSequence is > to set defaults and then the EnableSequence of currently active > devices run on top. It's ok, it's just a wrong use of the alsaucm command. The PA should activate HiFi verb only once. Those two commands run the verb EnableSequence twice. The correct test should be: alsaucm -i -c bytcht-es8316 set _verb HiFi alsaucm>> set _enadev Speaker alsaucm>> set _enadev Headphone alsaucm>> set _enadev Mic alsaucm>> set _enadev Headset ... you may run the enable commands with a delay for tests. The current UCM is designed for one task (application). I am playing with an idea to extend the alsactl state daemon to preserve the UCM settings and state for multiple applications using an RPC mechanism (dbus?). Jaroslav
boot enter ubuntu no sound at all if EnableSeq.conf disable all output. “ The PA should activate HiFi verb only once.” i don't think so. 2020-04-14 16:07 GMT+08:00, Jaroslav Kysela <perex@perex.cz>: > Dne 13. 04. 20 v 17:46 Hans de Goede napsal(a): >> Hi, >> >> On 4/13/20 4:59 PM, youling257 wrote: >>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Mic, Speaker will no >>> sound. >>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Headset, Headphone >>> will no sound. >> >> I see the problem, so the idea of the global EnableSequence is >> to set defaults and then the EnableSequence of currently active >> devices run on top. > > It's ok, it's just a wrong use of the alsaucm command. The PA should > activate > HiFi verb only once. Those two commands run the verb EnableSequence twice. > > The correct test should be: > > alsaucm -i -c bytcht-es8316 set _verb HiFi > alsaucm>> set _enadev Speaker > alsaucm>> set _enadev Headphone > alsaucm>> set _enadev Mic > alsaucm>> set _enadev Headset > > ... you may run the enable commands with a delay for tests. > > The current UCM is designed for one task (application). I am playing with an > > idea to extend the alsactl state daemon to preserve the UCM settings and > state > for multiple applications using an RPC mechanism (dbus?). > > Jaroslav > > -- > Jaroslav Kysela <perex@perex.cz> > Linux Sound Maintainer; ALSA Project; Red Hat, Inc. >
please remove es8316 cx2072x rt5672 Disable all inputs / outputs. 2020-04-15 14:26 GMT+08:00, youling 257 <youling257@gmail.com>: > boot enter ubuntu no sound at all if EnableSeq.conf disable all output. > “ The PA should activate HiFi verb only once.” i don't think so. > > > 2020-04-14 16:07 GMT+08:00, Jaroslav Kysela <perex@perex.cz>: >> Dne 13. 04. 20 v 17:46 Hans de Goede napsal(a): >>> Hi, >>> >>> On 4/13/20 4:59 PM, youling257 wrote: >>>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Mic, Speaker will >>>> no >>>> sound. >>>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Headset, Headphone >>>> will no sound. >>> >>> I see the problem, so the idea of the global EnableSequence is >>> to set defaults and then the EnableSequence of currently active >>> devices run on top. >> >> It's ok, it's just a wrong use of the alsaucm command. The PA should >> activate >> HiFi verb only once. Those two commands run the verb EnableSequence >> twice. >> >> The correct test should be: >> >> alsaucm -i -c bytcht-es8316 set _verb HiFi >> alsaucm>> set _enadev Speaker >> alsaucm>> set _enadev Headphone >> alsaucm>> set _enadev Mic >> alsaucm>> set _enadev Headset >> >> ... you may run the enable commands with a delay for tests. >> >> The current UCM is designed for one task (application). I am playing with >> an >> >> idea to extend the alsactl state daemon to preserve the UCM settings and >> state >> for multiple applications using an RPC mechanism (dbus?). >> >> Jaroslav >> >> -- >> Jaroslav Kysela <perex@perex.cz> >> Linux Sound Maintainer; ALSA Project; Red Hat, Inc. >> >
2020-04-13 23:46 GMT+08:00, Hans de Goede <hdegoede@redhat.com>: > Also you say you use pulseaudio for your testing, > AFAIK pulse does not suffer from the problem the > alsaucm commands you mention do, I said "remove the EnableSeq.conf Disable all output, i can get speaker and headphone work on ubuntu focal with pulseaudio." >so as long as > you only use pulse things should work. Disable all inputs / outputs, pulseaudio sound not work. > Regards, > > Hans > > > > >> >> Signed-off-by: youling257 <youling257@gmail.com> >> --- >> ucm2/codecs/es8316/EnableSeq.conf | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/ucm2/codecs/es8316/EnableSeq.conf >> b/ucm2/codecs/es8316/EnableSeq.conf >> index 551b79b..53bb115 100644 >> --- a/ucm2/codecs/es8316/EnableSeq.conf >> +++ b/ucm2/codecs/es8316/EnableSeq.conf >> @@ -1,6 +1,4 @@ >> # Disable all inputs / outputs >> -cset "name='Speaker Switch' off" >> -cset "name='Headphone Switch' off" >> cset "name='Headset Mic Switch' off" >> cset "name='Internal Mic Switch' off" >> >> > >
Hi, On 4/15/20 8:26 AM, youling 257 wrote: > boot enter ubuntu no sound at all if EnableSeq.conf disable all output. > “ The PA should activate HiFi verb only once.” i don't think so. Which version of pulseaudio does the Ubuntu you are using have? There was a bug in pulseaudio 13.99.1 wrt the handling of UCM profiles. The pulseaudio from Fedora 31 updates has a fix for this, maybe Ubuntu's version is missing this fix? Regards, Hans > > > 2020-04-14 16:07 GMT+08:00, Jaroslav Kysela <perex@perex.cz>: >> Dne 13. 04. 20 v 17:46 Hans de Goede napsal(a): >>> Hi, >>> >>> On 4/13/20 4:59 PM, youling257 wrote: >>>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Mic, Speaker will no >>>> sound. >>>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Headset, Headphone >>>> will no sound. >>> >>> I see the problem, so the idea of the global EnableSequence is >>> to set defaults and then the EnableSequence of currently active >>> devices run on top. >> >> It's ok, it's just a wrong use of the alsaucm command. The PA should >> activate >> HiFi verb only once. Those two commands run the verb EnableSequence twice. >> >> The correct test should be: >> >> alsaucm -i -c bytcht-es8316 set _verb HiFi >> alsaucm>> set _enadev Speaker >> alsaucm>> set _enadev Headphone >> alsaucm>> set _enadev Mic >> alsaucm>> set _enadev Headset >> >> ... you may run the enable commands with a delay for tests. >> >> The current UCM is designed for one task (application). I am playing with an >> >> idea to extend the alsactl state daemon to preserve the UCM settings and >> state >> for multiple applications using an RPC mechanism (dbus?). >> >> Jaroslav >> >> -- >> Jaroslav Kysela <perex@perex.cz> >> Linux Sound Maintainer; ALSA Project; Red Hat, Inc. >> >
Hi, On 4/15/20 8:30 AM, youling 257 wrote: > please remove es8316 cx2072x rt5672 Disable all inputs / outputs. No not going to happen. We need sane defaults. There is something wrong with your setup lets try to figure out what is wrong rather then change these UCM profiles in a way which only makes things worse. Regards, Hans > > 2020-04-15 14:26 GMT+08:00, youling 257 <youling257@gmail.com>: >> boot enter ubuntu no sound at all if EnableSeq.conf disable all output. >> “ The PA should activate HiFi verb only once.” i don't think so. >> >> >> 2020-04-14 16:07 GMT+08:00, Jaroslav Kysela <perex@perex.cz>: >>> Dne 13. 04. 20 v 17:46 Hans de Goede napsal(a): >>>> Hi, >>>> >>>> On 4/13/20 4:59 PM, youling257 wrote: >>>>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Mic, Speaker will >>>>> no >>>>> sound. >>>>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Headset, Headphone >>>>> will no sound. >>>> >>>> I see the problem, so the idea of the global EnableSequence is >>>> to set defaults and then the EnableSequence of currently active >>>> devices run on top. >>> >>> It's ok, it's just a wrong use of the alsaucm command. The PA should >>> activate >>> HiFi verb only once. Those two commands run the verb EnableSequence >>> twice. >>> >>> The correct test should be: >>> >>> alsaucm -i -c bytcht-es8316 set _verb HiFi >>> alsaucm>> set _enadev Speaker >>> alsaucm>> set _enadev Headphone >>> alsaucm>> set _enadev Mic >>> alsaucm>> set _enadev Headset >>> >>> ... you may run the enable commands with a delay for tests. >>> >>> The current UCM is designed for one task (application). I am playing with >>> an >>> >>> idea to extend the alsactl state daemon to preserve the UCM settings and >>> state >>> for multiple applications using an RPC mechanism (dbus?). >>> >>> Jaroslav >>> >>> -- >>> Jaroslav Kysela <perex@perex.cz> >>> Linux Sound Maintainer; ALSA Project; Red Hat, Inc. >>> >> >
2020-04-15 16:04 GMT+08:00, Hans de Goede <hdegoede@redhat.com>: > Hi, > > On 4/15/20 8:26 AM, youling 257 wrote: >> boot enter ubuntu no sound at all if EnableSeq.conf disable all output. >> “ The PA should activate HiFi verb only once.” i don't think so. > > Which version of pulseaudio does the Ubuntu you are using have? pulseaudio (1:13.99.1-1ubuntu2) packages.ubuntu.com/focal/pulseaudio what patch? > There was a bug in pulseaudio 13.99.1 wrt the handling of UCM > profiles. The pulseaudio from Fedora 31 updates has a fix > for this, maybe Ubuntu's version is missing this fix? > > Regards, > > Hans > > > > > > > >> >> >> 2020-04-14 16:07 GMT+08:00, Jaroslav Kysela <perex@perex.cz>: >>> Dne 13. 04. 20 v 17:46 Hans de Goede napsal(a): >>>> Hi, >>>> >>>> On 4/13/20 4:59 PM, youling257 wrote: >>>>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Mic, Speaker will >>>>> no >>>>> sound. >>>>> alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Headset, Headphone >>>>> will no sound. >>>> >>>> I see the problem, so the idea of the global EnableSequence is >>>> to set defaults and then the EnableSequence of currently active >>>> devices run on top. >>> >>> It's ok, it's just a wrong use of the alsaucm command. The PA should >>> activate >>> HiFi verb only once. Those two commands run the verb EnableSequence >>> twice. >>> >>> The correct test should be: >>> >>> alsaucm -i -c bytcht-es8316 set _verb HiFi >>> alsaucm>> set _enadev Speaker >>> alsaucm>> set _enadev Headphone >>> alsaucm>> set _enadev Mic >>> alsaucm>> set _enadev Headset >>> >>> ... you may run the enable commands with a delay for tests. >>> >>> The current UCM is designed for one task (application). I am playing with >>> an >>> >>> idea to extend the alsactl state daemon to preserve the UCM settings and >>> state >>> for multiple applications using an RPC mechanism (dbus?). >>> >>> Jaroslav >>> >>> -- >>> Jaroslav Kysela <perex@perex.cz> >>> Linux Sound Maintainer; ALSA Project; Red Hat, Inc. >>> >> > >
Hi, On 4/15/20 10:18 AM, youling 257 wrote: > 2020-04-15 16:04 GMT+08:00, Hans de Goede <hdegoede@redhat.com>: >> Hi, >> >> On 4/15/20 8:26 AM, youling 257 wrote: >>> boot enter ubuntu no sound at all if EnableSeq.conf disable all output. >>> “ The PA should activate HiFi verb only once.” i don't think so. >> >> Which version of pulseaudio does the Ubuntu you are using have? > pulseaudio (1:13.99.1-1ubuntu2) packages.ubuntu.com/focal/pulseaudio Ah yes then you are likely hitting the bug I was talking about. Can you please file a bug against Ubuntu for this? Here is the original Fedora bug for this: https://bugzilla.redhat.com/show_bug.cgi?id=1818883 And here are the patches fixing it: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/276/ Regards, Hans
2020-04-15 16:45 GMT+08:00, Hans de Goede <hdegoede@redhat.com>: > Hi, > > On 4/15/20 10:18 AM, youling 257 wrote: >> 2020-04-15 16:04 GMT+08:00, Hans de Goede <hdegoede@redhat.com>: >>> Hi, >>> >>> On 4/15/20 8:26 AM, youling 257 wrote: >>>> boot enter ubuntu no sound at all if EnableSeq.conf disable all output. >>>> “ The PA should activate HiFi verb only once.” i don't think so. >>> >>> Which version of pulseaudio does the Ubuntu you are using have? >> pulseaudio (1:13.99.1-1ubuntu2) packages.ubuntu.com/focal/pulseaudio > > Ah yes then you are likely hitting the bug I was talking about. > > Can you please file a bug against Ubuntu for this? open a bug, https://bugs.launchpad.net/ubuntu/+source/pulseaudio/+bug/1872970 > Here is the original Fedora bug for this: > https://bugzilla.redhat.com/show_bug.cgi?id=1818883 > > And here are the patches fixing it: > https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/276/ > > Regards, > > Hans > >
Hi, On 4/15/20 4:50 PM, youling 257 wrote: > 2020-04-15 16:45 GMT+08:00, Hans de Goede <hdegoede@redhat.com>: >> Hi, >> >> On 4/15/20 10:18 AM, youling 257 wrote: >>> 2020-04-15 16:04 GMT+08:00, Hans de Goede <hdegoede@redhat.com>: >>>> Hi, >>>> >>>> On 4/15/20 8:26 AM, youling 257 wrote: >>>>> boot enter ubuntu no sound at all if EnableSeq.conf disable all output. >>>>> “ The PA should activate HiFi verb only once.” i don't think so. >>>> >>>> Which version of pulseaudio does the Ubuntu you are using have? >>> pulseaudio (1:13.99.1-1ubuntu2) packages.ubuntu.com/focal/pulseaudio >> >> Ah yes then you are likely hitting the bug I was talking about. >> >> Can you please file a bug against Ubuntu for this? > > open a bug, https://bugs.launchpad.net/ubuntu/+source/pulseaudio/+bug/1872970 Great, thank you. Regards, Hans
diff --git a/ucm2/codecs/es8316/EnableSeq.conf b/ucm2/codecs/es8316/EnableSeq.conf index 551b79b..53bb115 100644 --- a/ucm2/codecs/es8316/EnableSeq.conf +++ b/ucm2/codecs/es8316/EnableSeq.conf @@ -1,6 +1,4 @@ # Disable all inputs / outputs -cset "name='Speaker Switch' off" -cset "name='Headphone Switch' off" cset "name='Headset Mic Switch' off" cset "name='Internal Mic Switch' off"
alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Mic, Speaker will no sound. alsaucm -c bytcht-es8316 set _verb HiFi set _enadev Headset, Headphone will no sound. Signed-off-by: youling257 <youling257@gmail.com> --- ucm2/codecs/es8316/EnableSeq.conf | 2 -- 1 file changed, 2 deletions(-)