[SeaBIOS] [PATCH v3 2/8] seabios: acpi: add mcfg table.

Kevin O'Connor kevin at koconnor.net
Sun Oct 28 00:38:38 CEST 2012

On Fri, Oct 19, 2012 at 04:40:59PM -0400, Jason Baron wrote:
> From: Isaku Yamahata <yamahata at valinux.co.jp>
> add mcfg table.
> mcfg isn't populated at the moment. dev-q35 will use it later.
> [jbaron at redhat.com: moved header from post.h -> acpi.h]

Thanks Jason.  Sorry for the delay in review.

I find the code flow of this patch to be confusing.  The code jumps
from build_mcfg to mch_mcfg_init and mch_mcfg_find (patch 5) setting
intermidiate variables (acpi_mcfg) which are then acted on by all of
build_mcfg and mch_mcfg_*.  I think it would be simpler if the code
just detected PCI_DEVICE_ID_INTEL_Q35_MCH and called a function which
did all the work necessary to setup the mcfg acpi table for the q35.


More information about the SeaBIOS mailing list