diff mbox series

[4/4] fwts_acpica: Remove unused variable "context"

Message ID 20230628204235.7141-5-W_Armin@gmx.de
State Accepted
Headers show
Series fwts: Fix various WMI issues | expand

Commit Message

Armin Wolf June 28, 2023, 8:42 p.m. UTC
The variable "context" is unused since commit 06e1d185d294
and thus triggers a compiler warning. Remove it to fix this
warning.

Fixes: 06e1d185d294 ("fwts_acpica: skip unneccessary null check")
Signed-off-by: Armin Wolf <W_Armin@gmx.de>
---
 src/libfwtsacpica/fwts_acpica.c | 3 ---
 1 file changed, 3 deletions(-)

--
2.30.2

Comments

ivanhu June 30, 2023, 2:31 a.m. UTC | #1
Thanks!

Acked-by: Ivan Hu <ivan.hu@canonical.com>

On 6/29/23 04:42, Armin Wolf wrote:
> The variable "context" is unused since commit 06e1d185d294
> and thus triggers a compiler warning. Remove it to fix this
> warning.
> 
> Fixes: 06e1d185d294 ("fwts_acpica: skip unneccessary null check")
> Signed-off-by: Armin Wolf <W_Armin@gmx.de>
> ---
>   src/libfwtsacpica/fwts_acpica.c | 3 ---
>   1 file changed, 3 deletions(-)
> 
> diff --git a/src/libfwtsacpica/fwts_acpica.c b/src/libfwtsacpica/fwts_acpica.c
> index d8bc54a6..a9ac8a75 100644
> --- a/src/libfwtsacpica/fwts_acpica.c
> +++ b/src/libfwtsacpica/fwts_acpica.c
> @@ -262,7 +262,6 @@ static ACPI_STATUS fwts_region_handler(
>   	UINT8                   *buffer = ACPI_CAST_PTR(UINT8, value);
>   	ACPI_SIZE               length;
>   	UINT32                  bytewidth;
> -	ACPI_CONNECTION_INFO    *context;
>   	int			i;
> 
>   	if (!regionobject)
> @@ -274,8 +273,6 @@ static ACPI_STATUS fwts_region_handler(
> 
>   	fwts_acpi_region_handler_called_set(true);
> 
> -	context = ACPI_CAST_PTR (ACPI_CONNECTION_INFO, handlercontext);
> -
>   	switch (regionobject->Region.SpaceId) {
>   	case ACPI_ADR_SPACE_SYSTEM_IO:
>   		switch (function & ACPI_IO_MASK) {
> --
> 2.30.2
> 
>
diff mbox series

Patch

diff --git a/src/libfwtsacpica/fwts_acpica.c b/src/libfwtsacpica/fwts_acpica.c
index d8bc54a6..a9ac8a75 100644
--- a/src/libfwtsacpica/fwts_acpica.c
+++ b/src/libfwtsacpica/fwts_acpica.c
@@ -262,7 +262,6 @@  static ACPI_STATUS fwts_region_handler(
 	UINT8                   *buffer = ACPI_CAST_PTR(UINT8, value);
 	ACPI_SIZE               length;
 	UINT32                  bytewidth;
-	ACPI_CONNECTION_INFO    *context;
 	int			i;

 	if (!regionobject)
@@ -274,8 +273,6 @@  static ACPI_STATUS fwts_region_handler(

 	fwts_acpi_region_handler_called_set(true);

-	context = ACPI_CAST_PTR (ACPI_CONNECTION_INFO, handlercontext);
-
 	switch (regionobject->Region.SpaceId) {
 	case ACPI_ADR_SPACE_SYSTEM_IO:
 		switch (function & ACPI_IO_MASK) {