Re: [PATCH 2/2] Move the EDID parsing to its own file

2013-05-17 Thread Bill Spitzak
Graeme Gill wrote: Bill Spitzak wrote: The Y of the primaries can be used as alternative method of specifying the whitepoint. (convert the 3 Yxy colors to XYZ, add them, then convert back to Yxy and the xy is the whitepoint, I think). Correct, but this assumes the display is perfectly

Re: [PATCH 2/2] Move the EDID parsing to its own file

2013-05-16 Thread Graeme Gill
Bill Spitzak wrote: The Y of the primaries can be used as alternative method of specifying the whitepoint. (convert the 3 Yxy colors to XYZ, add them, then convert back to Yxy and the xy is the whitepoint, I think). Correct, but this assumes the display is perfectly additive. Real world

Re: [PATCH 2/2] Move the EDID parsing to its own file

2013-05-06 Thread Bill Spitzak
The Y of the primaries can be used as alternative method of specifying the whitepoint. (convert the 3 Yxy colors to XYZ, add them, then convert back to Yxy and the xy is the whitepoint, I think). I think the reason rgb sets are specfied as 4 pairs of xy (the three primaries and the

Re: [PATCH 2/2] Move the EDID parsing to its own file

2013-05-04 Thread Richard Hughes
On 4 May 2013 00:16, Kai-Uwe Behrmann k...@gmx.de wrote: +struct weston_edid_color_Yxy { + double Y; + double x; + double y; +}; Why is the Y value set when it is all about primaries. No one will ever use that other than for assuming 1.0 . I assumed a standard type would

Re: [PATCH 2/2] Move the EDID parsing to its own file

2013-05-03 Thread Kai-Uwe Behrmann
Am 02.05.2013 23:33, schrieb Richard Hughes: --- src/Makefile.am | 2 + src/compositor-drm.c | 180 +-- src/compositor.c | 21 ++ src/compositor.h | 5 ++ src/edid.c | 175

[PATCH 2/2] Move the EDID parsing to its own file

2013-05-02 Thread Richard Hughes
--- src/Makefile.am | 2 + src/compositor-drm.c | 180 +-- src/compositor.c | 21 ++ src/compositor.h | 5 ++ src/edid.c | 175 + src/edid.h | 48