lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 04 Dec 2013 23:38:56 +0800 From: Hanjun Guo <hanjun.guo@...aro.org> To: Rob Herring <robherring2@...il.com> CC: "Rafael J. Wysocki" <rjw@...ysocki.net>, Catalin Marinas <catalin.marinas@....com>, Will Deacon <will.deacon@....com>, Russell King - ARM Linux <linux@....linux.org.uk>, Daniel Lezcano <daniel.lezcano@...aro.org>, Mark Rutland <mark.rutland@....com>, Matthew Garrett <mjg59@...f.ucam.org>, "linaro-kernel@...ts.linaro.org" <linaro-kernel@...ts.linaro.org>, Linaro Patches <patches@...aro.org>, Olof Johansson <olof@...om.net>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Rob Herring <rob.herring@...xeda.com>, linaro-acpi@...ts.linaro.org, linux-acpi@...r.kernel.org, Amit Daniel Kachhap <amit.daniel@...sung.com>, Grant Likely <grant.likely@...aro.org>, Bjorn Helgaas <bhelgaas@...gle.com>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org> Subject: Re: [RFC part2 PATCH 7/9] irqdomain: Add a new API irq_create_acpi_mapping() On 2013年12月04日 01:25, Rob Herring wrote: > On Tue, Dec 3, 2013 at 10:39 AM, Hanjun Guo <hanjun.guo@...aro.org> wrote: >> From: Amit Daniel Kachhap <amit.daniel@...sung.com> >> >> This patch introduces a new API for acpi based irq mapping. >> >> [hanjun: Rework this patch to delete the reference to >> gic_irq_domain_xlate() which can simplify the code a lot.] >> >> Signed-off-by: Amit Daniel Kachhap <amit.daniel@...sung.com> >> Signed-off-by: Hanjun Guo <hanjun.guo@...aro.org> >> --- >> include/linux/acpi.h | 2 ++ >> kernel/irq/irqdomain.c | 27 +++++++++++++++++++++++++++ >> 2 files changed, 29 insertions(+) >> >> diff --git a/include/linux/acpi.h b/include/linux/acpi.h >> index 1e6a0ac..edd5806 100644 >> --- a/include/linux/acpi.h >> +++ b/include/linux/acpi.h >> @@ -141,6 +141,8 @@ int acpi_map_lsapic(acpi_handle handle, int physid, int *pcpu); >> int acpi_unmap_lsapic(int cpu); >> #endif /* CONFIG_ACPI_HOTPLUG_CPU */ >> >> +unsigned int irq_create_acpi_mapping(irq_hw_number_t hwirq, >> + unsigned int type); >> int acpi_register_ioapic(acpi_handle handle, u64 phys_addr, u32 gsi_base); >> int acpi_unregister_ioapic(acpi_handle handle, u32 gsi_base); >> void acpi_irq_stats_init(void); >> diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c >> index cf68bb3..c661552 100644 >> --- a/kernel/irq/irqdomain.c >> +++ b/kernel/irq/irqdomain.c >> @@ -501,6 +501,33 @@ unsigned int irq_create_of_mapping(struct of_phandle_args *irq_data) >> } >> EXPORT_SYMBOL_GPL(irq_create_of_mapping); >> >> +#ifdef CONFIG_ACPI >> +unsigned int irq_create_acpi_mapping(irq_hw_number_t hwirq, >> + unsigned int type) >> +{ >> + struct irq_domain *domain; >> + unsigned int virq; >> + >> + domain = irq_default_domain; >> + if (!domain) { >> + pr_warn("no irq domain found !\n"); >> + return 0; >> + } >> + >> + /* Create mapping */ >> + virq = irq_create_mapping(domain, hwirq); >> + if (!virq) >> + return virq; >> + >> + /* Set type if specified and different than the current one */ >> + if (type != IRQ_TYPE_NONE && >> + type != irq_get_trigger_type(virq)) >> + irq_set_irq_type(virq, type); >> + return virq; >> +} >> +EXPORT_SYMBOL_GPL(irq_create_acpi_mapping); > There is nothing ACPI specific about this function. This is simply > irq_create_of_mapping w/o translating of_phandle_args to a hwirq and > type. So I expect the code to be re-factored here to mirror that. Sorry for my bad english, do you mean create a OF free function and call that from the OF function ? Thanks Hanjun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists