ACPICA: Remove redundant macro defines for ACPI_DEBUG_PRINT() and ACPI_DEBUG_PRINT_RAW()

PATCH In review Hanjun Guo 2 participants 3 messages May 25, 2013 – May 29, 2013

Subsystems (2)

ACPI ACPI component architecture (acpica)

lore

Share & tools

lore

Hanjun Guo

author

2013-05-25 09:13 UTC #0 lore raw

PATCH 1/1 ACPICA: Remove redundant macro defines for ACPI_DEBUG_PRINT() and ACPI_DEBUG_PRINT_RAW()

In commit 0377b5acba2 (ACPICA: Merge all debug output macros into a single file, acoutput.), the patch introduces redundant macro defines for ACPI_DEBUG_PRINT() and ACPI_DEBUG_PRINT_RAW(), just remove it.

Signed-off-by Hanjun Guo <[email protected]>

---
 include/acpi/acoutput.h | 2 --
 1 file changed, 2 deletions(-)

include/acpi/acoutput.h +0 -2

index 4f52ea7..e1c7474 100644
@@ -447,8 +447,6 @@
 #define ACPI_DUMP_TABLES(a, b)
 #define ACPI_DUMP_PATHNAME(a, b, c, d)
 #define ACPI_DUMP_BUFFER(a, b)
-#define ACPI_DEBUG_PRINT(pl)
-#define ACPI_DEBUG_PRINT_RAW(pl)
 #define ACPI_IS_DEBUG_ENABLED(level, component) 0
 
 /* Return macros must have a return statement at the minimum */

-- 1.8.2.1

Moore, Robert

2013-05-28 17:10 UTC #1 lore raw

This was fixed some time ago in the main ACPICA source, and should have made it into Linux by now.

Quoted 32 lines

-----Original Message-----
From: Hanjun Guo [mailto:[email protected]]
Sent: Saturday, May 25, 2013 2:13 AM
To: Zheng, Lv
Cc: Wysocki, Rafael J; Moore, Robert; [email protected]; [email protected]; [email protected]; Jiang Liu
Subject: [PATCH] ACPICA: Remove redundant macro defines for ACPI_DEBUG_PRINT() and ACPI_DEBUG_PRINT_RAW()
In commit 0377b5acba2 (ACPICA: Merge all debug output macros into a single file, acoutput.), the patch introduces redundant macro defines for ACPI_DEBUG_PRINT() and ACPI_DEBUG_PRINT_RAW(), just remove it.
Signed-off-by Hanjun Guo <[email protected]>
---
 include/acpi/acoutput.h | 2 --
 1 file changed, 2 deletions(-)
include/acpi/acoutput.h index renamed from include/acpi/acoutput.h +0 -0
4f52ea7..e1c7474 100644
include/acpi/acoutput.h +0 -3
@@ -447,8 +447,6 @@
 #define ACPI_DUMP_TABLES(a, b)
 #define ACPI_DUMP_PATHNAME(a, b, c, d)
 #define ACPI_DUMP_BUFFER(a, b)
-#define ACPI_DEBUG_PRINT(pl)
-#define ACPI_DEBUG_PRINT_RAW(pl)
 #define ACPI_IS_DEBUG_ENABLED(level, component) 0
 
 /* Return macros must have a return statement at the minimum */
-- 1.8.2.1

Hanjun Guo

author

2013-05-29 10:01 UTC #2 lore raw

Quoted 1 line · Robert

This was fixed some time ago in the main ACPICA source, and should have made it into Linux by now.

Ok, I noticed the ACPICA patch set sent out today by Lv Zheng, and it is fixed in this patch set.

Thanks
Hanjun Guo

Quoted 34 lines

Quoted 34 lines
-----Original Message-----
From: Hanjun Guo [mailto:[email protected]]
Sent: Saturday, May 25, 2013 2:13 AM
To: Zheng, Lv
Cc: Wysocki, Rafael J; Moore, Robert; [email protected]; [email protected]; [email protected]; Jiang Liu
Subject: [PATCH] ACPICA: Remove redundant macro defines for ACPI_DEBUG_PRINT() and ACPI_DEBUG_PRINT_RAW()
In commit 0377b5acba2 (ACPICA: Merge all debug output macros into a single file, acoutput.), the patch introduces redundant macro defines for ACPI_DEBUG_PRINT() and ACPI_DEBUG_PRINT_RAW(), just remove it.
Signed-off-by Hanjun Guo <[email protected]>
---
 include/acpi/acoutput.h | 2 --
 1 file changed, 2 deletions(-)
include/acpi/acoutput.h index renamed from include/acpi/acoutput.h +0 -0
4f52ea7..e1c7474 100644
include/acpi/acoutput.h +0 -3
@@ -447,8 +447,6 @@
 #define ACPI_DUMP_TABLES(a, b)
 #define ACPI_DUMP_PATHNAME(a, b, c, d)
 #define ACPI_DUMP_BUFFER(a, b)
-#define ACPI_DEBUG_PRINT(pl)
-#define ACPI_DEBUG_PRINT_RAW(pl)
 #define ACPI_IS_DEBUG_ENABLED(level, component) 0
 
 /* Return macros must have a return statement at the minimum */
-- 1.8.2.1