From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pali =?ISO-8859-1?Q?Roh=E1r?= Date: Sun, 14 Oct 2012 02:08:21 +0200 Subject: [U-Boot] [PATCH v3 2/5] arm bootm: Do not append zero ATAG_MEM In-Reply-To: <201210140145.06730.marex@denx.de> References: <1327415291-13260-1-git-send-email-pali.rohar@gmail.com> <1350156720-13387-3-git-send-email-pali.rohar@gmail.com> <201210140145.06730.marex@denx.de> Message-ID: <6352382.YMPOgPzKi8@pali> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sunday 14 October 2012 01:45:06 Marek Vasut wrote: > > > > for (i = 0; i < CONFIG_NR_DRAM_BANKS; i++) { > > > > + if (bd->bi_dram[i].size == 0) > > + continue; > > This doesn't look right at all, your board is misconfigured if > bd->bi_dram[i] == 0. Did you misconfigure CONFIG_NR_DRAM_BANKS > ? On some N900 devices there are two banks and on some only one. If there is only one it has size 256 MB and if there are two both has 128 MB. CONFIG_NR_DRAM_BANKS must be specified at compile time, but for N900 I need runtime detection. -- Pali Roh?r pali.rohar at gmail.com -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 198 bytes Desc: This is a digitally signed message part. URL: