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
<aouledam...@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 <aouledam...@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 <s...@chromium.org>

Can I suggest converting the doc to rST?

Regards,
Simon

Reply via email to