| Message ID | 1370644273-10495-6-git-send-email-yinghai@kernel.org |
|---|---|
| State | Not Applicable |
| Headers | show |
On Fri, Jun 07, 2013 at 03:30:51PM -0700, Yinghai Lu wrote:
> All others are using "-" instead of "_".
Who are "all others"? According to my grep it is 121 vs 44. So even without
your two here we still 42 doing it the other way around so it is not all.
However it might make sense to use _ instead of " " within a chip's name and
- as a delimiter for modes like edge or level. However that is way beyond the
scope of this patch and I am not sure if anyone finds this usefull at all.
Saying this, I am fine with this patch :)
Sebastian
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On Sun, Jun 9, 2013 at 8:16 AM, Sebastian Andrzej Siewior <sebastian@breakpoint.cc> wrote: > On Fri, Jun 07, 2013 at 03:30:51PM -0700, Yinghai Lu wrote: >> All others are using "-" instead of "_". > > Who are "all others"? According to my grep it is 121 vs 44. So even without > your two here we still 42 doing it the other way around so it is not all. I mean in arch/x86 arch/x86/kernel/apic/io_apic.c:struct irq_chip ioapic_chip __read_mostly = { arch/x86/kernel/apic/io_apic.c- .name = "IO-APIC", -- arch/x86/kernel/apic/io_apic.c:static struct irq_chip lapic_chip __read_mostly = { arch/x86/kernel/apic/io_apic.c- .name = "local-APIC", -- arch/x86/kernel/apic/io_apic.c:struct irq_chip msi_chip = { arch/x86/kernel/apic/io_apic.c- .name = "PCI-MSI", -- arch/x86/kernel/apic/io_apic.c:static struct irq_chip dmar_msi_type = { arch/x86/kernel/apic/io_apic.c- .name = "DMAR_MSI", -- arch/x86/kernel/apic/io_apic.c:struct irq_chip hpet_msi_type = { arch/x86/kernel/apic/io_apic.c- .name = "HPET_MSI", -- arch/x86/kernel/apic/io_apic.c:static struct irq_chip ht_irq_chip = { arch/x86/kernel/apic/io_apic.c- .name = "PCI-HT", -- arch/x86/kernel/i8259.c:struct irq_chip i8259A_chip = { arch/x86/kernel/i8259.c- .name = "XT-PIC", -- arch/x86/lguest/boot.c:static struct irq_chip lguest_irq_controller = { arch/x86/lguest/boot.c- .name = "lguest", -- arch/x86/platform/uv/uv_irq.c:static struct irq_chip uv_irq_chip = { arch/x86/platform/uv/uv_irq.c- .name = "UV-CORE", -- arch/x86/platform/visws/visws_quirks.c:static struct irq_chip cobalt_irq_type = { arch/x86/platform/visws/visws_quirks.c- .name = "Cobalt-APIC", -- arch/x86/platform/visws/visws_quirks.c:static struct irq_chip piix4_master_irq_type = { arch/x86/platform/visws/visws_quirks.c- .name = "PIIX4-master", -- arch/x86/platform/visws/visws_quirks.c:static struct irq_chip piix4_virtual_irq_type = { arch/x86/platform/visws/visws_quirks.c- .name = "PIIX4-virtual", -- > > However it might make sense to use _ instead of " " within a chip's name and > - as a delimiter for modes like edge or level. However that is way beyond the > scope of this patch and I am not sure if anyone finds this usefull at all. > > Saying this, I am fine with this patch :) ok. Thanks Yinghai -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c index f8d4d8d..436fb38 100644 --- a/arch/x86/kernel/apic/io_apic.c +++ b/arch/x86/kernel/apic/io_apic.c @@ -3199,7 +3199,7 @@ dmar_msi_set_affinity(struct irq_data *data, const struct cpumask *mask, } static struct irq_chip dmar_msi_type = { - .name = "DMAR_MSI", + .name = "DMAR-MSI", .irq_unmask = dmar_msi_unmask, .irq_mask = dmar_msi_mask, .irq_ack = ack_apic_edge, @@ -3247,7 +3247,7 @@ static int hpet_msi_set_affinity(struct irq_data *data, } struct irq_chip hpet_msi_type = { - .name = "HPET_MSI", + .name = "HPET-MSI", .irq_unmask = hpet_msi_unmask, .irq_mask = hpet_msi_mask, .irq_ack = ack_apic_edge,
All others are using "-" instead of "_". Change dmar_msi and hpet_msi to use "-". Signed-off-by: Yinghai Lu <yinghai@kernel.org> Cc: Joerg Roedel <joro@8bytes.org> Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Cc: Sebastian Andrzej Siewior <sebastian@breakpoint.cc> --- arch/x86/kernel/apic/io_apic.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)