status information

YhLu YhLu at
Thu Oct 28 18:06:00 CEST 2004

For S2885, I can not compile that under LOGLEVEL=8. And I have to comment
out severl printk_debug in auto.c stage.

Before Eric do sth about romcc, I think remove name member in chip_operation
would be fine.



-----Original Message-----
From: ebiederman at [mailto:ebiederman at] 
Sent: Thursday, October 28, 2004 3:44 PM
To: Ronald G. Minnich
Cc: Stefan Reinauer; linuxbios at
Subject: Re: status information

"Ronald G. Minnich" <rminnich at> writes:

> On Thu, 28 Oct 2004, Eric W. Biederman wrote:
> > I will (hopefully) commit later today, but I have found one general
> > technique that will cut down the code size but a small amount.
> > In the chip_operations struct we have a name for debugging.
> > Since we are no longer printing that name there is no point in
> > keeping it.
> hmm, this sounds like a lot of potential pain for a small gain. Can we 
> talk about this a bit first. 

Sure.    First here are two examples.
struct chip_opertations cpu_intel_socket_mPGA479M_control = {
	.name = "socket mPGA479M",

Killing the name for cpu sockets allows us to kill the whole

struct chip_operations northbridge_intel_e7501_ops = {
        .name      = "intel E7501 Northbridge",
	.enable_dev = enable_dev,

First note that all we did with the names was to print them when
we converted from the static to the dynamic tree.  Nothing
uses them anymore.  

Killing them is as simple as removing a structure member.  And
deleting a few lines of code.  Bloatwise strings appear to
be up towards the top of the list.

This is really a continuation of killing the static device tree.

If you like I will trade you gdb stubs for the currently useless
debugging string names. ;)  I just finished the code for that.
The conversations of the last couple of days inspired me. :)

Linuxbios mailing list
Linuxbios at

More information about the coreboot mailing list