Re: [PATCH] drm/pl111: Fix offset calculation for the primary plane.
Sean Paulwrites: > On Fri, Jun 02, 2017 at 06:57:33PM -0700, Eric Anholt wrote: >> If src_x/y were nonzero, we failed to shift them down by 16 to get the >> pixel offset. The recent CMA helper function gets it right. >> >> Signed-off-by: Eric Anholt >> Fixes: bed41005e617 ("drm/pl111: Initial drm/kms driver for pl111") >> Reported-by: Mircea Carausu > > > Reviewed-by: Sean Paul Pushed. Thanks! signature.asc Description: PGP signature ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH] drm/pl111: Fix offset calculation for the primary plane.
On Fri, Jun 02, 2017 at 06:57:33PM -0700, Eric Anholt wrote: > If src_x/y were nonzero, we failed to shift them down by 16 to get the > pixel offset. The recent CMA helper function gets it right. > > Signed-off-by: Eric Anholt> Fixes: bed41005e617 ("drm/pl111: Initial drm/kms driver for pl111") > Reported-by: Mircea Carausu Reviewed-by: Sean Paul > --- > drivers/gpu/drm/pl111/pl111_display.c | 15 ++- > 1 file changed, 2 insertions(+), 13 deletions(-) > > diff --git a/drivers/gpu/drm/pl111/pl111_display.c > b/drivers/gpu/drm/pl111/pl111_display.c > index 3e0a4fa73ddb..c6ca4f1bbd49 100644 > --- a/drivers/gpu/drm/pl111/pl111_display.c > +++ b/drivers/gpu/drm/pl111/pl111_display.c > @@ -50,17 +50,6 @@ irqreturn_t pl111_irq(int irq, void *data) > return status; > } > > -static u32 pl111_get_fb_offset(struct drm_plane_state *pstate) > -{ > - struct drm_framebuffer *fb = pstate->fb; > - struct drm_gem_cma_object *obj = drm_fb_cma_get_gem_obj(fb, 0); > - > - return (obj->paddr + > - fb->offsets[0] + > - fb->format->cpp[0] * pstate->src_x + > - fb->pitches[0] * pstate->src_y); > -} > - > static int pl111_display_check(struct drm_simple_display_pipe *pipe, > struct drm_plane_state *pstate, > struct drm_crtc_state *cstate) > @@ -73,7 +62,7 @@ static int pl111_display_check(struct > drm_simple_display_pipe *pipe, > return -EINVAL; > > if (fb) { > - u32 offset = pl111_get_fb_offset(pstate); > + u32 offset = drm_fb_cma_get_gem_addr(fb, pstate, 0); > > /* FB base address must be dword aligned. */ > if (offset & 3) > @@ -249,7 +238,7 @@ static void pl111_display_update(struct > drm_simple_display_pipe *pipe, > struct drm_framebuffer *fb = pstate->fb; > > if (fb) { > - u32 addr = pl111_get_fb_offset(pstate); > + u32 addr = drm_fb_cma_get_gem_addr(fb, pstate, 0); > > writel(addr, priv->regs + CLCD_UBAS); > } > -- > 2.11.0 > > ___ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Sean Paul, Software Engineer, Google / Chromium OS ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH] drm/pl111: Fix offset calculation for the primary plane.
If src_x/y were nonzero, we failed to shift them down by 16 to get the pixel offset. The recent CMA helper function gets it right. Signed-off-by: Eric AnholtFixes: bed41005e617 ("drm/pl111: Initial drm/kms driver for pl111") Reported-by: Mircea Carausu --- drivers/gpu/drm/pl111/pl111_display.c | 15 ++- 1 file changed, 2 insertions(+), 13 deletions(-) diff --git a/drivers/gpu/drm/pl111/pl111_display.c b/drivers/gpu/drm/pl111/pl111_display.c index 3e0a4fa73ddb..c6ca4f1bbd49 100644 --- a/drivers/gpu/drm/pl111/pl111_display.c +++ b/drivers/gpu/drm/pl111/pl111_display.c @@ -50,17 +50,6 @@ irqreturn_t pl111_irq(int irq, void *data) return status; } -static u32 pl111_get_fb_offset(struct drm_plane_state *pstate) -{ - struct drm_framebuffer *fb = pstate->fb; - struct drm_gem_cma_object *obj = drm_fb_cma_get_gem_obj(fb, 0); - - return (obj->paddr + - fb->offsets[0] + - fb->format->cpp[0] * pstate->src_x + - fb->pitches[0] * pstate->src_y); -} - static int pl111_display_check(struct drm_simple_display_pipe *pipe, struct drm_plane_state *pstate, struct drm_crtc_state *cstate) @@ -73,7 +62,7 @@ static int pl111_display_check(struct drm_simple_display_pipe *pipe, return -EINVAL; if (fb) { - u32 offset = pl111_get_fb_offset(pstate); + u32 offset = drm_fb_cma_get_gem_addr(fb, pstate, 0); /* FB base address must be dword aligned. */ if (offset & 3) @@ -249,7 +238,7 @@ static void pl111_display_update(struct drm_simple_display_pipe *pipe, struct drm_framebuffer *fb = pstate->fb; if (fb) { - u32 addr = pl111_get_fb_offset(pstate); + u32 addr = drm_fb_cma_get_gem_addr(fb, pstate, 0); writel(addr, priv->regs + CLCD_UBAS); } -- 2.11.0 ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel