On 11/14/19 4:16 PM, Kai Vehmanen wrote:
Hi Nikhil,
On Thu, 14 Nov 2019, Nikhil Mahale wrote:
This patch adds DP-MST jack support which will be used on NVIDIA platforms. Today, DP-MST audio is supported only if the codec has acomp support. This patch makes it possible to add DP-MST support for non-acomp codecs.
[...]
+++ b/sound/pci/hda/hda_generic.c @@ -4203,7 +4203,7 @@ static bool detect_pin_state(struct hda_codec *codec, hda_nid_t pin) { if (!is_jack_detectable(codec, pin)) return true;
- return snd_hda_jack_detect_state(codec, pin) != HDA_JACK_NOT_PRESENT;
- return snd_hda_jack_detect_state(codec, pin, 0) != HDA_JACK_NOT_PRESENT;
}
the patch adds quite a few checkpatch warnings. E.g. this line over 80 character limit. Run checkpatch yourself or take a look at e.g. https://travis-ci.org/thesofproject/linux/jobs/611787177?utm_medium=notifica...
I'll queued the series to SOF CI to do some testing against regressions on our platforms. I'll report later if anything major found.
Thanks Kai, I have fixed all errors reported by checkpatch.pl, in 2nd version of patch set. Sending out 2nd version of patch set for review.
Thanks, Nikhil Mahale
Br, Kai
----------------------------------------------------------------------------------- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message. -----------------------------------------------------------------------------------