Re: [patch] drm/vc4: fix a bounds check

2017-01-17 Thread Eric Anholt
Dan Carpenter writes: > We accidentally return success even if vc4_full_res_bounds_check() fails. > > Fixes: d5b1a78a772f ("drm/vc4: Add support for drawing 3D frames.") > Signed-off-by: Dan Carpenter Thanks. Reviewed, added Eric's review, and pushed to -fixes. signature.asc Description: PGP

Re: [patch] drm/vc4: fix a bounds check

2017-01-16 Thread Dan Carpenter
On Mon, Jan 16, 2017 at 11:40:10PM +, Eric Engestrom wrote: > > diff --git a/drivers/gpu/drm/vc4/vc4_render_cl.c > > b/drivers/gpu/drm/vc4/vc4_render_cl.c > > index 08886a3..5cdd003 100644 > > --- a/drivers/gpu/drm/vc4/vc4_render_cl.c > > +++ b/drivers/gpu/drm/vc4/vc4_render_cl.c > > @@ -461,7

Re: [patch] drm/vc4: fix a bounds check

2017-01-16 Thread Eric Engestrom
On Friday, 2017-01-13 10:49:00 +0300, Dan Carpenter wrote: > We accidentally return success even if vc4_full_res_bounds_check() fails. > > Fixes: d5b1a78a772f ("drm/vc4: Add support for drawing 3D frames.") > Signed-off-by: Dan Carpenter > --- > Not tested. It would be good to test it, but the p

[patch] drm/vc4: fix a bounds check

2017-01-12 Thread Dan Carpenter
We accidentally return success even if vc4_full_res_bounds_check() fails. Fixes: d5b1a78a772f ("drm/vc4: Add support for drawing 3D frames.") Signed-off-by: Dan Carpenter --- Not tested. diff --git a/drivers/gpu/drm/vc4/vc4_render_cl.c b/drivers/gpu/drm/vc4/vc4_render_cl.c index 08886a3..5cdd00