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 <kuleshovmail@xxxxxxxxx>; Moore, Robert > <robert.moore@xxxxxxxxx> > Cc: Wysocki, Rafael J <rafael.j.wysocki@xxxxxxxxx>; Len Brown > <lenb@xxxxxxxxxx>; linux-acpi@xxxxxxxxxxxxxxx; devel@xxxxxxxxxx; linux- > kernel@xxxxxxxxxxxxxxx > Subject: RE: [PATCH] ACPICA: Remove unnecessary '\n' in the end of > ACPI_INFO output > > > From: Alexander Kuleshov [mailto:kuleshovmail@xxxxxxxxx] > > 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 <lv.zheng@xxxxxxxxx> > However this patch should go through ACPICA upstream. > > Thanks > -Lv > > > > > Signed-off-by: Alexander Kuleshov <kuleshovmail@xxxxxxxxx> > > --- > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
