On Friday, May 26th, 2023 at 18:40, Sebastian Wick <sebastian.wick@xxxxxxxxxx> wrote: > On Thu, May 25, 2023 at 9:18 PM Harry Wentland harry.wentland@xxxxxxx wrote: > > > We want compositors to be able to set the output > > colorspace on DP and HDMI outputs, based on the > > caps reported from the receiver via EDID. > > > This commit message seems wrong for multiple reasons. The Colorspace > property documentation explicitly says that user space has to check > the sink EDID for support: > > > The expectation from userspace is that it should parse the EDID and get supported colorspaces. > > The code doesn't seem to care about EDID at all. Instead it exposes > the variants it knows how to support with e.g. the appropriate YCC > transform when necessary, independent of the sink support. The commit messages explains that compositors will parse the EDID. It doesn't state that the kernel does.