Hi Pratyush,

On Tue, Jun 14, 2022 at 1:49 AM Pratyush Yadav [off-list ref] wrote:

This is needed for TI's SoCs as well. APB and AHB accesses are
independent of each other on the interconnect and can be racy. I wrote a
couple patches [0][1] to fix this on TI's fork. I never got around to
sending them upstream. It would be great if you can pick those up. They
fix the race in all paths, not just indirect write.

I would also prefer if we do this unconditionally. I don't think it has
much downside even on platforms that do not strictly need this.

[0] https://git.ti.com/cgit/ti-linux-kernel/ti-linux-kernel/commit/drivers/spi/spi-cadence-quadspi.c?h=ti-linux-5.10.y&id=027f03a8512086e5ef05dc4e4ff53b2628848f95
[1] https://git.ti.com/cgit/ti-linux-kernel/ti-linux-kernel/commit/drivers/spi/spi-cadence-quadspi.c?h=ti-linux-5.10.y&id=4c367e58bab7d3f9c470c3778441f73546f20398

Let's get Elba specific support in first and then in a separate patch
go for the unconditional.  An extra op for devices for which its not
currently done will result in questions I can't answer.

Regards,
Brad