Bug 89348 - edid-decode reports sample rate units for bit rate
Summary: edid-decode reports sample rate units for bit rate
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: App/edid-decode (show other bugs)
Version: git
Hardware: All All
: medium normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-27 02:58 UTC by Kip Warner
Modified: 2018-08-13 17:50 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments

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.