Re: [PATCH 1/1] spi: mediatek: add power control when set_cs

From: luhua xu
Date: Thu Oct 24 2019 - 06:11:26 EST

On Thu, 2019-10-24 at 14:25 +0800, luhua xu wrote:
> On Wed, 2019-10-23 at 16:11 +0100, Mark Brown wrote:
> > On Wed, Oct 23, 2019 at 08:38:42AM -0400, Luhua Xu wrote:
> > > From: "luhua.xu" <luhua.xu@xxxxxxxxxxxx>
> >
> > > Use runtime PM to power spi when set_cs
> > > As set_cs may be called from interrupt context,
> > > set runtime PM IRQ safe for spi.
> >
> > Why might we be trying to set the chip select state while the device is
> > runtime idle? It seems like whatever is trying to set the chip select
> > should be dealing with this, not the chip select operation itself since
> > that's unlikely to be happening in isolation.
> Hi Mark,
> Spi framework provideds spi_setup() to modify spi settings for spi
> device (maybe spi is runtime idle now), and this will call
> spi_controller->set_cs() accessing registers.
> Other spi_controller callbacks that need to access hardware registers,
> are triggered by spi transfer. Spi framework will get and put runtime
> power in __spi_pump_message().

hi Mark,
If the spi slaver who wants to set cs need to power spi
itself , this means we need to provide spi power on/off APIs while spi
controller is runtime PM device.
And spi slaver needs to power spi itself when it wants to set cs,
and don't need to do so when do spi data transfer.

I see several case of spi driver with power control:
(1)No clk control in set_cs, such as spi-cadence.c
(2)Enable spi clk in probe before the first time hitting reg,
and never power down. Such as spi-sifive.c spi-armada-3700.c
(3)Enable spi clk in controller->set_cs callback, such as
My patch works the same as (3).For mtk platform, without this
patch, if user do spi_setup(), cs setting cannot take effect, and reg
access violation occurs.