Re: [PATCHv2] firmware/tegra: add COMPILE_TEST

Mikko Perttunen
2026-03-24T06:01:08Z

On Tuesday, March 24, 2026 10:57 AM Rosen Penev wrote:
> For this driver specifically, there's no arch specific stuff needed.
>
> Build the tegra clk stuff as tegra_bpmp_init_clocks is defined there.
> CLK_TEGRA_BPMP also depends on TEGRA_BPMP.
>
> Signed-off-by: Rosen Penev <rosenp@gmail.com>
> ---
> v2: also build the tegra clk stuff.
> drivers/clk/Makefile | 2 +-
> drivers/firmware/tegra/Kconfig | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile
> index 760d9643705f..334d5c522394 100644
> --- a/drivers/clk/Makefile
> +++ b/drivers/clk/Makefile
> @@ -154,7 +154,7 @@ obj-$(CONFIG_ARCH_STM32) += stm32/
> obj-y += starfive/
> obj-$(CONFIG_ARCH_SUNXI) += sunxi/
> obj-y += sunxi-ng/
> -obj-$(CONFIG_ARCH_TEGRA) += tegra/
> +obj-y += tegra/

clk/tegra/Makefile doesn't separately check for ARCH_TEGRA, so this will cause
much of it to get built on non-Tegra platforms as well.

Cheers,
Mikko

> obj-y += tenstorrent/
> obj-$(CONFIG_ARCH_THEAD) += thead/
> obj-y += ti/
> diff --git a/drivers/firmware/tegra/Kconfig b/drivers/firmware/tegra/Kconfig
> index 91f2320c0d0f..ae21f460807d 100644
> --- a/drivers/firmware/tegra/Kconfig
> +++ b/drivers/firmware/tegra/Kconfig
> @@ -3,7 +3,7 @@ menu "Tegra firmware driver"
>
> config TEGRA_IVC
> bool "Tegra IVC protocol" if COMPILE_TEST
> - depends on ARCH_TEGRA
> + depends on ARCH_TEGRA || COMPILE_TEST
> help
> IVC (Inter-VM Communication) protocol is part of the IPC
> (Inter Processor Communication) framework on Tegra. It maintains
the
> @@ -13,7 +13,7 @@ config TEGRA_IVC
>
> config TEGRA_BPMP
> bool "Tegra BPMP driver"
> - depends on ARCH_TEGRA && TEGRA_HSP_MBOX
> + depends on (ARCH_TEGRA && TEGRA_HSP_MBOX) || COMPILE_TEST
> depends on !CPU_BIG_ENDIAN
> select TEGRA_IVC
> help
> --
> 2.53.0