From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Thu, 29 Oct 2015 11:02:56 +0100 Subject: [U-Boot] [PATCH v0 5/5] spi: Add support for Armada 38x second controller In-Reply-To: References: <1446047056-16801-1-git-send-email-dirk.eibach@gdsys.cc> <1446047056-16801-6-git-send-email-dirk.eibach@gdsys.cc> <5630FA38.5090000@denx.de> Message-ID: <5631EED0.3020203@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 Hi Dirk, On 29.10.2015 10:54, Dirk Eibach wrote: > 2015-10-28 17:39 GMT+01:00 Stefan Roese : >> ... And please note that you can use the >> runtime SoC detection for this: >> >> if (mvebu_soc_family() == MVEBU_SOC_A38X) >> >> So no new #idefs are needed in such places. > > Just give me a quick update please. Why is runtime detection better? > Is it about code coverage? What about binary footprint? We try hard not to add more #idef's to the U-Boot source code if possible. This is definitely one of the reasons. Another is, that this runtime detection will enable support for multiple SoC's in one binary image. This is currently not the case, but we should try to work this way if its not too hard. And these places for the SoC detection are pretty easy to achieve. The image size will of course be affected. But this drawback is outweighed by the pros noted above. At least from my point of view. Thanks, Stefan