Re: [PATCH 2/2] bootm: Add a bootm command for type IH_OS_EFI

On 12/27/19 5:41 PM, Simon Glass wrote:

...

I think we should avoid transferring arguments as strings. It is preferable to separate do_bootefi() into an argument parser and an execution part.

Let's CC the developers who have contributed to common/bootm_os.c.

Yes I think it would be good to create a function to hold most of the code from do_bootefi(), call the function with appropriate parameters and changing do_bootefi() to call that new function.

Hello Simon,

you are responding to an outdated version of the patch. Please, see

https://patchwork.ozlabs.org/patch/1215251/ [v4,2/5] bootm: Add a bootm command for type IH_OS_EFI

Best regards

Heinrich