On 09/27/2017 06:54 AM, Hannes Schmelzer wrote: > On 09/22/2017 02:20 PM, Clément Péron wrote: >> Sorry these are my local commits you can find them here : >> >> https://patchwork.ozlabs.org/patch/765992/ >> https://patchwork.ozlabs.org/patch/765996/ >> https://patchwork.ozlabs.org/patch/765997/ >> https://patchwork.ozlabs.org/patch/765998/ > Hi, > just tested this on my cyclone5 board, but unfortunately without success. > > --- > U-Boot SPL 2017.09-00354-g824def8 (Sep 27 2017 - 06:47:19) > (....) > Hit any key to stop autoboot: 0 > => sf probe > SF: Detected n25q512 with page size 256 Bytes, erase size 64 KiB, total > 64 MiB > ### ERROR ### Please RESET the board ### > ---- > > same behavior as before.
Have you compared your dts to the other socfpga_cyclone5_*.dts files? I have tested this on the socrates board and it would also give me the above error until I change the "compatible" string of the flash chip from "n25q00" to "spi-flash". Without that, the sf code seems to try to use the first child node as flash chip which results in a divide-by-zero error because it passes a frequency of 0 Hz to the set_speed callback of the qspi driver, which it is not prepared to handle: If cadence_qspi_apb_config_baudrate_div is called with sclk_hz == 0, the DIV_ROUND_UP macro fails to check the denominator for zero. I wonder where this should be fixed: in the core sf code or in this driver... Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
