ALSA: hda - hdmi: Fix wrong baseline length in ATI/AMD generated ELD
authorAnssi Hannula <anssi.hannula@iki.fi>
Sun, 10 Nov 2013 20:57:42 +0000 (22:57 +0200)
committerTakashi Iwai <tiwai@suse.de>
Mon, 11 Nov 2013 16:08:39 +0000 (17:08 +0100)
According to the HDA specification the baseline ELD length is counted in
DW of 4 bytes instead of in bytes.

Fix the code accordingly.

Baseline length is not used by the kernel so only the ELD exported to
userspace was affected. No issues have been reported.

v2: Fixed so that eld_size is adjusted upwards accordingly as well.

Signed-off-by: Anssi Hannula <anssi.hannula@iki.fi>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/hda_eld.c

index 9b697a28cf53e52aa6ece03b40b376d05173cb9c..79ca80f6c77ada6d182a86b9880f3ce9125726f5 100644 (file)
@@ -792,12 +792,15 @@ int snd_hdmi_get_eld_ati(struct hda_codec *codec, hda_nid_t nid,
                /* else unknown/invalid or 0ms or video ahead of audio, so use zero */
        }
 
-       /* Baseline length */
-       buf[2] = pos - 4;
-
        /* SAD count */
        buf[5] |= ((pos - ELD_FIXED_BYTES - sink_desc_len) / 3) << 4;
 
+       /* Baseline ELD block length is 4-byte aligned */
+       pos = round_up(pos, 4);
+
+       /* Baseline ELD length (4-byte header is not counted in) */
+       buf[2] = (pos - 4) / 4;
+
        *eld_size = pos;
 
        return 0;