[PATCH 3/3] drm/doc: fix VRR_ENABLED casing

Kazlauskas, Nicholas Nicholas.Kazlauskas at amd.com
Wed Jan 30 16:57:48 UTC 2019


On 1/30/19 11:30 AM, Daniel Vetter wrote:
> Yes it's inconsitent with vrr_capable, but this is the actual uapi as
> exercise by igt.
> 
> Fixes: ab7a664f7a2d ("drm: Document variable refresh properties")
> Cc: Nicholas Kazlauskas <nicholas.kazlauskas at amd.com>
> Cc: Harry Wentland <harry.wentland at amd.com>
> Cc: Pekka Paalanen <pekka.paalanen at collabora.com>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>

Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas at amd.com>

It's what xf86-video-amdgpu expects too - the casing is intentional to 
match up with the other default CRTC properties.

Thanks!

Nicholas Kazlauskas

> ---
>   drivers/gpu/drm/drm_connector.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c
> index 847539645558..e3ff73695c32 100644
> --- a/drivers/gpu/drm/drm_connector.c
> +++ b/drivers/gpu/drm/drm_connector.c
> @@ -1367,7 +1367,7 @@ EXPORT_SYMBOL(drm_mode_create_scaling_mode_property);
>    *
>    *	Absence of the property should indicate absence of support.
>    *
> - * "vrr_enabled":
> + * "VRR_ENABLED":
>    *	Default &drm_crtc boolean property that notifies the driver that the
>    *	content on the CRTC is suitable for variable refresh rate presentation.
>    *	The driver will take this property as a hint to enable variable
> 



More information about the dri-devel mailing list