[PATCH 1/6] include: env: phytec: overlays: Add extension command

[Yannic Moog]

Hello Daniel,

On Sun, 2024-07-07 at 23:07 -0700, Daniel Schultz wrote:
> Add a new environment routine to apply extensions. Our SOM detection
> adds overlays via the extension framework to alter the kernel
> device-tree according to the flashed EEPROM image.
> 
> Signed-off-by: Daniel Schultz <d.schultz at phytec.de>
> ---
>  include/env/phytec/overlays.env | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
> 
> diff --git a/include/env/phytec/overlays.env b/include/env/phytec/overlays.env
> index 78721cde654..50110e731bd 100644
> --- a/include/env/phytec/overlays.env
> +++ b/include/env/phytec/overlays.env
> @@ -23,6 +23,16 @@ mmc_apply_overlays=
>  			fi;
>  		done;
>  	fi;
> +#ifdef CONFIG_CMD_EXTENSION
> +mmc_apply_extensions=
> +	setenv extension_overlay_addr ${fdtoverlay_addr_r};
> +	setenv extension_overlay_cmd 'load mmc ${mmcdev}:${mmcpart}
> +	  ${fdtoverlay_addr_r} ${extension_overlay_name}';
> +	extension scan;
> +	extension apply all;
> +#else
> +mmc_apply_extensions=echo "no extension command"

Do you think it makes sense to make this fail? I would think "apply_extensions" would be
unsuccessful if extensions are not working.

Yannic

> +#endif
>  net_load_bootenv=${get_cmd} ${bootenv_addr_r} ${bootenv}
>  net_load_overlay=${get_cmd} ${fdtoverlay_addr_r} ${overlay}
>  net_apply_overlays=
> @@ -36,3 +46,13 @@ net_apply_overlays=
>  			fi;
>  		done;
>  	fi;
> +#ifdef CONFIG_CMD_EXTENSION
> +net_apply_extensions=
> +	setenv extension_overlay_addr ${fdtoverlay_addr_r};
> +	setenv extension_overlay_cmd '${get_cmd} ${fdtoverlay_addr_r}
> +	 ${extension_overlay_name}';
> +	extension scan;
> +	extension apply all;
> +#else
> +net_apply_extensions=echo "no extension command"
> +#endif