Hi Casey,
On Mon, Feb 09, 2026 at 07:49:50PM +0100, Casey Connolly wrote:
> Hi Sumit,
>
> Thanks for respinning this.
>
> On 27/01/2026 07:23, Sumit Garg wrote:
> > From: Sumit Garg <[email protected]>
> >
> > Add support for OP-TEE live tree DT fixup support which enables U-Boot
> > OP-TEE driver to be probed. As well as the EFI DT fixup protocol allows
> > the live tree fixup to be carried over to the OS for the OP-TEE driver
> > in the OS to probe as well.
>
> The EFI DT fixup protocol hook seems to be missing here?
The EFI DT fixup for OP-TEE node is already taken care of by the common
OP-TEE library as part of this code path:
efi_dt_fixup() ->
image_setup_libfdt() ->
optee_copy_fdt_nodes()
-Sumit
> >
> > Note that this fixup only gets applied if OP-TEE support is detected via
> > checking for OP-TEE message UID.
> >
> > Signed-off-by: Sumit Garg <[email protected]>
> > ---
> > arch/arm/mach-snapdragon/of_fixup.c | 35 +++++++++++++++++++++++++++++
> > 1 file changed, 35 insertions(+)
> >
> > diff --git a/arch/arm/mach-snapdragon/of_fixup.c
> > b/arch/arm/mach-snapdragon/of_fixup.c
> > index 5b6076ea8e5..644afc22d88 100644
> > --- a/arch/arm/mach-snapdragon/of_fixup.c
> > +++ b/arch/arm/mach-snapdragon/of_fixup.c
> > @@ -25,6 +25,7 @@
> > #include <fdt_support.h>
> > #include <linux/errno.h>
> > #include <stdlib.h>
> > +#include <tee/optee.h>
> > #include <time.h>
> >
> > /* U-Boot only supports USB high-speed mode on Qualcomm platforms with DWC3
> > @@ -164,6 +165,37 @@ static void fixup_power_domains(struct device_node
> > *root)
> > }
> > }
> >
> > +static void add_optee_node(struct device_node *root)
> > +{
> > + struct device_node *fw = NULL, *optee = NULL;
> > + int ret;
> > +
> > + fw = of_find_node_by_path("/firmware");
> > + if (!fw) {
> > + log_err("Failed to find /firmware node\n");
> > + return;
> > + }
> > +
> > + ret = of_add_subnode(fw, "optee", strlen("optee") + 1, &optee);
> > + if (ret) {
> > + log_err("Failed to add 'optee' subnode: %d\n", ret);
> > + return;
> > + }
> > +
> > + ret = of_write_prop(optee, "compatible", strlen("linaro,optee-tz") + 1,
> > + "linaro,optee-tz");
> > + if (ret) {
> > + log_err("Failed to add optee 'compatible' property: %d\n", ret);
> > + return;
> > + }
> > +
> > + ret = of_write_prop(optee, "method", strlen("smc") + 1, "smc");
> > + if (ret) {
> > + log_err("Failed to add optee 'method' property: %d\n", ret);
> > + return;
> > + }
> > +}
> > +
> > #define time_call(func, ...) \
> > do { \
> > u64 start = timer_get_us(); \
> > @@ -178,6 +210,9 @@ static int qcom_of_fixup_nodes(void * __maybe_unused
> > ctx, struct event *event)
> > time_call(fixup_usb_nodes, root);
> > time_call(fixup_power_domains, root);
> >
> > + if (IS_ENABLED(CONFIG_OPTEE) && is_optee_smc_api())
> > + time_call(add_optee_node, root);
> > +
> > return 0;
> > }
> >
>
> --
> // Casey (she/her)
>
