On Mon, Nov 22, 2010 at 07:30:15PM +0100, Stefan Reinauer wrote:
* repository service <svn(a)coreboot.org> [101122
Date: Mon Nov 22 17:23:54 2010
New Revision: 6116
Drop unused ACPI_WRITE_MADT_IOAPIC #define.
This should probably be C code in some .c file anyway.
As a macro I think it belongs in a .h file.
If it shall remain a macro then yes. In general I think having code in
.h files as macro is not desirable, unless it cannot be avoided for some
reason. I'd prefer some function in C code instead of a macro.
Or, as you suggested, we do neither and add a devicetree.cb facility which
auto-generates the resp. entries/code then (in mptable, in MADT, etc.).
We don't really have IOAPICs in our device tree,
PCI_BASE_ADDRESS_0 sounds wrong.
Not necessarily wrong, but at the very least it's very specific to some
chipset. For example the 6700PXH 64bit PCI Hub has two integrated
IOAPICs, one at PCI device 0.1 and one at 0.3. Two BARs of those
two devices (PCI_BASE_ADDRESS_0 each) contain/determine to IOAPIC's
Offset 10h: MBAR--Memory Base Register (D0: F1, F3):
31:12 RW 0 Address (ADDR): These bits determine the base address
of the I/OxAPIC.
I guess a chipset like this lead to the PCI_BASE_ADDRESS_0 stuff in
How do we get IOAPICs into the tree? I
think the southbridges should add it where appropriate and I think we
need a new device type, as we have one for local APICs too.
Yep, sounds good. I'll start another thread about this.