Re: [coreboot] [PATCH 3/6] Intel EP80579 northbridge

[Ed Swierk]

On Wed, Sep 3, 2008 at 5:23 PM, Ed Swierk  wrote:
> Thanks for the feedback. I have addressed most of your comments;
> please see the attached patch.
>
> I will address your comment about the duplicate struct dimm_size
> declaration separately. Would it make sense to put it in a file common
> to all Intel northbridges, say, src/northbridge/intel/mem.h, and then
> remove it from the various raminit.c's?
>
> Signed-off-by: Ed Swierk

Pinged-by: Ed Swierk

;-)