Re: [PATCH 1/4] spi: Add option to wake a device by toggling CS
From: Andrey Pronin
Date: Fri Jul 01 2016 - 22:02:36 EST
On Fri, Jul 01, 2016 at 07:17:08PM +0200, Mark Brown wrote:
> On Fri, Jul 01, 2016 at 10:05:50AM -0700, Doug Anderson wrote:
>
> > I'm curious why you you need a timer at all. Can't you just keep
> > track of the jiffies that you last sent and do subtraction? ...or you
> > could get even more accurate and use a ktime_t. That avoids a whole
> > lot of synchronization / locking issues too...
>
> Yeah, that'd be a lot better.
>
> > Also: presumably you'll need to make sure that there's some margin in
> > this whole thing. I'd imagine that if the timeout is 10000
> > nanoseconds and you do the calculation and you last sent 9999
> > nanoseconds ago then you might decide that the other side isn't asleep
> > yet. ...but by the time the transfer starts it might be asleep...
>
> Indeed.
Lots of godd points in the feedback. Let me re-visit the whole idea.
And may be I'll move it to a more device-specific driver.