Skip to content

Commit

Permalink
ACPI: pfr_update: use ACPI_FREE() to free acpi_object
Browse files Browse the repository at this point in the history
acpi_evaluate_dsm_typed()/acpi_evaluate_dsm() should be coupled with
ACPI_FREE() to free the ACPI memory, because we need to track the
allocation of acpi_object when ACPI_DBG_TRACK_ALLOCATIONS enabled,
so use ACPI_FREE() instead of kfree().

Fixes: 0db89fa ("ACPI: Introduce Platform Firmware Runtime Update device driver")
Signed-off-by: Wang ShaoBo <bobo.shaobowang@huawei.com>
Reviewed-by: Chen Yu <yu.c.chen@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
  • Loading branch information
Wang ShaoBo authored and Rafael J. Wysocki committed Nov 23, 2022
1 parent 0f2aa7f commit e335bee
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions drivers/acpi/pfr_update.c
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ static int query_capability(struct pfru_update_cap_info *cap_hdr,
ret = 0;

free_acpi_buffer:
kfree(out_obj);
ACPI_FREE(out_obj);

return ret;
}
Expand Down Expand Up @@ -224,7 +224,7 @@ static int query_buffer(struct pfru_com_buf_info *info,
ret = 0;

free_acpi_buffer:
kfree(out_obj);
ACPI_FREE(out_obj);

return ret;
}
Expand Down Expand Up @@ -385,7 +385,7 @@ static int start_update(int action, struct pfru_device *pfru_dev)
ret = 0;

free_acpi_buffer:
kfree(out_obj);
ACPI_FREE(out_obj);

return ret;
}
Expand Down

0 comments on commit e335bee

Please sign in to comment.