[PATCH 1/3] Extract the EDID blob when adding a DRM output

Kai-Uwe Behrmann ku.b at gmx.de
Sat Apr 20 06:31:49 PDT 2013


Am 20.04.2013 13:12, schrieb David Herrmann:
> On Fri, Apr 19, 2013 at 9:19 PM, Kai-Uwe Behrmann <ku.b at gmx.de> wrote:
>> Am 19.04.2013 21:12, schrieb David Herrmann:
>>
>>> Why do you keep the blob around all the time? Just free it in
>>> create_output_for_connector() after you parsed it. If we somehow need
>>> it later, we can always change it again. But your patches don't depend
>>> on it being around all the time so lets just free it right away.
>>
>>
>> The EDID blob is exposed under X11, which is a good thing in many cases. Do
>> you have a special reason to suggest otherwise on Wayland/Weston?
>
> This patchset doesn't expose the blob.

Then the patch set is of limited use.

> Why do you want to keep it around then?

EDID is a pretty essential pice of information. It is more helpful to 
expose the EDID data blob completly, than only a few values selectively.

kind regards
Kai-Uwe


More information about the wayland-devel mailing list