[LinuxBIOS] [PATCH] v3: code documentation fixes
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Wed Sep 26 11:18:32 CEST 2007
Can somebody please commit?
Carl-Daniel
On 30.08.2007 17:11, ron minnich wrote:
> Acked-by: Ronald G. Minnich <rminnich at gmail.com>
>
>
> On 8/29/07, Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net> wrote:
>> Correct typo, fix stage2 code documentation, add dtsname printing to
>> differentiate between identically named objects during boot.
>>
>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>>
>> Index: LinuxBIOSv3/include/device/device.h
>> ===================================================================
>> --- LinuxBIOSv3/include/device/device.h (Revision 485)
>> +++ LinuxBIOSv3/include/device/device.h (Arbeitskopie)
>> @@ -202,7 +202,7 @@
>> struct resource resource[MAX_RESOURCES];
>> unsigned int resources;
>>
>> - /* link are (down sream) buses attached to the device, usually a leaf
>> + /* link are (downstream) buses attached to the device, usually a leaf
>> * device with no children have 0 buses attached and a bridge has 1 bus
>> */
>> struct bus link[MAX_LINKS];
>> Index: LinuxBIOSv3/device/device.c
>> ===================================================================
>> --- LinuxBIOSv3/device/device.c (Revision 485)
>> +++ LinuxBIOSv3/device/device.c (Arbeitskopie)
>> @@ -260,8 +260,10 @@
>> for (curdev = bus->children; curdev; curdev = curdev->sibling) {
>> unsigned int links;
>> int i;
>> - printk(BIOS_SPEW, "%s: %s(%s) have_resources %d enabled %d\n",
>> + printk(BIOS_SPEW,
>> + "%s: %s(%s) dtsname %s have_resources %d enabled %d\n",
>> __func__, bus->dev->dtsname, dev_path(bus->dev),
>> + curdev->dtsname,
>> curdev->have_resources, curdev->enabled);
>> if (curdev->have_resources) {
>> continue;
>> Index: LinuxBIOSv3/lib/stage2.c
>> ===================================================================
>> --- LinuxBIOSv3/lib/stage2.c (Revision 485)
>> +++ LinuxBIOSv3/lib/stage2.c (Arbeitskopie)
>> @@ -31,8 +31,9 @@
>> /**
>> * Main function of the DRAM part of LinuxBIOS.
>> *
>> - * LinuxBIOS is divided into pre-DRAM part and DRAM part. The phases before
>> - * this part are phase 0 and phase 1. This part contains phases x through y.
>> + * LinuxBIOS is divided into pre-DRAM part and DRAM part. The stages before
>> + * this part are stage 0 and stage 1. This part contains stage 2, which
>> + * consists of phases 1 through 6.
>> *
>> * Device Enumeration: in the dev_enumerate() phase.
>> *
>> @@ -53,6 +54,7 @@
>>
>> post_code(0x20);
>>
>> + /* TODO: Explain why we use printk here although it is impossible */
>> printk(BIOS_NOTICE, console_test);
>>
>> dev_init();
>>
>>
>>
>> --
>> linuxbios mailing list
>> linuxbios at linuxbios.org
>> http://www.linuxbios.org/mailman/listinfo/linuxbios
>>
>
--
http://www.hailfinger.org/
More information about the coreboot
mailing list