search for: drm_underscan_off

...ector_create_standard_properties(struct drm_device *dev) > @@ -1108,6 +1133,101 @@ int drm_mode_create_tv_properties(struct drm_device *dev, > } > EXPORT_SYMBOL(drm_mode_create_tv_properties); > > +static const struct drm_prop_enum_list drm_underscan_mode_enum_list[] = { > + { DRM_UNDERSCAN_OFF, "off" }, > + { DRM_UNDERSCAN_ON, "on" }, > + { DRM_UNDERSCAN_AUTO, "auto" }, > +}; > + > +/** > + * drm_connector_attach_underscan_properties - attach atomic underscan > + * properties > + * @connector: connector to attach underscan...

Hello, This is an attempt at providing generic support for underscan connector props. We already have 3 drivers defining the same underscan, underscan vborder and underscan hborder properties (amd, radeon and nouveau) and I am about to add a new one, hence my proposal to put the prop parsing code in the core and add ->underscan fields to drm_connector_state. Note that I use this new

...e bottom. */ int drm_connector_create_standard_properties(struct drm_device *dev) @@ -1108,6 +1133,101 @@ int drm_mode_create_tv_properties(struct drm_device *dev, } EXPORT_SYMBOL(drm_mode_create_tv_properties); +static const struct drm_prop_enum_list drm_underscan_mode_enum_list[] = { + { DRM_UNDERSCAN_OFF, "off" }, + { DRM_UNDERSCAN_ON, "on" }, + { DRM_UNDERSCAN_AUTO, "auto" }, +}; + +/** + * drm_connector_attach_underscan_properties - attach atomic underscan + * properties + * @connector: connector to attach underscan mode properties on. + * @mode_mask: bitm...

...pendent). */ int drm_connector_create_standard_properties(struct drm_device *dev) @@ -1241,6 +1273,101 @@ int drm_mode_create_tv_properties(struct drm_device *dev, } EXPORT_SYMBOL(drm_mode_create_tv_properties); +static const struct drm_prop_enum_list drm_underscan_mode_enum_list[] = { + { DRM_UNDERSCAN_OFF, "off" }, + { DRM_UNDERSCAN_ON, "on" }, + { DRM_UNDERSCAN_AUTO, "auto" }, +}; + +/** + * drm_connector_attach_underscan_properties - attach atomic underscan + * properties + * @connector: connector to attach underscan mode properties on. + * @mode_mask: bitm...

Hello, This is an attempt at providing generic support for underscan connector props. We already have 3 drivers defining the same underscan, underscan vborder and underscan hborder properties (amd, radeon and nouveau) and I am about to add a new one, hence my proposal to put the prop parsing code in the core and add ->underscan fields to drm_connector_state. This v3 was based on the "VC4

...es(struct drm_device *dev) > > @@ -1108,6 +1133,101 @@ int drm_mode_create_tv_properties(struct drm_device *dev, > > } > > EXPORT_SYMBOL(drm_mode_create_tv_properties); > > > > +static const struct drm_prop_enum_list drm_underscan_mode_enum_list[] = { > > + { DRM_UNDERSCAN_OFF, "off" }, > > + { DRM_UNDERSCAN_ON, "on" }, > > + { DRM_UNDERSCAN_AUTO, "auto" }, > > +}; > > + > > +/** > > + * drm_connector_attach_underscan_properties - attach atomic underscan > > + * properties > > + * @c...

...rm_connector_create_standard_properties(struct drm_device *dev) >@@ -1241,6 +1273,101 @@ int drm_mode_create_tv_properties(struct drm_device *dev, > } > EXPORT_SYMBOL(drm_mode_create_tv_properties); > >+static const struct drm_prop_enum_list drm_underscan_mode_enum_list[] = { >+ { DRM_UNDERSCAN_OFF, "off" }, >+ { DRM_UNDERSCAN_ON, "on" }, >+ { DRM_UNDERSCAN_AUTO, "auto" }, >+}; >+ >+/** >+ * drm_connector_attach_underscan_properties - attach atomic underscan >+ * properties >+ * @connector: connector to attach underscan mode pro...

Hello, This is an attempt at providing generic support for underscan connector props. We already have 3 drivers defining the same underscan, underscan vborder and underscan hborder properties (amd, radeon and nouveau) and I am about to add a new one, hence my proposal to put the prop parsing code in the core and add ->underscan fields to drm_connector_state. In this v2, I also converted the

...ector_create_standard_properties(struct drm_device *dev) > @@ -1108,6 +1133,101 @@ int drm_mode_create_tv_properties(struct drm_device *dev, > } > EXPORT_SYMBOL(drm_mode_create_tv_properties); > > +static const struct drm_prop_enum_list drm_underscan_mode_enum_list[] = { > + { DRM_UNDERSCAN_OFF, "off" }, > + { DRM_UNDERSCAN_ON, "on" }, > + { DRM_UNDERSCAN_AUTO, "auto" }, > +}; > + > +/** > + * drm_connector_attach_underscan_properties - attach atomic underscan > + * properties > + * @connector: connector to attach underscan...

...vborder limit is arbitrarily set to 1024 which + * should be more than enough for real use cases. Note that the actual + * limitation comes from the display mode: + * hborder < hdisplay && vborder < vdisplay + */ + drm_connector_attach_underscan_properties(connector, + BIT(DRM_UNDERSCAN_OFF) | + BIT(DRM_UNDERSCAN_ON), + 1024, 1024); + connector->polled = (DRM_CONNECTOR_POLL_CONNECT | DRM_CONNECTOR_POLL_DISCONNECT); @@ -408,6 +418,9 @@ static void vc4_hdmi_write_infoframe(struct drm_encoder *encoder, static void vc4_hdmi_set_avi_infoframe(struct drm_encod...

...vborder limit is arbitrarily set to 1024 which + * should be more than enough for real use cases. Note that the actual + * limitation comes from the display mode: + * hborder < hdisplay && vborder < vdisplay + */ + drm_connector_attach_underscan_properties(connector, + BIT(DRM_UNDERSCAN_OFF) | + BIT(DRM_UNDERSCAN_ON), + 1024, 1024); + connector->polled = (DRM_CONNECTOR_POLL_CONNECT | DRM_CONNECTOR_POLL_DISCONNECT); @@ -408,6 +418,9 @@ static void vc4_hdmi_write_infoframe(struct drm_encoder *encoder, static void vc4_hdmi_set_avi_infoframe(struct drm_encod...

...ses. Note that the actual > + * limitation comes from the display mode: > + * hborder < hdisplay && vborder < vdisplay > + */ > + drm_connector_attach_underscan_properties(connector, We should probably sprinkle __must_check over all these :-) -Daniel > + BIT(DRM_UNDERSCAN_OFF) | > + BIT(DRM_UNDERSCAN_ON), > + 1024, 1024); > + > connector->polled = (DRM_CONNECTOR_POLL_CONNECT | > DRM_CONNECTOR_POLL_DISCONNECT); > > @@ -408,6 +418,9 @@ static void vc4_hdmi_write_infoframe(struct drm_encoder *encoder, > static void vc4...