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

[Brian Norris]

On Fri, May 01, 2026 at 11:43:22AM +0200, Titouan Ameline de Cadeville wrote:
> 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.
> 
> Move ptr_entry increment before device_register(), log a warning on
> failure, and continue the loop rather than aborting.
> 
> Signed-off-by: Titouan Ameline de Cadeville <titouan.ameline@xxxxxxxxx>

Acked-by: Brian Norris <briannorris@xxxxxxxxxxxx>