From mboxrd@z Thu Jan  1 00:00:00 1970
From: Lokesh Vutla <lokeshvutla@ti.com>
Date: Mon, 25 Nov 2013 10:18:27 +0530
Subject: [U-Boot] [PATCH V2 06/14] ARM: AM43XX: Add
 CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG support
In-Reply-To: <CANacCWxa8uXrvUCBR0vfETUpYpKagcdeBLzXiEyKtCsWZgGF6g@mail.gmail.com>
References: <1385014699-7257-1-git-send-email-lokeshvutla@ti.com>
	<1385014699-7257-7-git-send-email-lokeshvutla@ti.com>
	<CANacCWxa8uXrvUCBR0vfETUpYpKagcdeBLzXiEyKtCsWZgGF6g@mail.gmail.com>
Message-ID: <5292D69B.7000003@ti.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Friday 22 November 2013 01:58 AM, Vaibhav Bedia wrote:
> On Thu, Nov 21, 2013 at 1:18 AM, Lokesh Vutla <lokeshvutla@ti.com> wrote:
> [...]
>> +#ifdef CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG
>> +       char safe_string[HDR_NAME_LEN + 1];
>> +       struct am43xx_board_id header;
>> +
>> +       if (read_eeprom(&header) < 0)
>> +               puts("Could not get board ID.\n");
> 
> Hmm... didn't the previous patch read the EEPROM?
This is as per suggested by Tom. If I populate a common structure it ll bread NOR boot.
I hope you understand what happens in NOR boot.

Thanks 
Lokesh
>