On Fri, 2022-03-04 at 15:57 +0100, Cezary Rojewski wrote: > Wrap elementary DSP-core operations and resource control into more > complex handlers. This is done to reduce the number of invocations of > wrapped operations throughout the driver as order of operations > matters - > most flows involve register manipulation and IPCs combined. > > Signed-off-by: Amadeusz Sławiński < > amadeuszx.slawinski@xxxxxxxxxxxxxxx> > Signed-off-by: Cezary Rojewski <cezary.rojewski@xxxxxxxxx> > --- > sound/soc/intel/avs/avs.h | 10 +++ > sound/soc/intel/avs/dsp.c | 170 > ++++++++++++++++++++++++++++++++++++++ > 2 files changed, 180 insertions(+) > > diff --git a/sound/soc/intel/avs/avs.h b/sound/soc/intel/avs/avs.h > index 02d7591d0eac..0034c075fa64 100644 > --- a/sound/soc/intel/avs/avs.h > +++ b/sound/soc/intel/avs/avs.h > @@ -89,6 +89,7 @@ struct avs_dev { > struct mutex modres_mutex; > struct ida ppl_ida; > struct list_head fw_list; > + int *core_refs; Is this a per core ref_count? a comment or explicitly calling it core_ref_count would help. Thanks,Ranjani