2005-01-27 06:54:31

by Dan Malek

[permalink] [raw]
Subject: [PATCH] add AMD Geode processor support



Hi Marcelo!

Here is a patch for 2.4 that adds the basic AMD Geode GX2/GX3
and GX1/SC1200 support. This patch updates configuration
scripts, defconfig, and setup files.

Signed-off-by: Dan Malek <[email protected]>


Attachments:
geode_x86.patch (7.12 kB)

2005-01-27 17:19:08

by Pavel Machek

[permalink] [raw]
Subject: Re: [PATCH] add AMD Geode processor support

Hi!

> Here is a patch for 2.4 that adds the basic AMD Geode GX2/GX3
> and GX1/SC1200 support. This patch updates configuration
> scripts, defconfig, and setup files.

Please inline patches...

We do not disable HIGHMEM_64GB for 486, I do not see why we should add
extra code to geode...

If you really want to do that, do it properly for all cpus...
Pavel
--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!

2005-01-27 17:40:01

by Dan Malek

[permalink] [raw]
Subject: Re: [PATCH] add AMD Geode processor support


On Jan 27, 2005, at 9:15 AM, Pavel Machek wrote:

> We do not disable HIGHMEM_64GB for 486, I do not see why we should add
> extra code to geode...

What about some of the other ones like MTRR and IOAPIC?
I was kinda passing this along from someone I thought knew
better than I, but I didn't like it either. It seems just setting these
booleans to 'n' should do the trick.

Thanks.

-- Dan

2005-01-27 18:54:34

by Pavel Machek

[permalink] [raw]
Subject: Re: [PATCH] add AMD Geode processor support

Hi!

> >We do not disable HIGHMEM_64GB for 486, I do not see why we should add
> >extra code to geode...
>
> What about some of the other ones like MTRR and IOAPIC?
> I was kinda passing this along from someone I thought knew
> better than I, but I didn't like it either. It seems just setting these
> booleans to 'n' should do the trick.

Actually sorry, no, you can't do that.

486 should mean "can run on 486 or better" (at least in 2.4.X
series). That means it should have IOAPIC support compiled in.

Just do not play with HIGHMEM_64GB; someone may want a kernel for
"geode or better".
Pavel
--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!

2005-01-31 17:07:04

by Alan

[permalink] [raw]
Subject: Re: [PATCH] add AMD Geode processor support

On Iau, 2005-01-27 at 06:54, Dan Malek wrote:
> Hi Marcelo!
>
> Here is a patch for 2.4 that adds the basic AMD Geode GX2/GX3
> and GX1/SC1200 support. This patch updates configuration
> scripts, defconfig, and setup files.

GX2 seems no different to the older Cyrix/NS settings. GX3 a little
different.
Also you might not want to magically force settings like highmem because
you want that for multi-target kernels - Geode is a sort of odd case
where it almost makes sense but its different enough to make me dubious.


2005-01-31 19:49:19

by Dan Malek

[permalink] [raw]
Subject: Re: [PATCH] add AMD Geode processor support


On Jan 31, 2005, at 11:01 AM, Alan Cox wrote:

> Also you might not want to magically force settings like highmem
> because
> you want that for multi-target kernels - Geode is a sort of odd case
> where it almost makes sense but its different enough to make me
> dubious.

I've already taken that out.

In fact, I've decided to wait until I get the 2.6 done first, then go
back
and properly update for 2.4. This was just a patch that has been around
for a while, I just wanted to get it off of my plate so I could do a
minor
update later, but I'll just wait.

Apologies for the wasted bandwidth.

Thanks.

-- Dan