[PATCH 1/2] Revert "drm: Introduce 'power saving policy' drm property"

Harry Wentland harry.wentland at amd.com
Fri Aug 2 15:18:58 UTC 2024


On 2024-08-02 10:59, Hamza Mahfooz wrote:
> This reverts commit 76299a557f36d624ca32500173ad7856e1ad93c0.
> 
> It was merged without meeting userspace requirements.
> 
> Signed-off-by: Hamza Mahfooz <hamza.mahfooz at amd.com>

Series is:
Reviewed-by: Harry Wentland <harry.wentland at amd.com>

Harry

> ---
>  drivers/gpu/drm/drm_connector.c | 48 ---------------------------------
>  include/drm/drm_connector.h     |  2 --
>  include/drm/drm_mode_config.h   |  5 ----
>  include/uapi/drm/drm_mode.h     |  7 -----
>  4 files changed, 62 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c
> index 7c44e3a1d8e0..b4f4d2f908d1 100644
> --- a/drivers/gpu/drm/drm_connector.c
> +++ b/drivers/gpu/drm/drm_connector.c
> @@ -1043,11 +1043,6 @@ static const struct drm_prop_enum_list drm_scaling_mode_enum_list[] = {
>  	{ DRM_MODE_SCALE_ASPECT, "Full aspect" },
>  };
>  
> -static const struct drm_prop_enum_list drm_power_saving_policy_enum_list[] = {
> -	{ __builtin_ffs(DRM_MODE_REQUIRE_COLOR_ACCURACY) - 1, "Require color accuracy" },
> -	{ __builtin_ffs(DRM_MODE_REQUIRE_LOW_LATENCY) - 1, "Require low latency" },
> -};
> -
>  static const struct drm_prop_enum_list drm_aspect_ratio_enum_list[] = {
>  	{ DRM_MODE_PICTURE_ASPECT_NONE, "Automatic" },
>  	{ DRM_MODE_PICTURE_ASPECT_4_3, "4:3" },
> @@ -1634,16 +1629,6 @@ EXPORT_SYMBOL(drm_hdmi_connector_get_output_format_name);
>   *
>   *	Drivers can set up these properties by calling
>   *	drm_mode_create_tv_margin_properties().
> - * power saving policy:
> - *	This property is used to set the power saving policy for the connector.
> - *	This property is populated with a bitmask of optional requirements set
> - *	by the drm master for the drm driver to respect:
> - *	- "Require color accuracy": Disable power saving features that will
> - *	  affect color fidelity.
> - *	  For example: Hardware assisted backlight modulation.
> - *	- "Require low latency": Disable power saving features that will
> - *	  affect latency.
> - *	  For example: Panel self refresh (PSR)
>   */
>  
>  int drm_connector_create_standard_properties(struct drm_device *dev)
> @@ -2146,39 +2131,6 @@ int drm_mode_create_scaling_mode_property(struct drm_device *dev)
>  }
>  EXPORT_SYMBOL(drm_mode_create_scaling_mode_property);
>  
> -/**
> - * drm_mode_create_power_saving_policy_property - create power saving policy property
> - * @dev: DRM device
> - * @supported_policies: bitmask of supported power saving policies
> - *
> - * Called by a driver the first time it's needed, must be attached to desired
> - * connectors.
> - *
> - * Returns: %0
> - */
> -int drm_mode_create_power_saving_policy_property(struct drm_device *dev,
> -						 uint64_t supported_policies)
> -{
> -	struct drm_property *power_saving;
> -
> -	if (dev->mode_config.power_saving_policy)
> -		return 0;
> -	WARN_ON((supported_policies & DRM_MODE_POWER_SAVING_POLICY_ALL) == 0);
> -
> -	power_saving =
> -		drm_property_create_bitmask(dev, 0, "power saving policy",
> -					    drm_power_saving_policy_enum_list,
> -					    ARRAY_SIZE(drm_power_saving_policy_enum_list),
> -					    supported_policies);
> -	if (!power_saving)
> -		return -ENOMEM;
> -
> -	dev->mode_config.power_saving_policy = power_saving;
> -
> -	return 0;
> -}
> -EXPORT_SYMBOL(drm_mode_create_power_saving_policy_property);
> -
>  /**
>   * DOC: Variable refresh properties
>   *
> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h
> index 5ad735253413..e3fa43291f44 100644
> --- a/include/drm/drm_connector.h
> +++ b/include/drm/drm_connector.h
> @@ -2267,8 +2267,6 @@ int drm_mode_create_dp_colorspace_property(struct drm_connector *connector,
>  					   u32 supported_colorspaces);
>  int drm_mode_create_content_type_property(struct drm_device *dev);
>  int drm_mode_create_suggested_offset_properties(struct drm_device *dev);
> -int drm_mode_create_power_saving_policy_property(struct drm_device *dev,
> -						 uint64_t supported_policies);
>  
>  int drm_connector_set_path_property(struct drm_connector *connector,
>  				    const char *path);
> diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h
> index 150f9a3b649f..ab0f167474b1 100644
> --- a/include/drm/drm_mode_config.h
> +++ b/include/drm/drm_mode_config.h
> @@ -969,11 +969,6 @@ struct drm_mode_config {
>  	 */
>  	struct drm_atomic_state *suspend_state;
>  
> -	/**
> -	 * @power_saving_policy: bitmask for power saving policy requests.
> -	 */
> -	struct drm_property *power_saving_policy;
> -
>  	const struct drm_mode_config_helper_funcs *helper_private;
>  };
>  
> diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
> index 880303c2ad97..d390011b89b4 100644
> --- a/include/uapi/drm/drm_mode.h
> +++ b/include/uapi/drm/drm_mode.h
> @@ -152,13 +152,6 @@ extern "C" {
>  #define DRM_MODE_SCALE_CENTER		2 /* Centered, no scaling */
>  #define DRM_MODE_SCALE_ASPECT		3 /* Full screen, preserve aspect */
>  
> -/* power saving policy options */
> -#define DRM_MODE_REQUIRE_COLOR_ACCURACY	BIT(0)	/* Compositor requires color accuracy */
> -#define DRM_MODE_REQUIRE_LOW_LATENCY	BIT(1)	/* Compositor requires low latency */
> -
> -#define DRM_MODE_POWER_SAVING_POLICY_ALL	(DRM_MODE_REQUIRE_COLOR_ACCURACY |\
> -						 DRM_MODE_REQUIRE_LOW_LATENCY)
> -
>  /* Dithering mode options */
>  #define DRM_MODE_DITHERING_OFF	0
>  #define DRM_MODE_DITHERING_ON	1



More information about the amd-gfx mailing list