Hi Tom,
On Fri, 11 Apr 2025 at 12:46, Tom Rini <tr...@konsulko.com> wrote:
>
> On Fri, Apr 11, 2025 at 12:32:12PM -0600, Simon Glass wrote:
> > Hi Jerome,
> >
> > On Fri, 11 Apr 2025 at 09:31, Jerome Forissier
> > <jerome.foriss...@linaro.org> wrote:
> > >
> > > CONFIG_BOOTSTAGE_REPORT is currently supported in the bootm command
> > > only. Add support to the EFI boot.
> > >
> > > Signed-off-by: Jerome Forissier <jerome.foriss...@linaro.org>
> > > ---
> > >
> > > lib/efi_loader/efi_boottime.c | 7 +++++++
> > > 1 file changed, 7 insertions(+)
> > >
> >
> > If you have time, could you take a look at unifying
> > announce_and_cleanup() with the duplicate code in EFI?
>
> And the duplicating code in arch/{arm,x86,riscv}. Maybe there's some
> good place we can stick the bootstage calls, board_quiesce_devices() and
> dm_remove_devices_active(). But maybe not, I'm not sure honestly.
Yes. I started a series at the weekend to drop board_quiesce_devices()
but in a few cases it is needed, so then I thought about creating an
event for those 3-4 cases, but then I didn't go any further. I will
see if I can at least send a WIP series this weekend to help.
Regards,
Simon
