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

ivanhu ivan.hu at canonical.com
Fri Jun 30 02:31:20 UTC 2023


Thanks!

Acked-by: Ivan Hu <ivan.hu at 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 at 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
> 
> 



More information about the fwts-devel mailing list