On Fri, Apr 24, 2020 at 11:48 PM Sagar Kadam sagar.kadam@sifive.com wrote:
Hello Jagan,
-----Original Message----- From: Jagan Teki jagan@amarulasolutions.com Sent: Thursday, April 23, 2020 10:31 PM To: u-boot@lists.denx.de Cc: Rick Chen rick@andestech.com; Bin Meng bmeng.cn@gmail.com; Bhargav Shah bhargavshah1988@gmail.com; Sagar Kadam sagar.kadam@sifive.com; linux-amarula@amarulasolutions.com; Jagan Teki jagan@amarulasolutions.com Subject: [PATCH v4 0/5] riscv: sifive/fu540: Enable SPI-NOR support
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
This is series v4 for SPI-NOR support on SiFive FU540 platform with HiFive Unleashed board.
Here is the previous version changes[1].
All patches on top of u-boot-spi/master.
Thanks for posting v4 for spi-nor support. Tested the series above u-boot-spi/master on HiFive Unleashed and was able to verify it for both spi-nor and mmc Additionally just confirmed the opcodes nor is configured with post spi_nor_scan: ====================================================== [QUAD mode in dt with spi-tx-bus-width: <4>] pp opcode = 0x34 [QUAD MODE] read opcode = 0x6c [QUAD MODE] erase opcode = 0x21
SPI-NOR:
- erase entire flash: Pass
- write entire flash: Pass
- read entire flash: Pass
- cmp 32MiB read back data: Pass
- MMC: Booted Linux and dtb from mmc (so as to confirm data integrity from mmc)
================================================================= [SPI MODE in dt with spi-tx-bus-width: <1> ] pp opcode = 0x12 [SPI MODE] read opcode = 0xc [SPI MODE] erase opcode = 0x21
SPI-NOR:
- erase entire flash: Pass
- write entire flash: Pass
- read entire flash: Pass
- cmp 32MiB read back data: Pass
- MMC: Booted Linux and dtb from mmc (so as to confirm data integrity from mmc)
Tested-by: Sagar Kadam sagar.kadam@sifive.com
Thanks for detailed testing of Sagar, I will include these use cases on git log,so-that we can find proper info in future.
Rick,
I would like to pick this via u-boot-spi, let me know if you have any concerns, thanks!
Jagan.