From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Fri, 09 Sep 2011 13:04:39 +0200 Subject: [U-Boot] [PATCH 01/31] iMX28: Initial support for iMX28 CPU In-Reply-To: <201109091250.55886.marek.vasut@gmail.com> References: <1315514579-19215-1-git-send-email-marek.vasut@gmail.com> <201109091212.09015.marek.vasut@gmail.com> <4E69EA6D.7060607@denx.de> <201109091250.55886.marek.vasut@gmail.com> Message-ID: <4E69F2C7.1000505@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 09/09/2011 12:50 PM, Marek Vasut wrote: >> >> Well, I do not know if on our Intel-PC this makes a so noticeable >> difference ;-) > > It's a point-of-view question, really. But what about people compiling stuff on > slower machines ? Yes, it is a point of view, and surely, compilation time is a drawback. However, if I compare the advantages between clear interface and compilation time, I choose the first one. >> What I remark here, it is to have a clear and identical interface among >> the several SOCs. This is not only easier for board developers, but also >> removes some nasty #ifdef CONFIG_MX from the drivers. > > Yes, I understand. btw there's no CONFIG_MX stuff. Not yet, but only because you are adding a new SOC without using IMX drivers. Things will be changed when the MX23 will be added. Or some newer SOC of the mxs family will be added by Freescale. Once we had only the MX31. Drivers were done only for this SOC. After using the same driver with different SOCs, a lot of CONFIG_MX was added, and most of code was really not readable - and it tooks time to remove most of them. Best regards, Stefano Babic -- ===================================================================== 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 =====================================================================