Bug 89348

Summary:	edid-decode reports sample rate units for bit rate
Product:	xorg	Reporter:	Kip Warner <Kip>
Component:	App/edid-decode	Assignee:	Xorg Project Team <xorg-team>
Status:	RESOLVED FIXED	QA Contact:	Xorg Project Team <xorg-team>
Severity:	normal
Priority:	medium	CC:	ajax, Kip
Version:	git
Hardware:	All
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Kip Warner 2015-02-27 02:58:12 UTC

The edid-decode utility reports sample rate for non-LPCM Short Audio Descriptors it detects when it meant bit rate.

Consider the following:

./edid-decode /sys/class/drm/card0/card0-HDMI-A-3/edid
...
    DTS, max channels 7
    Supported sample rates (kHz): 48 44.1
    Maximum bit rate: 1016 kHz
...

The Maximum bit rate units should be kilobits per second (kb/s), and not a frequency (kHz). According to the specification for the EIA / CEA-861 Extension Block all non-LPCM formats have the third byte in the SAD's bits 7..0 designate the maximum supported bit rate divided by 8 kbits/s.

Lastly, I am sorry for not having selected edid-decode for component in the bug report, but I wasn't able to find it in the list.

Comment 1 Alan Coopersmith 2018-08-12 20:34:30 UTC

Fixed in:
https://git.linuxtv.org/edid-decode.git/commit/?id=3b26b8a1a5a3c65caf9c6fd90f3e352e35e31441

Thanks for the report.

Comment 2 Kip Warner 2018-08-13 17:50:37 UTC

Thanks Alan. Since the patch was trivial I probably should have done this back in 2015. At the time I just didn't have time. Glad to see it fixed though.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.