[PATCH v2] cmd: pxe_utils: sysboot: add label override support

[Amjad Ouled-Ameur]

Hi,

Is there any news about this patch please ?

Regards,

Amjad

On 25/11/2021 01:12, Simon Glass wrote:
> Hi Amjad,
>
> On Sat, 13 Nov 2021 at 06:09, Amjad Ouled-Ameur
> <aouledameur at baylibre.com> wrote:
>> This will allow consumers to choose a pxe label at runtime instead of
>> having to prompt the user. One good use-case for this, is choosing
>> whether or not to apply a dtbo depending on the hardware configuration.
>> e.g: for TI's AM335x EVM, it would be convenient to apply a particular
>> dtbo only when the J9 jumper is on PRUSS mode. To achieve this, the
>> pxe menu should have 2 labels, one with the dtbo and the other without,
>> then the "pxe_label_override" env variable should point to the label with
>> the dtbo at runtime only when the jumper is on PRUSS mode.
>>
>> This change can be used for different use-cases and bring more
>> flexibilty to consumers who use sysboot/pxe_utils.
>>
>> if "pxe_label_override" is set but does not exist in the pxe menu,
>> the code should fallback to the default label if given, and no failure
>> is returned but rather a warning message.
>>
>>
>> Signed-off-by: Amjad Ouled-Ameur <aouledameur at baylibre.com>
>> ---
>>
>> Changes in v2:
>> - Use !strcmp() instead of strcmp()==0
>> - Add "pxe_label_override" documentation
>>
>>   boot/pxe_utils.c | 15 +++++++++++++++
>>   doc/README.pxe   |  6 ++++++
>>   2 files changed, 21 insertions(+)
> Reviewed-by: Simon Glass <sjg at chromium.org>
>
> Can I suggest converting the doc to rST?
>
> Regards,
> Simon