ACPI: NUMA: Remove the useless sub table pointer check
authorHanjun Guo <guohanjun@huawei.com>
Tue, 21 Jul 2020 09:59:05 +0000 (17:59 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 27 Jul 2020 13:19:12 +0000 (15:19 +0200)
In acpi_parse_entries_array(), the subtable entries (entry.hdr)
will never be NULL, so for ACPI subtable handler in struct
acpi_subtable_proc, will never handle NULL subtable entries.

Remove those useless subtable pointer checks in the callback
handlers.

Signed-off-by: Hanjun Guo <guohanjun@huawei.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/numa/srat.c

index 5be5a97..3d430b0 100644 (file)
@@ -291,8 +291,6 @@ acpi_parse_x2apic_affinity(union acpi_subtable_headers *header,
        struct acpi_srat_x2apic_cpu_affinity *processor_affinity;
 
        processor_affinity = (struct acpi_srat_x2apic_cpu_affinity *)header;
-       if (!processor_affinity)
-               return -EINVAL;
 
        acpi_table_print_srat_entry(&header->common);
 
@@ -309,8 +307,6 @@ acpi_parse_processor_affinity(union acpi_subtable_headers *header,
        struct acpi_srat_cpu_affinity *processor_affinity;
 
        processor_affinity = (struct acpi_srat_cpu_affinity *)header;
-       if (!processor_affinity)
-               return -EINVAL;
 
        acpi_table_print_srat_entry(&header->common);
 
@@ -327,8 +323,6 @@ acpi_parse_gicc_affinity(union acpi_subtable_headers *header,
        struct acpi_srat_gicc_affinity *processor_affinity;
 
        processor_affinity = (struct acpi_srat_gicc_affinity *)header;
-       if (!processor_affinity)
-               return -EINVAL;
 
        acpi_table_print_srat_entry(&header->common);
 
@@ -347,8 +341,6 @@ acpi_parse_memory_affinity(union acpi_subtable_headers * header,
        struct acpi_srat_mem_affinity *memory_affinity;
 
        memory_affinity = (struct acpi_srat_mem_affinity *)header;
-       if (!memory_affinity)
-               return -EINVAL;
 
        acpi_table_print_srat_entry(&header->common);