Re: [U-Boot] [PATCH] sf: Disable 4-KB erase command for SPANSION S25FS128

2016-08-06 Thread Michael Trimarchi
Hi On Sat, Aug 6, 2016 at 5:08 PM, Jagan Teki wrote: > On 6 August 2016 at 20:10, Michael Trimarchi > wrote: > > I understand your point but I don't have enough part number and I can > only to > test on this one. I will verify this patch too and put my review and drop > my

Re: [U-Boot] [PATCH] sf: Disable 4-KB erase command for SPANSION S25FS128

2016-08-06 Thread Jagan Teki
On 6 August 2016 at 20:10, Michael Trimarchi wrote: I understand your point but I don't have enough part number and I can only to test on this one. I will verify this patch too and put my review and drop my personal one. Anyway can you ask if the first 0x8000 are co

Re: [U-Boot] [PATCH] sf: Disable 4-KB erase command for SPANSION S25FS128

2016-08-06 Thread Jagan Teki
On 22 June 2016 at 15:44, Yao Yuan wrote: > On 06/22/2016 06:08 PM, Michael Trimarchi wrote: >> On Wed, Jun 22, 2016 at 10:00:49AM +, Yao Yuan wrote: >> > On 06/22/2016 03:59 PM, Michael Trimarchi wrote: >> > > The S25FS128 is part of S25FS-S family physical sectors may be >> > > configured as

Re: [U-Boot] [PATCH] sf: Disable 4-KB erase command for SPANSION S25FS128

2016-06-22 Thread Yao Yuan
On 06/22/2016 06:08 PM, Michael Trimarchi wrote: > On Wed, Jun 22, 2016 at 10:00:49AM +, Yao Yuan wrote: > > On 06/22/2016 03:59 PM, Michael Trimarchi wrote: > > > The S25FS128 is part of S25FS-S family physical sectors may be > > > configured as a hybrid combination of eight 4-kB parameter sec

Re: [U-Boot] [PATCH] sf: Disable 4-KB erase command for SPANSION S25FS128

2016-06-22 Thread Michael Trimarchi
Hi On Wed, Jun 22, 2016 at 10:00:49AM +, Yao Yuan wrote: > On 06/22/2016 03:59 PM, Michael Trimarchi wrote: > > The S25FS128 is part of S25FS-S family physical sectors may be configured > > as a > > hybrid combination of eight 4-kB parameter sectors at the top or bottom of > > the > > addres

Re: [U-Boot] [PATCH] sf: Disable 4-KB erase command for SPANSION S25FS128

2016-06-22 Thread Yao Yuan
On 06/22/2016 03:59 PM, Michael Trimarchi wrote: > The S25FS128 is part of S25FS-S family physical sectors may be configured as a > hybrid combination of eight 4-kB parameter sectors at the top or bottom of the > address space with all but one of the remaining sectors being uniform size. > This >

[U-Boot] [PATCH] sf: Disable 4-KB erase command for SPANSION S25FS128

2016-06-22 Thread Michael Trimarchi
The S25FS128 is part of S25FS-S family physical sectors may be configured as a hybrid combination of eight 4-kB parameter sectors at the top or bottom of the address space with all but one of the remaining sectors being uniform size. This rework a bit commit 80c1bfd2332e71dfe669cac53ba06b7435a7ca3