Re: [PATCH] PPC440EPx SDRAM width

2009-04-23 Thread Steven A. Falco
Valentine Barshak wrote: > > Josh Boyer wrote: >> On Thu, Apr 23, 2009 at 06:40:48PM +0400, Valentine Barshak wrote: >>> Stefan Roese wrote: On Thursday 23 April 2009, Josh Boyer wrote: > On Thu, Apr 23, 2009 at 09:36:12AM -0400, Steven A. Falco wrote: >> There is an error in the way

Re: [PATCH] PPC440EPx SDRAM width

2009-04-23 Thread Valentine Barshak
Stefan Roese wrote: On Thursday 23 April 2009, Josh Boyer wrote: On Thu, Apr 23, 2009 at 09:36:12AM -0400, Steven A. Falco wrote: There is an error in the way ibm4xx_denali_fixup_memsize calculates memory size. When testing the DDR_REDUC bit, the polarity is backwards. A "1" implies 32-bit wi

Re: [PATCH] PPC440EPx SDRAM width

2009-04-23 Thread Valentine Barshak
Josh Boyer wrote: On Thu, Apr 23, 2009 at 06:40:48PM +0400, Valentine Barshak wrote: Stefan Roese wrote: On Thursday 23 April 2009, Josh Boyer wrote: On Thu, Apr 23, 2009 at 09:36:12AM -0400, Steven A. Falco wrote: There is an error in the way ibm4xx_denali_fixup_memsize calculates memory siz

Re: [PATCH] PPC440EPx SDRAM width

2009-04-23 Thread Josh Boyer
On Thu, Apr 23, 2009 at 06:40:48PM +0400, Valentine Barshak wrote: > Stefan Roese wrote: >> On Thursday 23 April 2009, Josh Boyer wrote: >>> On Thu, Apr 23, 2009 at 09:36:12AM -0400, Steven A. Falco wrote: There is an error in the way ibm4xx_denali_fixup_memsize calculates memory size. W

Re: [PATCH] PPC440EPx SDRAM width

2009-04-23 Thread Stefan Roese
On Thursday 23 April 2009, Josh Boyer wrote: > On Thu, Apr 23, 2009 at 09:36:12AM -0400, Steven A. Falco wrote: > >There is an error in the way ibm4xx_denali_fixup_memsize calculates > >memory size. When testing the DDR_REDUC bit, the polarity is > >backwards. A "1" implies 32-bit wide memory whi

Re: [PATCH] PPC440EPx SDRAM width

2009-04-23 Thread Josh Boyer
On Thu, Apr 23, 2009 at 09:36:12AM -0400, Steven A. Falco wrote: >There is an error in the way ibm4xx_denali_fixup_memsize calculates >memory size. When testing the DDR_REDUC bit, the polarity is >backwards. A "1" implies 32-bit wide memory while a "0" implies >64-bit wide memory. > >For a 32-bit

[PATCH] PPC440EPx SDRAM width

2009-04-23 Thread Steven A. Falco
There is an error in the way ibm4xx_denali_fixup_memsize calculates memory size. When testing the DDR_REDUC bit, the polarity is backwards. A "1" implies 32-bit wide memory while a "0" implies 64-bit wide memory. For a 32-bit wide system, this bug causes twice the memory to be reported, leading