* [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message
@ 2016-03-28 17:59 Daniel Bristot de Oliveira
2016-03-29 19:09 ` [Devel] " Moore, Robert
0 siblings, 1 reply; 5+ messages in thread
From: Daniel Bristot de Oliveira @ 2016-03-28 17:59 UTC (permalink / raw)
To: Robert Moore, Lv Zheng, Rafael J. Wysocki, Len Brown
Cc: linux-acpi, devel, linux-kernel
A "\n" at the end of bellow ACPI_INFO message is causing a blank line
in the kernel log:
ACPI_INFO(("%u ACPI AML tables successfully acquired and loaded\n", tables_loaded));
This patch removes the "\n".
Kernel log before this patch:
ACPI: Core revision 20160108
ACPI: 2 ACPI AML tables successfully acquired and loaded
Security Framework initialized
Kernel log after this patch:
ACPI: Core revision 20160108
ACPI: 2 ACPI AML tables successfully acquired and loaded
Security Framework initialized
Signed-off-by: Daniel Bristot de Oliveira <bristot@redhat.com>
diff --git a/drivers/acpi/acpica/tbxfload.c b/drivers/acpi/acpica/tbxfload.c
index 3151968..e435b84 100644
--- a/drivers/acpi/acpica/tbxfload.c
+++ b/drivers/acpi/acpica/tbxfload.c
@@ -240,7 +240,8 @@ 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.5.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message
@ 2016-03-29 19:09 ` Moore, Robert
0 siblings, 0 replies; 5+ messages in thread
From: Moore, Robert @ 2016-03-29 19:09 UTC (permalink / raw)
To: Daniel Bristot de Oliveira, Zheng, Lv, Wysocki, Rafael J, Len Brown
Cc: linux-acpi, devel, linux-kernel
Actually, I did in fact put that there to break up the output after the tables are loaded. Is this a problem?
> -----Original Message-----
> From: Daniel Bristot de Oliveira [mailto:bristot@redhat.com]
> Sent: Monday, March 28, 2016 10:59 AM
> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J; Len Brown
> Cc: linux-acpi@vger.kernel.org; devel@acpica.org; linux-
> kernel@vger.kernel.org
> Subject: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot
> message
>
> A "\n" at the end of bellow ACPI_INFO message is causing a blank line in
> the kernel log:
>
> ACPI_INFO(("%u ACPI AML tables successfully acquired and loaded\n",
> tables_loaded));
>
> This patch removes the "\n".
>
> Kernel log before this patch:
> ACPI: Core revision 20160108
> ACPI: 2 ACPI AML tables successfully acquired and loaded
>
> Security Framework initialized
>
> Kernel log after this patch:
> ACPI: Core revision 20160108
> ACPI: 2 ACPI AML tables successfully acquired and loaded
> Security Framework initialized
>
> Signed-off-by: Daniel Bristot de Oliveira <bristot@redhat.com>
>
> diff --git a/drivers/acpi/acpica/tbxfload.c
> b/drivers/acpi/acpica/tbxfload.c index 3151968..e435b84 100644
> --- a/drivers/acpi/acpica/tbxfload.c
> +++ b/drivers/acpi/acpica/tbxfload.c
> @@ -240,7 +240,8 @@ 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.5.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Devel] [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message
@ 2016-03-29 19:09 ` Moore, Robert
0 siblings, 0 replies; 5+ messages in thread
From: Moore, Robert @ 2016-03-29 19:09 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 1736 bytes --]
Actually, I did in fact put that there to break up the output after the tables are loaded. Is this a problem?
> -----Original Message-----
> From: Daniel Bristot de Oliveira [mailto:bristot(a)redhat.com]
> Sent: Monday, March 28, 2016 10:59 AM
> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J; Len Brown
> Cc: linux-acpi(a)vger.kernel.org; devel(a)acpica.org; linux-
> kernel(a)vger.kernel.org
> Subject: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot
> message
>
> A "\n" at the end of bellow ACPI_INFO message is causing a blank line in
> the kernel log:
>
> ACPI_INFO(("%u ACPI AML tables successfully acquired and loaded\n",
> tables_loaded));
>
> This patch removes the "\n".
>
> Kernel log before this patch:
> ACPI: Core revision 20160108
> ACPI: 2 ACPI AML tables successfully acquired and loaded
>
> Security Framework initialized
>
> Kernel log after this patch:
> ACPI: Core revision 20160108
> ACPI: 2 ACPI AML tables successfully acquired and loaded
> Security Framework initialized
>
> Signed-off-by: Daniel Bristot de Oliveira <bristot(a)redhat.com>
>
> diff --git a/drivers/acpi/acpica/tbxfload.c
> b/drivers/acpi/acpica/tbxfload.c index 3151968..e435b84 100644
> --- a/drivers/acpi/acpica/tbxfload.c
> +++ b/drivers/acpi/acpica/tbxfload.c
> @@ -240,7 +240,8 @@ 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.5.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message
2016-03-29 19:09 ` [Devel] " Moore, Robert
(?)
@ 2016-03-31 1:11 ` Daniel Bristot de Oliveira
2016-03-31 4:27 ` Joe Perches
-1 siblings, 1 reply; 5+ messages in thread
From: Daniel Bristot de Oliveira @ 2016-03-31 1:11 UTC (permalink / raw)
To: Moore, Robert, Zheng, Lv, Wysocki, Rafael J, Len Brown
Cc: linux-acpi, devel, linux-kernel
On 03/29/2016 04:09 PM, Moore, Robert wrote:
> Actually, I did in fact put that there to break up the output after the tables are loaded. Is this a problem?
Well, I do not believe that there is a real problem on it.
On the other hand, it does not seem to be common to have blank lines in
the kernel log, and as there is no info about from where the black line
comes from, it does not even seems to be connected to the previous
message. So although my patch is about "cosmetics", IMHO it is worth as
pattern or best practices. Feel free to reject if it is not the case, np
:-).
-- Daniel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message
2016-03-31 1:11 ` Daniel Bristot de Oliveira
@ 2016-03-31 4:27 ` Joe Perches
0 siblings, 0 replies; 5+ messages in thread
From: Joe Perches @ 2016-03-31 4:27 UTC (permalink / raw)
To: Daniel Bristot de Oliveira, Moore, Robert, Zheng, Lv, Wysocki,
Rafael J, Len Brown
Cc: linux-acpi, devel, linux-kernel
On Wed, 2016-03-30 at 22:11 -0300, Daniel Bristot de Oliveira wrote:
> On 03/29/2016 04:09 PM, Moore, Robert wrote:
> > Actually, I did in fact put that there to break up the output after the tables are loaded. Is this a problem?
> Well, I do not believe that there is a real problem on it.
>
> On the other hand, it does not seem to be common to have blank lines in
> the kernel log, and as there is no info about from where the black line
> comes from, it does not even seems to be connected to the previous
> message. So although my patch is about "cosmetics", IMHO it is worth as
> pattern or best practices.
FWIW: I agree with Daniel.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-03-31 4:27 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-28 17:59 [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message Daniel Bristot de Oliveira
2016-03-29 19:09 ` Moore, Robert
2016-03-29 19:09 ` [Devel] " Moore, Robert
2016-03-31 1:11 ` Daniel Bristot de Oliveira
2016-03-31 4:27 ` Joe Perches
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.