diff mbox series

[2/2] media: imx7-media-csi: get rid of unused var

Message ID 91937229883824924f1a06ded49dfded4ca96d43.1550582690.git.mchehab+samsung@kernel.org (mailing list archive)
State New, archived
Headers show
Series [1/2] media: imx7-media-csi: don't store a floating pointer | expand

Commit Message

Mauro Carvalho Chehab Feb. 19, 2019, 1:24 p.m. UTC 
drivers/staging/media/imx/imx7-media-csi.c: In function 'imx7_csi_enum_mbus_code':
	drivers/staging/media/imx/imx7-media-csi.c:926:33: warning: variable 'in_cc' set but not used [-Wunused-but-set-variable]
	  const struct imx_media_pixfmt *in_cc;
	                                 ^~~~~

Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
---
 drivers/staging/media/imx/imx7-media-csi.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Rui Miguel Silva Feb. 20, 2019, 11:23 a.m. UTC | #1 
Oi Mauro,
On Tue 19 Feb 2019 at 13:24, Mauro Carvalho Chehab wrote:
> 	drivers/staging/media/imx/imx7-media-csi.c: In function 
> 'imx7_csi_enum_mbus_code':
> 	drivers/staging/media/imx/imx7-media-csi.c:926:33: 
> warning: variable 'in_cc' set but not used 
> [-Wunused-but-set-variable]
> 	  const struct imx_media_pixfmt *in_cc;
> 	                                 ^~~~~
>
> Signed-off-by: Mauro Carvalho Chehab 
> <mchehab+samsung@kernel.org>

Acked-by: Rui Miguel Silva <rmfrfs@gmail.com>

---
Cheers,
	Rui

> ---
>  drivers/staging/media/imx/imx7-media-csi.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/drivers/staging/media/imx/imx7-media-csi.c 
> b/drivers/staging/media/imx/imx7-media-csi.c
> index 0b1788d79ce9..3fba7c27c0ec 100644
> --- a/drivers/staging/media/imx/imx7-media-csi.c
> +++ b/drivers/staging/media/imx/imx7-media-csi.c
> @@ -923,7 +923,6 @@ static int imx7_csi_enum_mbus_code(struct 
> v4l2_subdev *sd,
>  				   struct 
>  v4l2_subdev_mbus_code_enum *code)
>  {
>  	struct imx7_csi *csi = v4l2_get_subdevdata(sd);
> -	const struct imx_media_pixfmt *in_cc;
>  	struct v4l2_mbus_framefmt *in_fmt;
>  	int ret = 0;
>  
> @@ -931,8 +930,6 @@ static int imx7_csi_enum_mbus_code(struct 
> v4l2_subdev *sd,
>  
>  	in_fmt = imx7_csi_get_format(csi, cfg, IMX7_CSI_PAD_SINK, 
>  code->which);
>  
> -	in_cc = imx_media_find_mbus_format(in_fmt->code, 
> CS_SEL_ANY, true);
> -
>  	switch (code->pad) {
>  	case IMX7_CSI_PAD_SINK:
>  		ret = imx_media_enum_mbus_format(&code->code, 
>  code->index,
diff mbox series

Patch

diff --git a/drivers/staging/media/imx/imx7-media-csi.c b/drivers/staging/media/imx/imx7-media-csi.c
index 0b1788d79ce9..3fba7c27c0ec 100644
--- a/drivers/staging/media/imx/imx7-media-csi.c
+++ b/drivers/staging/media/imx/imx7-media-csi.c
@@ -923,7 +923,6 @@  static int imx7_csi_enum_mbus_code(struct v4l2_subdev *sd,
 				   struct v4l2_subdev_mbus_code_enum *code)
 {
 	struct imx7_csi *csi = v4l2_get_subdevdata(sd);
-	const struct imx_media_pixfmt *in_cc;
 	struct v4l2_mbus_framefmt *in_fmt;
 	int ret = 0;
 
@@ -931,8 +930,6 @@  static int imx7_csi_enum_mbus_code(struct v4l2_subdev *sd,
 
 	in_fmt = imx7_csi_get_format(csi, cfg, IMX7_CSI_PAD_SINK, code->which);
 
-	in_cc = imx_media_find_mbus_format(in_fmt->code, CS_SEL_ANY, true);
-
 	switch (code->pad) {
 	case IMX7_CSI_PAD_SINK:
 		ret = imx_media_enum_mbus_format(&code->code, code->index,