[PATCH][RESUBMIT] acpi: acpidump: remove unnecessary variable n

Keng-Yu Lin kengyu at canonical.com
Wed Apr 11 08:17:47 UTC 2012


On Wed, Apr 11, 2012 at 4:08 PM, Colin King <colin.king at canonical.com> wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/acpi/acpidump/acpidump.c |    5 ++---
>  1 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/src/acpi/acpidump/acpidump.c b/src/acpi/acpidump/acpidump.c
> index 6b20ca7..28a43b0 100644
> --- a/src/acpi/acpidump/acpidump.c
> +++ b/src/acpi/acpidump/acpidump.c
> @@ -665,7 +665,6 @@ static void acpidump_xsdt(fwts_framework *fw, uint8_t *data, int length)
>  static void acpidump_madt(fwts_framework *fw, uint8_t *data, int length)
>  {
>        int i = 0;
> -       int n;
>        int offset = 0;
>
>        fwts_acpidump_field fields[] = {
> @@ -788,8 +787,8 @@ static void acpidump_madt(fwts_framework *fw, uint8_t *data, int length)
>                                };
>                                fwts_log_info_verbatum(fw, " Local SAPIC:");
>                                __acpi_dump_table_fields(fw, data, fields_madt_local_sapic, offset);
> -                               n = strlen(local_sapic->uid_string) + 1;
> -                               skip = (sizeof(fwts_acpi_madt_local_sapic) + n);
> +                               skip = (sizeof(fwts_acpi_madt_local_sapic) +
> +                                       strlen(local_sapic->uid_string) + 1);
>                        }
>                        break;
>                case 8: {
> --
> 1.7.9.1
>
Acked-by: Keng-Yu Lin <kengyu at canonical.com>


More information about the fwts-devel mailing list