-----Original Message----- From: Pratyush Yadav p.yadav@ti.com Sent: Wednesday, May 27, 2020 9:07 PM To: Tan, Ley Foon ley.foon.tan@intel.com Cc: u-boot@lists.denx.de; Jagan Teki jagan@amarulasolutions.com; Ley Foon Tan lftan.linux@gmail.com; Vignesh Raghavendra vigneshr@ti.com; See, Chin Liang chin.liang.see@intel.com; Simon Goldschmidt simon.k.r.goldschmidt@gmail.com; Ang, Chee Hong chee.hong.ang@intel.com Subject: Re: [PATCH] spi: cadence_spi: Add octal and quad write support
Hi Ley,
On 22/05/20 03:53PM, Ley Foon Tan wrote:
In Commit d64077202158 ("spi: cadence_qspi: Move to spi-mem framework") it removes setting to quad write bit by accident. This commit restores it back and also adding checking for octal support.
Fixes: d64077202158 ("spi: cadence_qspi: Move to spi-mem framework")
I sent a patch to add Octal DTR support for this controller [0]. It should also fix this problem (check cadence_qspi_set_protocol()).
[0] https://patchwork.ozlabs.org/project/uboot/patch/20200522124509.6901- 5-p.yadav@ti.com/
Hi
I have tried your patches with uboot/master, but it has compilation error below.
drivers/built-in.o: In function `spi_flash_std_remove': drivers/mtd/spi/sf_probe.c:160: undefined reference to `spi_nor_remove' drivers/mtd/spi/sf_probe.c:160:(.text.spi_flash_std_remove+0x10): relocation truncated to fit: R_AARCH64_JUMP26 against undefined symbol `spi_nor_remove' drivers/built-in.o:(.rodata.spi_nor_ids+0x258): undefined reference to `mt35xu512aba_fixups' drivers/built-in.o:(.rodata.spi_nor_ids+0x578): undefined reference to `s28hs512t_fixups'
Regards Ley Foon