Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Simon Goldschmidt
On 07.02.2018 22:18, York Sun wrote: On 02/07/2018 12:45 PM, Goldschmidt Simon wrote: On 02/07/2018 21:18, York Sun wrote: On 02/07/2018 12:43 AM, Maxime Ripard wrote: Hi, On Tue, Jan 30, 2018 at 11:02:49PM +, York Sun wrote: On 01/30/2018 12:16 PM, York Sun wrote: On 01/30/2018 11:40 A

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread York Sun
On 02/07/2018 12:45 PM, Goldschmidt Simon wrote: > On 02/07/2018 21:18, York Sun wrote: >> On 02/07/2018 12:43 AM, Maxime Ripard wrote: >>> Hi, >>> >>> On Tue, Jan 30, 2018 at 11:02:49PM +, York Sun wrote: On 01/30/2018 12:16 PM, York Sun wrote: > On 01/30/2018 11:40 AM, York Sun wrote

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Goldschmidt Simon
On 02/07/2018 21:18, York Sun wrote: > On 02/07/2018 12:43 AM, Maxime Ripard wrote: >> Hi, >> >> On Tue, Jan 30, 2018 at 11:02:49PM +, York Sun wrote: >>> On 01/30/2018 12:16 PM, York Sun wrote: On 01/30/2018 11:40 AM, York Sun wrote: > On 01/30/2018 12:19 AM, Simon Goldschmidt wrote:

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread York Sun
On 02/07/2018 12:43 AM, Maxime Ripard wrote: > Hi, > > On Tue, Jan 30, 2018 at 11:02:49PM +, York Sun wrote: >> On 01/30/2018 12:16 PM, York Sun wrote: >>> On 01/30/2018 11:40 AM, York Sun wrote: On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: > On 23.01.2018 21:16, Maxime Ripard wro

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Maxime Ripard
1;5002;0c On Wed, Feb 07, 2018 at 09:25:55AM +0100, Simon Goldschmidt wrote: > On 07.02.2018 09:19, Maxime Ripard wrote: > > On Tue, Feb 06, 2018 at 09:09:07AM +0100, Simon Goldschmidt wrote: > > > On 23.01.2018 21:16, Maxime Ripard wrote: > > > > Now that we have everything in place to support mul

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Maxime Ripard
On Wed, Feb 07, 2018 at 08:45:46AM +, Joakim Tjernlund wrote: > On Thu, 1970-01-01 at 00:00 +, Simon Goldschmidt wrote: > > CAUTION: This email originated from outside of the organization. Do not > > click links or open attachments unless you recognize the sender and know > > the content

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Joakim Tjernlund
On Thu, 1970-01-01 at 00:00 +, Simon Goldschmidt wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > On 06.02.2018 09:20, Joakim Tjernlund wrote: > > On Thu,

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Maxime Ripard
Hi, On Tue, Jan 30, 2018 at 11:02:49PM +, York Sun wrote: > On 01/30/2018 12:16 PM, York Sun wrote: > > On 01/30/2018 11:40 AM, York Sun wrote: > >> On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: > >>> On 23.01.2018 21:16, Maxime Ripard wrote: > Now that we have everything in place to s

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Joakim Tjernlund
On Thu, 1970-01-01 at 00:00 +, Maxime Ripard wrote: > Hi, > > On Tue, Feb 06, 2018 at 08:20:49AM +, Joakim Tjernlund wrote: > > On Thu, 1970-01-01 at 00:00 +, Simon Goldschmidt wrote: > > > > . > > > > Reviewed-by: Andre Przywara > > > > Reviewed-by: Simon Glass > > > > Signed-o

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Simon Goldschmidt
On 06.02.2018 09:20, Joakim Tjernlund wrote: On Thu, 1970-01-01 at 00:00 +, Simon Goldschmidt wrote: . Reviewed-by: Andre Przywara Reviewed-by: Simon Glass Signed-off-by: Maxime Ripard --- env/env.c | 80 +++- 1 file changed, 5

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Simon Goldschmidt
On 07.02.2018 09:19, Maxime Ripard wrote: On Tue, Feb 06, 2018 at 09:09:07AM +0100, Simon Goldschmidt wrote: On 23.01.2018 21:16, Maxime Ripard wrote: Now that we have everything in place to support multiple environment, let's make sure the current code can use it. I get more build errors test

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Maxime Ripard
On Tue, Feb 06, 2018 at 09:09:07AM +0100, Simon Goldschmidt wrote: > On 23.01.2018 21:16, Maxime Ripard wrote: > > Now that we have everything in place to support multiple environment, let's > > make sure the current code can use it. > > I get more build errors testing this feature: there's a glob

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-07 Thread Maxime Ripard
Hi, On Tue, Feb 06, 2018 at 08:20:49AM +, Joakim Tjernlund wrote: > On Thu, 1970-01-01 at 00:00 +, Simon Goldschmidt wrote: > > . > > > Reviewed-by: Andre Przywara > > > Reviewed-by: Simon Glass > > > Signed-off-by: Maxime Ripard > > > --- > > > env/env.c | 80 +++

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-06 Thread Joakim Tjernlund
On Thu, 1970-01-01 at 00:00 +, Simon Goldschmidt wrote: . > > Reviewed-by: Andre Przywara > > Reviewed-by: Simon Glass > > Signed-off-by: Maxime Ripard > > --- > > env/env.c | 80 +++- > > 1 file changed, 50 insertions(+), 30 deleti

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-02-06 Thread Simon Goldschmidt
On 23.01.2018 21:16, Maxime Ripard wrote: Now that we have everything in place to support multiple environment, let's make sure the current code can use it. I get more build errors testing this feature: there's a global variable 'env_ptr' declared in flash, nand, nvram and remote env drivers

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-30 Thread Simon Goldschmidt
On 31.01.2018 00:02, York Sun wrote: On 01/30/2018 12:16 PM, York Sun wrote: On 01/30/2018 11:40 AM, York Sun wrote: On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: On 23.01.2018 21:16, Maxime Ripard wrote: Now that we have everything in place to support multiple environment, let's make sure

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-30 Thread York Sun
On 01/30/2018 12:16 PM, York Sun wrote: > On 01/30/2018 11:40 AM, York Sun wrote: >> On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: >>> On 23.01.2018 21:16, Maxime Ripard wrote: Now that we have everything in place to support multiple environment, let's make sure the current code can us

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-30 Thread York Sun
On 01/30/2018 11:40 AM, York Sun wrote: > On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: >> On 23.01.2018 21:16, Maxime Ripard wrote: >>> Now that we have everything in place to support multiple environment, let's >>> make sure the current code can use it. >>> >>> The priority used between the va

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-30 Thread York Sun
On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: > On 23.01.2018 21:16, Maxime Ripard wrote: >> Now that we have everything in place to support multiple environment, let's >> make sure the current code can use it. >> >> The priority used between the various environment is the same one that was >> u

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-30 Thread Simon Goldschmidt
On 23.01.2018 21:16, Maxime Ripard wrote: Now that we have everything in place to support multiple environment, let's make sure the current code can use it. The priority used between the various environment is the same one that was used in the code previously. At read / init times, the highest

[U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-23 Thread Maxime Ripard
Now that we have everything in place to support multiple environment, let's make sure the current code can use it. The priority used between the various environment is the same one that was used in the code previously. At read / init times, the highest priority environment is going to be detected