From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anatolij Gustschin Date: Thu, 15 Apr 2010 14:32:14 +0200 Subject: [U-Boot] [PATCH v3 2/3] mpc5121: add support for PDM360NG board In-Reply-To: <20100414154840.E29F1E85054@gemini.denx.de> References: <20100321192306.BFE104C022@gemini.denx.de> <1271254909-20398-1-git-send-email-agust@denx.de> <1271254909-20398-2-git-send-email-agust@denx.de> <1271254909-20398-3-git-send-email-agust@denx.de> <20100414154840.E29F1E85054@gemini.denx.de> Message-ID: <20100415143214.3e10813d@wker> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Wolfgang, On Wed, 14 Apr 2010 17:48:40 +0200 Wolfgang Denk wrote: > In message <1271254909-20398-3-git-send-email-agust@denx.de> you wrote: > > PDM360NG is a MPC5121E based board by ifm ecomatic gmbh. > ... > > - don't use DDR RAM size config macro, get_ram_size() is > > used now in fixed_sdram() to determine RAM size (see > > previous patch 1/3) > > With this change, can we then not also get rid of manually defining > CONFIG_PDM360NG_BIG ? We should be able to auto-detect the > configuration from the actual RAM size now, right? But another configuration for 128 MB uses different values for DDR System Configuration Register and DDR Time Config[0-2] Registers. We still need this config option. Best regards, Anatolij