Re: [PATCH] media: usb: Remove duplicate declares

[Hans Verkuil]

Please specify in the subject line which driver this patch changes.

I also would prefer that this is split into two patches, one for each driver.

Just saying 'media: usb:' makes it look like you have a media/usb subsystem patch,
but that's not the case. Very confusing for code reviewers.

Regards,

	Hans

On 27/03/2021 08:42, Wan Jiabing wrote:
> struct cx231xx has been declared at 146th line.
> struct em28xx has been declared at 219th line.
> Remove the duplicate.
> 
> Signed-off-by: Wan Jiabing <wanjiabing@xxxxxxxx>
> ---
>  drivers/media/usb/cx231xx/cx231xx.h | 2 --
>  drivers/media/usb/em28xx/em28xx.h   | 2 --
>  2 files changed, 4 deletions(-)
> 
> diff --git a/drivers/media/usb/cx231xx/cx231xx.h b/drivers/media/usb/cx231xx/cx231xx.h
> index b32eab641793..6929e4d97067 100644
> --- a/drivers/media/usb/cx231xx/cx231xx.h
> +++ b/drivers/media/usb/cx231xx/cx231xx.h
> @@ -425,8 +425,6 @@ struct cx231xx_audio {
>  	u16 end_point_addr;
>  };
>  
> -struct cx231xx;
> -
>  /*****************************************************************/
>  /* set/get i2c */
>  /* 00--1Mb/s, 01-400kb/s, 10--100kb/s, 11--5Mb/s */
> diff --git a/drivers/media/usb/em28xx/em28xx.h b/drivers/media/usb/em28xx/em28xx.h
> index 6648e11f1271..43227111d410 100644
> --- a/drivers/media/usb/em28xx/em28xx.h
> +++ b/drivers/media/usb/em28xx/em28xx.h
> @@ -628,8 +628,6 @@ struct em28xx_audio {
>  	atomic_t       stream_started;	/* stream should be running if true */
>  };
>  
> -struct em28xx;
> -
>  enum em28xx_i2c_algo_type {
>  	EM28XX_I2C_ALGO_EM28XX = 0,
>  	EM28XX_I2C_ALGO_EM2800,
>