From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dirk Behme Date: Fri, 20 Jan 2012 08:06:05 +0100 Subject: [U-Boot] [PATCH 6/6] mx6q: mx6qsabrelite: Provide defaults for placing environment in serial flash In-Reply-To: References: <1326838190-13746-1-git-send-email-eric.nelson@boundarydevices.com> <1326838190-13746-7-git-send-email-eric.nelson@boundarydevices.com> Message-ID: <4F19125D.1080100@de.bosch.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 20.01.2012 04:27, Jason Hui wrote: > On Wed, Jan 18, 2012 at 6:09 AM, Eric Nelson > wrote: >> Signed-off-by: Eric Nelson >> --- >> include/configs/mx6qsabrelite.h | 12 +++++++++++- >> 1 files changed, 11 insertions(+), 1 deletions(-) >> >> diff --git a/include/configs/mx6qsabrelite.h b/include/configs/mx6qsabrelite.h >> index 44b028a..160894c 100644 >> --- a/include/configs/mx6qsabrelite.h >> +++ b/include/configs/mx6qsabrelite.h >> @@ -174,10 +174,20 @@ >> /* FLASH and environment organization */ >> #define CONFIG_SYS_NO_FLASH >> >> -#define CONFIG_ENV_OFFSET (6 * 64 * 1024) >> #define CONFIG_ENV_SIZE (8 * 1024) >> + >> #define CONFIG_ENV_IS_IN_MMC >> +/* #define CONFIG_ENV_IS_IN_SPI_FLASH */ > >>>From the commit log, it says the default is in serial flash, No, the commit log doesn't say this. It doesn't say 'it is'. It says it 'provides the defaults'. But it doesn't say that it actually uses these defaults. > but > apparently in the code > the env is default to MMC, which mismatch. please fix it. As mentioned above, I understand this differently. While I reviewed it some days ago, I found the description and the doing here quite fine. It enables the MMC env and additionally _provides_ everything needed to easily switch to SPI env by uncommenting one switch. This is fine and quite helpful, see e.g. [1]. I like this, please keep it as is. Best regards Dirk [1] http://lists.denx.de/pipermail/u-boot/2012-January/116266.html "you can place the environment in SPI-NOR as well by commenting out CONFIG_ENV_IS_IN_MMC, and un-commenting ..._IN_SPI_FLASH in include/configs/mx6qsabrelite.h." >> + >> +#if defined(CONFIG_ENV_IS_IN_MMC) >> +#define CONFIG_ENV_OFFSET (6 * 64 * 1024) >> #define CONFIG_SYS_MMC_ENV_DEV 0 >> +#elif defined(CONFIG_ENV_IS_IN_SPI_FLASH) >> +#define CONFIG_ENV_OFFSET (768 * 1024) >> +#define CONFIG_ENV_SECT_SIZE (8 * 1024) >> +#define CONFIG_ENV_SPI_CS 0x5300 > > I'm wondering how the CONFIG_ENV_SPI_CS could be 0x5300? Vague? > >> +#define CONFIG_ENV_SPI_MODE SPI_MODE_0 >> +#endif >> >> #define CONFIG_OF_LIBFDT >> >> -- >> 1.7.1 >> > -- ====================================================================== Dirk Behme Robert Bosch Car Multimedia GmbH CM-AI/PJ-CF32 Phone: +49 5121 49-3274 Dirk Behme Fax: +49 711 811 5053274 PO Box 77 77 77 mailto:dirk.behme at de.bosch.com D-31132 Hildesheim - Germany Bosch Group, Car Multimedia (CM) Automotive Navigation and Infotainment Systems (AI) ProJect - CoreFunctions (PJ-CF) Robert Bosch Car Multimedia GmbH - Ein Unternehmen der Bosch Gruppe Sitz: Hildesheim Registergericht: Amtsgericht Hildesheim HRB 201334 Aufsichtsratsvorsitzender: Volkmar Denner Gesch?ftsf?hrung: Uwe Thomas, Michael Bolle, Robby Drave, Egbert Hellwig ======================================================================