Hi Mark, On 7 August 2018 at 21:41, Mark Brown <broonie@xxxxxxxxxx> wrote: > On Tue, Aug 07, 2018 at 06:43:37PM +0800, Baolin Wang wrote: > >> +Optional properties: >> +- sprd,spi-interval: Specify the intervals of two SPI frames, which can be >> + converted to the delay clock cycles = interval number * 4 + 10. > > What's a frame here, and does it overlap with any of the existing delay > configuration we have? In general it's better for this stuff to be > configured at runtime by the device rather than at DT time by the > controller since that way if the device needs the delays we always do > them if we can and if they are only needed some of the time (eg, for > only one device on the bus or for only some operations) then we don't > take the performance hit when we don't need to. Sorry for confusing. Let me try to explain it explicitly. We can set the word size (bits_per_word) for each transmission, for our SPI controller, after every word size transmission, we need one interval time (hardware automatically) to make sure the slave has enough time to receive the whole data. Yes, I agree we should configure it at runtime by the device, but we did not find one member to use in 'struct spi_transfer', we just find one similar 'delay_usecs' member in 'struct spi_transfer' but not same. We can use 'delay_usecs' to set our hardware interval value, but we should clean it when transfer is done, since we do not need to delay after the transfer in spi_transfer_one _message(). Or can we add one new member maybe named 'word_interval' to indicate the interval time between word size transmission? What do you prefer or other better solution? Thanks. -- Baolin Wang Best Regards -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
