Re: [PATCH v2] coreboot_table: skip failing entries instead of aborting populate

From: Julius Werner

Date: Thu Apr 30 2026 - 16:47:30 EST


> coreboot_table_populate() registers devices one by one. If
> device_register() fails for one entry, the current code returns
> immediately, leaving previously registered devices orphaned on the
> coreboot bus with no cleanup path.
>
> Since coreboot table entries are independent of each other, a failure
> on one entry should not prevent the others from being registered.
> This mirrors the strategy used by of_platform_populate(), which skips
> individual failures rather than aborting.
>
> Log a warning and continue the loop on device_register() failure.
>
> Signed-off-by: Titouan Ameline de Cadeville <titouan.ameline@xxxxxxxxx>

Reviewed-by: Julius Werner <jwerner@xxxxxxxxxxxx>