Re: [PATCHv2] firmware/tegra: add COMPILE_TEST

[Mikko Perttunen <mperttunen@nvidia.com>]

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