[coreboot-gerrit] Change in coreboot[master]: SMBIOS: Allow skipping default SMBIOS generation

[Angel Pons (Code Review)]

Attention is currently required from: Nico Huber, Furquan Shaikh, Tim Wawrzynczak, Arthur Heymans, Michael Niewöhner, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57366 )

Change subject: SMBIOS: Allow skipping default SMBIOS generation
......................................................................

Patch Set 1:

(2 comments)

File src/arch/x86/smbios.c:

https://review.coreboot.org/c/coreboot/+/57366/comment/bc45b07e_fa3188cf 
PS1, Line 1223: 		} else {
> Sorry, I mixed that up a little. Your change is sound, just the […]

I see. I would've expected that disabled devices are never found here, though. But yes, skipping disabled devices makes sense.

So, should I make a patch that skips disabled devices?

https://review.coreboot.org/c/coreboot/+/57366/comment/217b411b_9e56c5bb 
PS1, Line 1225: 			len += smbios_generate_type41_from_devtree(dev, handle, current);
> Instead of exposing these as API, we'd usually have a default get_smbios_data() that […]

Sounds good.