linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH -next] [media] davinci: vpbe: remove unused variable in vpbe_initialize()
@ 2012-12-03  2:53 Wei Yongjun
  2012-12-03  6:00 ` Prabhakar Lad
  0 siblings, 1 reply; 2+ messages in thread
From: Wei Yongjun @ 2012-12-03  2:53 UTC (permalink / raw)
  To: manjunath.hadli, prabhakar.lad, mchehab
  Cc: yongjun_wei, linux-media, davinci-linux-open-source

From: Wei Yongjun <yongjun_wei@trendmicro.com.cn>

The variable 'output_index' is initialized but never used
otherwise, so remove the unused variable.

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
---
 drivers/media/platform/davinci/vpbe.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/media/platform/davinci/vpbe.c b/drivers/media/platform/davinci/vpbe.c
index 7f5cf9b..e0c79c1 100644
--- a/drivers/media/platform/davinci/vpbe.c
+++ b/drivers/media/platform/davinci/vpbe.c
@@ -584,7 +584,6 @@ static int vpbe_initialize(struct device *dev, struct vpbe_device *vpbe_dev)
 	struct v4l2_subdev **enc_subdev;
 	struct osd_state *osd_device;
 	struct i2c_adapter *i2c_adap;
-	int output_index;
 	int num_encoders;
 	int ret = 0;
 	int err;
@@ -731,7 +730,6 @@ static int vpbe_initialize(struct device *dev, struct vpbe_device *vpbe_dev)
 	/* set the current encoder and output to that of venc by default */
 	vpbe_dev->current_sd_index = 0;
 	vpbe_dev->current_out_index = 0;
-	output_index = 0;
 
 	mutex_unlock(&vpbe_dev->lock);
 



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH -next] [media] davinci: vpbe: remove unused variable in vpbe_initialize()
  2012-12-03  2:53 [PATCH -next] [media] davinci: vpbe: remove unused variable in vpbe_initialize() Wei Yongjun
@ 2012-12-03  6:00 ` Prabhakar Lad
  0 siblings, 0 replies; 2+ messages in thread
From: Prabhakar Lad @ 2012-12-03  6:00 UTC (permalink / raw)
  To: Wei Yongjun
  Cc: manjunath.hadli, prabhakar.lad, mchehab, yongjun_wei,
	davinci-linux-open-source, linux-media

Hi Wei,

Thanks for the patch.

On Mon, Dec 3, 2012 at 8:23 AM, Wei Yongjun <weiyj.lk@gmail.com> wrote:
> From: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
>
> The variable 'output_index' is initialized but never used
> otherwise, so remove the unused variable.
>
> Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>

Acked-by: Prabhakar Lad <prabhakar.lad@ti.com>

Regards,
--Prabhakar

> ---
>  drivers/media/platform/davinci/vpbe.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/media/platform/davinci/vpbe.c b/drivers/media/platform/davinci/vpbe.c
> index 7f5cf9b..e0c79c1 100644
> --- a/drivers/media/platform/davinci/vpbe.c
> +++ b/drivers/media/platform/davinci/vpbe.c
> @@ -584,7 +584,6 @@ static int vpbe_initialize(struct device *dev, struct vpbe_device *vpbe_dev)
>         struct v4l2_subdev **enc_subdev;
>         struct osd_state *osd_device;
>         struct i2c_adapter *i2c_adap;
> -       int output_index;
>         int num_encoders;
>         int ret = 0;
>         int err;
> @@ -731,7 +730,6 @@ static int vpbe_initialize(struct device *dev, struct vpbe_device *vpbe_dev)
>         /* set the current encoder and output to that of venc by default */
>         vpbe_dev->current_sd_index = 0;
>         vpbe_dev->current_out_index = 0;
> -       output_index = 0;
>
>         mutex_unlock(&vpbe_dev->lock);
>
>
>
> _______________________________________________
> Davinci-linux-open-source mailing list
> Davinci-linux-open-source@linux.davincidsp.com
> http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-12-03  6:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-03  2:53 [PATCH -next] [media] davinci: vpbe: remove unused variable in vpbe_initialize() Wei Yongjun
2012-12-03  6:00 ` Prabhakar Lad

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).