Am Freitag, den 20.11.2015, 16:14 +0800 schrieb Liu Ying: > To reduce code duplication, we can use the helper ipu_plane_cleanup() in > ipu_plane_destroy(). > > Signed-off-by: Liu Ying <Ying.Liu@xxxxxxxxxxxxx> > --- > This patch applies to the imx-drm/fixes branch of Philipp Zabel's open git. > > drivers/gpu/drm/imx/ipuv3-plane.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c > index e60b382..b3ed207 100644 > --- a/drivers/gpu/drm/imx/ipuv3-plane.c > +++ b/drivers/gpu/drm/imx/ipuv3-plane.c > @@ -369,8 +369,7 @@ static void ipu_plane_destroy(struct drm_plane *plane) > DRM_DEBUG_KMS("[%d] %s\n", __LINE__, __func__); > > ipu_disable_plane(plane); > - drm_plane_cleanup(plane); > - kfree(ipu_plane); > + ipu_plane_cleanup(ipu_plane); > } > > static struct drm_plane_funcs ipu_plane_funcs = { This could be merged into the first patch, but I don't think ipu_plane_cleanup is necessary at all. regards Philipp _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel