Re: [PATCH v2 0/7] drm/fb: width/height cleanups/fixes

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Rob,

Thank you for the patches.

On Wednesday 11 March 2015 10:23:07 Rob Clark wrote:
> In the process of fixing fb_width/fb_height problems with tiled displays
> (such as DP MST), I realized there were a few drivers having confusion
> about drm_fb_helper_surface_size.
> 
> So, document the struct, fix the drivers, simplify the logic in
> drm_fb_helper_single_fb_probe() that figures out the sizes, and finally
> fix the logic for tiled displays.
> 
> Oh and a minor completely unrelated kerneldoc fix for a typo that I
> noticed.
> 
> Rob Clark (7):
>   drm/fb: document drm_fb_helper_surface_size
>   drm/atomic: minor kerneldoc typo fix
>   drm/cma: use correct fb width/height
>   drm/exynos: use correct fb width/height
>   drm/rockchip: use correct fb width/height
>   drm/fb: small cleanup
>   drm/fb: handle tiled connectors better

For patches 1 to 5,

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

>  drivers/gpu/drm/drm_fb_cma_helper.c           |  2 +-
>  drivers/gpu/drm/drm_fb_helper.c               | 48 ++++++++++++++++--------
>  drivers/gpu/drm/exynos/exynos_drm_fbdev.c     |  5 +--
>  drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c |  2 +-
>  include/drm/drm_crtc.h                        |  2 +-
>  include/drm/drm_fb_helper.h                   | 19 +++++++++++
>  6 files changed, 60 insertions(+), 18 deletions(-)

-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel