ACK: [Vivid][SRU][patch] drm/i915: remove intel_pipe_set_base() (v4)

Brad Figg brad.figg at canonical.com
Tue Apr 28 13:15:22 UTC 2015


On Wed, Apr 22, 2015 at 06:57:42AM +0000, Zhang, Xiong Y wrote:
> BugLink: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1446979
> 
> commit 455a68086d1dfb801ad7c867d5ca0ed0e0f758b0
> Author: Gustavo Padovan <gustavo.padovan at collabora.co.uk>
> Date:   Mon Dec 1 15:40:11 2014 -0800
> 
>     drm/i915: remove intel_pipe_set_base() (v4)
> 
>     After some refactor intel_primary_plane_setplane() does the same
>     as intel_pipe_set_base() so we can get rid of it and replace the calls
>     with intel_primary_plane_setplane().
> 
>     v2: take Ville's comments:
>         - get the right arguments for update_plane()
>         - use drm_crtc_get_hv_timing()
> 
>     v3 (by Matt):
>      - Rebase to latest di-nightly codebase
>      - Use primary->funcs->update_plane() in __intel_set_mode()
>      - Use primary->funcs->disable_plane() in intel_crtc_disable()
> 
>     v4 (by Matt):
>      - Drop redundant calls to intel_crtc_wait_for_pending_flips() before
>        calling update_plane() (Ville)
> 
>     Signed-off-by: Gustavo Padovan <gustavo.padovan at collabora.co.uk>
>     Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
>     Acked-and-mourned-by: Jesse Barnes <jbarnes at virtuousgeek.org>
>     Reviewed-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira at intel.com>
>     Signed-off-by: Daniel Vetter daniel.vetter at ffwll.ch<mailto:daniel.vetter at ffwll.ch>
> 
> thanks
> 

> -- 
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team



-- 
Brad Figg brad.figg at canonical.com http://www.canonical.com




More information about the kernel-team mailing list