Re: [PATCH v6 09/16] ASoC: rsnd: ssui: Add RZ/G3E SSIU BUSIF support

[Kuninori Morimoto]

Hi John

> > 	- adding busif_status_count
> > 	- adding rag3e_id
> > 	- adding rstc
(snip)
>   09a/N: ASoC: rsnd: ssiu: Add shared SSI reset controller support
>          - just the devm_reset_control_get_optional_shared("ssi-all") in
>            rsnd_ssiu_probe() and passing it to rsnd_mod_init().
>          - No-op for non-RZ/G3E DTs.
> 
>   09b/N: ASoC: rsnd: ssiu: Add RZ/G3E BUSIF support
>          - introduces struct rsnd_ssiu_ctrl with busif_status_count and
>            parametrises the loops in rsnd_ssiu_busif_err_irq_ctrl() and
>            rsnd_ssiu_busif_err_status_clear().
>          - adds the rzg3e_id[] BUSIF index table and the SSI_MODE0 guard.
>          - sets RSND_SSIU_BUSIF_STATUS_COUNT_2 in the r9a09g047 match data.
> 
> I am keeping busif_status_count and rzg3e_id together because they are
> both keyed on RZ/G3E and form one feature ("RZ/G3E SSIU BUSIF layout").
> Splitting them would leave an intermediate state where the BUSIF index
> table and the error-status loop bound disagree on the SoC. Please let me
> know if you would prefer them as two separate patches anyway, then I
> will do a three-way split with the busif_status_count refactor placed
> first as a no-behavior-change preparation.

I have no objection if some of them are related.

Thank you for your help !!

Best regards
---
Kuninori Morimoto