[PATCH 02/17] ASoC: soc-dai: don't overwide dai->driver->ops
Mark Brown
broonie at kernel.org
Fri Apr 24 11:30:51 CEST 2020
On Fri, Apr 24, 2020 at 07:39:58AM +0900, Kuninori Morimoto wrote:
> > > From: Kuninori Morimoto <kuninori.morimoto.gx at renesas.com>
> > > Current ASoC overwides dai->driver->ops if it was NULL.
> > > But, it is not good idea, because dai->driver might be reused
> > > when modprobe/rmmod or bind/unbind, etc.
> > This is fine but it's not a correctness issue since we're always filling
> > the same value in - the big issue with putting stuff in structures is
> > when you end up using the same struct for two different different things
> > so you fill different values in.
> I see.
> But all my remaining patches are based on this patch,
> and rebase without it seems very difficult.
> So, I want to keep it.
> v2 patchset still has it, but adds different git-log.
Right, like I say the change is fine - it was just a note about the
changelog.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20200424/edd3ca4c/attachment.sig>
More information about the Alsa-devel
mailing list