diff options
author | Lv Zheng <lv.zheng@intel.com> | 2016-12-14 02:04:39 -0500 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-12-20 20:36:38 -0500 |
commit | 6b11d1d677132816252004426ef220ccd3c92d2f (patch) | |
tree | a512975fe6a4602264de5cbccd8926d0c121a0e6 /drivers/mailbox | |
parent | 66360faa4333babc53836c7b59a0cff68cb0a9c6 (diff) |
ACPI / osl: Remove acpi_get_table_with_size()/early_acpi_os_unmap_memory() users
This patch removes the users of the deprectated APIs:
acpi_get_table_with_size()
early_acpi_os_unmap_memory()
The following APIs should be used instead of:
acpi_get_table()
acpi_put_table()
The deprecated APIs are invented to be a replacement of acpi_get_table()
during the early stage so that the early mapped pointer will not be stored
in ACPICA core and thus the late stage acpi_get_table() won't return a
wrong pointer. The mapping size is returned just because it is required by
early_acpi_os_unmap_memory() to unmap the pointer during early stage.
But as the mapping size equals to the acpi_table_header.length
(see acpi_tb_init_table_descriptor() and acpi_tb_validate_table()), when
such a convenient result is returned, driver code will start to use it
instead of accessing acpi_table_header to obtain the length.
Thus this patch cleans up the drivers by replacing returned table size with
acpi_table_header.length, and should be a no-op.
Reported-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/mailbox')
-rw-r--r-- | drivers/mailbox/pcc.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/mailbox/pcc.c b/drivers/mailbox/pcc.c index 08c87fadca8c..0661987fdc1a 100644 --- a/drivers/mailbox/pcc.c +++ b/drivers/mailbox/pcc.c | |||
@@ -446,7 +446,6 @@ static int pcc_parse_subspace_irq(int id, | |||
446 | */ | 446 | */ |
447 | static int __init acpi_pcc_probe(void) | 447 | static int __init acpi_pcc_probe(void) |
448 | { | 448 | { |
449 | acpi_size pcct_tbl_header_size; | ||
450 | struct acpi_table_header *pcct_tbl; | 449 | struct acpi_table_header *pcct_tbl; |
451 | struct acpi_subtable_header *pcct_entry; | 450 | struct acpi_subtable_header *pcct_entry; |
452 | struct acpi_table_pcct *acpi_pcct_tbl; | 451 | struct acpi_table_pcct *acpi_pcct_tbl; |
@@ -455,9 +454,7 @@ static int __init acpi_pcc_probe(void) | |||
455 | acpi_status status = AE_OK; | 454 | acpi_status status = AE_OK; |
456 | 455 | ||
457 | /* Search for PCCT */ | 456 | /* Search for PCCT */ |
458 | status = acpi_get_table_with_size(ACPI_SIG_PCCT, 0, | 457 | status = acpi_get_table(ACPI_SIG_PCCT, 0, &pcct_tbl); |
459 | &pcct_tbl, | ||
460 | &pcct_tbl_header_size); | ||
461 | 458 | ||
462 | if (ACPI_FAILURE(status) || !pcct_tbl) { | 459 | if (ACPI_FAILURE(status) || !pcct_tbl) { |
463 | pr_warn("PCCT header not found.\n"); | 460 | pr_warn("PCCT header not found.\n"); |