Hi Christian,

On 2026-10-27T12:38:55, Christian Marangi <[email protected]> wrote:

> In such context U-Boot is loaded as BL31 and the PHY firmware

This still says BL31. It should be BL33, since U-Boot runs as the
non-secure payload.

> Then there is the request_firmware_size() new OP to get only
> the size of the firmware.

One thing I'd like to check from last time: patch 2 still only renames
doc/develop/driver-model/firmware_loader.rst and makes no other
changes to it. It is still titled 'File System Firmware Loader',
covers only the filesystem loader and the /chosen lookup, and says
nothing about the FIP loader, request_firmware_size() or
get_fw_loader_from_node(). Please can you add a doc patch so the API
description matches the DT binding? Also, the binding patch still
comes before the FIP driver it describes, so it would be worth moving
it after patch 7.

Regards,
Simon

Reply via email to