From: Jagan Teki <jagannadh.teki@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] sf: Fix s25fs512s erase size and remove SECT_4K flag
Date: Wed, 26 Apr 2017 13:00:39 +0530 [thread overview]
Message-ID: <CAD6G_RRWDee7bTjwUFS2a5rWOdzhs7DJZ1Z1AJ=PyHx=kAMnOA@mail.gmail.com> (raw)
In-Reply-To: <1493112099-30909-3-git-send-email-suresh.gupta@nxp.com>
On Tue, Apr 25, 2017 at 2:51 PM, Suresh Gupta <suresh.gupta@nxp.com> wrote:
> As per data sheet, S25FS512S support Uniform sector option
> or erase size of 256 kbytes and Page Programming buffer of
> 256 or 512 Bytes. So, flag SECT_4K have no significance
> for this flash.
>
> Signed-off-by: Suresh Gupta <suresh.gupta@nxp.com>
> ---
> drivers/mtd/spi/spi_flash_ids.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/spi/spi_flash_ids.c b/drivers/mtd/spi/spi_flash_ids.c
> index edca94e..7ca33e8 100644
> --- a/drivers/mtd/spi/spi_flash_ids.c
> +++ b/drivers/mtd/spi/spi_flash_ids.c
> @@ -101,7 +101,7 @@ const struct spi_flash_info spi_flash_ids[] = {
> {"s25fl256s_256k", INFO(0x010219, 0x4d00, 256 * 1024, 128, RD_FULL | WR_QPP) },
> {"s25fl256s_64k", INFO(0x010219, 0x4d01, 64 * 1024, 512, RD_FULL | WR_QPP) },
> {"s25fs256s_64k", INFO6(0x010219, 0x4d0181, 64 * 1024, 512, RD_FULL | WR_QPP | SECT_4K) },
> - {"s25fs512s", INFO6(0x010220, 0x4d0081, 128 * 1024, 512, RD_FULL | WR_QPP | SECT_4K) },
> + {"s25fs512s", INFO6(0x010220, 0x4d0081, 256 * 1024, 256, RD_FULL | WR_QPP) },
Did you verify this? because we have an issue about this part number[1]
[1] https://lists.denx.de/pipermail/u-boot/2016-December/276032.html
thanks!
--
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
next prev parent reply other threads:[~2017-04-26 7:30 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-25 9:21 [U-Boot] [PATCH] config: remove CONFIG_SPI_FLASH_BAR from some platforms Suresh Gupta
2017-04-25 9:21 ` [U-Boot] [PATCH] LS1012A: change the size of flash Suresh Gupta
2017-05-23 16:20 ` york sun
2017-05-24 5:52 ` Suresh Gupta
2017-05-25 15:01 ` york sun
2017-04-25 9:21 ` [U-Boot] [PATCH] sf: Fix s25fs512s erase size and remove SECT_4K flag Suresh Gupta
2017-04-26 7:30 ` Jagan Teki [this message]
2017-04-26 7:47 ` Suresh Gupta
2017-04-26 23:05 ` York Sun
2017-04-27 5:17 ` Suresh Gupta
2017-04-27 16:47 ` York Sun
2017-04-28 5:26 ` Suresh Gupta
2017-05-12 17:53 ` York Sun
2017-05-15 6:47 ` Suresh Gupta
2017-06-07 7:03 ` Jagan Teki
2017-06-07 7:41 ` Suresh Gupta
2017-06-07 9:37 ` Jagan Teki
2017-06-08 7:30 ` Suresh Gupta
2017-06-12 6:07 ` Jagan Teki
2017-06-12 8:48 ` Suresh Gupta
2017-06-12 14:03 ` Suresh Gupta
2017-09-13 7:56 ` Suresh Gupta
2017-05-25 14:59 ` [U-Boot] [PATCH] config: remove CONFIG_SPI_FLASH_BAR from some platforms york sun
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='CAD6G_RRWDee7bTjwUFS2a5rWOdzhs7DJZ1Z1AJ=PyHx=kAMnOA@mail.gmail.com' \
--to=jagannadh.teki@gmail.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.