Re: [RFC PATCH 03/15] spi: make `cs_change_delay` the first user of the `spi_delay` logic
From: Ardelean, Alexandru
Date: Mon Sep 16 2019 - 09:04:59 EST
On Mon, 2019-09-16 at 13:47 +0100, Mark Brown wrote:
> [External]
>
> On Mon, Sep 16, 2019 at 12:37:12PM +0000, Ardelean, Alexandru wrote:
>
> > > This breaks the build as there is a user of this interface.
> > Ack.
> > Jonathan pointed this out.
> > There's a V3 that changes both this and it's user (in IIO).
>
> That v3 seems to be a small subset of this series?
Ack.
V3 is the first 4 patches from this series.
Well, patches 3 & 4 are squashed.
I am 100% convinced that the entire series is a good idea.
In the sense that a `struct spi_delay` may be a good idea, but at the same time, it may be un-needed.
All I wanted to do, was to add another delay somewhere, and got lost in the rework of current delays.
I thought about proposing just the first 4 patches [on their own], but I thought that showing the current series as-is
now, may be a good idea as well [to gather some feedback].