Hi Tom, On 2026-05-19T15:18:15, Tom Rini <[email protected]> wrote:
> This series fixes bootdev_get_bootflow to not require the ops > field in the driver to be set and then removes the empty struct from the > places which had been creating one. Two more empty bootdev_ops structures fit this pattern: nvme_bootdev_ops and usb_bootdev_ops. Please can you fold those into v2 so we clean them all up together? > As part of Marek Vasut's work to add static and/or const to structs that > should have it, but do not, it exposed a small problem with the bootdev > uclass. BTW for patch 1 it would be useful to update the comment for bootdev_get_ops() to indicate it can return NULL - normally, we don't allow this. Regards, Simon
