From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Tue, 12 May 2009 06:39:24 +0200 Subject: [U-Boot] [PATCH] mtd: Update MTD infrastructure to support 64bit device size In-Reply-To: <20090511175910.E1079832E416@gemini.denx.de> References: <1242050635-7152-1-git-send-email-sr@denx.de> <200905111838.45552.sr@denx.de> <20090511175910.E1079832E416@gemini.denx.de> Message-ID: <200905120639.24172.sr@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Monday 11 May 2009 19:59:10 Wolfgang Denk wrote: > > > really wish there was a define to control this. large devices are > > > fine, but it sucks when majority of people dont hit this limit. > > > > What is the drawback of this code version? I have to admit that I didn't > > compare code size, but this is the only possible drawback coming to my > > mind. > > Can you please make such a comparison? I, too, would like to know how > much of a difference that makes - I share Mike's concerns. OK, here we go: Canyonlands (using NAND) without this patch: text data bss dec hex filename 339324 20044 336384 695752 a9dc8 ./u-boot and with this patch: text data bss dec hex filename 339796 20044 336384 696224 a9fa0 ./u-boot So it's 472 bytes on this platform. Best regards, Stefan ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de =====================================================================