2015-04-27 12:51:18

by Krzysztof Kozłowski

[permalink] [raw]
Subject: [PATCH] ARM: OMAP: Constify irq_domain_ops

The irq_domain_ops are not modified by the driver and the irqdomain core
code accepts pointer to a const data.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm/mach-omap2/omap-wakeupgen.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-omap2/omap-wakeupgen.c b/arch/arm/mach-omap2/omap-wakeupgen.c
index 3b56722dfd8a..8e52621b5a6b 100644
--- a/arch/arm/mach-omap2/omap-wakeupgen.c
+++ b/arch/arm/mach-omap2/omap-wakeupgen.c
@@ -444,7 +444,7 @@ static int wakeupgen_domain_alloc(struct irq_domain *domain,
return irq_domain_alloc_irqs_parent(domain, virq, nr_irqs, &parent_args);
}

-static struct irq_domain_ops wakeupgen_domain_ops = {
+static const struct irq_domain_ops wakeupgen_domain_ops = {
.xlate = wakeupgen_domain_xlate,
.alloc = wakeupgen_domain_alloc,
.free = irq_domain_free_irqs_common,
--
2.1.0


2015-05-20 17:14:41

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH] ARM: OMAP: Constify irq_domain_ops

* Krzysztof Kozlowski <[email protected]> [150427 05:52]:
> The irq_domain_ops are not modified by the driver and the irqdomain core
> code accepts pointer to a const data.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Applying into omap-for-v4.2/cleanup thanks.

Tony

> ---
> arch/arm/mach-omap2/omap-wakeupgen.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-omap2/omap-wakeupgen.c b/arch/arm/mach-omap2/omap-wakeupgen.c
> index 3b56722dfd8a..8e52621b5a6b 100644
> --- a/arch/arm/mach-omap2/omap-wakeupgen.c
> +++ b/arch/arm/mach-omap2/omap-wakeupgen.c
> @@ -444,7 +444,7 @@ static int wakeupgen_domain_alloc(struct irq_domain *domain,
> return irq_domain_alloc_irqs_parent(domain, virq, nr_irqs, &parent_args);
> }
>
> -static struct irq_domain_ops wakeupgen_domain_ops = {
> +static const struct irq_domain_ops wakeupgen_domain_ops = {
> .xlate = wakeupgen_domain_xlate,
> .alloc = wakeupgen_domain_alloc,
> .free = irq_domain_free_irqs_common,
> --
> 2.1.0
>