Hi, Thanks again for that thorough review :)
On Thu, May 23, 2024 at 01:22:56PM GMT, Dmitry Baryshkov wrote:
> On Tue, May 21, 2024 at 12:13:50PM +0200, Maxime Ripard wrote:
> > The i915 driver has a property to force the RGB range of an HDMI output.
> > The vc4 driver then implemented the same property with the same
> > semantics. KWin has support for it, and a PR for mutter is also there to
> > support it.
> >
> > Both drivers implementing the same property with the same semantics,
> > plus the userspace having support for it, is proof enough that it's
> > pretty much a de-facto standard now and we can provide helpers for it.
> >
> > Let's plumb it into the newly created HDMI connector.
> >
> > Reviewed-by: Dave Stevenson <dave.steven...@raspberrypi.com>
> > Acked-by: Pekka Paalanen <pekka.paala...@collabora.com>
> > Reviewed-by: Sebastian Wick <sebastian.w...@redhat.com>
> > Signed-off-by: Maxime Ripard <mrip...@kernel.org>
> > ---
> > Documentation/gpu/kms-properties.csv | 1 -
> > drivers/gpu/drm/display/drm_hdmi_state_helper.c | 4 +-
> > drivers/gpu/drm/drm_atomic.c | 2 +
> > drivers/gpu/drm/drm_atomic_uapi.c | 4 ++
> > drivers/gpu/drm/drm_connector.c | 88
> > +++++++++++++++++++++++++
> > include/drm/drm_connector.h | 36 ++++++++++
> > 6 files changed, 133 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/gpu/kms-properties.csv
> > b/Documentation/gpu/kms-properties.csv
> > index 0f9590834829..caef14c532d4 100644
> > --- a/Documentation/gpu/kms-properties.csv
> > +++ b/Documentation/gpu/kms-properties.csv
> > @@ -15,11 +15,10 @@ Owner Module/Drivers,Group,Property Name,Type,Property
> > Values,Object attached,De
> > ,,“saturation”,RANGE,"Min=0, Max=100",Connector,TBD
> > ,,“hue”,RANGE,"Min=0, Max=100",Connector,TBD
> > ,Virtual GPU,“suggested X”,RANGE,"Min=0,
> > Max=0xffffffff",Connector,property to suggest an X offset for a connector
> > ,,“suggested Y”,RANGE,"Min=0, Max=0xffffffff",Connector,property to
> > suggest an Y offset for a connector
> > ,Optional,"""aspect ratio""",ENUM,"{ ""None"", ""4:3"", ""16:9""
> > }",Connector,TDB
> > -i915,Generic,"""Broadcast RGB""",ENUM,"{ ""Automatic"", ""Full"",
> > ""Limited 16:235"" }",Connector,"When this property is set to Limited
> > 16:235 and CTM is set, the hardware will be programmed with the result of
> > the multiplication of CTM by the limited range matrix to ensure the pixels
> > normally in the range 0..1.0 are remapped to the range 16/255..235/255."
>
> Should it still be defined as a generic property?
I'm not sure what you mean here, sorry. It's being documented as a
connector property now, so it's very much still listed as a generic
property?
Maxime
signature.asc
Description: PGP signature
