On Thu, Dec 12, 2024 at 12:55 PM Xiong Yining <xiongyining1480@xxxxxxxxxxxxxx> wrote: > > > > 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? Because they can be enabled through dynamic debug which IMV is superior to a compile-time switch.