ACK: [PATCH] acpi: madt: reduce scope of variable 'listint'

ivanhu ivan.hu at canonical.com
Thu Aug 18 02:40:02 UTC 2016



On 2016年07月26日 04:54, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Minor change to fix cppcheck style warnings, no functional change
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/acpi/madt/madt.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/acpi/madt/madt.c b/src/acpi/madt/madt.c
> index 3505f62..b087d27 100644
> --- a/src/acpi/madt/madt.c
> +++ b/src/acpi/madt/madt.c
> @@ -303,10 +303,9 @@ static ACPI_OBJECT_TYPE madt_find_processor_uid(fwts_framework *fw,
>  {
>  	char table_label[64];
>  	fwts_list_link *item;
> -	struct acpi_integer *listint;
>
>  	fwts_list_foreach(item, &processor_uids) {
> -		listint = fwts_list_data(struct acpi_integer *, item);
> +		struct acpi_integer *listint = fwts_list_data(struct acpi_integer *, item);
>  		if (uid == listint->value) {
>  			fwts_passed(fw, "MADT %s has matching processor "
>  				    "UID %" PRIu64 ".", table_name, uid);
>

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



More information about the fwts-devel mailing list