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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20140527125622.GA14841@phenom.ffwll.local>
Date:	Tue, 27 May 2014 14:56:22 +0200
From:	Daniel Vetter <daniel@...ll.ch>
To:	Stephen Rothwell <sfr@...b.auug.org.au>
Cc:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Vincent Stehlé <vincent.stehle@...oste.net>,
	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org,
	linux-next@...r.kernel.org, Daniel Vetter <daniel.vetter@...ll.ch>,
	Andrzej Hajda <a.hajda@...sung.com>,
	Dave Airlie <airlied@...ux.ie>
Subject: Re: [PATCH linux-next] imx-drm: imx-tve: remove unused variable

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@...uxfoundation.org> 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@...oste.net>
> 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@...oste.net>
> Cc: Andrzej Hajda <a.hajda@...sung.com>
> Cc: Daniel Vetter <daniel.vetter@...ll.ch>
> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> ---
> 
> 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@...b.auug.org.au



-- 
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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/