[PATCH 7/8] ASoC: Intel: sof_ssp_amp: rename driver and support cs35l41 amplifier

Mark Brown broonie at kernel.org
Wed Mar 2 17:10:22 CET 2022


On Wed, Mar 02, 2022 at 05:05:05PM +0100, Amadeusz Sławiński wrote:

> I think you've misunderstood me, I'm not talking about squashing, but
> splitting patch 7 into two separate patches - one doing rename and another
> adding/changing things.

> I was looking at:
> +	links = devm_kzalloc(dev, sizeof(struct snd_soc_dai_link) *
> +					sof_ssp_amp_card.num_links, GFP_KERNEL);
> +	cpus = devm_kzalloc(dev, sizeof(struct snd_soc_dai_link_component) *
> +					sof_ssp_amp_card.num_links, GFP_KERNEL);

> and wondered where sof_ssp_amp_card.num_links comes from, and it only shows
> up 4 times in removed file, but 6 times in added one, which tells me that
> things changed, but it is hard to see what, when the diff shows whole files
> being moved instead of things that changed.

That would've been a bit nicer but TBH it was fine - I've already queued
the patches.
-------------- 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/20220302/a09147be/attachment.sig>


More information about the Alsa-devel mailing list