Re: [PATCH v4 03/10] memory: ti-aemif: Wrap CS timings into a struct

[Miquel Raynal]

On 15/11/2024 at 14:26:24 +01, Bastien Curutchet <bastien.curutchet@xxxxxxxxxxx> wrote:

> CS timings are store in the struct aemif_cs_data along with other CS
> parameters. It isn't convenient for exposing CS timings to other drivers
> without also exposing the other parameters.
>
> Wrap the CS timings in a new the struct aemif_cs_timings to simplify

s/the//                        ^^^

> their export in upcoming patches.
>
> Signed-off-by: Bastien Curutchet <bastien.curutchet@xxxxxxxxxxx>

But otherwise this feels very sensible.

Reviewed-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>