[LinuxBIOS] [patch 2/4] Geode LX: Norwich mainboard

Marc Jones marc.jones at amd.com
Wed May 2 21:49:25 CEST 2007



ron minnich wrote:
> On 5/1/07, Uwe Hermann <uwe at hermann-uwe.de> wrote:
>> On Tue, May 01, 2007 at 04:48:29PM -0600, Marc Jones wrote:
>> > This patch adds support for the AMD Norwich development platform
>> > based on the Geode LX processor.  The Norwich is the canonical
>> > Geode reference, and will server as a good basis for other
>> > Geode based platforms.
>>
>> So is this a physically existing product or merely a development
>> board/prototype? I'm asking because I'd like to know into which
>> category this should go later in the wiki:
>>
>> http://linuxbios.org/Supported_Motherboards
> 
> I'd say put it in as supported.
> 
>> > +#define POST_CODE(x) outb(x, 0x80)
>>
>> Why this? We have a post_code() function in src/console/console.c.
> 
> Marc, is it possible to use the standard one? (There may be a reason
> for the macro, if so, leave it in)
> 
This was for pre-stack code when we were using romcc. Now that it is CAR 
code I think it could be changed to use console.c.

>> > +     SystemPreInit();
>> [...]
>> > +     pll_reset(ManualConf);
>> > +
>> > +     cpuRegInit();
> 
> Those names may be the fault of Ollie or me, not sure; Marc, change if 
> you wish.
>
I think this would be best as a follow-on patch.

> So, I'm with Peter, I'd go for the
> Acked-by: Ronald G. Minnich <rminnich at gmail.com>
> and, Marc, consider either a new patch with Uwe's recommendations or
> fixup later. But I don't want to delay getting this in; there are some
> customers waiting for it, literally daily.
> 
> thanks
> 
> ron
> 
> 
Marc

-- 
Marc Jones
Senior Software Engineer
(970) 226-9684 Office
mailto:Marc.Jones at amd.com
http://www.amd.com/embeddedprocessors






More information about the coreboot mailing list