[U-Boot] [PATCH 00/12] cmd_sf: Add support for read and write instructions

[Jagan Teki]

Hi Tom,

On Tue, Mar 5, 2013 at 10:38 PM, Tom Rini <trini at ti.com> wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 03/05/2013 12:00 PM, Jagan Teki wrote:
>> Hi Tom,
>>
>> On Tue, Mar 5, 2013 at 2:38 AM, Tom Rini <trini at ti.com> wrote:
>>> On Sat, Mar 02, 2013 at 01:59:38PM +0530, Jagan Teki wrote:
>>>
>>> [snip]
>>>> Since these changes I have sent long back, I am just
>>>> re-modified the framework to add new features at the same time
>>>> with backward comparability for current commands.
>>>>
>>>> Current command setup: sf write sf read sf update
>>>>
>>>> Changed command set: [no changes in the argument count] sf
>>>> write     ---  current command sf write.pp --  same as sf
>>>> write sf write.qp -- quad program
>>>>
>>>> sf read   -- current read sf read.af --- array flast read, same
>>>> as sf read sf read.as -- array slow read sf read.do --- dual
>>>> out sf read.qo -- quad out sf read.dio -- dual io sf read.qio
>>>> -- quad io
>>>>
>>>> sf update  -- current update sf update.pp.af -- write page
>>>> program, read array fast, same as sf update sf update.pp.as -
>>>> write page program, read array slow sf update.pp.do - write
>>>> page program, read dual out sf update.pp.qo - write page
>>>> program, read quad out sf update.pp.dio - write page program,
>>>> read dual io sf update.pp.qio - write page program, read quad
>>>> io sf update.qp.af - write quad program, read array fast sf
>>>> update.qp.as - write quad program, read array slow sf
>>>> update.qp.do - write quad program, read dual out sf
>>>> update.qp.qo - write quad program, read quad out sf
>>>> update.qp.dio - write quad program, read dual io sf
>>>> update.qp.qio - write quad program, read quad io
>>>>
>>>> Though it seems to be lengthy, but may useful with lot of
>>>> combinations from user. My intention is to use the existing
>>>> argument count with changes in the command set.
>>>
>>> Are there cases where for the current device we're operating on
>>> that can handle more than one of these, aside from fast or slow?
>>> And do we really need to offer both fast and slow?
>>
>> Yes as per as I know spansion, numonyx and winbond flashes are
>> supporting all the operation modes that I listed above.
>>
>> These are very generic w.r.t above flashes.or I can say these are
>> commonly available modes.
>
> And when hooked up via whichever SPI controller you have, all of those
> modes are also available and we can't really "guess" about using a
> faster mode?
I didn't get you. In general, u-boot should provide all the possible
operation modes
and the choice should be up to user.
Please elaborate as I seem to be missing something.

Thanks,
Jagan.

>
> - --
> Tom
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.11 (GNU/Linux)
> Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
>
> iQIcBAEBAgAGBQJRNiaFAAoJENk4IS6UOR1WMykQAJy9vm+WJIDxVvUdP1jEBksH
> mhOCcaSN7H7cbDxVvJ/9PBcN/9204X7Bf5Rg3k24rLO3QKIwJrtKP3hK9vQhGZRD
> ikaS6tqOY9auJlww7ZZSbA8Kh7WbzXQ3h8TNUl/tKTU/CqqhXnZyA0N6ML547HQA
> wn1+qRMSwduPZ7w0/276Ec2fRAhEK0IhXQ7KLBFaR8mman5sUB/IaRuE5oEtewaV
> rpnOiwoUkih3lSq3oT51ooJFtQR4q8LvH0v272dMjFteFqjirpEdsTRY8MB1l+pb
> GIGEFq/WHGkbrpzwEed9miTY1MhTZHQSRULErfgfsGck96a84J6CqGH1Se1GbRIP
> 5qjX4n9FyRyRMJ94wNju4Q3GMO9H04cQrxt+DACKPNAj0XoEkWZOa740fhn3rcaJ
> a+pbEja9c0PiPzoSsO4Pa3jTmvsvUcwNGlBxubdcIEDENmii63aeH8LzaOCSKTkO
> 3ckE9pIzBJO+Oe5CW/7x9srGGSTsbyNgKVbgfTesS0DQYzxJ401o7Cq7hbojqTI9
> TXy8RoJDbnxuIupm43K4Ci1dRZGX2g6/XrCexv7KIyiWgN1TQJwhJKXl74OH1RhL
> 4ZetFaJOHbGavaKGZK3ZTouQ3IjfYrnZ1FcijVW36fZredzHVFgG5HxGCE1VgKv7
> vRG65lCA3cSh7jyu8BYH
> =jrxq
> -----END PGP SIGNATURE-----