Re: [PATCH] mailbox: ARM_MHU_V3 should depend on ARM64

Geert Uytterhoeven
2024-05-29T11:36:57Z

Hi Cristian,

On Wed, May 29, 2024 at 12:13 PM Cristian Marussi
<cristian.marussi@arm.com> wrote:
> On Wed, May 29, 2024 at 09:30:45AM +0200, Geert Uytterhoeven wrote:
> > The ARM MHUv3 controller is only present on ARM64 SoCs. Hence add a
> > dependency on ARM64, to prevent asking the user about this driver when
> > configuring a kernel for a different architecture than ARM64.
>
> the ARM64 dependency was dropped on purpose after a few iterations of
> this series since, despite this being an ARM IP, it has really no technical
> dependency on ARM arch, not even the usual one on ARM AMBA bus, being this a
> platform driver, so it seemed an uneeded artificial restriction to impose...
> ...having said that, surely my live testing were performed only on arm64 models
> as of now.

For that, we have COMPILE_TEST=y.

> So, I am not saying that I am against this proposed fix but what is the
> issue that is trying to solve, have you seen any compilation error ? or
> is it just to avoid the user-prompting ?

I did not see a compile error (I didn't enable it on any non-ARM
platform).

But it is rather futile to ask the user about (thousands of) drivers
for hardware that cannot possibly be present on the system he is
configuring a kernel for.

> > --- a/drivers/mailbox/Kconfig
> > +++ b/drivers/mailbox/Kconfig
> > @@ -25,6 +25,7 @@ config ARM_MHU_V2
> >
> > config ARM_MHU_V3
> > tristate "ARM MHUv3 Mailbox"
> > + depends on ARM64 || COMPILE_TEST
> > depends on HAS_IOMEM || COMPILE_TEST
> > depends on OF
> > help

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds