On Tue, Dec 10, 2019 at 04:58:37PM +0100, Takashi Iwai wrote:
> On Tue, 10 Dec 2019 16:49:28 +0100,
> Takashi Sakamoto wrote:
> >
> > Hi,
> >
> > On Tue, Dec 10, 2019 at 07:11:27AM +0100, Takashi Iwai wrote:
> > > Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
> > > ---
> > > sound/firewire/bebob/bebob_pcm.c | 2 --
> > > sound/firewire/dice/dice-pcm.c | 2 --
> > > sound/firewire/digi00x/digi00x-pcm.c | 2 --
> > > sound/firewire/fireface/ff-pcm.c | 2 --
> > > sound/firewire/fireworks/fireworks_pcm.c | 2 --
> > > sound/firewire/isight.c | 1 -
> > > sound/firewire/motu/motu-pcm.c | 2 --
> > > sound/firewire/oxfw/oxfw-pcm.c | 2 --
> > > sound/firewire/tascam/tascam-pcm.c | 2 --
> > > 9 files changed, 17 deletions(-)
> >
> > I understand that this patch is based on the below commit:
> > * fc033cbf6fb7("ALSA: pcm: Allow NULL ioctl ops")[1]
> >
> > All of the changes are fine to me.
> >
> > Acked-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx>
> >
> > [1] https://mailman.alsa-project.org/pipermail/alsa-devel/2019-November/158692.html
>
> Thanks. It's good idea to put the background information. I'll add
> it to the changelog.
>
> Actually I seem to have forgotten to fill the changelog text in this
> commit. Below is the revised patch to be committed.
>
>
> Takashi
>
> -- 8< --
> Subject: [PATCH] ALSA: firewire: Drop superfluous ioctl PCM ops
>
> All the PCM ioctl ops of ALSA FireWire drivers do nothing but calling
> the default handler.
>
> Now PCM core accepts NULL as the default ioctl ops(*), so let's drop
> altogether.
>
> (*) commit fc033cbf6fb7 ("ALSA: pcm: Allow NULL ioctl ops")
>
> Acked-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx>
> Link: https://lore.kernel.org/r/20191210061145.24641-6-tiwai@xxxxxxx
> Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
It's preferrable. Thanks for your extra work ;)
Thanks
Takashi Sakamoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
