Re: [PATCH 09/11] bootm: Allow updating the bootargs in a buffer

2020-10-21 Thread Simon Glass
Hi Wolfgang, On Wed, 21 Oct 2020 at 00:57, Wolfgang Denk wrote: > > Dear Simon, > > In message > you > wrote: > > > > > I think this is the wrong way to address a problem. > > > > > > Instead, zimage should use bootargs as well. > > > > OK I can add that. But for testing purposes it is still g

Re: [PATCH 09/11] bootm: Allow updating the bootargs in a buffer

2020-10-20 Thread Wolfgang Denk
Dear Simon, In message you wrote: > > > I think this is the wrong way to address a problem. > > > > Instead, zimage should use bootargs as well. > > OK I can add that. But for testing purposes it is still good to be > able to have the feature in a function that can be called without > going thro

Re: [PATCH 09/11] bootm: Allow updating the bootargs in a buffer

2020-10-20 Thread Simon Glass
Hi Wolfgang, On Mon, 19 Oct 2020 at 08:47, Wolfgang Denk wrote: > > Dear Simon Glass, > > In message <20201019135602.3943835-10-...@chromium.org> you wrote: > > At present we only support updating the 'bootargs' environment > > variable. Add another function to update a buffer instead. This will

Re: [PATCH 09/11] bootm: Allow updating the bootargs in a buffer

2020-10-19 Thread Wolfgang Denk
Dear Simon Glass, In message <20201019135602.3943835-10-...@chromium.org> you wrote: > At present we only support updating the 'bootargs' environment > variable. Add another function to update a buffer instead. This will > allow zimage to use this feature. I think this is the wrong way to address

[PATCH 09/11] bootm: Allow updating the bootargs in a buffer

2020-10-19 Thread Simon Glass
At present we only support updating the 'bootargs' environment variable. Add another function to update a buffer instead. This will allow zimage to use this feature. Also add a lot more tests to cover various cases. Signed-off-by: Simon Glass --- common/bootm.c | 18 +++- include/bootm.h