After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 694989 - "Analog Output" in volume OSD
"Analog Output" in volume OSD
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: media-keys
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
3.8?
Depends on:
Blocks:
 
 
Reported: 2013-03-02 12:46 UTC by William Jon McCann
Modified: 2013-04-29 17:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot (150.17 KB, image/png)
2013-03-02 12:46 UTC, William Jon McCann
  Details
media-keys: don't show label for analog-output ports (1.30 KB, patch)
2013-03-02 15:30 UTC, William Jon McCann
committed Details | Review

Description William Jon McCann 2013-03-02 12:46:27 UTC
Created attachment 237788 [details]
screenshot

On my x220 laptop I get "Analog Output" in the volume OSD.
Comment 1 William Jon McCann 2013-03-02 15:18:34 UTC
port->port = "analog-output"
form-factor = "internal"
Comment 2 William Jon McCann 2013-03-02 15:30:45 UTC
Created attachment 237799 [details] [review]
media-keys: don't show label for analog-output ports
Comment 3 Bastien Nocera 2013-03-02 23:34:51 UTC
CC:ed David on this so he can give his opinion.
I'm afraid this patch might cause headphones to not have a label in some cases.
Comment 4 David Henningsson 2013-03-04 08:23:03 UTC
I think the patch is OK. I don't know what the rules are for what should be shown or not; but if the intention is to hide everything that's labelled "Analog Output", then the patch is correct.

If you plug the headphones in on an X220 and then try to set the volume, it will still show "Analog Output"...I think? I can't find any alsa-info of it right now, so not sure.
Anyway, if you add "options snd-hda-intel model=auto" to a modprobe conf file (in Ubuntu, that would be /etc/modprobe.d/alsa-base.conf, don't know if Fedora works the same) and reboot, chances are you would get analog-output-speaker and analog-headphones just like most of the other computers.
If you do this, find that it works, and attach your alsa-info, I'll send a patch upstream for making this the default in future kernels. Thanks!
Comment 5 Bastien Nocera 2013-03-04 10:11:39 UTC
(In reply to comment #4)
> I think the patch is OK. I don't know what the rules are for what should be
> shown or not; but if the intention is to hide everything that's labelled
> "Analog Output", then the patch is correct.

We want to hide the label for built-in speaker outputs. I was wondering whether there's a chance of a false positive here and have a headphones output with a hidden label instead.
Comment 6 David Henningsson 2013-03-04 10:14:56 UTC
In the X220 (and other) cases where there are just one analog-output, instead of analog-output-headphones and analog-output-speaker, "Analog Output" will be the active port regardless of whether headphones are plugged in or not.
Comment 7 Bastien Nocera 2013-03-04 10:31:36 UTC
Review of attachment 237799 [details] [review]:

As per David's comment, looks fine to commit though you'll need to get the driver fixed to see the headphones label.
Comment 8 William Jon McCann 2013-03-04 20:34:29 UTC
David, I can confirm that adding "options snd-hda-intel model=auto" makes the jack detection work. Awesome!
Comment 9 William Jon McCann 2013-03-04 20:35:13 UTC
Attachment 237799 [details] pushed as 20b7828 - media-keys: don't show label for analog-output ports
Comment 10 William Jon McCann 2013-04-24 21:05:34 UTC
I suppose this should get pushed to 3.8 too.

David, has this fix been submitted to the upstream kernel yet?
Comment 11 David Henningsson 2013-04-25 06:17:51 UTC
@William, no, you never submitted any alsa-info (see http://git.alsa-project.org/?p=alsa-driver.git;a=blob_plain;f=alsa/utils/alsa-info.sh;hb=refs/heads/build ). 
I need alsa-info information in order to write a patch.
Comment 12 Bastien Nocera 2013-04-25 08:22:21 UTC
(In reply to comment #10)
> I suppose this should get pushed to 3.8 too.

Adapted to gnome-3-8.
Comment 15 William Jon McCann 2013-04-29 17:02:36 UTC
Yay. Thanks!