[U-Boot] [PATCH 6/6] mx6q: mx6qsabrelite: Provide defaults for placing environment in serial flash

Dirk Behme
2012-01-20T07:06:05Z

On 20.01.2012 04:27, Jason Hui wrote:
> On Wed, Jan 18, 2012 at 6:09 AM, Eric Nelson
> <eric.nelson@boundarydevices.com> wrote:
>> Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
>> ---
>> 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
======================================================================