-----Original Message----- From: Jagan Teki [mailto:jagannadh.teki@gmail.com] Sent: Monday, June 12, 2017 11:37 AM To: Suresh Gupta suresh.gupta@nxp.com Cc: Jagan Teki jagan@amarulasolutions.com; york sun york.sun@nxp.com; u-boot@lists.denx.de; Jagan Teki jagan@openedev.com; Yao Yuan yao.yuan@nxp.com Subject: Re: [U-Boot] [PATCH] sf: Fix s25fs512s erase size and remove SECT_4K flag
Hi Suresh,
On Thu, Jun 8, 2017 at 1:00 PM, Suresh Gupta suresh.gupta@nxp.com wrote:
<snip> > > > > So best way is to disable hybrid mode and configure flash in > > Uniform Sector Architecture > > OK, then send patches to Linux and U-Boot will review and take other > suggestions too. >
Thanks Jagan, I will send U-boot patches (after testing) by early next week.
My next point :) to discuss is "32-bit address options". This particular flash do not support below commands /* Bank addr access commands */ #ifdef CONFIG_SPI_FLASH_BAR # define CMD_BANKADDR_BRWR 0x17 # define CMD_BANKADDR_BRRD 0x16 # define CMD_EXTNADDR_WREAR 0xC5 # define CMD_EXTNADDR_RDEAR 0xC8 #endif
Instead of above S25FS512S flash support 32-bit address commands like 4READ, 4P4E and many more which require 32 bit address directly pass on
wires.
Jagan, So, Do we have any plan to fix this? Yao Yuan already send one patch [1] for same. I think this is a decent approach [1]. If required I will resend this patch with
little changes.
Don't know whether you tried this or not.
- Erase bottom 4K sectors using 4K
- Erase middle sectors using SE
- Erase top 4K sectors using 4K
I will try all combinations and let you know the results.
thanks!
Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India.