Hi, On Thu, Jul 5, 2018 at 2:46 PM, Girish Mahadevan <girishm@xxxxxxxxxxxxxx> wrote: > New driver for Qualcomm QuadSPI(QSPI) controller that is used to > communicate with slaves such flash memory devices. The QSPI controller > can operate in 2 or 4 wire mode but only supports SPI Mode 0 and SPI > Mode 3. The controller can also operate in Single or Dual data rate modes. > > Signed-off-by: Girish Mahadevan <girishm@xxxxxxxxxxxxxx> > --- > drivers/spi/Kconfig | 7 + > drivers/spi/Makefile | 1 + > drivers/spi/spi-qcom-qspi.c | 728 ++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 736 insertions(+) I spent a few days poking at this driver and trying to see if I could get it to do anything other than crash and corrupt memory. I finally was able to do so. I've posted my diffs at: https://chromium.googlesource.com/chromiumos/third_party/kernel/+/6670275e22b5901a1c4c11982cdafd10c2820771%5E%21/#F0 ...if it's helpful I'm happy to post a full patch here too (just let me know), but I'd just expect my patch to get squashed into the next version and it can be reviewed then. With my patch applied my testing on sdm845 on a device that can only support dual SPI seems to work pretty well. NOTE: as I reply to this, I realized that I hadn't gone back and tried to address Mark and Boris's comments. Some of them are handled because I found the same problem that they did, but before re-posting you should definitely handle them. -Doug -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html