Re: [coreboot] [PATCH 4/5] Intel 3100, version 3: northbridge and RAM controller support

[Peter Stuge]

On Thu, Mar 13, 2008 at 02:45:25PM -0700, Ed Swierk wrote:
> This patch implements support for the Intel 3100 integrated
> northbridge and RAM controller. This is largely based on the
> existing Intel E7520/E7525 code.

The coding style isn't completely clean. Long lines, missing
whitespace here and there, and #if 1.

I feel like acking anyway. :)

Can you commit or should I? Should we wait for more comments?

//Peter