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 <kuleshovm...@gmail.com>; Moore, Robert > <robert.mo...@intel.com> > Cc: Wysocki, Rafael J <rafael.j.wyso...@intel.com>; Len Brown > <l...@kernel.org>; linux-a...@vger.kernel.org; de...@acpica.org; linux- > ker...@vger.kernel.org > Subject: RE: [PATCH] ACPICA: Remove unnecessary '\n' in the end of > ACPI_INFO output > > > From: Alexander Kuleshov [mailto:kuleshovm...@gmail.com] > > 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.zh...@intel.com> > However this patch should go through ACPICA upstream. > > Thanks > -Lv > > > > > Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> > > --- > > 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
