From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Mon, 11 May 2009 18:38:45 +0200 Subject: [U-Boot] [PATCH] mtd: Update MTD infrastructure to support 64bit device size In-Reply-To: <200905111227.03607.vapier@gentoo.org> References: <1242050635-7152-1-git-send-email-sr@denx.de> <200905111227.03607.vapier@gentoo.org> Message-ID: <200905111838.45552.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 18:27:02 Mike Frysinger wrote: > On Monday 11 May 2009 10:03:55 Stefan Roese wrote: > > This patch brings the U-Boot MTD infrastructure in sync with the current > > Linux MTD version (2.6.30-rc3). Biggest change is the 64bit device size > > support and a resync of the mtdpart.c file which has seen multiple fixes > > meanwhile. > > 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. And the advantages are big enough from my point of view. Being in sync with Linux once again not only brings 64bit device size support but removes quite a lot of bugs from the mtd infrastructure (for example used by UBI). And frankly, cluttering the files with another set of #ifdef's doesn't really sound promising. Additionally this would make future sync's with the Linux version more troublesome. 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 =====================================================================