On 06/19/2017 11:10 PM, Vidya Srinivas wrote:
From: Chandra Konduru <[email protected]>
This patch adds NV12 as supported format
to intel_framebuffer_init and performs various checks.
v2:
-Fix an issue in checks added (Chandra Konduru)
v3: rebased (me)
v4: Review comments by Ville addressed
Added platform check for NV12 in intel_framebuffer_init
Removed offset checks for NV12 case
Signed-off-by: Chandra Konduru <[email protected]>
Signed-off-by: Nabendu Maiti <[email protected]>
Signed-off-by: Vidya Srinivas <[email protected]>
---
drivers/gpu/drm/i915/intel_display.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index 83b20fd..56fd9ae 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -14765,6 +14765,10 @@ static int intel_framebuffer_init(struct
intel_framebuffer *intel_fb,
goto err;
}
break;
+ case DRM_FORMAT_NV12:
+ if (INTEL_GEN(dev_priv) >= 9)
+ break;
+ goto err;
This NV12 support only correctly works on SKL. Plane color space
conversion is different on GLK and later platforms causing the colors to
display incorrectly. Ville's plane color space property patch series in
review will fix this issue.
Tested-by: Clinton Taylor <[email protected]>
Reviewed-by: Clinton Taylor <[email protected]>
-Clint
default:
DRM_DEBUG_KMS("unsupported pixel format: %s\n",
drm_get_format_name(mode_cmd->pixel_format,
&format_name));
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
