diff options
author | Bob Moore <robert.moore@intel.com> | 2016-02-19 01:16:42 -0500 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-02-24 08:14:49 -0500 |
commit | 05fb04b54f6b67f4eb901e99dd5b4604b46ffa0f (patch) | |
tree | 26a2d68518a0fcf85eb671a783c3caf0330910a3 | |
parent | c340e5f0f8fe12079219ddf3c012bbc989b6f45b (diff) |
ACPICA: Remove unnecessary arguments to ACPI_INFO
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>
-rw-r--r-- | drivers/acpi/acpica/dbcmds.c | 2 | ||||
-rw-r--r-- | drivers/acpi/acpica/dsmethod.c | 3 | ||||
-rw-r--r-- | drivers/acpi/acpica/dsobject.c | 3 | ||||
-rw-r--r-- | drivers/acpi/acpica/evgpeblk.c | 3 | ||||
-rw-r--r-- | drivers/acpi/acpica/evgpeinit.c | 2 | ||||
-rw-r--r-- | drivers/acpi/acpica/exconfig.c | 4 | ||||
-rw-r--r-- | drivers/acpi/acpica/nseval.c | 3 | ||||
-rw-r--r-- | drivers/acpi/acpica/tbinstal.c | 5 | ||||
-rw-r--r-- | drivers/acpi/acpica/tbprint.c | 7 | ||||
-rw-r--r-- | drivers/acpi/acpica/tbutils.c | 4 | ||||
-rw-r--r-- | drivers/acpi/acpica/tbxfload.c | 6 | ||||
-rw-r--r-- | drivers/acpi/acpica/uttrack.c | 2 | ||||
-rw-r--r-- | drivers/acpi/acpica/utxferror.c | 3 | ||||
-rw-r--r-- | include/acpi/acpixf.h | 6 |
14 files changed, 20 insertions, 33 deletions
diff --git a/drivers/acpi/acpica/dbcmds.c b/drivers/acpi/acpica/dbcmds.c index 7ec62c461280..772178c96ccf 100644 --- a/drivers/acpi/acpica/dbcmds.c +++ b/drivers/acpi/acpica/dbcmds.c | |||
@@ -348,7 +348,7 @@ void acpi_db_display_table_info(char *table_arg) | |||
348 | } else { | 348 | } else { |
349 | /* If the pointer is null, the table has been unloaded */ | 349 | /* If the pointer is null, the table has been unloaded */ |
350 | 350 | ||
351 | ACPI_INFO((AE_INFO, "%4.4s - Table has been unloaded", | 351 | ACPI_INFO(("%4.4s - Table has been unloaded", |
352 | table_desc->signature.ascii)); | 352 | table_desc->signature.ascii)); |
353 | } | 353 | } |
354 | } | 354 | } |
diff --git a/drivers/acpi/acpica/dsmethod.c b/drivers/acpi/acpica/dsmethod.c index 6a72047aae1c..1982310e6d83 100644 --- a/drivers/acpi/acpica/dsmethod.c +++ b/drivers/acpi/acpica/dsmethod.c | |||
@@ -809,8 +809,7 @@ acpi_ds_terminate_control_method(union acpi_operand_object *method_desc, | |||
809 | if (method_desc->method. | 809 | if (method_desc->method. |
810 | info_flags & ACPI_METHOD_SERIALIZED_PENDING) { | 810 | info_flags & ACPI_METHOD_SERIALIZED_PENDING) { |
811 | if (walk_state) { | 811 | if (walk_state) { |
812 | ACPI_INFO((AE_INFO, | 812 | ACPI_INFO(("Marking method %4.4s as Serialized " |
813 | "Marking method %4.4s as Serialized " | ||
814 | "because of AE_ALREADY_EXISTS error", | 813 | "because of AE_ALREADY_EXISTS error", |
815 | walk_state->method_node->name. | 814 | walk_state->method_node->name. |
816 | ascii)); | 815 | ascii)); |
diff --git a/drivers/acpi/acpica/dsobject.c b/drivers/acpi/acpica/dsobject.c index c303e9d9266f..a91de2b4603c 100644 --- a/drivers/acpi/acpica/dsobject.c +++ b/drivers/acpi/acpica/dsobject.c | |||
@@ -524,8 +524,7 @@ acpi_ds_build_internal_package_obj(struct acpi_walk_state *walk_state, | |||
524 | arg = arg->common.next; | 524 | arg = arg->common.next; |
525 | } | 525 | } |
526 | 526 | ||
527 | ACPI_INFO((AE_INFO, | 527 | ACPI_INFO(("Actual Package length (%u) is larger than " |
528 | "Actual Package length (%u) is larger than " | ||
529 | "NumElements field (%u), truncated", | 528 | "NumElements field (%u), truncated", |
530 | i, element_count)); | 529 | i, element_count)); |
531 | } else if (i < element_count) { | 530 | } else if (i < element_count) { |
diff --git a/drivers/acpi/acpica/evgpeblk.c b/drivers/acpi/acpica/evgpeblk.c index 9275e626ed8d..447fa1cac64f 100644 --- a/drivers/acpi/acpica/evgpeblk.c +++ b/drivers/acpi/acpica/evgpeblk.c | |||
@@ -499,8 +499,7 @@ acpi_ev_initialize_gpe_block(struct acpi_gpe_xrupt_info *gpe_xrupt_info, | |||
499 | } | 499 | } |
500 | 500 | ||
501 | if (gpe_enabled_count) { | 501 | if (gpe_enabled_count) { |
502 | ACPI_INFO((AE_INFO, | 502 | ACPI_INFO(("Enabled %u GPEs in block %02X to %02X", |
503 | "Enabled %u GPEs in block %02X to %02X", | ||
504 | gpe_enabled_count, (u32)gpe_block->block_base_number, | 503 | gpe_enabled_count, (u32)gpe_block->block_base_number, |
505 | (u32)(gpe_block->block_base_number + | 504 | (u32)(gpe_block->block_base_number + |
506 | (gpe_block->gpe_count - 1)))); | 505 | (gpe_block->gpe_count - 1)))); |
diff --git a/drivers/acpi/acpica/evgpeinit.c b/drivers/acpi/acpica/evgpeinit.c index 9fdd8d09141b..7dc75474c897 100644 --- a/drivers/acpi/acpica/evgpeinit.c +++ b/drivers/acpi/acpica/evgpeinit.c | |||
@@ -281,7 +281,7 @@ void acpi_ev_update_gpes(acpi_owner_id table_owner_id) | |||
281 | } | 281 | } |
282 | 282 | ||
283 | if (walk_info.count) { | 283 | if (walk_info.count) { |
284 | ACPI_INFO((AE_INFO, "Enabled %u new GPEs", walk_info.count)); | 284 | ACPI_INFO(("Enabled %u new GPEs", walk_info.count)); |
285 | } | 285 | } |
286 | 286 | ||
287 | (void)acpi_ut_release_mutex(ACPI_MTX_EVENTS); | 287 | (void)acpi_ut_release_mutex(ACPI_MTX_EVENTS); |
diff --git a/drivers/acpi/acpica/exconfig.c b/drivers/acpi/acpica/exconfig.c index 011df210b7b2..f74161301037 100644 --- a/drivers/acpi/acpica/exconfig.c +++ b/drivers/acpi/acpica/exconfig.c | |||
@@ -252,7 +252,7 @@ acpi_ex_load_table_op(struct acpi_walk_state *walk_state, | |||
252 | 252 | ||
253 | status = acpi_get_table_by_index(table_index, &table); | 253 | status = acpi_get_table_by_index(table_index, &table); |
254 | if (ACPI_SUCCESS(status)) { | 254 | if (ACPI_SUCCESS(status)) { |
255 | ACPI_INFO((AE_INFO, "Dynamic OEM Table Load:")); | 255 | ACPI_INFO(("Dynamic OEM Table Load:")); |
256 | acpi_tb_print_table_header(0, table); | 256 | acpi_tb_print_table_header(0, table); |
257 | } | 257 | } |
258 | 258 | ||
@@ -472,7 +472,7 @@ acpi_ex_load_op(union acpi_operand_object *obj_desc, | |||
472 | 472 | ||
473 | /* Install the new table into the local data structures */ | 473 | /* Install the new table into the local data structures */ |
474 | 474 | ||
475 | ACPI_INFO((AE_INFO, "Dynamic OEM Table Load:")); | 475 | ACPI_INFO(("Dynamic OEM Table Load:")); |
476 | (void)acpi_ut_acquire_mutex(ACPI_MTX_TABLES); | 476 | (void)acpi_ut_acquire_mutex(ACPI_MTX_TABLES); |
477 | 477 | ||
478 | status = acpi_tb_install_standard_table(ACPI_PTR_TO_PHYSADDR(table), | 478 | status = acpi_tb_install_standard_table(ACPI_PTR_TO_PHYSADDR(table), |
diff --git a/drivers/acpi/acpica/nseval.c b/drivers/acpi/acpica/nseval.c index 65d58bea4320..5d59cfcef6f4 100644 --- a/drivers/acpi/acpica/nseval.c +++ b/drivers/acpi/acpica/nseval.c | |||
@@ -378,8 +378,7 @@ void acpi_ns_exec_module_code_list(void) | |||
378 | acpi_ut_remove_reference(prev); | 378 | acpi_ut_remove_reference(prev); |
379 | } | 379 | } |
380 | 380 | ||
381 | ACPI_INFO((AE_INFO, | 381 | ACPI_INFO(("Executed %u blocks of module-level executable AML code", |
382 | "Executed %u blocks of module-level executable AML code", | ||
383 | method_count)); | 382 | method_count)); |
384 | 383 | ||
385 | ACPI_FREE(info); | 384 | ACPI_FREE(info); |
diff --git a/drivers/acpi/acpica/tbinstal.c b/drivers/acpi/acpica/tbinstal.c index b661a1e013fb..4dc6108de4ff 100644 --- a/drivers/acpi/acpica/tbinstal.c +++ b/drivers/acpi/acpica/tbinstal.c | |||
@@ -267,8 +267,7 @@ acpi_tb_install_standard_table(acpi_physical_address address, | |||
267 | if (!reload && | 267 | if (!reload && |
268 | acpi_gbl_disable_ssdt_table_install && | 268 | acpi_gbl_disable_ssdt_table_install && |
269 | ACPI_COMPARE_NAME(&new_table_desc.signature, ACPI_SIG_SSDT)) { | 269 | ACPI_COMPARE_NAME(&new_table_desc.signature, ACPI_SIG_SSDT)) { |
270 | ACPI_INFO((AE_INFO, | 270 | ACPI_INFO(("Ignoring installation of %4.4s at %8.8X%8.8X", |
271 | "Ignoring installation of %4.4s at %8.8X%8.8X", | ||
272 | new_table_desc.signature.ascii, | 271 | new_table_desc.signature.ascii, |
273 | ACPI_FORMAT_UINT64(address))); | 272 | ACPI_FORMAT_UINT64(address))); |
274 | goto release_and_exit; | 273 | goto release_and_exit; |
@@ -432,7 +431,7 @@ finish_override: | |||
432 | return; | 431 | return; |
433 | } | 432 | } |
434 | 433 | ||
435 | ACPI_INFO((AE_INFO, "%4.4s 0x%8.8X%8.8X" | 434 | ACPI_INFO(("%4.4s 0x%8.8X%8.8X" |
436 | " %s table override, new table: 0x%8.8X%8.8X", | 435 | " %s table override, new table: 0x%8.8X%8.8X", |
437 | old_table_desc->signature.ascii, | 436 | old_table_desc->signature.ascii, |
438 | ACPI_FORMAT_UINT64(old_table_desc->address), | 437 | ACPI_FORMAT_UINT64(old_table_desc->address), |
diff --git a/drivers/acpi/acpica/tbprint.c b/drivers/acpi/acpica/tbprint.c index fd4146d4ff49..26d61dbace0a 100644 --- a/drivers/acpi/acpica/tbprint.c +++ b/drivers/acpi/acpica/tbprint.c | |||
@@ -132,7 +132,7 @@ acpi_tb_print_table_header(acpi_physical_address address, | |||
132 | 132 | ||
133 | /* FACS only has signature and length fields */ | 133 | /* FACS only has signature and length fields */ |
134 | 134 | ||
135 | ACPI_INFO((AE_INFO, "%-4.4s 0x%8.8X%8.8X %06X", | 135 | ACPI_INFO(("%-4.4s 0x%8.8X%8.8X %06X", |
136 | header->signature, ACPI_FORMAT_UINT64(address), | 136 | header->signature, ACPI_FORMAT_UINT64(address), |
137 | header->length)); | 137 | header->length)); |
138 | } else if (ACPI_VALIDATE_RSDP_SIG(header->signature)) { | 138 | } else if (ACPI_VALIDATE_RSDP_SIG(header->signature)) { |
@@ -144,7 +144,7 @@ acpi_tb_print_table_header(acpi_physical_address address, | |||
144 | ACPI_OEM_ID_SIZE); | 144 | ACPI_OEM_ID_SIZE); |
145 | acpi_tb_fix_string(local_header.oem_id, ACPI_OEM_ID_SIZE); | 145 | acpi_tb_fix_string(local_header.oem_id, ACPI_OEM_ID_SIZE); |
146 | 146 | ||
147 | ACPI_INFO((AE_INFO, "RSDP 0x%8.8X%8.8X %06X (v%.2d %-6.6s)", | 147 | ACPI_INFO(("RSDP 0x%8.8X%8.8X %06X (v%.2d %-6.6s)", |
148 | ACPI_FORMAT_UINT64(address), | 148 | ACPI_FORMAT_UINT64(address), |
149 | (ACPI_CAST_PTR(struct acpi_table_rsdp, header)-> | 149 | (ACPI_CAST_PTR(struct acpi_table_rsdp, header)-> |
150 | revision > | 150 | revision > |
@@ -158,8 +158,7 @@ acpi_tb_print_table_header(acpi_physical_address address, | |||
158 | 158 | ||
159 | acpi_tb_cleanup_table_header(&local_header, header); | 159 | acpi_tb_cleanup_table_header(&local_header, header); |
160 | 160 | ||
161 | ACPI_INFO((AE_INFO, | 161 | ACPI_INFO(("%-4.4s 0x%8.8X%8.8X" |
162 | "%-4.4s 0x%8.8X%8.8X" | ||
163 | " %06X (v%.2d %-6.6s %-8.8s %08X %-4.4s %08X)", | 162 | " %06X (v%.2d %-6.6s %-8.8s %08X %-4.4s %08X)", |
164 | local_header.signature, ACPI_FORMAT_UINT64(address), | 163 | local_header.signature, ACPI_FORMAT_UINT64(address), |
165 | local_header.length, local_header.revision, | 164 | local_header.length, local_header.revision, |
diff --git a/drivers/acpi/acpica/tbutils.c b/drivers/acpi/acpica/tbutils.c index 3269bef371d7..9240c76d2823 100644 --- a/drivers/acpi/acpica/tbutils.c +++ b/drivers/acpi/acpica/tbutils.c | |||
@@ -174,9 +174,7 @@ struct acpi_table_header *acpi_tb_copy_dsdt(u32 table_index) | |||
174 | ACPI_TABLE_ORIGIN_INTERNAL_VIRTUAL, | 174 | ACPI_TABLE_ORIGIN_INTERNAL_VIRTUAL, |
175 | new_table); | 175 | new_table); |
176 | 176 | ||
177 | ACPI_INFO((AE_INFO, | 177 | ACPI_INFO(("Forced DSDT copy: length 0x%05X copied locally, original unmapped", new_table->length)); |
178 | "Forced DSDT copy: length 0x%05X copied locally, original unmapped", | ||
179 | new_table->length)); | ||
180 | 178 | ||
181 | return (new_table); | 179 | return (new_table); |
182 | } | 180 | } |
diff --git a/drivers/acpi/acpica/tbxfload.c b/drivers/acpi/acpica/tbxfload.c index 278666e39563..c89943b3ff3c 100644 --- a/drivers/acpi/acpica/tbxfload.c +++ b/drivers/acpi/acpica/tbxfload.c | |||
@@ -206,9 +206,7 @@ acpi_status acpi_tb_load_namespace(void) | |||
206 | } | 206 | } |
207 | 207 | ||
208 | if (!tables_failed) { | 208 | if (!tables_failed) { |
209 | ACPI_INFO((AE_INFO, | 209 | ACPI_INFO(("%u ACPI AML tables successfully acquired and loaded\n", tables_loaded)); |
210 | "%u ACPI AML tables successfully acquired and loaded\n", | ||
211 | tables_loaded)); | ||
212 | } else { | 210 | } else { |
213 | ACPI_ERROR((AE_INFO, | 211 | ACPI_ERROR((AE_INFO, |
214 | "%u table load failures, %u successful", | 212 | "%u table load failures, %u successful", |
@@ -301,7 +299,7 @@ acpi_status acpi_load_table(struct acpi_table_header *table) | |||
301 | 299 | ||
302 | /* Install the table and load it into the namespace */ | 300 | /* Install the table and load it into the namespace */ |
303 | 301 | ||
304 | ACPI_INFO((AE_INFO, "Host-directed Dynamic ACPI Table Load:")); | 302 | ACPI_INFO(("Host-directed Dynamic ACPI Table Load:")); |
305 | (void)acpi_ut_acquire_mutex(ACPI_MTX_TABLES); | 303 | (void)acpi_ut_acquire_mutex(ACPI_MTX_TABLES); |
306 | 304 | ||
307 | status = acpi_tb_install_standard_table(ACPI_PTR_TO_PHYSADDR(table), | 305 | status = acpi_tb_install_standard_table(ACPI_PTR_TO_PHYSADDR(table), |
diff --git a/drivers/acpi/acpica/uttrack.c b/drivers/acpi/acpica/uttrack.c index c7c2bb8f3559..60c406a8efcb 100644 --- a/drivers/acpi/acpica/uttrack.c +++ b/drivers/acpi/acpica/uttrack.c | |||
@@ -712,7 +712,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module) | |||
712 | /* Print summary */ | 712 | /* Print summary */ |
713 | 713 | ||
714 | if (!num_outstanding) { | 714 | if (!num_outstanding) { |
715 | ACPI_INFO((AE_INFO, "No outstanding allocations")); | 715 | ACPI_INFO(("No outstanding allocations")); |
716 | } else { | 716 | } else { |
717 | ACPI_ERROR((AE_INFO, "%u(0x%X) Outstanding allocations", | 717 | ACPI_ERROR((AE_INFO, "%u(0x%X) Outstanding allocations", |
718 | num_outstanding, num_outstanding)); | 718 | num_outstanding, num_outstanding)); |
diff --git a/drivers/acpi/acpica/utxferror.c b/drivers/acpi/acpica/utxferror.c index 6fe59597b599..d9f15cbcd8a0 100644 --- a/drivers/acpi/acpica/utxferror.c +++ b/drivers/acpi/acpica/utxferror.c | |||
@@ -175,8 +175,7 @@ ACPI_EXPORT_SYMBOL(acpi_warning) | |||
175 | * TBD: module_name and line_number args are not needed, should be removed. | 175 | * TBD: module_name and line_number args are not needed, should be removed. |
176 | * | 176 | * |
177 | ******************************************************************************/ | 177 | ******************************************************************************/ |
178 | void ACPI_INTERNAL_VAR_XFACE | 178 | void ACPI_INTERNAL_VAR_XFACE acpi_info(const char *format, ...) |
179 | acpi_info(const char *module_name, u32 line_number, const char *format, ...) | ||
180 | { | 179 | { |
181 | va_list arg_list; | 180 | va_list arg_list; |
182 | 181 | ||
diff --git a/include/acpi/acpixf.h b/include/acpi/acpixf.h index c96621e87c19..17556979dc79 100644 --- a/include/acpi/acpixf.h +++ b/include/acpi/acpixf.h | |||
@@ -897,11 +897,9 @@ ACPI_MSG_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(3) | |||
897 | acpi_warning(const char *module_name, | 897 | acpi_warning(const char *module_name, |
898 | u32 line_number, | 898 | u32 line_number, |
899 | const char *format, ...)) | 899 | const char *format, ...)) |
900 | ACPI_MSG_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(3) | 900 | ACPI_MSG_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(1) |
901 | void ACPI_INTERNAL_VAR_XFACE | 901 | void ACPI_INTERNAL_VAR_XFACE |
902 | acpi_info(const char *module_name, | 902 | acpi_info(const char *format, ...)) |
903 | u32 line_number, | ||
904 | const char *format, ...)) | ||
905 | ACPI_MSG_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(3) | 903 | ACPI_MSG_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(3) |
906 | void ACPI_INTERNAL_VAR_XFACE | 904 | void ACPI_INTERNAL_VAR_XFACE |
907 | acpi_bios_error(const char *module_name, | 905 | acpi_bios_error(const char *module_name, |