[PATCH 017/114] x86, genapic: cleanup 32-bit apic_summit template

From: Ingo Molnar
Date: Wed Jan 28 2009 - 18:50:54 EST


Clean up the APIC driver template:

- order fields properly
- use the macro names explicitly (so that they can be renamed later)
- fill in NULL entries as well

Signed-off-by: Ingo Molnar <mingo@xxxxxxx>
---
arch/x86/mach-generic/summit.c | 58 +++++++++++++++++++++++++++++++++++++++-
1 files changed, 57 insertions(+), 1 deletions(-)

diff --git a/arch/x86/mach-generic/summit.c b/arch/x86/mach-generic/summit.c
index 2821ffc..479c1d4 100644
--- a/arch/x86/mach-generic/summit.c
+++ b/arch/x86/mach-generic/summit.c
@@ -37,4 +37,60 @@ static void vector_allocation_domain(int cpu, cpumask_t *retmask)
*retmask = (cpumask_t){ { [0] = APIC_ALL_CPUS, } };
}

-struct genapic apic_summit = APIC_INIT("summit", probe_summit);
+struct genapic apic_summit = {
+
+ .name = "summit",
+ .probe = probe_summit,
+ .acpi_madt_oem_check = acpi_madt_oem_check,
+ .apic_id_registered = apic_id_registered,
+
+ .int_delivery_mode = INT_DELIVERY_MODE,
+ .int_dest_mode = INT_DEST_MODE,
+
+ .target_cpus = target_cpus,
+ .ESR_DISABLE = esr_disable,
+ .apic_destination_logical = APIC_DEST_LOGICAL,
+ .check_apicid_used = check_apicid_used,
+ .check_apicid_present = check_apicid_present,
+
+ .no_balance_irq = NO_BALANCE_IRQ,
+ .no_ioapic_check = 0,
+
+ .vector_allocation_domain = vector_allocation_domain,
+ .init_apic_ldr = init_apic_ldr,
+
+ .ioapic_phys_id_map = ioapic_phys_id_map,
+ .setup_apic_routing = setup_apic_routing,
+ .multi_timer_check = multi_timer_check,
+ .apicid_to_node = apicid_to_node,
+ .cpu_to_logical_apicid = cpu_to_logical_apicid,
+ .cpu_present_to_apicid = cpu_present_to_apicid,
+ .apicid_to_cpu_present = apicid_to_cpu_present,
+ .setup_portio_remap = setup_portio_remap,
+ .check_phys_apicid_present = check_phys_apicid_present,
+ .enable_apic_mode = enable_apic_mode,
+ .phys_pkg_id = phys_pkg_id,
+ .mps_oem_check = mps_oem_check,
+
+ .get_apic_id = get_apic_id,
+ .set_apic_id = NULL,
+ .apic_id_mask = APIC_ID_MASK,
+
+ .cpu_mask_to_apicid = cpu_mask_to_apicid,
+ .cpu_mask_to_apicid_and = cpu_mask_to_apicid_and,
+
+ .send_IPI_mask = send_IPI_mask,
+ .send_IPI_mask_allbutself = NULL,
+ .send_IPI_allbutself = send_IPI_allbutself,
+ .send_IPI_all = send_IPI_all,
+ .send_IPI_self = NULL,
+
+ .wakeup_cpu = NULL,
+ .trampoline_phys_low = TRAMPOLINE_PHYS_LOW,
+ .trampoline_phys_high = TRAMPOLINE_PHYS_HIGH,
+ .wait_for_init_deassert = wait_for_init_deassert,
+ .smp_callin_clear_local_apic = smp_callin_clear_local_apic,
+ .store_NMI_vector = store_NMI_vector,
+ .restore_NMI_vector = restore_NMI_vector,
+ .inquire_remote_apic = inquire_remote_apic,
+};
--
1.6.0.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/