From mboxrd@z Thu Jan 1 00:00:00 1970 From: Detlev Zundel Date: Fri, 15 Feb 2008 17:35:56 +0100 Subject: [U-Boot-Users] [PATCH] Add 'imload' command In-Reply-To: (Grant Likely's message of "Wed, 13 Feb 2008 18:13:40 -0700") References: <47B2C244.1050500@semihalf.com> <4268AEEB-CDE1-4110-A979-5D0FBD64B079@kernel.crashing.org> <47B34B23.3040706@semihalf.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Grant, > Ugh, please don't continue down that path. Dataflash is a serial > flash technology, but the driver pretends that it is memory mapped. > It is not a good abstraction that I really think needs to be removed. > I don't think it is a good idea to add that mis-feature into new > commands. Seconded. That abstraction always irritated me a lot and made the code only harder to read. Cheers Detlev -- Markov does it in chains. -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de