[alsa-devel] [PATCH for-5.6 14/14] media: Drop superfluous ioctl PCM ops

Ismael Luceno ismael at iodev.co.uk
Tue Dec 17 00:43:17 CET 2019


On 10/Dec/2019 14:58, Takashi Iwai wrote:
> PCM core deals the empty ioctl field now as default.
> Let's kill the redundant lines.
> 
> Cc: Mauro Carvalho Chehab <mchehab at kernel.org>
> Cc: Bluecherry Maintainers <maintainers at bluecherrydvr.com>
> Cc: Anton Sviridenko <anton at corp.bluecherry.net>
> Cc: Andrey Utkin <andrey.utkin at corp.bluecherry.net>
> Cc: Ismael Luceno <ismael at iodev.co.uk>
> Cc: Ezequiel Garcia <ezequiel at vanguardiasur.com.ar>
> Cc: Hans Verkuil <hverkuil-cisco at xs4all.nl>
> Signed-off-by: Takashi Iwai <tiwai at suse.de>
> ---
>  drivers/media/pci/cx23885/cx23885-alsa.c   | 1 -
>  drivers/media/pci/cx25821/cx25821-alsa.c   | 1 -
>  drivers/media/pci/cx88/cx88-alsa.c         | 1 -
>  drivers/media/pci/saa7134/saa7134-alsa.c   | 1 -
>  drivers/media/pci/solo6x10/solo6x10-g723.c | 1 -
>  drivers/media/pci/tw686x/tw686x-audio.c    | 1 -
>  drivers/media/usb/cx231xx/cx231xx-audio.c  | 1 -
>  drivers/media/usb/em28xx/em28xx-audio.c    | 1 -
>  drivers/media/usb/go7007/snd-go7007.c      | 1 -
>  drivers/media/usb/tm6000/tm6000-alsa.c     | 1 -
>  drivers/media/usb/usbtv/usbtv-audio.c      | 1 -
>  11 files changed, 11 deletions(-)
> 
> diff --git a/drivers/media/pci/cx23885/cx23885-alsa.c b/drivers/media/pci/cx23885/cx23885-alsa.c
> index a8e980c6dacb..df44ed7393a0 100644
> --- a/drivers/media/pci/cx23885/cx23885-alsa.c
> +++ b/drivers/media/pci/cx23885/cx23885-alsa.c
> @@ -495,7 +495,6 @@ static struct page *snd_cx23885_page(struct snd_pcm_substream *substream,
>  static const struct snd_pcm_ops snd_cx23885_pcm_ops = {
>  	.open = snd_cx23885_pcm_open,
>  	.close = snd_cx23885_close,
> -	.ioctl = snd_pcm_lib_ioctl,
>  	.hw_params = snd_cx23885_hw_params,
>  	.hw_free = snd_cx23885_hw_free,
>  	.prepare = snd_cx23885_prepare,
> diff --git a/drivers/media/pci/cx25821/cx25821-alsa.c b/drivers/media/pci/cx25821/cx25821-alsa.c
> index c2f2d7c782c7..301616426d8a 100644
> --- a/drivers/media/pci/cx25821/cx25821-alsa.c
> +++ b/drivers/media/pci/cx25821/cx25821-alsa.c
> @@ -639,7 +639,6 @@ static struct page *snd_cx25821_page(struct snd_pcm_substream *substream,
>  static const struct snd_pcm_ops snd_cx25821_pcm_ops = {
>  	.open = snd_cx25821_pcm_open,
>  	.close = snd_cx25821_close,
> -	.ioctl = snd_pcm_lib_ioctl,
>  	.hw_params = snd_cx25821_hw_params,
>  	.hw_free = snd_cx25821_hw_free,
>  	.prepare = snd_cx25821_prepare,
> diff --git a/drivers/media/pci/cx88/cx88-alsa.c b/drivers/media/pci/cx88/cx88-alsa.c
> index e1e71ae293ed..7d7aceecc985 100644
> --- a/drivers/media/pci/cx88/cx88-alsa.c
> +++ b/drivers/media/pci/cx88/cx88-alsa.c
> @@ -585,7 +585,6 @@ static struct page *snd_cx88_page(struct snd_pcm_substream *substream,
>  static const struct snd_pcm_ops snd_cx88_pcm_ops = {
>  	.open = snd_cx88_pcm_open,
>  	.close = snd_cx88_close,
> -	.ioctl = snd_pcm_lib_ioctl,
>  	.hw_params = snd_cx88_hw_params,
>  	.hw_free = snd_cx88_hw_free,
>  	.prepare = snd_cx88_prepare,
> diff --git a/drivers/media/pci/saa7134/saa7134-alsa.c b/drivers/media/pci/saa7134/saa7134-alsa.c
> index 0385127dd7ff..544ca57eee75 100644
> --- a/drivers/media/pci/saa7134/saa7134-alsa.c
> +++ b/drivers/media/pci/saa7134/saa7134-alsa.c
> @@ -865,7 +865,6 @@ static struct page *snd_card_saa7134_page(struct snd_pcm_substream *substream,
>  static const struct snd_pcm_ops snd_card_saa7134_capture_ops = {
>  	.open =			snd_card_saa7134_capture_open,
>  	.close =		snd_card_saa7134_capture_close,
> -	.ioctl =		snd_pcm_lib_ioctl,
>  	.hw_params =		snd_card_saa7134_hw_params,
>  	.hw_free =		snd_card_saa7134_hw_free,
>  	.prepare =		snd_card_saa7134_capture_prepare,
> diff --git a/drivers/media/pci/solo6x10/solo6x10-g723.c b/drivers/media/pci/solo6x10/solo6x10-g723.c
> index ca349d447610..d6d16e8fd997 100644
> --- a/drivers/media/pci/solo6x10/solo6x10-g723.c
> +++ b/drivers/media/pci/solo6x10/solo6x10-g723.c
> @@ -259,7 +259,6 @@ static int snd_solo_pcm_copy_kernel(struct snd_pcm_substream *ss, int channel,
>  static const struct snd_pcm_ops snd_solo_pcm_ops = {
>  	.open = snd_solo_pcm_open,
>  	.close = snd_solo_pcm_close,
> -	.ioctl = snd_pcm_lib_ioctl,
>  	.prepare = snd_solo_pcm_prepare,
>  	.trigger = snd_solo_pcm_trigger,
>  	.pointer = snd_solo_pcm_pointer,
> diff --git a/drivers/media/pci/tw686x/tw686x-audio.c b/drivers/media/pci/tw686x/tw686x-audio.c
> index f5065c72bb28..54144e23a487 100644
> --- a/drivers/media/pci/tw686x/tw686x-audio.c
> +++ b/drivers/media/pci/tw686x/tw686x-audio.c
> @@ -258,7 +258,6 @@ static snd_pcm_uframes_t tw686x_pcm_pointer(struct snd_pcm_substream *ss)
>  static const struct snd_pcm_ops tw686x_pcm_ops = {
>  	.open = tw686x_pcm_open,
>  	.close = tw686x_pcm_close,
> -	.ioctl = snd_pcm_lib_ioctl,
>  	.prepare = tw686x_pcm_prepare,
>  	.trigger = tw686x_pcm_trigger,
>  	.pointer = tw686x_pcm_pointer,
> diff --git a/drivers/media/usb/cx231xx/cx231xx-audio.c b/drivers/media/usb/cx231xx/cx231xx-audio.c
> index d46efea53370..de42db6f6ad1 100644
> --- a/drivers/media/usb/cx231xx/cx231xx-audio.c
> +++ b/drivers/media/usb/cx231xx/cx231xx-audio.c
> @@ -551,7 +551,6 @@ static snd_pcm_uframes_t snd_cx231xx_capture_pointer(struct snd_pcm_substream
>  static const struct snd_pcm_ops snd_cx231xx_pcm_capture = {
>  	.open = snd_cx231xx_capture_open,
>  	.close = snd_cx231xx_pcm_close,
> -	.ioctl = snd_pcm_lib_ioctl,
>  	.prepare = snd_cx231xx_prepare,
>  	.trigger = snd_cx231xx_capture_trigger,
>  	.pointer = snd_cx231xx_capture_pointer,
> diff --git a/drivers/media/usb/em28xx/em28xx-audio.c b/drivers/media/usb/em28xx/em28xx-audio.c
> index a5215fb1d36a..6833b5bfe293 100644
> --- a/drivers/media/usb/em28xx/em28xx-audio.c
> +++ b/drivers/media/usb/em28xx/em28xx-audio.c
> @@ -626,7 +626,6 @@ static int em28xx_cvol_new(struct snd_card *card, struct em28xx *dev,
>  static const struct snd_pcm_ops snd_em28xx_pcm_capture = {
>  	.open      = snd_em28xx_capture_open,
>  	.close     = snd_em28xx_pcm_close,
> -	.ioctl     = snd_pcm_lib_ioctl,
>  	.prepare   = snd_em28xx_prepare,
>  	.trigger   = snd_em28xx_capture_trigger,
>  	.pointer   = snd_em28xx_capture_pointer,
> diff --git a/drivers/media/usb/go7007/snd-go7007.c b/drivers/media/usb/go7007/snd-go7007.c
> index 40dbf081ef6b..ae27e988e578 100644
> --- a/drivers/media/usb/go7007/snd-go7007.c
> +++ b/drivers/media/usb/go7007/snd-go7007.c
> @@ -175,7 +175,6 @@ static snd_pcm_uframes_t go7007_snd_pcm_pointer(struct snd_pcm_substream *substr
>  static const struct snd_pcm_ops go7007_snd_capture_ops = {
>  	.open		= go7007_snd_capture_open,
>  	.close		= go7007_snd_capture_close,
> -	.ioctl		= snd_pcm_lib_ioctl,
>  	.hw_params	= go7007_snd_hw_params,
>  	.hw_free	= go7007_snd_hw_free,
>  	.prepare	= go7007_snd_pcm_prepare,
> diff --git a/drivers/media/usb/tm6000/tm6000-alsa.c b/drivers/media/usb/tm6000/tm6000-alsa.c
> index 2bfa5abc1b49..c26a0ff60a64 100644
> --- a/drivers/media/usb/tm6000/tm6000-alsa.c
> +++ b/drivers/media/usb/tm6000/tm6000-alsa.c
> @@ -306,7 +306,6 @@ static snd_pcm_uframes_t snd_tm6000_pointer(struct snd_pcm_substream *substream)
>  static const struct snd_pcm_ops snd_tm6000_pcm_ops = {
>  	.open = snd_tm6000_pcm_open,
>  	.close = snd_tm6000_close,
> -	.ioctl = snd_pcm_lib_ioctl,
>  	.prepare = snd_tm6000_prepare,
>  	.trigger = snd_tm6000_card_trigger,
>  	.pointer = snd_tm6000_pointer,
> diff --git a/drivers/media/usb/usbtv/usbtv-audio.c b/drivers/media/usb/usbtv/usbtv-audio.c
> index b27009875758..b57e94fb1977 100644
> --- a/drivers/media/usb/usbtv/usbtv-audio.c
> +++ b/drivers/media/usb/usbtv/usbtv-audio.c
> @@ -312,7 +312,6 @@ static snd_pcm_uframes_t snd_usbtv_pointer(struct snd_pcm_substream *substream)
>  static const struct snd_pcm_ops snd_usbtv_pcm_ops = {
>  	.open = snd_usbtv_pcm_open,
>  	.close = snd_usbtv_pcm_close,
> -	.ioctl = snd_pcm_lib_ioctl,
>  	.prepare = snd_usbtv_prepare,
>  	.trigger = snd_usbtv_card_trigger,
>  	.pointer = snd_usbtv_pointer,
> -- 
> 2.16.4
> 

Signed-off-by: Ismael Luceno <ismael at iodev.co.uk>


More information about the Alsa-devel mailing list