On Mon, 2019-09-16 at 13:25 +0100, Mark Brown wrote: > [External] > > On Fri, Sep 13, 2019 at 02:45:38PM +0300, Alexandru Ardelean wrote: > > > - u16 cs_change_delay; > > - u8 cs_change_delay_unit; > > + struct spi_delay cs_change_delay; > > 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). V3: https://lore.kernel.org/linux-iio/20190916071024.21447-1-alexandru.ardelean@xxxxxxxxxx/T/#t V2: https://lore.kernel.org/linux-iio/20190913115549.3823-1-alexandru.ardelean@xxxxxxxxxx/T/#t [ archive is from the IIO list ] Well, I'm hoping you are referring to the same user. On a general note: I apologise for the amount of noise/spam I am doing here. Still adjusting to how to do things/changes that touch 2 subsystems, especially when trees are not quite in-sync.