Re: [PATCH 3/7] firmware: cs_dsp: Add ability to load multiple coefficient files

[Fredrik Treven]

On 2/4/25 04:10, Charles Keepax wrote:
> On Fri, Jan 31, 2025 at 01:56:34PM -0600, Fred Treven wrote:
> > Add cs_dsp_power_up_multiple() which accepts an array of
> > cs_dsp_coeff_desc firmware-filename pairs to load.
> >
> > This enables the user to load more than one tuning file
> > along with the associated firmware.
> >
> > Change cs_dsp_power_up() to make use of the new function
> > with a single coefficient file.
> >
> > Signed-off-by: Fred Treven <ftreven@xxxxxxxxxxxxxxxxxxxxx>
> > ---
>
> Can't say I love loading multiple bin files, feels like a tooling
> issue, but I understand you guys have become fairly dependent on
> it and the fallout isn't too bad in the code so:
>
> Reviewed-by: Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx>
>
> Thanks,
> Charles

I agree, and it is technically possible to combine the bin files into one.
We have strongly suggested doing this to consumers of this driver, but
they are adamantly against it, sighting the fact that they want to
maintain the coefficient files separately on a per-algorithm basis.

Thanks for your understanding,
Fred