[U-Boot] [PATCHv5 4/4] Add support the Avionic Design Meerkat COM and Kein Baseboard
Stephen Warren
swarren at wwwdotorg.org
Tue Sep 6 19:15:35 CEST 2016
On 09/05/2016 07:29 AM, Julian Scheel wrote:
> Add support for platforms based on the Meerkat COM module. Includes support
> for the minimal reference platform called Kein Baseboard, which in fact is
> sufficient to run most existing Meerkat carriers.
> diff --git a/arch/arm/dts/tegra124-meerkat.dtsi b/arch/arm/dts/tegra124-meerkat.dtsi
> @@ -0,0 +1,409 @@
> +
> +#include "tegra124.dtsi"
There's an unnecessary blank line at the top of the file.
> diff --git a/board/avionic-design/common/meerkat.c b/board/avionic-design/common/meerkat.c
> +void pinmux_init(void)
> +{
> + pinmux_set_tristate_input_clamping();
That should be pinmux_clear_tristate_input_clamping();
gpio_config_table() is missing here.
> + pinmux_config_pingrp_table(meerkat_pingrps,
> + ARRAY_SIZE(meerkat_pingrps));
> +
> + pinmux_config_drvgrp_table(meerkat_drvgrps,
> + ARRAY_SIZE(meerkat_drvgrps));
pinmux_config_mipipadctrlgrp_table() is missing here.
Most/all of these are related to not using the latest
tegra-pinmux-scripts to generate the pin config table; see the comments
on that below.
> diff --git a/board/avionic-design/common/pinmux-config-meerkat.h b/board/avionic-design/common/pinmux-config-meerkat.h
> +/*
> + * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
> + * Copyright (c) 2015, Avionic Design GmbH
> + *
> + * SPDX-License-Identifier: GPL-2.0+
> + */
Can you please also add support for this board to tegra-pinmux-scripts,
so that anyone can generate this file? That will also allow you to
re-generate the file using the latest version of tegra-pinmux-scripts
which will add (a) the missing "this file is auto-generated" notice, (b)
the GPIO initialization table, (c) the MIPI padctl initialization table.
> diff --git a/configs/kein-baseboard_defconfig b/configs/kein-baseboard_defconfig
> +CONFIG_CMD_EXT4=y
Relative to Jetson TK1, CONFIG_CMD_EXT4_WRITE is missing. Was that
deliberate? I'd rather keep all the Tegra configs as similar as
possible, at least in the upstream code.
> +CONFIG_DM_I2C_COMPAT=y
That's not required on any Tegra board these days. Is it necessary?
> +CONFIG_E1000=y
I notice that CONFIG_CMD_MII isn't present, yet Ethernet is. For
consistency with other Tegra boards, does it make sense to add
CONFIG_CMD_MII?
> +CONFIG_USB_STORAGE=y
For Jetson TK1, this is defined in include/configs/jetson-tk1.h. I'd
expect the two board configs to work the same way.
CONFIG_USE_PRIVATE_LIBGCC=y is missing.
More information about the U-Boot
mailing list