[PATCH 1/2] Geode GX2 cleanup again

[Nils]

For gx2/northbridgeinit.c
-some white space fixes
-some codingstyle fixes
-some comment fixes.

Signed-off-by: Nils Jacobs

Thanks, Nils.

[Stefan Reinauer]

On 12/30/10 10:48 PM, Nils wrote:
> {0xffffffff, 				{0xffffffff, 0xffffffff}},
> {0xFFFFFFFF, 				{0xFFFFFFFF, 0xFFFFFFFF}},

I don't think we should capitalize hexadecimal numbers. We never did (on
purpose) anywhere else.

Bikeshed, anyone?

[Uwe Hermann]

On Fri, Dec 31, 2010 at 07:57:34AM +0100, Stefan Reinauer wrote:
> On 12/30/10 10:48 PM, Nils wrote:
> > {0xffffffff, 				{0xffffffff, 0xffffffff}},
> > {0xFFFFFFFF, 				{0xFFFFFFFF, 0xFFFFFFFF}},
>
> I don't think we should capitalize hexadecimal numbers. We never did (on
> purpose) anywhere else.
>
> Bikeshed, anyone?

Yup, I also prefer non-capitalized, but that's probably a matter of
taste. I'm relatively sure the majority of our code-base uses lower-case
right now.

Uwe.

[Marc Bertens]

Uwe,

i did not know if you saw my message on IRC, so to be sure by mail too.

Are there any guidelines for me to read how to use the wiki pages, 
I was just keeping the mainpage of the Nokia-ip530 related to the 
hardware and status of the implementation. And keep other things 
away from here, like installation descriptions. This was basically 
due to that there was not clear page regarding to installing 
coreboot on a system, or even special systems. So maybe it would 
be a good idea to make a more general page for installing coreboot 
(stable release) on a system with even some simple pointers howto 
install a OS. This to make it easier for newbies (like I was when 
I started in april 2010) life a little easier. And this makes our 
lives easier too (as (more) experienced guys :-)) And maybe i'm 
somewhat of an odd guy how like to make documentation under the 
KISS principal ;-)

Let me know what you think

Marc