Re: [PATCH 5/5] ARM: make ARCH_MULTIPLATFORM user-visible

From: Geert Uytterhoeven
Date: Wed Sep 28 2022 - 02:50:34 EST


Hi Arnd,

On Tue, Sep 27, 2022 at 10:19 PM Arnd Bergmann <arnd@xxxxxxxx> wrote:
> On Tue, Sep 27, 2022, at 3:31 PM, Geert Uytterhoeven wrote:
> > On Thu, Aug 18, 2022 at 4:56 PM Arnd Bergmann <arnd@xxxxxxxxxx> wrote:
> >> --- a/arch/arm/Kconfig.debug
> >> +++ b/arch/arm/Kconfig.debug
> >> @@ -1904,6 +1904,7 @@ config DEBUG_UART_8250_PALMCHIP
> >>
> >> config DEBUG_UNCOMPRESS
> >> bool "Enable decompressor debugging via DEBUG_LL output"
> >> + depends on !ARCH_MULTIPLATFORM
> >
> > Shouldn't DEBUG_LL itself depend on !ARCH_MULTIPLATFORM instead?
>
> That would also be possible, but I prefer to keep limiting only
> the DEBUG_UNCOMPRESS. The idea of DEBUG_LL is that while it's
> hardwired to a particular hardware address, it does not actually
> access this address unless you specify the 'earlyprintk'
> argument on the command line.

... or unless something goes really wrong, and the kernel tries to
inform the user using early_print()?

Note that the I/O region for the debug serial port is mapped regardless.
Any chance this can cause conflicts?

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

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