diff options
| author |   Anssi Hannula <anssi.hannula@iki.fi> | 2011-02-20 20:07:21 +0200 |
|---|---|---|
| committer |   Matthew Garrett <mjg@redhat.com> | 2011-03-28 06:07:18 -0400 |
| commit | 25bb067a08c5db70cd8bcf9e160ac81718ea075c (patch) | |
| tree | 58241bf2f9f787e70bbd02615fa04d5efd560cf2 /drivers/platform/x86/hp-wmi.c | |
| parent | hp-wmi: check query return value in hp_wmi_perform_query (diff) | |
| download | linux-dev-25bb067a08c5db70cd8bcf9e160ac81718ea075c.tar.xz linux-dev-25bb067a08c5db70cd8bcf9e160ac81718ea075c.zip | |
hp-wmi: remove a variable that is never read
Remove the status variable from hp_wmi_perform_query which holds the
return value from wmi_evaluate_method(). It is never checked as the
function bails out if the output buffer hasn't been allocated which
indicates the call failed.
Signed-off-by: Anssi Hannula <anssi.hannula@iki.fi>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
Diffstat (limited to 'drivers/platform/x86/hp-wmi.c')
| -rw-r--r-- | drivers/platform/x86/hp-wmi.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/platform/x86/hp-wmi.c b/drivers/platform/x86/hp-wmi.c index 87dfb2489762..8e27c27900a3 100644 --- a/drivers/platform/x86/hp-wmi.c +++ b/drivers/platform/x86/hp-wmi.c @@ -153,7 +153,6 @@ static int hp_wmi_perform_query(int query, int write, u32 *buffer, int buffersize) { struct bios_return bios_return; - acpi_status status; union acpi_object *obj; struct bios_args args = { .signature = 0x55434553, @@ -165,7 +164,7 @@ static int hp_wmi_perform_query(int query, int write, u32 *buffer, struct acpi_buffer input = { sizeof(struct bios_args), &args }; struct acpi_buffer output = { ACPI_ALLOCATE_BUFFER, NULL }; - status = wmi_evaluate_method(HPWMI_BIOS_GUID, 0, 0x3, &input, &output); + wmi_evaluate_method(HPWMI_BIOS_GUID, 0, 0x3, &input, &output); obj = output.pointer; |