* Re:[PATCH -next] ALSA: hda - fix error return code in patch_alc662()
@ 2013-04-18 4:09 Wang Shilong
2013-04-18 4:14 ` [PATCH " Wang Shilong
0 siblings, 1 reply; 2+ messages in thread
From: Wang Shilong @ 2013-04-18 4:09 UTC (permalink / raw)
To: yongjun_wei, Wei Yongjun; +Cc: linux-kernel
Hello Wei,
+++ b/sound/pci/hda/patch_realtek.c
@@ -3787,7 +3787,8 @@ static int patch_alc662(struct hda_codec *codec)
if ((alc_get_coef0(codec) & (1 << 14)) &&
codec->bus->pci->subsystem_vendor == 0x1025 &&
spec->cdefine.platform_type == 1) {
- if (alc_codec_rename(codec, "ALC272X") < 0)
I don't think we should fix this problem. And your changelog is wrong!
Thanks,
Wang
+ err = alc_codec_rename(codec, "ALC272X");
+ if (err < 0)
goto error;
}
--
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH -next] ALSA: hda - fix error return code in patch_alc662()
2013-04-18 4:09 Re:[PATCH -next] ALSA: hda - fix error return code in patch_alc662() Wang Shilong
@ 2013-04-18 4:14 ` Wang Shilong
0 siblings, 0 replies; 2+ messages in thread
From: Wang Shilong @ 2013-04-18 4:14 UTC (permalink / raw)
To: Wang Shilong; +Cc: yongjun_wei, Wei Yongjun, linux-kernel
Sorry, i miss something..
Thanks,
wang
> Hello Wei,
>
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -3787,7 +3787,8 @@ static int patch_alc662(struct hda_codec *codec)
> if ((alc_get_coef0(codec) & (1 << 14)) &&
> codec->bus->pci->subsystem_vendor == 0x1025 &&
> spec->cdefine.platform_type == 1) {
> - if (alc_codec_rename(codec, "ALC272X") < 0)
>
> I don't think we should fix this problem. And your changelog is wrong!
>
> Thanks,
> Wang
> + err = alc_codec_rename(codec, "ALC272X");
> + if (err < 0)
> goto error;
> }
>
>
> --
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-04-18 4:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-18 4:09 Re:[PATCH -next] ALSA: hda - fix error return code in patch_alc662() Wang Shilong
2013-04-18 4:14 ` [PATCH " Wang Shilong
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox