On Tue, Mar 28, 2017 at 04:07:58PM +0200, Arnd Bergmann wrote:
> The newly introduced function causes a harmless build warning:
>
> drivers/gpu/drm/arm/malidp_planes.c: In function
> 'malidp_plane_atomic_print_state':
> drivers/gpu/drm/arm/malidp_planes.c:98:23: error: unused variable 'mp'
> [-Werror=unused-variable]
>
> The variable serves no purpose here and can be removed.
Ouch, shame on me for not noticing it :(
Thanks for this!
Acked-by: Liviu Dudau <liviu.du...@arm.com>
>
> Fixes: 242f43b69c61 ("drm: mali-dp: add atomic_print_state for planes")
> Signed-off-by: Arnd Bergmann <a...@arndb.de>
> ---
> drivers/gpu/drm/arm/malidp_planes.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/arm/malidp_planes.c
> b/drivers/gpu/drm/arm/malidp_planes.c
> index 5b0bad87f0b5..afd59e0f7918 100644
> --- a/drivers/gpu/drm/arm/malidp_planes.c
> +++ b/drivers/gpu/drm/arm/malidp_planes.c
> @@ -95,7 +95,6 @@ static void malidp_plane_atomic_print_state(struct
> drm_printer *p,
> const struct drm_plane_state *state)
> {
> struct malidp_plane_state *ms = to_malidp_plane_state(state);
> - struct malidp_plane *mp = to_malidp_plane(state->plane);
>
> drm_printf(p, "\trotmem_size=%u\n", ms->rotmem_size);
> drm_printf(p, "\tformat_id=%u\n", ms->format);
> --
> 2.9.0
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
¯\_(ツ)_/¯
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
