Messages in this thread | | | Date | Mon, 30 Jan 2023 09:08:10 +0100 | From | "Arnd Bergmann" <> | Subject | Re: [PATCH] ARM: Allow DEBUG_UNCOMPRESS on ARCH_MULTIPLATFORM |
| |
On Sun, Jan 29, 2023, at 17:00, Jonathan Neuschäfer wrote: > A kernel with DEBUG_UNCOMPRESS=y will only work on one platform, but > that is already the case with DEBUG_LL=y (and documented in its help > text), and DEBUG_UNCOMPRESS depends on DEBUG_LL. Therefore, I don't > think users need to be prevented from enabling DEBUG_UNCOMPRESS on > ARCH_MULTIPLATFORM kernels. > > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
I think we should instead change the help text for CONFIG_DEBUG_LL.
There is a very significant difference between the requirements of DEBUG_LL compared to DEBUG_UNCOMPRESS. The former only provides the platform specific functions but does not call them unless a user explicitly passes the "earlyprintk" command line argument or adds explicit function calls to printascii(), while the latter makes the kernel immediately unbootable.
Arnd
| |