[U-Boot] [PATCH 38/60] ARM: tegra: remove tegra_get_chip()

Simon Glass
2016-04-23T17:14:32Z

Hi Stephen,

On 19 April 2016 at 14:59, Stephen Warren <swarren@wwwdotorg.org> wrote:
> From: Stephen Warren <swarren@nvidia.com>
>
> U-Boot is compiled for a single board, which in turn uses a specific SoC.
> There's no need to make runtime decisions based on SoC ID. While there's
> certainly an argument for making the code support different SoCs at
> run-time, the Tegra code is so far from that possible ideal that the
> existing runtime code is an anomaly. If this changes in the future, all
> runtime decisions should likely be based on DT anyway.
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> arch/arm/mach-tegra/ap.c | 106 ++++++++++-----------------------
> arch/arm/mach-tegra/cache.c | 20 +++----
> arch/arm/mach-tegra/cpu.c | 16 ++---
> arch/arm/mach-tegra/cpu.h | 6 --
> arch/arm/mach-tegra/tegra20/warmboot.c | 20 ++-----
> 5 files changed, 51 insertions(+), 117 deletions(-)

What exactly is missing to prevent multi-arch support? Shouldn't we
head towards that rather than making it harder?

Regards,
Simon