[LinuxBIOS] [PATCH] beautify m57sli mptable.c (Was: Fix FireWire...)

Torsten Duwe duwe at lst.de
Mon Jan 7 12:21:42 CET 2008


On Sunday 06 January 2008, Carl-Daniel Hailfinger wrote:

> This is a much needed simplification and readability improvement. Thanks!
> Generated code seems to be unchanged.
>
> If you drop the #undef lines (or explain why we absolutely need them),
> you can take the ack from below.

I consider it good practise to undef such readability macros as soon as they 
are not needed any more, like freeing unused memory. But I really have no 
strong opinion about them, no problem.

> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Thanks, revision 3035.

>
> Bonus points if you manage to convert all mptable.c files for MCP55 (or
> even, if that task is not too daring, all mptable.c files).

You mean something like perl -pe 
s:smp_write_intsrc\(mc,\s*mp_INT,\s*MP_IRQ[^,]*,\s*bus_isa\s*,(([^,]*),[^,]*,
([^)]*)):ISA_INT($2, $3):g;
and so on? Turn it into a full-blown script and run it per board?
Maybe.

	Torsten






More information about the coreboot mailing list