]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
ALSA: hda - Revert the check of NO_PRESENCE pincfg default bit
authorTakashi Iwai <tiwai@suse.de>
Sun, 6 Nov 2011 13:01:58 +0000 (14:01 +0100)
committerTakashi Iwai <tiwai@suse.de>
Sun, 6 Nov 2011 13:07:37 +0000 (14:07 +0100)
The implementation on commit [08a1f5eb: ALSA: hda - Check NO_PRESENCE
pincfg default bit] seems like a mis-interpretation of specification.
The spec gives the reversed bit definition.  But, following the spec
also causes to change so many existing device configurations, thus we
can't change it so easily for now.  For 3.2-rc1, it's safer to revert
this check (actually this patch comments out the code).

We may re-introduced the fixed version once after the wider test-case
coverages are done.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/hda_local.h

index 79f49e2e8cbc4f4366a180f5811b68fb21659c99..dcbea0da0fa2e5fea3681897cce0fe04e91ef3ef 100644 (file)
@@ -511,8 +511,11 @@ int snd_hda_jack_detect(struct hda_codec *codec, hda_nid_t nid);
 static inline bool is_jack_detectable(struct hda_codec *codec, hda_nid_t nid)
 {
        return (snd_hda_query_pin_caps(codec, nid) & AC_PINCAP_PRES_DETECT) &&
-               !(get_defcfg_misc(snd_hda_codec_get_pincfg(codec, nid) &
-                 AC_DEFCFG_MISC_NO_PRESENCE)) &&
+               /* disable MISC_NO_PRESENCE check because it may break too
+                * many devices
+                */
+               /*(get_defcfg_misc(snd_hda_codec_get_pincfg(codec, nid) &
+                 AC_DEFCFG_MISC_NO_PRESENCE)) &&*/
                (get_wcaps(codec, nid) & AC_WCAP_UNSOL_CAP);
 }