On Thu, Aug 21, 2014 at 06:25:06PM +0300, Grygorii Strashko wrote: > +- ti,davinci-spi-wdelay : delay between transmissions.
I don't understand why this is here - there is already standard support in the SPI core for client drivers specifying inter-transfer delays. If there is a need to provide platform configuration for this in addition to that then it should also be a standard property, there is nothing device specific about these. > +- ti,davinci-spi-odd-parity : odd partity enabled > + OR > + ti,davinci-spi-even-parity : even parity enabled What do these mean? > +- ti,davinci-spi-io-type: io type (check platform_data/spi-davinci.h) The bindings should be independent of the kernel, the values need to be included here (and the defines moved to include/dt-bindings so they can be used when writing DTs). > +- ti,davinci-spi-disable-timer: disable CS timer (SPIFMTn) > +- ti,davinci-spi-c2t-delay: c2t delay > +- ti,davinci-spi-t2c-delay: t2c delay > +- ti,davinci-spi-t2e-delay: t2e delay > +- ti,davinci-spi-c2e-delay: c2e delay What are all these timers/delays - at least some reference to the datasheet please?
signature.asc
Description: Digital signature
_______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
