Re: [PATCH] spi: cadence: Correct initialisation of runtime PM

[Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx>]

On Mon, Jan 07, 2019 at 10:23:17AM +0000, Shubhrajyoti Datta wrote:
> > -----Original Message-----
> > From: Charles Keepax [mailto:ckeepax@xxxxxxxxxxxxxxxxxxxxx]
> > Sent: Monday, January 7, 2019 3:27 PM
> > To: Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxxxx>
> > Cc: Mark Brown <broonie@xxxxxxxxxx>; Shubhrajyoti Datta
> > <shubhraj@xxxxxxxxxx>; linux-spi@xxxxxxxxxxxxxxx
> > Subject: Re: [PATCH] spi: cadence: Correct initialisation of runtime PM
> > 
> > On Mon, Jan 07, 2019 at 03:07:23PM +0530, Shubhrajyoti Datta wrote:
> > > Hi Charles,
> > > BTW  after the probe the clocks are left enabled ?
> > >
> > 
> > Assuming you mean after my patch, the clocks are left enabled but once the
> > auto suspend timeout triggers it will turn them off. So they should be abled
> > for about 3mS after probe.
> 
> Yes can can you check if there are no transactions if the clock are  off?

Already tested before I sent the patch I can see both
cnds_runtime_resume and cnds_runtime_suspend being called as SPI
transactions happen.

Thanks,
Charles