On Friday, December 11, 2020 7:50:12 AM CET Tian Tao wrote: > printk(KERNEL_INFO) is same with pr_info, so used pr_info instead of > printk. > > Signed-off-by: Tian Tao <tiantao6@xxxxxxxxxxxxx> > --- > drivers/acpi/ac.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/acpi/ac.c b/drivers/acpi/ac.c > index 46a64e9..5e4b9c1 100644 > --- a/drivers/acpi/ac.c > +++ b/drivers/acpi/ac.c > @@ -278,9 +278,9 @@ static int acpi_ac_add(struct acpi_device *device) > goto end; > } > > - printk(KERN_INFO PREFIX "%s [%s] (%s)\n", > - acpi_device_name(device), acpi_device_bid(device), > - ac->state ? "on-line" : "off-line"); > + pr_info(PREFIX "%s [%s] (%s)\n", > + acpi_device_name(device), acpi_device_bid(device), > + ac->state ? "on-line" : "off-line"); What about defining a proper pr_fmt() for this file instead of using PREFIX here? > > ac->battery_nb.notifier_call = acpi_ac_battery_notify; > register_acpi_notifier(&ac->battery_nb); >
