Re: [U-Boot] [PATCH] Revert "env_eeprom: Assign default environment during board_init_f"

2015-09-16 Thread Ludger Dreier
Hi Tom, Am 2015-09-15 18:54, schrieb Tom Rini: > Was there perhaps some sort of init ordering issue in a board which > did have EEPROM used as the backing store for env? I could see a > case where perhaps (and Ludger, can you test this as well please?) > the problem is that for env_eeprom env_ini

Re: [U-Boot] [PATCH] Revert "env_eeprom: Assign default environment during board_init_f"

2015-09-16 Thread Tom Rini
On Wed, Sep 16, 2015 at 01:46:34PM +0200, Ludger Dreier wrote: > Hi Tom, > > Am 2015-09-15 18:54, schrieb Tom Rini: > > Was there perhaps some sort of init ordering issue in a board which > > did have EEPROM used as the backing store for env? I could see a > > case where perhaps (and Ludger, can

Re: [U-Boot] [PATCH] Revert "env_eeprom: Assign default environment during board_init_f"

2015-09-15 Thread Tom Rini
On Tue, Sep 15, 2015 at 11:35:35AM +0200, Ludger Dreier wrote: > This reverts commit ed6a5d4f880ac248530dbf64683b2257dbe54b64. > > The original patch uses ENV_IS_EMBEDDED to decide if the default > environment should be used or the one actually read from EEPROM. The > code in environment.h allows

[U-Boot] [PATCH] Revert "env_eeprom: Assign default environment during board_init_f"

2015-09-15 Thread Ludger Dreier
This reverts commit ed6a5d4f880ac248530dbf64683b2257dbe54b64. The original patch uses ENV_IS_EMBEDDED to decide if the default environment should be used or the one actually read from EEPROM. The code in environment.h allows setting of ENV_IS_EMBEDDED only for a subset of flash types. EEPROM is no