ACPI: HMAT: remove unnecessary variable initialization
authorVishal Verma <vishal.l.verma@intel.com>
Wed, 16 Nov 2022 23:37:36 +0000 (16:37 -0700)
committerDan Williams <dan.j.williams@intel.com>
Sat, 19 Nov 2022 07:55:40 +0000 (23:55 -0800)
In hmat_register_target_initiators(), the variable 'best' gets
initialized in the outer per-locality-type for loop. The initialization
just before setting up 'Access 1' targets was unnecessary. Remove it.

Cc: Rafael J. Wysocki <rafael@kernel.org>
Cc: Liu Shixin <liushixin2@huawei.com>
Cc: Dan Williams <dan.j.williams@intel.com>
Acked-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
Link: https://lore.kernel.org/r/20221116-acpi_hmat_fix-v2-1-3712569be691@intel.com
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
drivers/acpi/numa/hmat.c

index 23f49a2..144a84f 100644 (file)
@@ -644,7 +644,6 @@ static void hmat_register_target_initiators(struct memory_target *target)
        /* Access 1 ignores Generic Initiators */
        bitmap_zero(p_nodes, MAX_NUMNODES);
        list_sort(p_nodes, &initiators, initiator_cmp);
-       best = 0;
        for (i = WRITE_LATENCY; i <= READ_BANDWIDTH; i++) {
                loc = localities_types[i];
                if (!loc)