Hi Manorit, Thank you for the patch.
On lun., janv. 08, 2024 at 13:48, Manorit Chawdhry <[email protected]> wrote: > By default this variable is unset and this causes the test condition to > fail on devices that don't have this defined. Set a default value for > this. > > => boot > ## Error: "main_cpsw0_qsgmii_phyinit" not defined > [...] > > Fixes: 0d72b0f2f83b ("include: env: ti: ti_common: Run > main_cpsw0_qsgmii_phyinit conditionally") > > Signed-off-by: Manorit Chawdhry <[email protected]> Reviewed-by: Mattijs Korpershoek <[email protected]> > --- > include/env/ti/ti_common.env | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/env/ti/ti_common.env b/include/env/ti/ti_common.env > index 5473f633aa7d..02b410c3adc4 100644 > --- a/include/env/ti/ti_common.env > +++ b/include/env/ti/ti_common.env > @@ -22,6 +22,7 @@ get_fit_overlaystring= > done; > get_fit_config=setexpr name_fit_config gsub / _ conf-${fdtfile} > run_fit=run get_fit_config; bootm > ${addr_fit}#${name_fit_config}${overlaystring} > +do_main_cpsw0_qsgmii_phyinit=0 > bootcmd_ti_mmc= > run findfdt; run init_${boot}; > #if CONFIG_CMD_REMOTEPROC > > -- > 2.43.0

