From mboxrd@z Thu Jan 1 00:00:00 1970 From: Valentin Longchamp Date: Mon, 04 Jun 2012 17:36:30 +0200 Subject: [U-Boot] [PATCH] Kirkwood: Add support for Ka-Ro TK71 In-Reply-To: <201206011503.43239.marex@denx.de> References: <1338462428-11915-1-git-send-email-marex@denx.de> <201206011503.43239.marex@denx.de> Message-ID: <4FCCD5FE.6050707@keymile.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Marek and Prafulla, On 06/01/2012 03:03 PM, Marek Vasut wrote: > Dear Prafulla Wadaskar, > >>> -----Original Message----- >>> From: Marek Vasut [mailto:marex at denx.de] >>> Sent: 31 May 2012 16:37 >>> To: u-boot at lists.denx.de >>> Cc: Marek Vasut; Prafulla Wadaskar; Wolfgang Denk >>> Subject: [PATCH] Kirkwood: Add support for Ka-Ro TK71 >>> >>> Signed-off-by: Marek Vasut >>> Cc: Prafulla Wadaskar >>> Cc: Wolfgang Denk >>> --- >>> >>> board/karo/tk71/Makefile | 45 ++++++++++ >>> board/karo/tk71/kwbimage-256.cfg | 174 >>> >>> ++++++++++++++++++++++++++++++++++++++ >>> >>> board/karo/tk71/kwbimage-512.cfg | 174 >>> >>> ++++++++++++++++++++++++++++++++++++++ >> >> Dear Marek >> Just for DRAM size change do not add one more cfg file, configure by >> default 256MB of RAM in default kwbimg.cfg file and in function >> board_early_init_f() tune it to 512 for your other board version. > > There's only one single bit flipped between those two kwb configs. Do you think > it'd work if we just configured the system for 512MB RAM and ran get_ram_size() > to see if it has only 256MB? That'd eliminate two board entries for this tk71. > I would like to have your advice on this as well Prafulla. We have tested this on km_arm (we will have the same boards with 1/2 the RAM size) and with the above get_ram_size() it works as expected. We still should, however, at some point (board_early_init_f() is a good candidate) then reduce the corresponding RAM CS size (reg @1504 for CS0 so that the window is the same size as what was detected by get_ram_size). What do you guys think ? Valentin