On Tue, Jun 07, 2016 at 05:50:02PM +0100, Eric Engestrom wrote: > On Mon, Jun 06, 2016 at 09:14:51PM +0300, Laurent Pinchart wrote: > > The function has no side effect and its returned values are ignored, > > don't call it. > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com> > > Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
Both of Laurent's patches applied, thanks. -Daniel > > > --- > > drivers/gpu/drm/qxl/qxl_fb.c | 4 ---- > > 1 file changed, 4 deletions(-) > > > > diff --git a/drivers/gpu/drm/qxl/qxl_fb.c b/drivers/gpu/drm/qxl/qxl_fb.c > > index 5ea57f6320b8..df2657051afd 100644 > > --- a/drivers/gpu/drm/qxl/qxl_fb.c > > +++ b/drivers/gpu/drm/qxl/qxl_fb.c > > @@ -131,10 +131,6 @@ static int qxlfb_create_pinned_object(struct qxl_fbdev > > *qfbdev, > > int ret; > > int aligned_size, size; > > int height = mode_cmd->height; > > - int bpp; > > - int depth; > > - > > - drm_fb_get_bpp_depth(mode_cmd->pixel_format, &bpp, &depth); > > > > size = mode_cmd->pitches[0] * height; > > aligned_size = ALIGN(size, PAGE_SIZE); > > -- > > Regards, > > > > Laurent Pinchart > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel at lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > _______________________________________________ > dri-devel mailing list > dri-devel at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch