r3263 build service

[coreboot information]

Dear coreboot readers!

This is the automated build check service of coreboot.

The developer "mjones" checked in revision 3263 to
the coreboot source repository and caused the following 
changes:

Change Log:
Add CPUID processor name string support for Fam10 CPUs.
Peter did a nice job cleaning up my initial patch. Thanks!

Signed-off-by: Marc Jones 
Signed-off-by: Peter Stuge

Acked-by: Marc Jones

Build Log:
Compilation of amd:serengeti_cheetah_fam10 has been broken
See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=3263&device=serengeti_c...

If something broke during this checkin please be a pain 
in mjones's neck until the issue is fixed.

If this issue is not fixed within 24h the revision should 
be backed out.

Best regards,
     coreboot automatic build system

[Marc Jones]

OK, I'm going to get rid of some inline things today.....

Marc

coreboot information wrote:
> Dear coreboot readers!
>
> This is the automated build check service of coreboot.
>
> The developer "mjones" checked in revision 3263 to
> the coreboot source repository and caused the following 
> changes:
>
> Change Log:
> Add CPUID processor name string support for Fam10 CPUs.
> Peter did a nice job cleaning up my initial patch. Thanks!
>
> Signed-off-by: Marc Jones 
> Signed-off-by: Peter Stuge
>
> Acked-by: Marc Jones
>
> Build Log:
> Compilation of amd:serengeti_cheetah_fam10 has been broken
> See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=3263&device=serengeti_c...
>
> If something broke during this checkin please be a pain 
> in mjones's neck until the issue is fixed.
>
> If this issue is not fixed within 24h the revision should 
> be backed out.
>
> Best regards,
>      coreboot automatic build system

[Carl-Daniel Hailfinger]

On 25.04.2008 00:00, Marc Jones wrote:
> OK, I'm going to get rid of some inline things today.....

Cool, thanks. By the way, if you manage to save 702 bytes, the breakage
will be gone.

Regards,
Carl-Daniel
> coreboot information wrote:
> > Dear coreboot readers!
> >
> > This is the automated build check service of coreboot.
> >
> > The developer "mjones" checked in revision 3263 to
> > the coreboot source repository and caused the following 
> > changes:
> >
> > Change Log:
> > Add CPUID processor name string support for Fam10 CPUs.
> > Peter did a nice job cleaning up my initial patch. Thanks!
> >
> > Signed-off-by: Marc Jones 
> > Signed-off-by: Peter Stuge
> >
> > Acked-by: Marc Jones
> >
> > Build Log:
> > Compilation of amd:serengeti_cheetah_fam10 has been broken
> > See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=3263&device=serengeti_c...

[Marc Jones]

Carl-Daniel Hailfinger wrote:
> On 25.04.2008 00:00, Marc Jones wrote:
> > OK, I'm going to get rid of some inline things today.....
>
> Cool, thanks. By the way, if you manage to save 702 bytes, the breakage
> will be gone.
>
> Regards,
> Carl-Daniel

I still think that we should increase the size to match the normal 
build. The rom_image_size will continue to grow as we add more features 
just as it has on the last several patches. I don't really see a reason 
to constrict the size.

Marc