[PATCH] Revert "cmd: pxe_utils: Check fdtcontroladdr in label_boot"
Quentin Schulz
quentin.schulz at theobroma-systems.com
Tue Dec 13 15:34:45 CET 2022
Hi Tom,
On 12/13/22 15:29, Tom Rini wrote:
> With the change here, all extlinux.conf files with only "KERNEL
> /fitImage" don't work anymore. One common example of this would be those
> files generated by thee Poky/OE WIC bootimg-partition bootloader
> partition generator.
>
> This reverts commit d5ba6188dfbf6bb68354bec86e483623f1f6dae2.
>
FYI, this patch has been in U-Boot for three releases already, hence why
I was reluctant to ask for a revert when I discovered the issue (that
Neil already had reported in August, well before me).
Adding people who worked on the reverted patch in Cc so they can give
their 2¢ on it.
Cheers,
Quentin
> Reported-by: Neil Armstrong <neil.armstrong at linaro.org>
> Reported-by: Quentin Schulz <quentin.schulz at theobroma-systems.com>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> boot/pxe_utils.c | 8 +-------
> drivers/net/tsec.c | 2 +-
> 2 files changed, 2 insertions(+), 8 deletions(-)
>
> diff --git a/boot/pxe_utils.c b/boot/pxe_utils.c
> index 8133006875f9..96528aa14c03 100644
> --- a/boot/pxe_utils.c
> +++ b/boot/pxe_utils.c
> @@ -616,10 +616,7 @@ static int label_boot(struct pxe_context *ctx, struct pxe_label *label)
> * Scenario 2: If there is an fdt_addr specified, pass it along to
> * bootm, and adjust argc appropriately.
> *
> - * Scenario 3: If there is an fdtcontroladdr specified, pass it along to
> - * bootm, and adjust argc appropriately.
> - *
> - * Scenario 4: fdt blob is not available.
> + * Scenario 3: fdt blob is not available.
> */
> bootm_argv[3] = env_get("fdt_addr_r");
>
> @@ -724,9 +721,6 @@ static int label_boot(struct pxe_context *ctx, struct pxe_label *label)
> if (!bootm_argv[3])
> bootm_argv[3] = env_get("fdt_addr");
>
> - if (!bootm_argv[3])
> - bootm_argv[3] = env_get("fdtcontroladdr");
> -
> if (bootm_argv[3]) {
> if (!bootm_argv[2])
> bootm_argv[2] = "-";
> diff --git a/drivers/net/tsec.c b/drivers/net/tsec.c
> index d69a9ff47736..519ea14b070e 100644
> --- a/drivers/net/tsec.c
> +++ b/drivers/net/tsec.c
> @@ -156,7 +156,7 @@ static int tsec_mcast_addr(struct udevice *dev, const u8 *mcast_mac, int join)
> return 0;
> }
>
> -static int __maybe_unused tsec_set_promisc(struct udevice *dev, bool enable)
> +static int tsec_set_promisc(struct udevice *dev, bool enable)
> {
> struct tsec_private *priv = dev_get_priv(dev);
> struct tsec __iomem *regs = priv->regs;
More information about the U-Boot
mailing list