Skip to contentSkip to conversation

sound/pci/hda: Remove CD control from model=benq for CX20549

PATCH In review Michael Karcher 2 participants 2 messages Apr 5, 2012 Sound

lore
Share & tools
lore
#0 lore raw

PATCH 4/6 sound/pci/hda: Remove CD control from model=benq for CX20549

The ID used for detection of the BenQ R55E actually identifies the Quana TW3 ODM design, which is also used for the Gigabyte W551 laptop series. Schematics on the internet clearly indicate that the "Port C" (analog input connected to record source #4 and mixer input #4) is unconnected.

sound/pci/hda/patch_conexant.c +0 -4
index bd7e7ce..8d7a977 100644
@@ -695,7 +695,6 @@ static const struct hda_input_mux cxt5045_capture_source = {
 static const struct hda_input_mux cxt5045_capture_source_benq = {
 	.num_items = 5,
 	.items = {
-		{ "CD",     0x4 },
 		{ "Internal Mic", 0x1 },
 		{ "Mic", 0x2 },
 		{ "Line In", 0x3 },
@@ -820,9 +819,6 @@ static const struct snd_kcontrol_new cxt5045_mixers[] = {
 };
 
 static const struct snd_kcontrol_new cxt5045_benq_mixers[] = {
-	HDA_CODEC_VOLUME("CD Playback Volume", 0x17, 0x4, HDA_INPUT),
-	HDA_CODEC_MUTE("CD Playback Switch", 0x17, 0x4, HDA_INPUT),
-
 	HDA_CODEC_VOLUME("Line In Playback Volume", 0x17, 0x3, HDA_INPUT),
 	HDA_CODEC_MUTE("Line In Playback Switch", 0x17, 0x3, HDA_INPUT),
 
Signature · 1 line
1.7.9.1
#1 lore raw

At Thu, 05 Apr 2012 15:30:25 +0200,

Quoted 6 lines · Michael Karcher

The ID used for detection of the BenQ R55E actually identifies the Quana TW3 ODM design, which is also used for the Gigabyte W551 laptop series. Schematics on the internet clearly indicate that the "Port C" (analog input connected to record source #4 and mixer input #4) is unconnected.

Well, such an action needs to be done carefully since the documentation often lies. But I also doubt that there is any analog CD line. If this is confirmed that actually it's not present on the real machine, fine to remove it indeed.

Takashi

Quoted 27 lines
sound/pci/hda/patch_conexant.c +0 -4
index bd7e7ce..8d7a977 100644
@@ -695,7 +695,6 @@ static const struct hda_input_mux cxt5045_capture_source = {
 static const struct hda_input_mux cxt5045_capture_source_benq = {
 	.num_items = 5,
 	.items = {
-		{ "CD",     0x4 },
 		{ "Internal Mic", 0x1 },
 		{ "Mic", 0x2 },
 		{ "Line In", 0x3 },
@@ -820,9 +819,6 @@ static const struct snd_kcontrol_new cxt5045_mixers[] = {
 };
 
 static const struct snd_kcontrol_new cxt5045_benq_mixers[] = {
-	HDA_CODEC_VOLUME("CD Playback Volume", 0x17, 0x4, HDA_INPUT),
-	HDA_CODEC_MUTE("CD Playback Switch", 0x17, 0x4, HDA_INPUT),
-
 	HDA_CODEC_VOLUME("Line In Playback Volume", 0x17, 0x3, HDA_INPUT),
 	HDA_CODEC_MUTE("Line In Playback Switch", 0x17, 0x3, HDA_INPUT),
 
Signature · 1 line
1.7.9.1