On Wed, Jan 15, 2014 at 9:11 PM, Marek Vasut <[email protected]> wrote:
> On Wednesday, January 15, 2014 at 04:17:55 PM, Jagan Teki wrote:
>> On Wed, Jan 15, 2014 at 8:06 PM, Marek Vasut <[email protected]> wrote:
>> > On Wednesday, January 15, 2014 at 03:29:43 PM, Marek Vasut wrote:
>> >> Both of these chips have 256kB big sectors, thus the _256K suffix,
>> >> compared to their _64K counterparts, which have 64kB sectors. Also,
>> >> they have four times less sectors than their _64K counterparts.
>> >>
>> >> Signed-off-by: Marek Vasut <[email protected]>
>> >> Cc: Jagannadha Sutradharudu Teki <[email protected]>
>> >> ---
>> >>
>> >>  drivers/mtd/spi/sf_params.c | 4 ++--
>> >>  1 file changed, 2 insertions(+), 2 deletions(-)
>> >>
>> >> Note: Would be nice if someone actually tested this fix as I go by the
>> >>
>> >>       datasheet and by the old code that _was_ in U-Boot before the
>> >>       rework.
>> >
>> > btw. would be nice to get this one into current release to prevent it
>> > being broken. But I would _really_ appreciate some real-hardware testing
>> > here.
>>
>> Yes - I'll try it and let us know.
>
> Thanks!

Tested on S25FL256S_256K, works fine!

-- 
Thanks,
Jagan.
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to