Den 29.09.2022 18.31, skrev Maxime Ripard: > The TV mode property has been around for a while now to select and get the > current TV mode output on an analog TV connector. > > Despite that property name being generic, its content isn't and has been > driver-specific which makes it hard to build any generic behaviour on top > of it, both in kernel and user-space. > > Let's create a new enum tv norm property, that can contain any of the > analog TV standards currently supported by kernel drivers. Each driver can > then pass in a bitmask of the modes it supports, and the property > creation function will filter out the modes not supported. > > We'll then be able to phase out the older tv mode property. > > Signed-off-by: Maxime Ripard <maxime@xxxxxxxxxx> > > --- > Changes in v4: > - Add property documentation to kms-properties.csv > - Fix documentation > --- > Documentation/gpu/kms-properties.csv | 1 + > drivers/gpu/drm/drm_atomic_uapi.c | 4 +++ > drivers/gpu/drm/drm_connector.c | 57 +++++++++++++++++++++++++++++++- > include/drm/drm_connector.h | 64 ++++++++++++++++++++++++++++++++++++ > include/drm/drm_mode_config.h | 8 +++++ > 5 files changed, 133 insertions(+), 1 deletion(-) > > diff --git a/Documentation/gpu/kms-properties.csv b/Documentation/gpu/kms-properties.csv > index 45c12e3e82f4..3498bd5d5856 100644 > --- a/Documentation/gpu/kms-properties.csv > +++ b/Documentation/gpu/kms-properties.csv > @@ -91,6 +91,7 @@ omap,Generic,“zorder”,RANGE,"Min=0, Max=3","CRTC, Plane",TBD > qxl,Generic,"“hotplug_mode_update""",RANGE,"Min=0, Max=1",Connector,TBD > radeon,DVI-I,“coherent”,RANGE,"Min=0, Max=1",Connector,TBD > ,DAC enable load detect,“load detection”,RANGE,"Min=0, Max=1",Connector,TBD > +,TV Mode,"""TV Mode""",ENUM,"{ ""NTSC"", ""NTSC-443"", ""NTSC-J"", ""PAL"", ""PAL-M"", ""PAL-N"", ""SECAM"" }",Connector,TBD > ,legacy TMDS PLL detect,"""tmds_pll""",ENUM,"{ ""driver"", ""bios"" }",-,TBD > ,Underscan,"""underscan""",ENUM,"{ ""off"", ""on"", ""auto"" }",Connector,TBD > ,,"""underscan hborder""",RANGE,"Min=0, Max=128",Connector,TBD Turns out I was wrong about adding the property to this file, Daniel says it's deprecated in f0f0657b108c ("drm/doc: Drop "content type" from the legacy kms property table"). If you look at the Fixes commit it adds a kernel doc HDMI property section and TV should probably have something like that. Noralf.