Hi, On 1/18/26 9:11 PM, Marco Felsch wrote: > Hi, > > On 26-01-15, Ahmad Fatoum wrote: >> Hi, >> >> On 11/10/25 9:34 PM, Marco Felsch wrote: >>> config OF_FIXUP_OPTEE >>> def_bool y >>> depends on HAVE_OPTEE && ARM >>> + select LIBFDT >> >> AFAIK, we use LIBFDT only in PBL as we don't want to unflatten the whole >> tree there. Why would you want to use it in barebox proper? > > I used it due to the fdt_check_header() check within > optee_register_overlay(). I think we can drop this check and the LIBFDT > since barebox PBL is creating the HANDOFF_DATA_BL32_DT_OVL entry.
As mentioned before, I prefer we do the device tree fragment stuff in barebox proper if at all possible. In that case, the select here makes sense. Thanks, Ahmad > > Regards, > Marco > > >> >> Thanks, >> Ahmad >> >> -- >> Pengutronix e.K. | | >> Steuerwalder Str. 21 | http://www.pengutronix.de/ | >> 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | >> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | >> >> > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
