Re: [PATCH v2 2/6] memory: ti-aemif: Export aemif_set_cs_timings()

From: Miquel Raynal
Date: Mon Nov 11 2024 - 14:21:52 EST



Hello Bastien,

On 06/11/2024 at 09:55:03 +01, Bastien Curutchet <bastien.curutchet@xxxxxxxxxxx> wrote:

> Export the aemif_set_cs_timing() symbol so it can be used by other
> drivers
>
> Add a spinlock to protect the CS configuration register from concurrent
> accesses.

What concurrent accesses are you trying to protect yourself against?
I fail to see the use case, but TBH I haven't tried hard enough maybe.

Also, what justifies the use of a spin-lock in this case?

Thanks,
Miquèl