From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [UBOOT][PATCH 3/5] configs: dra7-evm: Add mtd parts info for qspi.
Date: Wed, 2 Apr 2014 07:43:01 -0400 [thread overview]
Message-ID: <20140402114301.GA16360@bill-the-cat> (raw)
In-Reply-To: <1396434973-26354-4-git-send-email-sourav.poddar@ti.com>
On Wed, Apr 02, 2014 at 04:06:11PM +0530, Sourav Poddar wrote:
> Add MTD partition info for qspi on dra7 evm
>
> Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
[snip]
> +/*
> + * Default to using SPI for environment, etc.
> + * 0x000000 - 0x010000 : QSPI.SPL (64KiB)
> + * 0x010000 - 0x020000 : QSPI.SPL.backup1 (64KiB)
> + * 0x020000 - 0x030000 : QSPI.SPL.backup2 (64KiB)
> + * 0x030000 - 0x040000 : QSPI.SPL.backup3 (64KiB)
> + * 0x040000 - 0x140000 : QSPI.u-boot (1MiB)
> + * 0x140000 - 0x150000 : QSPI.u-boot-spl-os (64KiB)
This is too small, I found out when adding falcon mode support for this
board. We need to bump this to 512KiB and then re-adjust what follows.
> +#if defined(CONFIG_QSPI_BOOT)
> +#undef CONFIG_ENV_IS_IN_MMC
... This needs adjusting too, we should only be setting
CONFIG_ENV_IS_IN_MMC on !QSPI_BOOT, up in the MMC area.
And related, we need to fix ti_omap5_common.h to not set CONFIG_ENV_SIZE
because right now it's setting 128KiB and our redundant location never
would have valid data.
I'll pick this all up in a v2 and address it, along with a few other
things. Thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140402/0619d8bc/attachment.pgp>
next prev parent reply other threads:[~2014-04-02 11:43 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-02 10:36 [U-Boot] [UBOOT][PATCH 0/5] dra7: Add partition table entry and qspi boot config Sourav Poddar
2014-04-02 10:36 ` [U-Boot] [UBOOT][PATCH 1/5] board.cfg: dra7: Add QSPI " Sourav Poddar
2014-04-02 10:36 ` [U-Boot] [UBOOT][PATCH 2/5] qspi: add support for qspi4 device Sourav Poddar
2014-04-02 10:36 ` [U-Boot] [UBOOT][PATCH 3/5] configs: dra7-evm: Add mtd parts info for qspi Sourav Poddar
2014-04-02 11:43 ` Tom Rini [this message]
2014-04-03 6:29 ` Jagan Teki
2014-04-03 6:30 ` Sourav Poddar
2014-04-02 10:36 ` [U-Boot] [UBOOT][PATCH 4/5] configs: dra7-evm: change uboot offset Sourav Poddar
2014-04-02 10:36 ` [U-Boot] [UBOOT][PATCH 5/5] spi: ti_qspi: Add delay for successful bulk erase Sourav Poddar
2014-04-03 6:00 ` Jagan Teki
2014-04-03 6:16 ` Sourav Poddar
2014-04-03 6:42 ` Jagan Teki
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=20140402114301.GA16360@bill-the-cat \
--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.