I put that \n in there to break up the output from AcpiExec. This comes up every year or so, I think I'll give up -- without really seeing the harm in an extra \n.
> -----Original Message----- > From: Zheng, Lv > Sent: Sunday, August 7, 2016 5:56 PM > To: Alexander Kuleshov <[email protected]>; Moore, Robert > <[email protected]> > Cc: Wysocki, Rafael J <[email protected]>; Len Brown > <[email protected]>; [email protected]; [email protected]; linux- > [email protected] > Subject: RE: [PATCH] ACPICA: Remove unnecessary '\n' in the end of > ACPI_INFO output > > > From: Alexander Kuleshov [mailto:[email protected]] > > Subject: [PATCH] ACPICA: Remove unnecessary '\n' in the end of > > ACPI_INFO output > > > > as the ACPI_INFO already prints `\n` in the end itself. > [Lv Zheng] > Looks good. > Acked-by: Lv Zheng <[email protected]> > However this patch should go through ACPICA upstream. > > Thanks > -Lv > > > > > Signed-off-by: Alexander Kuleshov <[email protected]> > > --- > > drivers/acpi/acpica/tbxfload.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/acpi/acpica/tbxfload.c > > b/drivers/acpi/acpica/tbxfload.c index ac71abc..e7119b7 100644 > > --- a/drivers/acpi/acpica/tbxfload.c > > +++ b/drivers/acpi/acpica/tbxfload.c > > @@ -240,7 +240,7 @@ acpi_status acpi_tb_load_namespace(void) > > } > > > > if (!tables_failed) { > > - ACPI_INFO(("%u ACPI AML tables successfully acquired and > > loaded\n", tables_loaded)); > > + ACPI_INFO(("%u ACPI AML tables successfully acquired and > > loaded", tables_loaded)); > > } else { > > ACPI_ERROR((AE_INFO, > > "%u table load failures, %u successful", > > -- > > 2.8.0.rc3.1353.gea9bdc0
