Hi Peng, On 21 March 2016 at 02:29, Peng Fan <van.free...@gmail.com> wrote: > 1. Support compatible string "spi-gpio" which is used by Linux > Linux use different bindings, so use UBOOT_COMPAT and > LINUX_COMPAT to differentiate them.
There is only one user - instead of this, can you please drop the U-Boot binding and move the universal board over to use your new binding? There should be only one. > 2. Bug fix > When doing xfer, should use device->parent, but not device > When doing bit xfer, should use "!!(tmpdout & 0x80)", but not > "(tmpdout & 0x80)" > 3. Introduce SPI_MASTER_NO_RX and SPI_MASTER_NO_TX to handle > no rx or no tx case. Perhaps put these two in separate patches? > 4. Tested on i.MX6 UltraLite board with 74LV595 spi-gpio chip. > > Signed-off-by: Peng Fan <van.free...@gmail.com> > Cc: Simon Glass <s...@chromium.org> > Cc: Jagan Teki <jt...@openedev.com> > Cc: Stefano Babic <sba...@denx.de> > --- > drivers/spi/soft_spi.c | 75 > ++++++++++++++++++++++++++++++++++++++------------ > 1 file changed, 57 insertions(+), 18 deletions(-) Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot