Author: uwe
Date: 2009-05-22 20:17:06 +0200 (Fri, 22 May 2009)
New Revision: 4296

Modified:
   trunk/util/getpir/code_gen.c
Log:
Make the getpir output look less crappy and add a license
header template, as people keep forgetting them.

Signed-off-by: Uwe Hermann <[email protected]>
Acked-by: Uwe Hermann <[email protected]>



Modified: trunk/util/getpir/code_gen.c
===================================================================
--- trunk/util/getpir/code_gen.c        2009-05-22 16:03:04 UTC (rev 4295)
+++ trunk/util/getpir/code_gen.c        2009-05-22 18:17:06 UTC (rev 4296)
@@ -3,21 +3,34 @@
 #include "pirq_routing.h"
 
 static char *preamble[] = {
-       "/* This file was generated by getpir.c, do not modify!\n",
-       " * (but if you do, please run checkpir on it to verify)\n",
+
+       "/*\n",
+       " * This file is part of the coreboot project.\n",
        " *\n",
-       " * Contains the IRQ Routing Table dumped directly from your\n",
-       " * memory, which BIOS sets up.\n",
+       " * Copyright (C) 200x TODO <t...@todo>\n",
        " *\n",
-       " * Documentation at: 
http://www.microsoft.com/whdc/archive/pciirq.mspx\n */\n\n",
-       "#ifdef GETPIR\n",
-       "#include \"pirq_routing.h\"\n",
-       "#else\n"
+       " * This program is free software; you can redistribute it and/or 
modify\n",
+       " * it under the terms of the GNU General Public License as published 
by\n",
+       " * the Free Software Foundation; either version 2 of the License, 
or\n",
+       " * (at your option) any later version.\n",
+       " *\n",
+       " * This program is distributed in the hope that it will be useful,\n",
+       " * but WITHOUT ANY WARRANTY; without even the implied warranty of\n",
+       " * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the\n",
+       " * GNU General Public License for more details.\n",
+       " *\n",
+       " * You should have received a copy of the GNU General Public 
License\n",
+       " * along with this program; if not, write to the Free Software\n",
+       " * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  
02110-1301 USA\n",
+       " */\n\n",
+       "#ifdef GETPIR                  /* TODO: Drop this when copying to 
coreboot. */\n",
+       "#include \"pirq_routing.h\"    /* TODO: Drop this when copying to 
coreboot. */\n",
+       "#else                          /* TODO: Drop this when copying to 
coreboot. */\n"
        "#include <arch/pirq_routing.h>\n",
-       "#endif\n\n"
+       "#endif                         /* TODO: Drop this when copying to 
coreboot. */\n\n"
        "const struct irq_routing_table intel_irq_routing_table = {\n",
-       "\tPIRQ_SIGNATURE,  /* u32 signature */\n",
-       "\tPIRQ_VERSION,    /* u16 version   */\n",
+       "\tPIRQ_SIGNATURE,              /* u32 signature */\n",
+       "\tPIRQ_VERSION,                /* u16 version */\n",
        0
 };
 
@@ -36,25 +49,25 @@
        while (*code)
                fprintf(fpir, "%s", *code++);
 
-       fprintf(fpir, "\t32+16*%d,       /* There can be total %d devices on 
the bus */\n",
+       fprintf(fpir, "\t32 + 16 * %d,          /* Max. number of devices on 
the bus */\n",
                ts, ts);
-       fprintf(fpir, "\t0x%02x,                 /* Where the interrupt router 
lies (bus) */\n",
+       fprintf(fpir, "\t0x%02x,                        /* Interrupt router bus 
*/\n",
                rt->rtr_bus);
-       fprintf(fpir, "\t(0x%02x<<3)|0x%01x,   /* Where the interrupt router 
lies (dev) */\n",
+       fprintf(fpir, "\t(0x%02x << 3) | 0x%01x,        /* Interrupt router dev 
*/\n",
                rt->rtr_devfn >> 3, rt->rtr_devfn & 7);
-       fprintf(fpir, "\t%#x,            /* IRQs devoted exclusively to PCI 
usage */\n",
+       fprintf(fpir, "\t%#x,                   /* IRQs devoted exclusively to 
PCI usage */\n",
                rt->exclusive_irqs);
-       fprintf(fpir, "\t%#x,            /* Vendor */\n", rt->rtr_vendor);
-       fprintf(fpir, "\t%#x,            /* Device */\n", rt->rtr_device);
-       fprintf(fpir, "\t%#x,            /* Miniport */\n",
+       fprintf(fpir, "\t%#x,                   /* Vendor */\n", 
rt->rtr_vendor);
+       fprintf(fpir, "\t%#x,                   /* Device */\n", 
rt->rtr_device);
+       fprintf(fpir, "\t%#x,                   /* Miniport */\n",
                rt->miniport_data);
        fprintf(fpir, "\t{ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 }, /* u8 rfu[11] 
*/\n");
-       fprintf(fpir, "\t%#x,            /* u8 checksum. This has to be set to 
some\n                       value that would give 0 after the sum of all\n     
                     bytes for this structure (including checksum) */\n",
+       fprintf(fpir, "\t%#x,                   /* Checksum (has to be set to 
some value that\n                          * would give 0 after the sum of all 
bytes\n                             * for this structure (including 
checksum).\n                                 */\n",
                rt->checksum);
        fprintf(fpir, "\t{\n");
-       fprintf(fpir, "\t\t/* bus,     dev|fn,   {link, bitmap}, {link, 
bitmap}, {link, bitmap}, {link, bitmap},  slot, rfu */\n");
+       fprintf(fpir, "\t\t/* bus,        dev | fn,   {link, bitmap}, {link, 
bitmap}, {link, bitmap}, {link, bitmap}, slot, rfu */\n");
        for (i = 0; i < ts; i++) {
-               fprintf(fpir, "\t\t{0x%02x,(0x%02x<<3)|0x%01x, {{0x%02x, 
0x%04x}, {0x%02x, 0x%04x}, {0x%02x, 0x%04x}, {0x%02x, 0x%04x}}, 0x%x, 0x%x},\n",
+               fprintf(fpir, "\t\t{0x%02x, (0x%02x << 3) | 0x%01x, {{0x%02x, 
0x%04x}, {0x%02x, 0x%04x}, {0x%02x, 0x%04x}, {0x%02x, 0x%04x}}, 0x%x, 0x%x},\n",
                        (se_arr+i)->bus, (se_arr+i)->devfn >> 3,
                        (se_arr+i)->devfn & 7, (se_arr+i)->irq[0].link,
                        (se_arr+i)->irq[0].bitmap, (se_arr+i)->irq[1].link,


-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to