Hi Tomi, Thank you for the patch. On Friday 19 February 2016 11:47:41 Tomi Valkeinen wrote: > From: Jyri Sarha <jsarha@xxxxxx> > > drm_atomic_get_plane_state() may return ERR_PTR. Handle > drm_atomic_get_plane_state() return values right in > omap_crtc_atomic_set_property(). > > Signed-off-by: Jyri Sarha <jsarha@xxxxxx> > Acked-by: Tomi Valkeinen <tomi.valkeinen@xxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/omapdrm/omap_crtc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c > b/drivers/gpu/drm/omapdrm/omap_crtc.c index 7dd3d44a93e5..f5b19d18fa8b > 100644 > --- a/drivers/gpu/drm/omapdrm/omap_crtc.c > +++ b/drivers/gpu/drm/omapdrm/omap_crtc.c > @@ -433,8 +433,8 @@ static int omap_crtc_atomic_set_property(struct drm_crtc > *crtc, */ > > plane_state = drm_atomic_get_plane_state(state->state, plane); > - if (!plane_state) > - return -EINVAL; > + if (IS_ERR(plane_state)) > + return PTR_ERR(plane_state); > > return drm_atomic_plane_set_property(plane, plane_state, property, val); > } -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
