[coreboot] [PATCH]Move generation of ISA bus mptable entries togeneric code

[Patrick Georgi]

Am 20.05.2010 17:00, schrieb Myles Watson:
>>  #define IO_LOCAL_INT(type, intr, apicid, pin) \
>>  	smp_write_intsrc(mc, (type), MP_IRQ_TRIGGER_EDGE |
> MP_IRQ_POLARITY_HIGH, bus_isa, (intr), (apicid), (pin));
>>  
>> -	IO_LOCAL_INT(mp_ExtINT, 0x0, apicid_sb700, 0x0);
>> +	mptable_add_isa_interrupts(mc, bus_isa, apicid_sb700, 0);
> 
> It looks like IO_LOCAL_INT should be removed, since it uses bus_isa.  Same
> for tilapia & probably other similar mainboards.
It's still used for some other table entry, so I kept it. Once that
entry is gone (I think it can be made generic, too), I'll remove the macro.

With all done, it might be possible to drop board specific mptable.c's
completely, which would also fix this issue.

>> +		   /*I/O Ints:	Type	Polarity    Trigger
> Bus ID	 IRQ	APIC ID	PIN# */
>>  	smp_write_intsrc(mc, mp_INT,
> MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_mcp55[0], ((sbdn+1)<<2)|1,
> m->apicid_mcp55, 0xa);
> 
> In some places you reformat the comment, in others you just remove it.
And the comment doesn't even exist in all files - mptable.c isn't very
consistent to begin with. I guess this happened because some files had
/* comment here
*/ code I removed on the same line as the comment end

Committed as r5575, thank you for the Ack.


Patrick