[LinuxBIOS] [PATCH] v3: code documentation fixes

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Sun Sep 2 22:05:18 CEST 2007


On 02.09.2007 18:43, Stefan Reinauer wrote:
> * Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net> [070830 02:05]:
>> -		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),
> 
> These kind of indent changes look rather ugly in my opinion.

This is not really an indent change, the message became 18 chars longer
so I had to either violate the 80 column limit or move the message to a
new line. I chose the second option.

>> +	/* TODO: Explain why we use printk here although it is impossible */
> 
> What? Impossible? Why? That commend looks pretty bogus.

newboot.lyx chapter 5.3 says stage 2 phase 1 makes printk work. That
implies printk will not work before that.

>>  	printk(BIOS_NOTICE, console_test);

Carl-Daniel




More information about the coreboot mailing list