Hi Dirk,
On 29.10.2015 10:54, Dirk Eibach wrote:
2015-10-28 17:39 GMT+01:00 Stefan Roese sr@denx.de:
... 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