On Wed, Jan 30, 2019 at 04:57:48PM +0000, Kazlauskas, Nicholas wrote:
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@amd.com Cc: Harry Wentland harry.wentland@amd.com Cc: Pekka Paalanen pekka.paalanen@collabora.com Cc: Alex Deucher alexander.deucher@amd.com Signed-off-by: Daniel Vetter daniel.vetter@intel.com
Reviewed-by: Nicholas Kazlauskas nicholas.kazlauskas@amd.com
It's what xf86-video-amdgpu expects too - the casing is intentional to match up with the other default CRTC properties.
Thanks!
Applied, thanks for your review. Can I motivate you to take a look at the other patches in this series too?
Thanks, Daniel
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