Re: [PATCH 2/2] [media] cx231xx: get rid of CX231XX_VMUX_DEBUG

[Hans Verkuil <hverkuil@xs4all.nl>]

On 02/12/2016 12:21 PM, Mauro Carvalho Chehab wrote:
> This macro is not used inside the driver. get rid of it.
> 
> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>

Acked-by: Hans Verkuil <hans.verkuil@cisco.com>

> ---
>  drivers/media/usb/cx231xx/cx231xx-video.c | 3 +--
>  drivers/media/usb/cx231xx/cx231xx.h       | 1 -
>  2 files changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/media/usb/cx231xx/cx231xx-video.c b/drivers/media/usb/cx231xx/cx231xx-video.c
> index 7222b1c27d40..6414188ffdfa 100644
> --- a/drivers/media/usb/cx231xx/cx231xx-video.c
> +++ b/drivers/media/usb/cx231xx/cx231xx-video.c
> @@ -1103,7 +1103,6 @@ static const char *iname[] = {
>  	[CX231XX_VMUX_TELEVISION] = "Television",
>  	[CX231XX_VMUX_CABLE]      = "Cable TV",
>  	[CX231XX_VMUX_DVB]        = "DVB",
> -	[CX231XX_VMUX_DEBUG]      = "for debug only",
>  };
>  
>  void cx231xx_v4l2_create_entities(struct cx231xx *dev)
> @@ -1136,7 +1135,7 @@ void cx231xx_v4l2_create_entities(struct cx231xx *dev)
>  			if (dev->tuner_type == TUNER_ABSENT)
>  				continue;
>  			/* fall though */
> -		default: /* CX231XX_VMUX_DEBUG */
> +		default: /* just to shut up a gcc warning */
>  			ent->function = MEDIA_ENT_F_CONN_RF;
>  			break;
>  		}
> diff --git a/drivers/media/usb/cx231xx/cx231xx.h b/drivers/media/usb/cx231xx/cx231xx.h
> index 60e14776a6cd..69f6d20870f5 100644
> --- a/drivers/media/usb/cx231xx/cx231xx.h
> +++ b/drivers/media/usb/cx231xx/cx231xx.h
> @@ -281,7 +281,6 @@ enum cx231xx_itype {
>  	CX231XX_VMUX_CABLE,
>  	CX231XX_RADIO,
>  	CX231XX_VMUX_DVB,
> -	CX231XX_VMUX_DEBUG
>  };
>  
>  enum cx231xx_v_input {
>