On 02/22/2012 06:14 AM, Thomas Abraham wrote:
> Add device tree support for gpio wakeup source interrupt controller
> on Exynos4.
> 
> Cc: Rob Herring <[email protected]>
> Cc: Grant Likely <[email protected]>
> Signed-off-by: Thomas Abraham <[email protected]>

For the series:

Acked-by: Rob Herring <[email protected]>

Rob

> ---
>  .../bindings/arm/samsung/wakeup-eint.txt           |   37 +++++++++++++++++
>  arch/arm/mach-exynos/common.c                      |   43 +++++++++++++------
>  2 files changed, 66 insertions(+), 14 deletions(-)
>  create mode 100644 
> Documentation/devicetree/bindings/arm/samsung/wakeup-eint.txt
> 
> diff --git a/Documentation/devicetree/bindings/arm/samsung/wakeup-eint.txt 
> b/Documentation/devicetree/bindings/arm/samsung/wakeup-eint.txt
> new file mode 100644
> index 0000000..ac9db41
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/samsung/wakeup-eint.txt
> @@ -0,0 +1,37 @@
> +* Samsung Exynos4 GPIO Wakeup Interrupt Source Controller
> +
> +Samsung Exynos4 processor supports 32 external wakeup interrupt sources. 
> First
> +16 of these interrupts are directly connected to GIC and the rest 16 of the
> +interrupts are grouped together to deliver a single interrupt to GIC.
> +
> +Required properties:
> +
> +- compatible: should be "samsung,exynos4210-wakeup-eint".
> +- interrupt-controller: Identifies the node as an interrupt controller.
> +- interrupt-cells: Specifies the number of cells required to specify the
> +  interrupt source number. The value of should be <2>. The first cell
> +  represents the wakeup interrupt source number and the second cell
> +  should be zero (currently unused).
> +- interrupts: List of interrupts generated by the gpio wakeup interrupt
> +    controller which are connected to a parent interrupt controller. The
> +    format of the interrupt specifier depends on the interrupt parent
> +    controller.
> +
> +Optional properties:
> +- interrupt-parent: phandle of the parent interrupt controller, required if
> +  not inheriting the interrupt parent from the parent node.
> +
> +Example:
> +
> +     The following example is from the Exynos4210 dtsi file.
> +
> +     wakeup_eint: interrupt-controller-wakeup-eint {
> +             compatible = "samsung,exynos4210-wakeup-eint";
> +             #interrupt-cells = <2>;
> +             interrupt-controller;
> +             interrupts = <0 16 0>, <0 17 0>, <0 18 0>, <0 19 0>,
> +                             <0 20 0>, <0 21 0>, <0 22 0>, <0 23 0>,
> +                             <0 24 0>, <0 25 0>, <0 26 0>, <0 27 0>,
> +                             <0 28 0>, <0 29 0>, <0 30 0>, <0 31 0>,
> +                             <0 32 0>;
> +     };
> diff --git a/arch/arm/mach-exynos/common.c b/arch/arm/mach-exynos/common.c
> index 888e703..615168e 100644
> --- a/arch/arm/mach-exynos/common.c
> +++ b/arch/arm/mach-exynos/common.c
> @@ -49,6 +49,9 @@
>  
>  #include "common.h"
>  
> +static int exynos4_init_irq_eint(struct device_node *np,
> +                                     struct device_node *parent);
> +
>  static const char name_exynos4210[] = "EXYNOS4210";
>  static const char name_exynos4212[] = "EXYNOS4212";
>  static const char name_exynos4412[] = "EXYNOS4412";
> @@ -81,8 +84,6 @@ static struct cpu_table cpu_ids[] __initdata = {
>       },
>  };
>  
> -static int exynos4_init_irq_eint(void);
> -
>  /* Initial IO mappings */
>  
>  static struct map_desc exynos_iodesc[] __initdata = {
> @@ -461,6 +462,8 @@ static const struct of_device_id exynos4_dt_irq_match[] = 
> {
>       { .compatible = "arm,cortex-a9-gic", .data = gic_of_init, },
>       { .compatible = "samsung,exynos4210-combiner",
>                       .data = combiner_of_init, },
> +     { .compatible = "samsung,exynos4210-wakeup-eint",
> +             .data = exynos4_init_irq_eint, },
>       {},
>  };
>  #endif
> @@ -478,8 +481,10 @@ void __init exynos4_init_irq(void)
>               of_irq_init(exynos4_dt_irq_match);
>  #endif
>  
> -     if (!of_have_populated_dt())
> +     if (!of_have_populated_dt()) {
>               combiner_init(S5P_VA_COMBINER_BASE, NULL);
> +             exynos4_init_irq_eint(NULL, NULL);
> +     }
>  
>       /*
>        * The parameters of s5p_init_irq() are for VIC init.
> @@ -487,7 +492,6 @@ void __init exynos4_init_irq(void)
>        * uses GIC instead of VIC.
>        */
>       s5p_init_irq(NULL, 0);
> -     exynos4_init_irq_eint();
>  }
>  
>  struct bus_type exynos4_subsys = {
> @@ -745,9 +749,20 @@ static struct irq_domain_ops exynos4_eint_irq_domain_ops 
> = {
>       .map = exynos4_eint_irq_domain_map,
>  };
>  
> -static int __init exynos4_init_irq_eint(void)
> +static int __init exynos4_eint_to_irq(struct device_node *np, int hwirq)
> +{
> +#ifdef CONFIG_OF
> +     return np ? irq_of_parse_and_map(np, hwirq) :
> +                     exynos4_irq_eint_to_gic_irq(hwirq);
> +#else
> +     return exynos4_irq_eint_to_gic_irq(hwirq);
> +#endif
> +}
> +
> +static int __init exynos4_init_irq_eint(struct device_node *np,
> +                                             struct device_node *parent)
>  {
> -     int eint, irq_base;
> +     int eint, irq_base, irq;
>       struct irq_domain *irq_domain;
>  
>       irq_base = irq_alloc_descs(IRQ_EINT(0), 1, EXYNOS4_EINT_NR, 0);
> @@ -757,7 +772,7 @@ static int __init exynos4_init_irq_eint(void)
>                       "Continuing with %d as linux irq base\n", irq_base);
>       }
>  
> -     irq_domain = irq_domain_add_legacy(NULL, EXYNOS4_EINT_NR, irq_base, 0,
> +     irq_domain = irq_domain_add_legacy(np, EXYNOS4_EINT_NR, irq_base, 0,
>                                       &exynos4_eint_irq_domain_ops, NULL);
>       if (WARN_ON(!irq_domain)) {
>               pr_warning("exynos4_init_irq_eint: irq domain init failed\n");
> @@ -765,16 +780,16 @@ static int __init exynos4_init_irq_eint(void)
>       }
>  
>       eint_data.irq_domain = irq_domain;
> -     eint_data.gic_irq_base = exynos4_irq_eint_to_gic_irq(0);
> +     eint_data.gic_irq_base = exynos4_eint_to_irq(np, 0);
>  
>       for (eint = 0 ; eint <= 15 ; eint++) {
> -             irq_set_handler_data(exynos4_irq_eint_to_gic_irq(eint),
> -                                     &eint_data);
> -             irq_set_chained_handler(exynos4_irq_eint_to_gic_irq(eint),
> -                                     exynos4_irq_eint0_15);
> +             irq = exynos4_eint_to_irq(np, eint);
> +             irq_set_handler_data(irq, &eint_data);
> +             irq_set_chained_handler(irq, exynos4_irq_eint0_15);
>       }
>  
> -     irq_set_chained_handler(IRQ_EINT16_31, exynos4_irq_demux_eint16_31);
> -     irq_set_handler_data(IRQ_EINT16_31, &eint_data);
> +     irq = exynos4_eint_to_irq(np, eint);
> +     irq_set_chained_handler(irq, exynos4_irq_demux_eint16_31);
> +     irq_set_handler_data(irq, &eint_data);
>       return 0;
>  }

_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss

Reply via email to