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

[Jason Hui]

On Fri, Jan 20, 2012 at 3:06 PM, Dirk Behme <dirk.behme at de.bosch.com> wrote:
> On 20.01.2012 04:27, Jason Hui wrote:
>>
>> On Wed, Jan 18, 2012 at 6:09 AM, Eric Nelson
>> <eric.nelson at boundarydevices.com> wrote:
>>>
>>> Signed-off-by: Eric Nelson <eric.nelson at 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.

OK, I get it, then I'm fine with it too.

>
> 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.
>
[...]

>>> +#define CONFIG_ENV_SPI_CS              0x5300
>>
>>
>> I'm wondering how the CONFIG_ENV_SPI_CS  could be 0x5300? Vague?

Then the left open question is only above one.

Jason