[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