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 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.