Patchwork [U-Boot,2/6] am335x_evm: Add MTDPARTS info for SPI flash

login
register
mail settings
Submitter Tom Rini
Date May 8, 2013, 10:45 p.m.
Message ID <1368053157-28317-2-git-send-email-trini@ti.com>
Download mbox | patch
Permalink /patch/242656/
State Superseded
Delegated to: Tom Rini
Headers show

Comments

Tom Rini - May 8, 2013, 10:45 p.m.
Signed-off-by: Tom Rini <trini@ti.com>
---
 include/configs/am335x_evm.h |    7 +++++++
 1 file changed, 7 insertions(+)
Peter Korsgaard - May 12, 2013, 9:10 p.m.
>>>>> "Tom" == Tom Rini <trini@ti.com> writes:

 Tom> Signed-off-by: Tom Rini <trini@ti.com>
 Tom> ---
 Tom>  include/configs/am335x_evm.h |    7 +++++++
 Tom>  1 file changed, 7 insertions(+)

 Tom> diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h
 Tom> index 945ec09..004a06a 100644
 Tom> --- a/include/configs/am335x_evm.h
 Tom> +++ b/include/configs/am335x_evm.h
 Tom> @@ -436,6 +436,13 @@
 Tom>  # define CONFIG_ENV_SPI_MAX_HZ		CONFIG_SF_DEFAULT_SPEED
 Tom>  # define CONFIG_ENV_OFFSET		(892 << 10) /* 892 KiB in */
 Tom>  # define CONFIG_ENV_SECT_SIZE		(4 << 10) /* 4 KB sectors */
 Tom> +#define CONFIG_CMD_MTDPARTS
 Tom> +#define MTDIDS_DEFAULT			"nor0=physmap-flash.0"
 Tom> +#define MTDPARTS_DEFAULT		"mtdparts=physmap-flash.0:128k(SPL)," \

physmap for a spi flash? I would have expected to see m25p80?


 Tom> +					"128k(SPL.backup1)," \
 Tom> +					"128k(SPL.backup2)," \
 Tom> +					"128k(SPL.backup3),384k(u-boot)," \
 Tom> +					"4k(u-boot-env),3464k(kernel),-(rootfs)"
 Tom>  #endif /* SPI support */
 
 Tom>  /* Unsupported features */
 Tom> -- 
 Tom> 1.7.9.5

 Tom> _______________________________________________
 Tom> U-Boot mailing list
 Tom> U-Boot@lists.denx.de
 Tom> http://lists.denx.de/mailman/listinfo/u-boot
Tom Rini - May 13, 2013, 12:24 p.m.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 05/12/2013 05:10 PM, Peter Korsgaard wrote:
>>>>>> "Tom" == Tom Rini <trini@ti.com> writes:
> 
> Tom> Signed-off-by: Tom Rini <trini@ti.com> Tom> --- Tom>
> include/configs/am335x_evm.h |    7 +++++++ Tom>  1 file changed, 7
> insertions(+)
> 
> Tom> diff --git a/include/configs/am335x_evm.h
> b/include/configs/am335x_evm.h Tom> index 945ec09..004a06a 100644 
> Tom> --- a/include/configs/am335x_evm.h Tom> +++
> b/include/configs/am335x_evm.h Tom> @@ -436,6 +436,13 @@ Tom>  #
> define CONFIG_ENV_SPI_MAX_HZ		CONFIG_SF_DEFAULT_SPEED Tom>  #
> define CONFIG_ENV_OFFSET		(892 << 10) /* 892 KiB in */ Tom>  #
> define CONFIG_ENV_SECT_SIZE		(4 << 10) /* 4 KB sectors */ Tom>
> +#define CONFIG_CMD_MTDPARTS Tom> +#define MTDIDS_DEFAULT
> "nor0=physmap-flash.0" Tom> +#define MTDPARTS_DEFAULT
> "mtdparts=physmap-flash.0:128k(SPL)," \
> 
> physmap for a spi flash? I would have expected to see m25p80?

Yes, I'd swear it's physmap when I've tested it on the vendor kernel tree.

