<div dir="ltr"><div dir="ltr">On Sun, Jan 24, 2021 at 9:15 AM Simon Ser <<a href="mailto:contact@emersion.fr">contact@emersion.fr</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sunday, January 24th, 2021 at 5:40 AM, Mario Kleiner <<a href="mailto:mario.kleiner.de@gmail.com" target="_blank">mario.kleiner.de@gmail.com</a>> wrote:<br>
<br>
> According to the CTA 861.G spec, HDMI_STATIC_METADATA_TYPE1 is<br>
> not 1, but zero, so fix this enum.<br>
><br>
> While this doesn't cause problems in the kernel yet, as the<br>
> constant isn't actively used by drivers yet, it did create<br>
> confusion while debugging HDR problems in yours truly, and<br>
> also potential bugs in userspace components, as the wrong<br>
> enum propagates to components, e.g., like it did already<br>
> into intel-gpu-tools (tests/kms_hdr.c) or is used as wrong<br>
> reference when writing future new userspace HDR components<br>
> like compositors.<br>
><br>
> Fixes: fbb5d0353c62 ("drm: Add HDR source metadata property")<br>
> Signed-off-by: Mario Kleiner <<a href="mailto:mario.kleiner.de@gmail.com" target="_blank">mario.kleiner.de@gmail.com</a>><br>
> Cc: Uma Shankar <<a href="mailto:uma.shankar@intel.com" target="_blank">uma.shankar@intel.com</a>><br>
> Cc: Shashank Sharma <<a href="mailto:shashank.sharma@intel.com" target="_blank">shashank.sharma@intel.com</a>><br>
> Cc: Ville Syrjälä <<a href="mailto:ville.syrjala@linux.intel.com" target="_blank">ville.syrjala@linux.intel.com</a>><br>
<br>
Sadly I think it's too late for this. Some user-space (like Kodi)<br>
already has copied over the kernel definitions, and libdrm already has<br>
a release with the wrong value.<br></blockquote><div><br></div><div>But it still needs to be fixed if we want working HDR. I thought libdrm copies the definitions from the kernel periodically, so the fix should propagate?</div><div> <br></div></div></div>