From: "Cédric Le Goater" <clg@kaod.org>
To: marcin.krzeminski@nokia.com, qemu-devel@nongnu.org
Cc: crosthwaitepeter@gmail.com, pawel.lenkow@itlen.com,
peter.maydell@linaro.org
Subject: Re: [Qemu-devel] [PATCH 9/9] m25p80: New flash devices.
Date: Thu, 16 Jun 2016 09:20:42 +0200 [thread overview]
Message-ID: <5762534A.4060507@kaod.org> (raw)
In-Reply-To: <1465998071-7355-10-git-send-email-marcin.krzeminski@nokia.com>
On 06/15/2016 03:41 PM, marcin.krzeminski@nokia.com wrote:
> From: Marcin Krzeminski <marcin.krzeminski@nokia.com>
>
> Macronix: mx66u51235f and mx66u1g45g
> Micron: mt25ql01g and mt25qu01g
> Spansion: s25fs512s and s70fs01gs
>
> Signed-off-by: Marcin Krzeminski <marcin.krzeminski@nokia.com>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
> ---
> hw/block/m25p80.c | 19 +++++++++++++------
> 1 file changed, 13 insertions(+), 6 deletions(-)
>
> diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c
> index 7bc0e03..e1a7698 100644
> --- a/hw/block/m25p80.c
> +++ b/hw/block/m25p80.c
> @@ -192,6 +192,8 @@ static const FlashPartInfo known_devices[] = {
> { INFO("mx25l12855e", 0xc22618, 0, 64 << 10, 256, 0) },
> { INFO("mx25l25635e", 0xc22019, 0, 64 << 10, 512, 0) },
> { INFO("mx25l25655e", 0xc22619, 0, 64 << 10, 512, 0) },
> + { INFO("mx66u51235f", 0xc2253a, 0, 64 << 10, 1024, ER_4K | ER_32K) },
> + { INFO("mx66u1g45g", 0xc2253b, 0, 64 << 10, 2048, ER_4K | ER_32K) },
>
> /* Micron */
> { INFO("n25q032a11", 0x20bb16, 0, 64 << 10, 64, ER_4K) },
> @@ -202,6 +204,11 @@ static const FlashPartInfo known_devices[] = {
> { INFO("n25q128a13", 0x20ba18, 0, 64 << 10, 256, ER_4K) },
> { INFO("n25q256a11", 0x20bb19, 0, 64 << 10, 512, ER_4K) },
> { INFO("n25q256a13", 0x20ba19, 0, 64 << 10, 512, ER_4K) },
> + { INFO("n25q128", 0x20ba18, 0, 64 << 10, 256, 0) },
> + { INFO("n25q256a", 0x20ba19, 0, 64 << 10, 512, ER_4K) },
> + { INFO("n25q512a", 0x20ba20, 0, 64 << 10, 1024, ER_4K) },
> + { INFO("mt25ql01g", 0x20ba21, 0, 64 << 10, 2048, ER_4K) },
> + { INFO("mt25qu01g", 0x20bb21, 0, 64 << 10, 2048, ER_4K) },
>
> /* Spansion -- single (large) sector size only, at least
> * for the chips listed here (without boot sectors).
> @@ -210,8 +217,8 @@ static const FlashPartInfo known_devices[] = {
> { INFO("s25sl064p", 0x010216, 0x4d00, 64 << 10, 128, ER_4K) },
> { INFO("s25fl256s0", 0x010219, 0x4d00, 256 << 10, 128, 0) },
> { INFO("s25fl256s1", 0x010219, 0x4d01, 64 << 10, 512, 0) },
> - { INFO("s25fl512s", 0x010220, 0x4d00, 256 << 10, 256, 0) },
> - { INFO("s70fl01gs", 0x010221, 0x4d00, 256 << 10, 256, 0) },
> + { INFO6("s25fl512s", 0x010220, 0x4d0080, 256 << 10, 256, 0) },
> + { INFO6("s70fl01gs", 0x010221, 0x4d0080, 256 << 10, 512, 0) },
> { INFO("s25sl12800", 0x012018, 0x0300, 256 << 10, 64, 0) },
> { INFO("s25sl12801", 0x012018, 0x0301, 64 << 10, 256, 0) },
> { INFO("s25fl129p0", 0x012018, 0x4d00, 256 << 10, 64, 0) },
> @@ -224,6 +231,10 @@ static const FlashPartInfo known_devices[] = {
> { INFO("s25fl016k", 0xef4015, 0, 64 << 10, 32, ER_4K | ER_32K) },
> { INFO("s25fl064k", 0xef4017, 0, 64 << 10, 128, ER_4K | ER_32K) },
>
> + /* Spansion -- boot sectors support */
> + { INFO6("s25fs512s", 0x010220, 0x4d0081, 256 << 10, 256, 0) },
> + { INFO6("s70fs01gs", 0x010221, 0x4d0081, 256 << 10, 512, 0) },
> +
> /* SST -- large erase sizes are "overlays", "sectors" are 4<< 10 */
> { INFO("sst25vf040b", 0xbf258d, 0, 64 << 10, 8, ER_4K) },
> { INFO("sst25vf080b", 0xbf258e, 0, 64 << 10, 16, ER_4K) },
> @@ -274,10 +285,6 @@ static const FlashPartInfo known_devices[] = {
> { INFO("w25q80", 0xef5014, 0, 64 << 10, 16, ER_4K) },
> { INFO("w25q80bl", 0xef4014, 0, 64 << 10, 16, ER_4K) },
> { INFO("w25q256", 0xef4019, 0, 64 << 10, 512, ER_4K) },
> -
> - { INFO("n25q128", 0x20ba18, 0, 64 << 10, 256, 0) },
> - { INFO("n25q256a", 0x20ba19, 0, 64 << 10, 512, ER_4K) },
> - { INFO("n25q512a", 0x20ba20, 0, 64 << 10, 1024, ER_4K) },
> };
>
> typedef enum {
>
prev parent reply other threads:[~2016-06-16 7:20 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-15 13:41 [Qemu-devel] [PATCH 0/9] m25p80: Add new 512Mbit and 1Gbit devices marcin.krzeminski
2016-06-15 13:41 ` [Qemu-devel] [PATCH 1/9] m25p80: Replace JEDEC ID masking with function marcin.krzeminski
2016-06-15 14:05 ` Cédric Le Goater
2016-06-15 17:09 ` [Qemu-devel] Odp.: " Krzeminski, Marcin (Nokia - PL/Wroclaw)
2016-06-15 13:41 ` [Qemu-devel] [PATCH 2/9] m25p80: Make a table for JEDEC ID marcin.krzeminski
2016-06-15 14:17 ` Cédric Le Goater
2016-06-15 13:41 ` [Qemu-devel] [PATCH 3/9] m25p80: Allow more than four banks marcin.krzeminski
2016-06-16 7:09 ` Cédric Le Goater
2016-06-15 13:41 ` [Qemu-devel] [PATCH 4/9] m25p80: Introduce COLLECTING_VAR_LEN_DATA state marcin.krzeminski
2016-06-16 7:13 ` Cédric Le Goater
2016-06-16 7:43 ` Krzeminski, Marcin (Nokia - PL/Wroclaw)
2016-06-16 8:13 ` Cédric Le Goater
2016-06-15 13:41 ` [Qemu-devel] [PATCH 5/9] m25p80: Add additional flash commands: marcin.krzeminski
2016-06-16 7:14 ` Cédric Le Goater
2016-06-15 13:41 ` [Qemu-devel] [PATCH 6/9] m25p80: Introduce quad and equad modes marcin.krzeminski
2016-06-15 14:25 ` Cédric Le Goater
2016-06-15 17:40 ` [Qemu-devel] Odp.: " Krzeminski, Marcin (Nokia - PL/Wroclaw)
2016-06-17 12:43 ` Cédric Le Goater
2016-06-15 13:41 ` [Qemu-devel] [PATCH 7/9] m25p80: Introduce configuration registers marcin.krzeminski
2016-06-16 7:24 ` Cédric Le Goater
2016-06-16 7:52 ` Krzeminski, Marcin (Nokia - PL/Wroclaw)
2016-06-16 8:05 ` Cédric Le Goater
2016-06-17 10:31 ` Krzeminski, Marcin (Nokia - PL/Wroclaw)
2016-06-15 13:41 ` [Qemu-devel] [PATCH 8/9] m25p80: Fast read commands family changes marcin.krzeminski
2016-06-16 7:19 ` Cédric Le Goater
2016-06-16 7:53 ` Krzeminski, Marcin (Nokia - PL/Wroclaw)
2016-06-15 13:41 ` [Qemu-devel] [PATCH 9/9] m25p80: New flash devices marcin.krzeminski
2016-06-16 7:20 ` Cédric Le Goater [this message]
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=5762534A.4060507@kaod.org \
--to=clg@kaod.org \
--cc=crosthwaitepeter@gmail.com \
--cc=marcin.krzeminski@nokia.com \
--cc=pawel.lenkow@itlen.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).