diff mbox

[v2,01/20] drm: omapdrm: fb: Limit number of planes per framebuffer to two

Message ID 1465256545-533-2-git-send-email-laurent.pinchart@ideasonboard.com (mailing list archive)
State New, archived
Headers show

Commit Message

Laurent Pinchart June 6, 2016, 11:42 p.m. UTC
The only multi-planar format supported by the driver is NV12, there will
thus never be more than two planes per framebuffer.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 drivers/gpu/drm/omapdrm/omap_fb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Tomi Valkeinen June 10, 2016, 11:58 a.m. UTC | #1
On 07/06/16 02:42, Laurent Pinchart wrote:
> The only multi-planar format supported by the driver is NV12, there will
> thus never be more than two planes per framebuffer.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>  drivers/gpu/drm/omapdrm/omap_fb.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/omapdrm/omap_fb.c b/drivers/gpu/drm/omapdrm/omap_fb.c
> index 530567cc25b7..4214138d5795 100644
> --- a/drivers/gpu/drm/omapdrm/omap_fb.c
> +++ b/drivers/gpu/drm/omapdrm/omap_fb.c
> @@ -36,7 +36,7 @@ struct format {
>  	struct {
>  		int stride_bpp;           /* this times width is stride */
>  		int sub_y;                /* sub-sample in y dimension */
> -	} planes[4];
> +	} planes[2];
>  	bool yuv;
>  };
>  
> @@ -90,7 +90,7 @@ struct omap_framebuffer {
>  	struct drm_framebuffer base;
>  	int pin_count;
>  	const struct format *format;
> -	struct plane planes[4];
> +	struct plane planes[2];
>  	/* lock for pinning (pin_count and planes.paddr) */
>  	struct mutex lock;
>  };
> 

Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

 Tomi
diff mbox

Patch

diff --git a/drivers/gpu/drm/omapdrm/omap_fb.c b/drivers/gpu/drm/omapdrm/omap_fb.c
index 530567cc25b7..4214138d5795 100644
--- a/drivers/gpu/drm/omapdrm/omap_fb.c
+++ b/drivers/gpu/drm/omapdrm/omap_fb.c
@@ -36,7 +36,7 @@  struct format {
 	struct {
 		int stride_bpp;           /* this times width is stride */
 		int sub_y;                /* sub-sample in y dimension */
-	} planes[4];
+	} planes[2];
 	bool yuv;
 };
 
@@ -90,7 +90,7 @@  struct omap_framebuffer {
 	struct drm_framebuffer base;
 	int pin_count;
 	const struct format *format;
-	struct plane planes[4];
+	struct plane planes[2];
 	/* lock for pinning (pin_count and planes.paddr) */
 	struct mutex lock;
 };