[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <27eefbf2-fc2c-7800-1397-8acfea7ed7e8@csgroup.eu>
Date: Fri, 24 Dec 2021 06:10:33 +0000
From: Christophe Leroy <christophe.leroy@...roup.eu>
To: Lukas Bulwahn <lukas.bulwahn@...il.com>,
Michael Ellerman <mpe@...erman.id.au>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Paul Mackerras <paulus@...ba.org>,
"linuxppc-dev@...ts.ozlabs.org" <linuxppc-dev@...ts.ozlabs.org>
CC: kernel-janitors <kernel-janitors@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: code conditional on non-existing PPC_EARLY_DEBUG_MICROWATT.
Le 23/12/2021 à 11:21, Lukas Bulwahn a écrit :
> Dear Benjamin, dear Paul, dear Michael,
>
> with commit 48b545b8018d ("powerpc/microwatt: Use standard 16550 UART
> for console"), you have some code in arch/powerpc/kernel/udbg_16550.c,
> conditional on the Kconfig symbol PPC_EARLY_DEBUG_MICROWATT. However,
> since then, the definition of this Kconfig symbol was never introduced
> to the mainline repository or current linux-next, nor am I finding any
> pending patch for that.
>
> Are you going to add this config definition soon? Or did you identify
> that this setup code in udbg_16550.c is not actually needed and can we
> simply drop this code again?
>
> This issue was identified with the script ./scripts/checkkconfigsymbols.py.
>
Was it forgotten when handling comments to
https://patchwork.ozlabs.org/project/linuxppc-dev/patch/20200509050340.GD1464954@thinks.paulus.ozlabs.org/
?
Powered by blists - more mailing lists