From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 00/12] cmd_sf: Add support for read and write instructions
Date: Tue, 5 Mar 2013 12:45:15 -0500 [thread overview]
Message-ID: <51362F2B.9030006@ti.com> (raw)
In-Reply-To: <CAD6G_RQkODc5212CSYfFibbv8O0Zuy53HDzez0YPDNg5trztOg@mail.gmail.com>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 03/05/2013 12:21 PM, Jagan Teki wrote:
> Hi Tom,
>
> On Tue, Mar 5, 2013 at 10:38 PM, Tom Rini <trini@ti.com> wrote: On
> 03/05/2013 12:00 PM, Jagan Teki wrote:
>>>> Hi Tom,
>>>>
>>>> On Tue, Mar 5, 2013 at 2:38 AM, Tom Rini <trini@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.
Unless I'm missing something (or we're talking about different
things), you have a SPI flash that do quad (or dual) read/write. But
it also has to be connected to a controller that talks quad (or dual).
So is there any way we can make sf read/write/update be the fastest
supported (and why do we want fast and slow quad programming?) by the
controller and chip? I can see having to do "sf read.qp.qf ..." and
so forth leading to some less than desired user interaction.
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQIcBAEBAgAGBQJRNi8rAAoJENk4IS6UOR1WkAsP/3T7bdlRJAWtYFuUFoqF/Ofu
4LZZGcab75WwW7xUlHxct7oH8j4DE3c+PYT+RmxdpbhW7Av6VQJ9acDWRJE6upao
3M4WvTlDD2uyUEyMeWQWtgdcLNMP5eJcZ1CHP7MKYklW2qsdIbCZ4SnqseDqCIyb
7u2PPtCNWSVIn6tpihaM7hEho8KWa1YH7nnOfqF7V2629Tyah9IxX83VFLyP4Ujz
k/qRenQQN7CcKsyaSEnDNH3ZWH1FN0asrXZyxOcEUVaIca+pq4Z7QtmufS2I+e/b
DV2HUmI5UsC0USQrmaD21ais4taz4hZVNowEGcvw8Go0AnLkB7r+epksFr+Qf/T0
jm1XT3+pNEN4kt+Rgsuf+97ovOlNBWvIGmrYO104aDVg3+IVc8lkeb7STAKIAXah
d+WAA8kq36O0gj0enR4wsf96bzmn0aVUqBtuKmdwFP+W1U+q1Es/JodFW5niKMWn
i2VEMFZ0Oww+7VzWcIg5tT5t2FtkLkHRg+7QkFP0BIvd1+7rKpuEE+ntd9UBjFpr
ZzaC65D/aPtv9LvmTqOfRlk6nGxXmFJQqWbd0sv5hFEPp4pHMlkr8queKdRItW1E
dYu98kvcETdAI3AfgLcsyzjNOzPODRdlUiUmCDhcHW2Waxby7pmreeEdSJoHGeZt
QszhTVaNWgVq0qAFQVIc
=1CRc
-----END PGP SIGNATURE-----
next prev parent reply other threads:[~2013-03-05 17:45 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-31 11:13 [U-Boot] [PATCH 00/12] cmd_sf: Add support for read and write instructions Jagannadha Sutradharudu Teki
2012-12-31 11:13 ` [U-Boot] [PATCH 01/12] cmd_sf: Add wr_inst argument to 'sf write' command Jagannadha Sutradharudu Teki
2013-01-11 2:11 ` Simon Glass
[not found] ` <CAD6G_RSQXL3RsSuK=4QkRGRNfzuKvU-s7JPrsuB+o5O7GRxJuA@mail.gmail.com>
2013-02-14 14:35 ` Simon Glass
2012-12-31 11:13 ` [U-Boot] [PATCH 02/12] cmd_sf: Add rd_inst argument to 'sf read' command Jagannadha Sutradharudu Teki
2012-12-31 11:13 ` [U-Boot] [PATCH 03/12] cmd_sf: Add wr_inst argument to 'sf update' command Jagannadha Sutradharudu Teki
2012-12-31 11:13 ` [U-Boot] [PATCH 04/12] cmd_sf: Add rd_inst " Jagannadha Sutradharudu Teki
2012-12-31 11:13 ` [U-Boot] [PATCH 05/12] cmd_sf: Define a functions for parsing read and write instructions Jagannadha Sutradharudu Teki
2013-01-11 2:18 ` Simon Glass
2012-12-31 11:13 ` [U-Boot] [PATCH 06/12] cmd_sf: Add QPP(Quad-input Page Program) write instruction support Jagannadha Sutradharudu Teki
2012-12-31 11:13 ` [U-Boot] [PATCH 07/12] cmd_sf: Add ASR(Array Slow Read) read " Jagannadha Sutradharudu Teki
2012-12-31 11:13 ` [U-Boot] [PATCH 08/12] cmd_sf: Add DOFR(Dual Output Fast " Jagannadha Sutradharudu Teki
2012-12-31 11:13 ` [U-Boot] [PATCH 09/12] cmd_sf: Add QOFR(Quad " Jagannadha Sutradharudu Teki
2013-01-11 2:16 ` [U-Boot] [PATCH 00/12] cmd_sf: Add support for read and write instructions Simon Glass
2013-01-11 2:16 ` Simon Glass
2013-03-02 8:29 ` Jagan Teki
2013-03-04 21:02 ` Jagan Teki
2013-03-04 21:08 ` Tom Rini
2013-03-05 17:00 ` Jagan Teki
2013-03-05 17:08 ` Tom Rini
2013-03-05 17:21 ` Jagan Teki
2013-03-05 17:45 ` Tom Rini [this message]
2013-03-05 18:34 ` Jagan Teki
2013-03-05 19:59 ` Tom Rini
2013-03-08 12:22 ` Michal Simek
2013-03-08 23:55 ` Simon Glass
-- strict thread matches above, loose matches on Subject: below --
2013-01-10 14:33 Langer Thomas
2013-01-11 2:28 ` Simon Glass
2013-01-16 7:47 ` Jagan Teki
2013-01-16 19:51 ` Langer Thomas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51362F2B.9030006@ti.com \
--to=trini@ti.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.