Re: [PATCH 19/23] security: optee: add helpers to apply OP-TEE FDTO

[Ahmad Fatoum <a.fatoum@pengutronix.de>]

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 |