On 04/27/2017 10:26 PM, Suresh Gupta wrote:


-----Original Message-----
From: York Sun [mailto:york....@nxp.com]
Sent: Thursday, April 27, 2017 10:18 PM
To: Suresh Gupta <suresh.gu...@nxp.com>; Jagan Teki
<jagannadh.t...@gmail.com>
Cc: u-boot@lists.denx.de; Jagan Teki <ja...@openedev.com>; Yao Yuan
<yao.y...@nxp.com>
Subject: Re: [U-Boot] [PATCH] sf: Fix s25fs512s erase size and remove SECT_4K
flag

On 04/26/2017 10:17 PM, Suresh Gupta wrote:



<snip>


Suresh,

Did you test your patch after removing CONFIG_SPI_FLASH_BAR? Why
doesn't this flash support BAR commands? How did it work before broken?


York, the flash s25fs512s support 32 bit address to handle >16M memory
access.
With and without CONFIG_SPI_FLASH_BAR the u-boot only access up to 16M
of flash memory for this particular flash which is installed on our boards
(LS1012ARDB, LS1046ARDB, LS1088ARDB, LS2088ARDB).
So, anyhow we need to remove this CONFIG_SPI_FLASH_BAR from our boards
configs.

For accessing >16M of flash memory (to support 32 bit address) I
already start discussion [2] with Jagan. Hopefully We have a solution for that
in near future.
[2] https://patchwork.ozlabs.org/patch/676152/


How did it (>16MB) work before?

York

According to me for this s25fs512s  particular flash,  >16M should not work in 
open source code.
If you have any commit id where it works, let me know.
I will try at my end and update you


Suresh,

Where are we on this? The internal U-Boot has it working (previous SDK release). Did you check?

York

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to