On 07/27/2015 06:52 PM, Takashi Iwai wrote:
> On Mon, 27 Jul 2015 12:34:31 +0200,
> <email address hidden> wrote:
>> From: Woodrow Shen <email address hidden>
>>
>> The new Dell laptop with codec 256 can't detect headset mic when headset was
>> inserted on the machine. From alsa-info, we check init_pin_configs and need to
>> define the new register value for pin 0x1d & 0x1e because the original macro
>> ALC256_STANDARD_PINS can't match pin definition. Also, the macro ALC256_STANDARD_PINS
>> is simplified by removing them. This makes headset mic works on laptop.
>>
>> Codec: Realtek ALC256
>> Vendor Id: 0x10ec0256
>> Subsystem Id: 0x102806f2
>>
>> BugLink: https://bugs.launchpad.net/bugs/1478497
>> Signed-off-by: Woodrow Shen <email address hidden>
> I applied this as is now. But the code there is already messy, and I
> wonder whether we should treat all 0x4xxxxxxx equally. So far, all
> pincfgs are regarded as a kind of fingerprint. But, judging from the
> actual values, the difference of 0x4xxxxxxx might be just a leftover
> of wastes by BIOS programmers.
>
> I don't mind any other way, but a sane cleanup would be appreciated.
>
>
> thanks,
>
> Takashi
Something like below?
remove all 0x4xxxxxxx from pin quirk table
rewrite the pin_config_match()
- first, compare all pins in a quirk table with the corresponding pin
on the machine, if all pins get a matching,
- second, compare all pins on the machine with the corresponding pin
in the quirk table
if the pin is non-0x4xxxxxxx, it needs match a corresponding
pin in the quirk table, otherwise, it return false
if the pin is 0x4xxxxxxx, the corresponding pin in the quirk
table is 0x4xxxxxxxx as well, it matches; if the corresponding pin in
the quirk table is non-0x4xxxxxxx, it return false
if the pin is 0x4xxxxxxx, and can't find the corresponding pin
in the quirk table, it matches.
diff --git a/sound/pci/hda/hda_auto_parser.c
b/sound/pci/hda/hda_auto_parser.c
index 03b7399..2639fc3 100644
--- a/sound/pci/hda/hda_auto_parser.c
+++ b/sound/pci/hda/hda_auto_parser.c
@@ -887,11 +887,38 @@ EXPORT_SYMBOL_GPL(snd_hda_apply_fixup);
static bool pin_config_match(struct hda_codec *codec, const struct hda_pintbl *pins)
{
- for (; pins->nid; pins++) {
- u32 def_conf = snd_hda_codec_get_pincfg(codec, pins->nid);
- if (pins->val != def_conf)
+ const struct hda_pintbl *t_pins = pins;
+ int i;
+
+ for (; t_pins->nid; t_pins++) {
+ u32 def_conf = snd_hda_codec_get_pincfg(codec, t_pins->nid);
+ if (t_pins->val != def_conf)
+ return false;
+ }
+
+ for (i = 0; i < codec->init_pins.used; i++) {
+ struct hda_pincfg *pin =
snd_array_elem(&codec->init_pins, i);
+ hda_nid_t nid = pin->nid;
+ u32 cfg = pin->cfg;
+ int found;
+
+ t_pins = pins;
+ found = 0;
+ for (; t_pins->nid; t_pins++) {
+ if (t_pins->nid == nid) {
+ found = 1;
+ if (t_pins->val == cfg)
+ break;
+ else if ((cfg & 0xf0000000) ==
0x40000000 && (t_pins->val & 0xf0000000) == 0x40000000)
+ break;
+ else
+ return false;
+ }
+ }
+ if (!found && (cfg & 0xf0000000) != 0x40000000) return false;
}
+
return true;
}
On 07/27/2015 06:52 PM, Takashi Iwai wrote: STANDARD_ PINS can't match pin definition. Also, the macro ALC256_ STANDARD_ PINS /bugs.launchpad .net/bugs/ 1478497
> On Mon, 27 Jul 2015 12:34:31 +0200,
> <email address hidden> wrote:
>> From: Woodrow Shen <email address hidden>
>>
>> The new Dell laptop with codec 256 can't detect headset mic when headset was
>> inserted on the machine. From alsa-info, we check init_pin_configs and need to
>> define the new register value for pin 0x1d & 0x1e because the original macro
>> ALC256_
>> is simplified by removing them. This makes headset mic works on laptop.
>>
>> Codec: Realtek ALC256
>> Vendor Id: 0x10ec0256
>> Subsystem Id: 0x102806f2
>>
>> BugLink: https:/
>> Signed-off-by: Woodrow Shen <email address hidden>
> I applied this as is now. But the code there is already messy, and I
> wonder whether we should treat all 0x4xxxxxxx equally. So far, all
> pincfgs are regarded as a kind of fingerprint. But, judging from the
> actual values, the difference of 0x4xxxxxxx might be just a leftover
> of wastes by BIOS programmers.
>
> I don't mind any other way, but a sane cleanup would be appreciated.
>
>
> thanks,
>
> Takashi
Something like below?
remove all 0x4xxxxxxx from pin quirk table
rewrite the pin_config_match()
- first, compare all pins in a quirk table with the corresponding pin
on the machine, if all pins get a matching,
- second, compare all pins on the machine with the corresponding pin
in the quirk table
if the pin is non-0x4xxxxxxx, it needs match a corresponding
pin in the quirk table, otherwise, it return false
if the pin is 0x4xxxxxxx, the corresponding pin in the quirk
table is 0x4xxxxxxxx as well, it matches; if the corresponding pin in
the quirk table is non-0x4xxxxxxx, it return false
if the pin is 0x4xxxxxxx, and can't find the corresponding pin
in the quirk table, it matches.
diff --git a/sound/ pci/hda/ hda_auto_ parser. c pci/hda/ hda_auto_ parser. c pci/hda/ hda_auto_ parser. c pci/hda/ hda_auto_ parser. c SYMBOL_ GPL(snd_ hda_apply_ fixup); match(struct hda_codec *codec,
const struct hda_pintbl *pins) codec_get_ pincfg( codec, pins->nid); codec_get_ pincfg( codec, t_pins->nid); init_pins. used; i++) { elem(&codec- >init_pins, i);
return false;
b/sound/
index 03b7399..2639fc3 100644
--- a/sound/
+++ b/sound/
@@ -887,11 +887,38 @@ EXPORT_
static bool pin_config_
{
- for (; pins->nid; pins++) {
- u32 def_conf = snd_hda_
- if (pins->val != def_conf)
+ const struct hda_pintbl *t_pins = pins;
+ int i;
+
+ for (; t_pins->nid; t_pins++) {
+ u32 def_conf = snd_hda_
+ if (t_pins->val != def_conf)
+ return false;
+ }
+
+ for (i = 0; i < codec->
+ struct hda_pincfg *pin =
snd_array_
+ hda_nid_t nid = pin->nid;
+ u32 cfg = pin->cfg;
+ int found;
+
+ t_pins = pins;
+ found = 0;
+ for (; t_pins->nid; t_pins++) {
+ if (t_pins->nid == nid) {
+ found = 1;
+ if (t_pins->val == cfg)
+ break;
+ else if ((cfg & 0xf0000000) ==
0x40000000 && (t_pins->val & 0xf0000000) == 0x40000000)
+ break;
+ else
+ return false;
+ }
+ }
+ if (!found && (cfg & 0xf0000000) != 0x40000000)
}
+
return true;
}
diff --git a/sound/ pci/hda/ patch_realtek. c b/sound/ pci/hda/ patch_realtek. c pci/hda/ patch_realtek. c pci/hda/ patch_realtek. c fixup_models[ ] = {
index 18ae17e..0fda442 100644
--- a/sound/
+++ b/sound/
@@ -5386,399 +5386,207 @@ static const struct hda_model_fixup
alc269_
{}
};
-#define ALC255_ STANDARD_ PINS \ STANDARD_ PINS \
- {0x18, 0x411111f0}, \
- {0x19, 0x411111f0}, \
- {0x1a, 0x411111f0}, \
- {0x1b, 0x411111f0}, \
- {0x1e, 0x411111f0}
-
#define ALC256_
{0x12, 0x90a60140}, \
{0x14, 0x90170110}, \
- {0x19, 0x411111f0}, \
- {0x1a, 0x411111f0}, \
- {0x1b, 0x411111f0}, \
{0x21, 0x02211020}
#define ALC282_ STANDARD_ PINS \ STANDARD_ PINS \
- {0x14, 0x90170110}, \
- {0x18, 0x411111f0}, \
- {0x1a, 0x411111f0}, \
- {0x1b, 0x411111f0}, \
- {0x1e, 0x411111f0}
-
-#define ALC288_
- {0x17, 0x411111f0}, \
- {0x18, 0x411111f0}, \
- {0x19, 0x411111f0}, \
- {0x1a, 0x411111f0}, \
- {0x1e, 0x411111f0}
+ {0x14, 0x90170110}
#define ALC290_ STANDARD_ PINS \
- {0x12, 0x99a30130}, \
- {0x13, 0x40000000}, \
- {0x16, 0x411111f0}, \
- {0x17, 0x411111f0}, \
- {0x19, 0x411111f0}, \
- {0x1b, 0x411111f0}, \
- {0x1e, 0x411111f0}
+ {0x12, 0x99a30130}
#define ALC292_ STANDARD_ PINS \ STANDARD_ PINS \
{0x14, 0x90170110}, \
- {0x15, 0x0221401f}, \
- {0x1a, 0x411111f0}, \
- {0x1b, 0x411111f0}, \
- {0x1d, 0x40700001}
-
-#define ALC298_
- {0x18, 0x411111f0}, \
- {0x19, 0x411111f0}, \
- {0x1a, 0x411111f0}, \
- {0x1e, 0x411111f0}, \
- {0x1f, 0x411111f0}
+ {0x15, 0x0221401f}
static const struct snd_hda_pin_quirk alc269_ pin_fixup_ tbl[] = {
SND_HDA_ PIN_QUIRK( 0x10ec0255, 0x1028, "Dell", FIXUP_DELL2_ MIC_NO_ PRESENCE, STANDARD_ PINS,
{0x14, 0x90170110},
{0x21, 0x02211020}),
SND_HDA_ PIN_QUIRK( 0x10ec0255, 0x1028, "Dell", FIXUP_DELL1_ MIC_NO_ PRESENCE, STANDARD_ PINS,
{0x12, 0x90a60140},
{0x14, 0x90170110},
{0x21, 0x02211020}),
SND_HDA_ PIN_QUIRK( 0x10ec0255, 0x1028, "Dell", FIXUP_DELL1_ MIC_NO_ PRESENCE, STANDARD_ PINS,
{0x12, 0x90a60160},
{0x14, 0x90170120},
{0x21, 0x02211030}),
SND_HDA_ PIN_QUIRK( 0x10ec0255, 0x1028, "Dell", FIXUP_DELL1_ MIC_NO_ PRESENCE,
{0x14, 0x90170130},
{0x1b, 0x01014020},
{0x21, 0x0221103f}),
SND_HDA_ PIN_QUIRK( 0x10ec0255, 0x1028, "Dell", FIXUP_DELL1_ MIC_NO_ PRESENCE,
{0x14, 0x90170150},
{0x1b, 0x02011020},
{0x21, 0x0221105f}),
ALC255_
- ALC255_
- {0x12, 0x40300000},
- {0x17, 0x411111f0},
- {0x1d, 0x40538029},
ALC255_
- ALC255_
- {0x17, 0x40000000},
- {0x1d, 0x40700001},
ALC255_
- ALC255_
- {0x17, 0x40000000},
- {0x1d, 0x40700001},
ALC255_
- {0x12, 0x40000000},
- {0x17, 0x411111f0},
- {0x18, 0x411111f0},
- {0x19, 0x411111f0},
- {0x1a, 0x411111f0},
- {0x1d, 0x4054c029},
- {0x1e, 0x411111f0},
ALC255_
- {0x12, 0x40000000},
- {0x17, 0x411111f0},
- {0x18, 0x411111f0},
- {0x19, 0x411111f0},
- {0x1a, 0x411111f0},
- {0x1d, 0x4054c029},
- {0x1e, 0x411111f0},
..........
>> --- hda/patch_ realtek. c | 15 +++++++++++---- pci/hda/ patch_realtek. c b/sound/ pci/hda/ patch_realtek. c pci/hda/ patch_realtek. c pci/hda/ patch_realtek. c fixup_models[ ] = { STANDARD_ PINS \ pin_fixup_ tbl[] = { PIN_QUIRK( 0x10ec0256, 0x1028, "Dell", ALC255_ FIXUP_DELL1_ MIC_NO_ PRESENCE, STANDARD_ PINS, PIN_QUIRK( 0x10ec0256, 0x1028, "Dell", ALC255_ FIXUP_DELL1_ MIC_NO_ PRESENCE, STANDARD_ PINS, PIN_QUIRK( 0x10ec0256, 0x1028, "Dell", ALC255_ FIXUP_DELL1_ MIC_NO_ PRESENCE, STANDARD_ PINS, PIN_QUIRK( 0x10ec0280, 0x103c, "HP", ALC280_ FIXUP_HP_ GPIO4, _______ _______ _______ _______ _______ _____ mailman. alsa-project. org/mailman/ listinfo/ alsa-devel
>> sound/pci/
>> 1 file changed, 11 insertions(+), 4 deletions(-)
>>
>> diff --git a/sound/
>> index d94c0e3..4ae877c 100644
>> --- a/sound/
>> +++ b/sound/
>> @@ -5398,8 +5398,6 @@ static const struct hda_model_fixup alc269_
>> {0x19, 0x411111f0}, \
>> {0x1a, 0x411111f0}, \
>> {0x1b, 0x411111f0}, \
>> - {0x1d, 0x40700001}, \
>> - {0x1e, 0x411111f0}, \
>> {0x21, 0x02211020}
>>
>> #define ALC282_
>> @@ -5556,10 +5554,19 @@ static const struct snd_hda_pin_quirk alc269_
>> {0x21, 0x02211030}),
>> SND_HDA_
>> ALC256_
>> - {0x13, 0x40000000}),
>> + {0x13, 0x40000000},
>> + {0x1d, 0x40700001},
>> + {0x1e, 0x411111f0}),
>> SND_HDA_
>> ALC256_
>> - {0x13, 0x411111f0}),
>> + {0x13, 0x411111f0},
>> + {0x1d, 0x40700001},
>> + {0x1e, 0x411111f0}),
>> + SND_HDA_
>> + ALC256_
>> + {0x13, 0x411111f0},
>> + {0x1d, 0x4077992d},
>> + {0x1e, 0x411111ff}),
>> SND_HDA_
>> {0x12, 0x90a60130},
>> {0x13, 0x40000000},
>> --
>> 2.1.4
>>
> _______
> Alsa-devel mailing list
> <email address hidden>
> http://
>