Re: [PATCH linux-next] imx-drm: imx-tve: remove unused variable

[Daniel Vetter]

On Mon, May 26, 2014 at 09:12:52AM +1000, Stephen Rothwell wrote:
> Hi all,
> 
> On Sun, 25 May 2014 15:12:35 -0700 Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> >
> > On Sun, May 25, 2014 at 11:54:14PM +0200, Vincent Stehlà wrote:
> > > Commit f9b0e251dfbf 'drm: make mode_valid callback optional' left variable ret
> > > unused; remove it.
> > > 
> > > This fixes the following compilation warning:
> > > 
> > >   drivers/staging/imx-drm/imx-tve.c: In function âimx_tve_connector_mode_validâ:
> > >   drivers/staging/imx-drm/imx-tve.c:252:6: warning: unused variable âretâ [-Wunused-variable]
> > 
> > It doesn't apply to my tree :(
> 
> Yeah, commit f9b0e251dfbf 'drm: make mode_valid callback optional' is
> in the drm tree, so this patch needs to go to Dave Airlie (cc'd).
> Though Daniel Vetter may want to send it on.
> 
> For Dave's benefit, here is the patch again:

I've pulled it into my drm-stuff branch so it doesn't get lost. Should
land in 3.16 one way or the other.

Thanks, Daniel

> 
> From:	Vincent Stehlà <vincent.stehle@xxxxxxxxxxx>
> Subject: [PATCH linux-next] imx-drm: imx-tve: remove unused variable
> Date:	Sun, 25 May 2014 23:54:14 +0200
> 
> Commit f9b0e251dfbf 'drm: make mode_valid callback optional' left variable ret
> unused; remove it.
> 
> This fixes the following compilation warning:
> 
>   drivers/staging/imx-drm/imx-tve.c: In function âimx_tve_connector_mode_validâ:
>   drivers/staging/imx-drm/imx-tve.c:252:6: warning: unused variable âretâ [-Wunused-variable]
> 
> Signed-off-by: Vincent Stehlà <vincent.stehle@xxxxxxxxxxx>
> Cc: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
> Cc: Daniel Vetter <daniel.vetter@xxxxxxxx>
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> ---
> 
> Hi,
> 
> This can be seen with e.g. linux next-20140523 and arm allmodconfig.
> 
> Best regards,
> 
> V.
> 
>  drivers/staging/imx-drm/imx-tve.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/staging/imx-drm/imx-tve.c b/drivers/staging/imx-drm/imx-tve.c
> index 3e8b0a1..4caef2b1 100644
> --- a/drivers/staging/imx-drm/imx-tve.c
> +++ b/drivers/staging/imx-drm/imx-tve.c
> @@ -249,7 +249,6 @@ static int imx_tve_connector_mode_valid(struct drm_connector *connector,
>  {
>  	struct imx_tve *tve = con_to_tve(connector);
>  	unsigned long rate;
> -	int ret;
>  
>  	/* pixel clock with 2x oversampling */
>  	rate = clk_round_rate(tve->clk, 2000UL * mode->clock) / 2000;
> -- 
> 2.0.0.rc2
> 
> -- 
> Cheers,
> Stephen Rothwell                    sfr@xxxxxxxxxxxxxxxx



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/