Re: [PATCH v11 1/5] firmware: cs_dsp: Add write sequence interface

[James Ogletree <James.Ogletree@xxxxxxxxxx>]

Hi Charles,

> On Jun 10, 2024, at 10:11 AM, Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx> wrote:
> 
> On Wed, Jun 05, 2024 at 01:52:45PM +0000, James Ogletree wrote:
>> A write sequence is a sequence of register addresses
>> and values executed by some Cirrus DSPs following
>> certain power state transitions.
>> 
>> Add support for Cirrus drivers to update or add to a
>> write sequence present in firmware.
>> 
>> Reviewed-by: Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx>
>> Signed-off-by: James Ogletree <jogletre@xxxxxxxxxxxxxxxxxxxxx>
>> ---
> 
> I would suggest we just apply this patch, it is unlikely to
> change from review comments on the rest of the series and could
> well be useful to other parts in the meantime. It would also
> mean less traffic each time this series is sent up.
> 
> Thanks,
> Charles

Agreed with your points. I am okay with applying this separately.

Best,
James