From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Denk Date: Wed, 10 Feb 2016 10:36:54 +0100 Subject: [U-Boot] [PATCH] Revert "common/memsize.c: Simplify RAM size detection" In-Reply-To: <56BAF2B1.3040507@redhat.com> References: <1455053911-7530-1-git-send-email-hdegoede@redhat.com> <56BAF2B1.3040507@redhat.com> Message-ID: <20160210093654.41B4D3841F3@gemini.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 Dear Hans, In message <56BAF2B1.3040507@redhat.com> you wrote: > > It seems this code was as it was for a reason and that a revert > of your simplification is probably the best solution. Indeed. Over the past 15 years I have seen several attempts to improve or simplify the mesize code. Most did not make it into mainline as they worked for the poster, but caused problems on some other boards. We had a similar "simplification" in 2012, see commit b8496cce and revert in 3ab270d5, but all these attempts caused problems of some kind, so the basic structure of the code is still exactly the same as in the original version of 2004 (commit c83bf6a2). Probably this is just Perfect Code (TM) which cannot improved further :-) Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de The word "fit", as I understand it, means "appropriate to a purpose", and I would say the body of the Dean is supremely appropriate to the purpose of sitting around all day and eating big heavy meals. - Terry Pratchett, _Moving Pictures_