[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