- -- 
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJRkNt3AAoJENk4IS6UOR1WyzoP/A8pbRwsOTOHx0dlISVP9fBz
Kh08OPhplgATd/PgbE/OvVGJo43CDK8gmdT0hvWDtIq5GSTfpL1iE7hUZiTMYLkj
ir6xlE2lV0MUuD675VQu/r4d2sN/6+MuAffdjgV2vfeyeRcSRM2pjXGr0HwicQvX
iYX9ZDQMhGDNlk5tAjX5cTsAY3r5rvYL5Fr3vtHBLa69GnYF9/aZC5uuYsLwYCYB
icBprOnK5oT2imsUqGjrMCFP3NP6MOOYp8ywRper3ssHujKXu6tP6K/nvbJZUEAn
W9YcluLIIbVGKapw7VErBfXzf2iwPArWPRGHEKMcwyavhPd34dgZCaiDc7/OdYQg
1vLPhlJ+E0lbFTX97CYQqW0lNB8y6dxbIf4bshsey8rj7bC569Kboo2wqu+W+Boh
cy0Aq6Tc64izoGQahx6eK+i4LQiWdI2pVw/4BT6BzY8ih4bjLUujskLy2koTkLDu
iRzfnHytLKu0i2bBE9zsqmWg2vD8mSyUkwUXt9JuqWvIFn9XRnKhXCp+L4S+ST9n
uUmMBfNireQXfPWXuJHgzFuyzl/wPMiTKZ1BGzisfMNo6usHKFcrB6wc4bdgLtO4
+gMEQOKykf7OoS74MxVAC1eObuOuSQJGhBNqr08cq4jUTjdalBbw5CTrFfk8Jgdq
8DgGRk0rwxoteTmSPjBY
=r89g
-----END PGP SIGNATURE-----
Tom Rini - May 13, 2013, 4:10 p.m.
On Mon, May 13, 2013 at 08:24:23AM -0400, Tom Rini wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 05/12/2013 05:10 PM, Peter Korsgaard wrote:
> >>>>>> "Tom" == Tom Rini <trini@ti.com> writes:
> > 
> > Tom> Signed-off-by: Tom Rini <trini@ti.com> Tom> --- Tom>
> > include/configs/am335x_evm.h |    7 +++++++ Tom>  1 file changed, 7
> > insertions(+)
> > 
> > Tom> diff --git a/include/configs/am335x_evm.h
> > b/include/configs/am335x_evm.h Tom> index 945ec09..004a06a 100644 
> > Tom> --- a/include/configs/am335x_evm.h Tom> +++
> > b/include/configs/am335x_evm.h Tom> @@ -436,6 +436,13 @@ Tom>  #
> > define CONFIG_ENV_SPI_MAX_HZ		CONFIG_SF_DEFAULT_SPEED Tom>  #
> > define CONFIG_ENV_OFFSET		(892 << 10) /* 892 KiB in */ Tom>  #
> > define CONFIG_ENV_SECT_SIZE		(4 << 10) /* 4 KB sectors */ Tom>
> > +#define CONFIG_CMD_MTDPARTS Tom> +#define MTDIDS_DEFAULT
> > "nor0=physmap-flash.0" Tom> +#define MTDPARTS_DEFAULT
> > "mtdparts=physmap-flash.0:128k(SPL)," \
> > 
> > physmap for a spi flash? I would have expected to see m25p80?
> 
> Yes, I'd swear it's physmap when I've tested it on the vendor kernel tree.

Or I must be misrecalling things, changed.

Patch

diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h
index 945ec09..004a06a 100644
--- a/include/configs/am335x_evm.h
+++ b/include/configs/am335x_evm.h
@@ -436,6 +436,13 @@ 
 # define CONFIG_ENV_SPI_MAX_HZ		CONFIG_SF_DEFAULT_SPEED
 # define CONFIG_ENV_OFFSET		(892 << 10) /* 892 KiB in */
 # define CONFIG_ENV_SECT_SIZE		(4 << 10) /* 4 KB sectors */
+#define CONFIG_CMD_MTDPARTS
+#define MTDIDS_DEFAULT			"nor0=physmap-flash.0"
+#define MTDPARTS_DEFAULT		"mtdparts=physmap-flash.0:128k(SPL)," \
+					"128k(SPL.backup1)," \
+					"128k(SPL.backup2)," \
+					"128k(SPL.backup3),384k(u-boot)," \
+					"4k(u-boot-env),3464k(kernel),-(rootfs)"
 #endif /* SPI support */
 
 /* Unsupported features */