[coreboot] Warnings

Nils njacobs8 at hetnet.nl
Sun May 9 22:10:27 CEST 2010


Op zondag 9 mei 2010 00:36:49 schreef u:
> On 5/1/10 1:34 AM, Nils wrote:
> > Hi Stefan,
> > First of all thanks for the great  improvements in Geode (GX2).
> >
> > On 4/30/10 7:50 PM, Stefan Reinauer wrote:
> >> src/northbridge/amd/gx2/chipsetinit.c:271: warning: suggest
> >> parentheses around '-' inside '<<'
> >
> > This would need help from someone with a GX2 (or willing to check out
> > the data sheets ;-)
> >
> > I would be happy if i could be of any help with this, I have a GX2 board
> > i can test on.
> 
> Can you test the following two patches:
> 
> Index: northbridge/amd/gx2/chipsetinit.c
> ===================================================================
> --- northbridge/amd/gx2/chipsetinit.c    (revision 5532)
> +++ northbridge/amd/gx2/chipsetinit.c    (working copy)
> @@ -268,7 +268,7 @@
>      if ((msr.lo&0xff) == 0x11)
>          return;
> 
> -    totalmem = sizeram() << 20 - 1;
> +    totalmem = (sizeram() << 20) - 1;
>      totalmem >>= 12;
>      totalmem = ~totalmem;
>      totalmem &= 0xfffff;
> 
> 
> and this one
> 
> Index: northbridge/amd/gx2/chipsetinit.c
> ===================================================================
> --- northbridge/amd/gx2/chipsetinit.c    (revision 5532)
> +++ northbridge/amd/gx2/chipsetinit.c    (working copy)
> @@ -268,7 +268,7 @@
>      if ((msr.lo&0xff) == 0x11)
>          return;
> 
> -    totalmem = sizeram() << 20 - 1;
> +    totalmem = sizeram() << (20 - 1);
>      totalmem >>= 12;
>      totalmem = ~totalmem;
>      totalmem &= 0xfffff;
> 
> 
> and see if any of them causes a failure?
> 
> It would seem the first patch is correct and the second is not, but i am
> not sure.

I tested both patches on rev 5470 and didn`t see a difference, both seem to 
work fine.
See attached patches.

> 
> > I saw your discussion about the warning before and it inspired me to
> > dedicate my spare free time to again update my working rev5446 patches to
> > current trunk.
> > But unfortunately i can`t get it to work anymore on rev5120 and some
> > other rev`s i tried.
> > (Linux errors out with: "hda: lost interrupt")
> > I will send the details in a separate mail.
> 
> We do need to make sure we know the exact revision that broke this.
> 
> Stefan
> 
The exact revision that broke vsa was 5471.

Thanks,Nils.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: (sizeram() << 20) - 1.log
Type: text/x-log
Size: 42395 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100509/9ed2e90a/attachment.log>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sizeram() << (20 - 1).log
Type: text/x-log
Size: 42478 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100509/9ed2e90a/attachment-0001.log>


More information about the coreboot mailing list