Re: [U-Boot] [PATCH 1/2] env: Fix env_load_location

2018-02-08 Thread York Sun
On 02/08/2018 02:05 AM, Simon Goldschmidt wrote: > On 08.02.2018 09:38, Maxime Ripard wrote: >> Hi, >> >> Thanks for your patch >> >> On Wed, Feb 07, 2018 at 02:17:11PM -0800, York Sun wrote: >>> Commit 7d714a24d725 ("env: Support multiple environments") added >>> static variable env_load_location.

Re: [U-Boot] [PATCH 1/2] env: Fix env_load_location

2018-02-08 Thread Simon Goldschmidt
On 08.02.2018 09:38, Maxime Ripard wrote: Hi, Thanks for your patch On Wed, Feb 07, 2018 at 02:17:11PM -0800, York Sun wrote: Commit 7d714a24d725 ("env: Support multiple environments") added static variable env_load_location. When saving environmental variables, this variable is presumed to ha

Re: [U-Boot] [PATCH 1/2] env: Fix env_load_location

2018-02-08 Thread Maxime Ripard
Hi, Thanks for your patch On Wed, Feb 07, 2018 at 02:17:11PM -0800, York Sun wrote: > Commit 7d714a24d725 ("env: Support multiple environments") added > static variable env_load_location. When saving environmental > variables, this variable is presumed to have the value set before. > In case the

[U-Boot] [PATCH 1/2] env: Fix env_load_location

2018-02-07 Thread York Sun
Commit 7d714a24d725 ("env: Support multiple environments") added static variable env_load_location. When saving environmental variables, this variable is presumed to have the value set before. In case the value was set before relocation and U-Boot runs from a NOR flash, this variable wasn't writabl