[PATCH v3 0/7] ASoC: topology: Change to resource managed memory
Amadeusz Sławiński
amadeuszx.slawinski at linux.intel.com
Mon Nov 2 12:24:10 CET 2020
On 10/30/2020 3:23 PM, Rojewski, Cezary wrote:
> On 2020-10-30 3:54 PM, Amadeusz Sławiński wrote:
>> Almost all other allocations in ASoC API are resource managed, the only
>> exception is soc-topology.c.
>>
>> This patchset clean ups few unnecessary functions in preparation for
>> change and then changes to devm_ functions for allocation.
>>
>
> These address some of the long standing issues of resource allocation
> for topology, thanks for your work.
> Nitpick:
> *the 0/7* looks weird given that only 6 patches are actually part of the
> series.
Yes that was a copy&paste mistake, similarly to already mentioned v3.
There is only 6 patches.
It is a problem caused by manually filling cover letter and copying
title from old cover letter to new one.
I did some reading, and apparently if one operates on branches to
prepare patch sets, they can do something like:
$ git branch --edit-description
$ git format-patch -6 --cover-letter --cover-from-description=subject
And git will automatically populate cover letter with text from branch
description. I will try to remember to try it when sending another
series. Hopefully this will allow me to avoid copy&paste mistakes, as
filling cover letter title and description will be automated ;)
Mark, do let me know if you want me to resend.
>
> For the series:
>
> Reviewed-by: Cezary Rojewski <cezary.rojewski at intel.com>
>
> Thanks,
> Czarek
>
>> Amadeusz Sławiński (6):
>> ASoC: topology: Remove unused functions from topology API
>> ASoC: topology: Remove multistep topology loading
>> ASoC: topology: Unify all device references
>> ASoC: topology: Change allocations to resource managed
>> ASoC: topology: Remove empty functions
>> ASoC: topology: Simplify remove_widget function
>>
>> include/sound/soc-topology.h | 13 +-
>> sound/soc/intel/skylake/skl-topology.c | 9 +-
>> sound/soc/soc-topology.c | 302 +++++--------------------
>> sound/soc/sof/pcm.c | 2 +-
>> sound/soc/sof/topology.c | 4 +-
>> 5 files changed, 57 insertions(+), 273 deletions(-)
>>
More information about the Alsa-devel
mailing list