> > In the API module, there is no unified configuration switch to control de= bug > You mean ACPI I suppose? Oh~, Sorry for the typo. > > output, and the current approach is to enable debugging by adding "define= DEBUG" > > in the file, which is both cumbersome and difficult to manage. a global d= ebug config > > to control the debug output of the ACPI module will be more easily and cl= early. > So there is only one KERN_DEBUG printk() statement in the entire drivers/acpi/ directory, the rest is pr_debug() or dev_dbg() that shouldn't need this change. The original intention of this patch is to manage the printing function of all KERN_DEBUG levels in drivers/acpi, including acpi_handle_debug(), pr_debug(), and dev_debg(). Why don't pr_debug() need this change? > > diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig > > index cee82b473dc5..ea198ead57d7 100644 > > --- a/drivers/acpi/Kconfig > > +++ b/drivers/acpi/Kconfig > > @@ -479,6 +479,12 @@ config ACPI_REDUCED_HARDWARE_ONLY > > > > If you are unsure what to do, do not enable this option. > > > > +config ACPI_KERN_DEBUG > > + bool "Acpi kernel debugging" > Please always spell ACPI in capitals. Got it, thanks. > > diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile > > index eaa09bf52f17..d0a417e73071 100644 > > --- a/drivers/acpi/Makefile > > +++ b/drivers/acpi/Makefile > > ccflags-$(CONFIG_ACPI_DEBUG) += -DACPI_DEBUG_OUTPUT > > +ccflags-$(CONFIG_ACPI_KERN_DEBUG) := -DDEBUG > Isn't this kind of confusing? Indeed, i will change. Thanks.
