On Fri, Dec 15, 2023 at 11:06 AM Adam Ford <aford...@gmail.com> wrote: > > On Fri, Dec 15, 2023 at 12:41 PM Fabio Estevam <feste...@gmail.com> wrote: > > > > Hi Tim, > > > > On Fri, Dec 15, 2023 at 3:34 PM Tim Harvey <thar...@gateworks.com> wrote: > > > > > Fabio, > > > > > > The commit log details are not valid for upstream. I was basing this > > > off of 8d060e4a66d6884341fbb3d8ab1d837a3f173d47 which made it upstream > > > with the same message. > > > > > > I can submit a v2 if necessary. > > > > Yes, please submit a v2 and I will queue this series and the TPM one > > to u-boot-imx next. > > This node appears to already be in the imx8mm-u-boot.dtsi encapsulated > by an #ifdef looking for optee. Can this ifdef be expanded to include > CONFIG_SECURE_BOOT? >
Adam, Thanks for reminding me of that... that was my submission. This patch just needs to be dropped... it was a remnant from before bb6a4c55363c3 "(arm: dts: imx8mm: move firmware/optee node to common imx8mm-u-boot.dtsi") I think it makes sense that the firmware/optee node is protected within a CONFIG_OPTEE Best Regards, Tim
