Hi Laurent, I plan to take this patch in my pull request that will be done next week.
BR Vincent On 09/08/2016 04:44 PM, Laurent Pinchart wrote: > The driver needs the number of bytes per pixel, not the bpp and depth > info meant for fbdev compatibility. Use the right API. > > Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com> > Acked-by: Vincent Abriou <vincent.abriou at st.com> > --- > drivers/gpu/drm/sti/sti_gdp.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > Cc: Benjamin Gaignard <benjamin.gaignard at linaro.org> > Cc: Vincent Abriou <vincent.abriou at st.com> > > diff --git a/drivers/gpu/drm/sti/sti_gdp.c b/drivers/gpu/drm/sti/sti_gdp.c > index b8d942ca45e8..3fc62c1ea9c2 100644 > --- a/drivers/gpu/drm/sti/sti_gdp.c > +++ b/drivers/gpu/drm/sti/sti_gdp.c > @@ -719,7 +719,7 @@ static void sti_gdp_atomic_update(struct drm_plane > *drm_plane, > u32 dma_updated_top; > u32 dma_updated_btm; > int format; > - unsigned int depth, bpp; > + unsigned int bpp; > u32 ydo, xdo, yds, xds; > > if (!crtc || !fb) > @@ -758,9 +758,9 @@ static void sti_gdp_atomic_update(struct drm_plane > *drm_plane, > (unsigned long)cma_obj->paddr); > > /* pixel memory location */ > - drm_fb_get_bpp_depth(fb->pixel_format, &depth, &bpp); > + bpp = drm_format_plane_cpp(fb->pixel_format, 0); > top_field->gam_gdp_pml = (u32)cma_obj->paddr + fb->offsets[0]; > - top_field->gam_gdp_pml += src_x * (bpp >> 3); > + top_field->gam_gdp_pml += src_x * bpp; > top_field->gam_gdp_pml += src_y * fb->pitches[0]; > > /* output parameters (clamped / cropped) */ >
