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

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

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

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

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

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:

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

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

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

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

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

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

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

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

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 >>

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