Re: [PATCHv2] firmware/tegra: add COMPILE_TEST

[Rosen Penev]

On Mon, Mar 23, 2026 at 11:01 PM Mikko Perttunen <mperttunen@xxxxxxxxxx> wrote:
>
> 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@xxxxxxxxx>
> > ---
> >  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.
Sure. It builds fine.
>
> 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
>
>
>
>