ACPICA: Remove unnecessary arguments to ACPI_INFO
authorBob Moore <robert.moore@intel.com>
Fri, 19 Feb 2016 06:16:42 +0000 (14:16 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 24 Feb 2016 13:14:49 +0000 (14:14 +0100)
ACPICA commit 181f56605a771e0b91e24b0648d2565ca70bea20

This is used as a purely infomation message, without module name
and line number information. Therefore, these arguments are
not needed and they are unnecessary overhead.
Arguments are removed.
ACPICA BZ 872.

Link: https://github.com/acpica/acpica/commit/181f5660
Link: https://bugs.acpica.org/show_bug.cgi?id=872
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
14 files changed:
drivers/acpi/acpica/dbcmds.c
drivers/acpi/acpica/dsmethod.c
drivers/acpi/acpica/dsobject.c
drivers/acpi/acpica/evgpeblk.c
drivers/acpi/acpica/evgpeinit.c
drivers/acpi/acpica/exconfig.c
drivers/acpi/acpica/nseval.c
drivers/acpi/acpica/tbinstal.c
drivers/acpi/acpica/tbprint.c
drivers/acpi/acpica/tbutils.c
drivers/acpi/acpica/tbxfload.c
drivers/acpi/acpica/uttrack.c
drivers/acpi/acpica/utxferror.c
include/acpi/acpixf.h

index 7ec62c46128013c43104190d868764fb58e83541..772178c96ccf25a90a003135bb0f33ab89bcf104 100644 (file)
@@ -348,7 +348,7 @@ void acpi_db_display_table_info(char *table_arg)
                } else {
                        /* If the pointer is null, the table has been unloaded */
 
-                       ACPI_INFO((AE_INFO, "%4.4s - Table has been unloaded",
+                       ACPI_INFO(("%4.4s - Table has been unloaded",
                                   table_desc->signature.ascii));
                }
        }
index 6a72047aae1c4c0c1db21db5739172011bf4f217..1982310e6d83a3aa6a6e510e8f72682fb1c05226 100644 (file)
@@ -809,8 +809,7 @@ acpi_ds_terminate_control_method(union acpi_operand_object *method_desc,
                if (method_desc->method.
                    info_flags & ACPI_METHOD_SERIALIZED_PENDING) {
                        if (walk_state) {
-                               ACPI_INFO((AE_INFO,
-                                          "Marking method %4.4s as Serialized "
+                               ACPI_INFO(("Marking method %4.4s as Serialized "
                                           "because of AE_ALREADY_EXISTS error",
                                           walk_state->method_node->name.
                                           ascii));
index c303e9d9266f6e5ec4afed89dc825f0c2fac449f..a91de2b4603c78cfe7a627c092e3e37b922bbdd8 100644 (file)
@@ -524,8 +524,7 @@ acpi_ds_build_internal_package_obj(struct acpi_walk_state *walk_state,
                        arg = arg->common.next;
                }
 
-               ACPI_INFO((AE_INFO,
-                          "Actual Package length (%u) is larger than "
+               ACPI_INFO(("Actual Package length (%u) is larger than "
                           "NumElements field (%u), truncated",
                           i, element_count));
        } else if (i < element_count) {
index 9275e626ed8d8d04e4c920ff494e35ac3c236e73..447fa1cac64fdb3c3bdde2f3199e0ae326a8d953 100644 (file)
@@ -499,8 +499,7 @@ acpi_ev_initialize_gpe_block(struct acpi_gpe_xrupt_info *gpe_xrupt_info,
        }
 
        if (gpe_enabled_count) {
-               ACPI_INFO((AE_INFO,
-                          "Enabled %u GPEs in block %02X to %02X",
+               ACPI_INFO(("Enabled %u GPEs in block %02X to %02X",
                           gpe_enabled_count, (u32)gpe_block->block_base_number,
                           (u32)(gpe_block->block_base_number +
                                 (gpe_block->gpe_count - 1))));
index 9fdd8d09141b89e52d059cc6d9bad027fa9fe89d..7dc75474c8979ceb974e50d2eb66898bb87c3293 100644 (file)
@@ -281,7 +281,7 @@ void acpi_ev_update_gpes(acpi_owner_id table_owner_id)
        }
 
        if (walk_info.count) {
-               ACPI_INFO((AE_INFO, "Enabled %u new GPEs", walk_info.count));
+               ACPI_INFO(("Enabled %u new GPEs", walk_info.count));
        }
 
        (void)acpi_ut_release_mutex(ACPI_MTX_EVENTS);
index 011df210b7b2339323866f71737680b7e1ed8ead..f7416130103798549e0b95f0f80069e381c30195 100644 (file)
@@ -252,7 +252,7 @@ acpi_ex_load_table_op(struct acpi_walk_state *walk_state,
 
        status = acpi_get_table_by_index(table_index, &table);
        if (ACPI_SUCCESS(status)) {
-               ACPI_INFO((AE_INFO, "Dynamic OEM Table Load:"));
+               ACPI_INFO(("Dynamic OEM Table Load:"));
                acpi_tb_print_table_header(0, table);
        }
 
@@ -472,7 +472,7 @@ acpi_ex_load_op(union acpi_operand_object *obj_desc,
 
        /* Install the new table into the local data structures */
 
-       ACPI_INFO((AE_INFO, "Dynamic OEM Table Load:"));
+       ACPI_INFO(("Dynamic OEM Table Load:"));
        (void)acpi_ut_acquire_mutex(ACPI_MTX_TABLES);
 
        status = acpi_tb_install_standard_table(ACPI_PTR_TO_PHYSADDR(table),
index 65d58bea43207a80b61805bb51cd9bb1744c5bfd..5d59cfcef6f4eb8a5e8fa0a1b1fdaee40fff1dba 100644 (file)
@@ -378,8 +378,7 @@ void acpi_ns_exec_module_code_list(void)
                acpi_ut_remove_reference(prev);
        }
 
-       ACPI_INFO((AE_INFO,
-                  "Executed %u blocks of module-level executable AML code",
+       ACPI_INFO(("Executed %u blocks of module-level executable AML code",
                   method_count));
 
        ACPI_FREE(info);
index b661a1e013fbed639da4aab35002670c5fdc2f4e..4dc6108de4ffbf83a37ea714e0342cfcd8847d6c 100644 (file)
@@ -267,8 +267,7 @@ acpi_tb_install_standard_table(acpi_physical_address address,
        if (!reload &&
            acpi_gbl_disable_ssdt_table_install &&
            ACPI_COMPARE_NAME(&new_table_desc.signature, ACPI_SIG_SSDT)) {
-               ACPI_INFO((AE_INFO,
-                          "Ignoring installation of %4.4s at %8.8X%8.8X",
+               ACPI_INFO(("Ignoring installation of %4.4s at %8.8X%8.8X",
                           new_table_desc.signature.ascii,
                           ACPI_FORMAT_UINT64(address)));
                goto release_and_exit;
@@ -432,7 +431,7 @@ finish_override:
                return;
        }
 
-       ACPI_INFO((AE_INFO, "%4.4s 0x%8.8X%8.8X"
+       ACPI_INFO(("%4.4s 0x%8.8X%8.8X"
                   " %s table override, new table: 0x%8.8X%8.8X",
                   old_table_desc->signature.ascii,
                   ACPI_FORMAT_UINT64(old_table_desc->address),
index fd4146d4ff4902660996add0e78ca9582d07e5b3..26d61dbace0a1fa67b8706cc1ee0d124ac3d7e01 100644 (file)
@@ -132,7 +132,7 @@ acpi_tb_print_table_header(acpi_physical_address address,
 
                /* FACS only has signature and length fields */
 
-               ACPI_INFO((AE_INFO, "%-4.4s 0x%8.8X%8.8X %06X",
+               ACPI_INFO(("%-4.4s 0x%8.8X%8.8X %06X",
                           header->signature, ACPI_FORMAT_UINT64(address),
                           header->length));
        } else if (ACPI_VALIDATE_RSDP_SIG(header->signature)) {
@@ -144,7 +144,7 @@ acpi_tb_print_table_header(acpi_physical_address address,
                       ACPI_OEM_ID_SIZE);
                acpi_tb_fix_string(local_header.oem_id, ACPI_OEM_ID_SIZE);
 
-               ACPI_INFO((AE_INFO, "RSDP 0x%8.8X%8.8X %06X (v%.2d %-6.6s)",
+               ACPI_INFO(("RSDP 0x%8.8X%8.8X %06X (v%.2d %-6.6s)",
                           ACPI_FORMAT_UINT64(address),
                           (ACPI_CAST_PTR(struct acpi_table_rsdp, header)->
                            revision >
@@ -158,8 +158,7 @@ acpi_tb_print_table_header(acpi_physical_address address,
 
                acpi_tb_cleanup_table_header(&local_header, header);
 
-               ACPI_INFO((AE_INFO,
-                          "%-4.4s 0x%8.8X%8.8X"
+               ACPI_INFO(("%-4.4s 0x%8.8X%8.8X"
                           " %06X (v%.2d %-6.6s %-8.8s %08X %-4.4s %08X)",
                           local_header.signature, ACPI_FORMAT_UINT64(address),
                           local_header.length, local_header.revision,
index 3269bef371d7ddd21a32ac18f58d404cefdeb44b..9240c76d2823e333d2b30ca2d24ad32bfae3c5cf 100644 (file)
@@ -174,9 +174,7 @@ struct acpi_table_header *acpi_tb_copy_dsdt(u32 table_index)
                                      ACPI_TABLE_ORIGIN_INTERNAL_VIRTUAL,
                                      new_table);
 
-       ACPI_INFO((AE_INFO,
-                  "Forced DSDT copy: length 0x%05X copied locally, original unmapped",
-                  new_table->length));
+       ACPI_INFO(("Forced DSDT copy: length 0x%05X copied locally, original unmapped", new_table->length));
 
        return (new_table);
 }
index 278666e39563e7fc3cacdc4ae6e36a5a7bdc3c93..c89943b3ff3ccf2fe577058ec1ec26f3acb1f660 100644 (file)
@@ -206,9 +206,7 @@ acpi_status acpi_tb_load_namespace(void)
        }
 
        if (!tables_failed) {
-               ACPI_INFO((AE_INFO,
-                          "%u ACPI AML tables successfully acquired and loaded\n",
-                          tables_loaded));
+               ACPI_INFO(("%u ACPI AML tables successfully acquired and loaded\n", tables_loaded));
        } else {
                ACPI_ERROR((AE_INFO,
                            "%u table load failures, %u successful",
@@ -301,7 +299,7 @@ acpi_status acpi_load_table(struct acpi_table_header *table)
 
        /* Install the table and load it into the namespace */
 
-       ACPI_INFO((AE_INFO, "Host-directed Dynamic ACPI Table Load:"));
+       ACPI_INFO(("Host-directed Dynamic ACPI Table Load:"));
        (void)acpi_ut_acquire_mutex(ACPI_MTX_TABLES);
 
        status = acpi_tb_install_standard_table(ACPI_PTR_TO_PHYSADDR(table),
index c7c2bb8f3559cc71c07fbb8933b13a53d6874ac2..60c406a8efcb3b533b6d2fd1f1f5c9f75f13831e 100644 (file)
@@ -712,7 +712,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
        /* Print summary */
 
        if (!num_outstanding) {
-               ACPI_INFO((AE_INFO, "No outstanding allocations"));
+               ACPI_INFO(("No outstanding allocations"));
        } else {
                ACPI_ERROR((AE_INFO, "%u(0x%X) Outstanding allocations",
                            num_outstanding, num_outstanding));
index 6fe59597b5999dc340cd25590e8c6b00a0cc14b6..d9f15cbcd8a0b9228382042abdd802b4322d11ba 100644 (file)
@@ -175,8 +175,7 @@ ACPI_EXPORT_SYMBOL(acpi_warning)
  * TBD: module_name and line_number args are not needed, should be removed.
  *
  ******************************************************************************/
-void ACPI_INTERNAL_VAR_XFACE
-acpi_info(const char *module_name, u32 line_number, const char *format, ...)
+void ACPI_INTERNAL_VAR_XFACE acpi_info(const char *format, ...)
 {
        va_list arg_list;
 
index c96621e87c196bf8fa6aa7c9f5033c625efc4cb2..17556979dc79bffdb26b928d6f6348edfae4d095 100644 (file)
@@ -897,11 +897,9 @@ ACPI_MSG_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(3)
                                acpi_warning(const char *module_name,
                                             u32 line_number,
                                             const char *format, ...))
-ACPI_MSG_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(3)
+ACPI_MSG_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(1)
                                void ACPI_INTERNAL_VAR_XFACE
-                               acpi_info(const char *module_name,
-                                         u32 line_number,
-                                         const char *format, ...))
+                               acpi_info(const char *format, ...))
 ACPI_MSG_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(3)
                                void ACPI_INTERNAL_VAR_XFACE
                                acpi_bios_error(const char *module_name,