[PATCH] drm/omap: check plane size

2016-04-29 Thread Daniel Vetter
On Thu, Apr 28, 2016 at 11:52 PM, Laurent Pinchart wrote: >> > I thought turning the plane off was done by setting >> > the framebuffer to NULL (in which case the src and crtc coordinates must >> > of course be ignored) ? >> >> That's another way. However setting the fb to 0 is a bit different, as

[PATCH] drm/omap: check plane size

2016-04-29 Thread Laurent Pinchart
Hi Ville, On Thursday 28 Apr 2016 21:30:18 Ville Syrjälä wrote: > On Wed, Apr 27, 2016 at 11:02:17PM +0300, Laurent Pinchart wrote: > > On Wednesday 27 Apr 2016 20:29:24 Ville Syrjälä wrote: > >> On Wed, Apr 27, 2016 at 06:30:19PM +0300, Laurent Pinchart wrote: > >>> On Tuesday 26 Apr 2016 13:

[PATCH] drm/omap: check plane size

2016-04-28 Thread Ville Syrjälä
On Wed, Apr 27, 2016 at 11:02:17PM +0300, Laurent Pinchart wrote: > Hi Ville, > > On Wednesday 27 Apr 2016 20:29:24 Ville Syrjälä wrote: > > On Wed, Apr 27, 2016 at 06:30:19PM +0300, Laurent Pinchart wrote: > > > Hi Tomi, > > > > > > (CC'ing Daniel) > > > > > > Thank you for the patch. > > >

[PATCH] drm/omap: check plane size

2016-04-27 Thread Laurent Pinchart
Hi Ville, On Wednesday 27 Apr 2016 20:29:24 Ville Syrjälä wrote: > On Wed, Apr 27, 2016 at 06:30:19PM +0300, Laurent Pinchart wrote: > > Hi Tomi, > > > > (CC'ing Daniel) > > > > Thank you for the patch. > > > > On Tuesday 26 Apr 2016 13:16:42 Tomi Valkeinen wrote: > > > At the moment we don't

[PATCH] drm/omap: check plane size

2016-04-27 Thread Ville Syrjälä
On Wed, Apr 27, 2016 at 06:30:19PM +0300, Laurent Pinchart wrote: > Hi Tomi, > > (CC'ing Daniel) > > Thank you for the patch. > > On Tuesday 26 Apr 2016 13:16:42 Tomi Valkeinen wrote: > > At the moment we don't check the plane input/output sizes, which can > > lead to DSS HW errors when invalid

[PATCH] drm/omap: check plane size

2016-04-27 Thread Laurent Pinchart
Hi Tomi, (CC'ing Daniel) Thank you for the patch. On Tuesday 26 Apr 2016 13:16:42 Tomi Valkeinen wrote: > At the moment we don't check the plane input/output sizes, which can > lead to DSS HW errors when invalid values are given from the userspace. > > Add a check so that the sizes are > 0. >

[PATCH] drm/omap: check plane size

2016-04-26 Thread Tomi Valkeinen
At the moment we don't check the plane input/output sizes, which can lead to DSS HW errors when invalid values are given from the userspace. Add a check so that the sizes are > 0. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/omapdrm/omap_plane.c | 6 ++ 1 file changed, 6 insertions(+)