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 addi

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 whitepoint)

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 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 +