[patch v3 2/3] arm: pmu: allow platform specific irq enable/disable handling

Jean Pihet jean.pihet at newoldbits.com
Mon Mar 7 04:52:08 EST 2011


Some minor comments: checkpatch issues some warnings:

WARNING: please, no space before tabs
#76: FILE: arch/arm/include/asm/pmu.h:26:
+ * ^Iinterrupt and passed the address of the low level handler,$

WARNING: please, no space before tabs
#77: FILE: arch/arm/include/asm/pmu.h:27:
+ * ^Iand can be used to implement any platform specific handling$

WARNING: please, no space before tabs
#78: FILE: arch/arm/include/asm/pmu.h:28:
+ * ^Ibefore or after calling it.$

WARNING: please, no space before tabs
#80: FILE: arch/arm/include/asm/pmu.h:30:
+ * ^Irequest_irq and be used to handle some platform specific$

WARNING: please, no space before tabs
#81: FILE: arch/arm/include/asm/pmu.h:31:
+ * ^Iirq enablement$

WARNING: please, no space before tabs
#83: FILE: arch/arm/include/asm/pmu.h:33:
+ * ^Ifree_irq and be used to handle some platform specific$

WARNING: please, no space before tabs
#84: FILE: arch/arm/include/asm/pmu.h:34:
+ * ^Iirq disablement$

total: 0 errors, 7 warnings, 62 lines checked

Thanks,
Jean

On Mon, Mar 7, 2011 at 10:51 AM, Jean Pihet <jean.pihet at newoldbits.com> wrote:
> On Thu, Mar 3, 2011 at 11:53 AM,  <tom.leiming at gmail.com> wrote:
>> From: Ming Lei <tom.leiming at gmail.com>
>>
>> This patch introduces .enable_irq and .disable_irq into
>> struct arm_pmu_platdata, so platform specific irq enablement
>> can be handled after request_irq, and platform specific irq
>> disablement can be handled before free_irq.
>>
>> This patch is for support of  pmu irq routed from CTI on omap4.
>>
>> Reviewed-by: Will Deacon <will.deacon at arm.com>
>> Signed-off-by: Ming Lei <tom.leiming at gmail.com>
>
> Acked-by: Jean Pihet <j-pihet at ti.com>
>
> Thanks,
> Jean
>
>> ---
>>  arch/arm/include/asm/pmu.h   |   15 ++++++++++++---
>>  arch/arm/kernel/perf_event.c |   15 ++++++++++++---
>>  2 files changed, 24 insertions(+), 6 deletions(-)
>>
>> diff --git a/arch/arm/include/asm/pmu.h b/arch/arm/include/asm/pmu.h
>> index 7544ce6..e50c542 100644
>> --- a/arch/arm/include/asm/pmu.h
>> +++ b/arch/arm/include/asm/pmu.h
>> @@ -22,13 +22,22 @@ enum arm_pmu_type {
>>  /*
>>  * struct arm_pmu_platdata - ARM PMU platform data
>>  *
>> - * @handle_irq: an optional handler which will be called from the interrupt and
>> - * passed the address of the low level handler, and can be used to implement
>> - * any platform specific handling before or after calling it.
>> + * @handle_irq: an optional handler which will be called from the
>> + *     interrupt and passed the address of the low level handler,
>> + *     and can be used to implement any platform specific handling
>> + *     before or after calling it.
>> + * @enable_irq: an optional handler which will be called after
>> + *     request_irq and be used to handle some platform specific
>> + *     irq enablement
>> + * @disable_irq: an optional handler which will be called before
>> + *     free_irq and be used to handle some platform specific
>> + *     irq disablement
>>  */
>>  struct arm_pmu_platdata {
>>        irqreturn_t (*handle_irq)(int irq, void *dev,
>>                                  irq_handler_t pmu_handler);
>> +       void (*enable_irq)(int irq);
>> +       void (*disable_irq)(int irq);
>>  };
>>
>>  #ifdef CONFIG_CPU_HAS_PMU
>> diff --git a/arch/arm/kernel/perf_event.c b/arch/arm/kernel/perf_event.c
>> index 22e194eb..61ff471 100644
>> --- a/arch/arm/kernel/perf_event.c
>> +++ b/arch/arm/kernel/perf_event.c
>> @@ -422,14 +422,18 @@ armpmu_reserve_hardware(void)
>>                        pr_warning("unable to request IRQ%d for ARM perf "
>>                                "counters\n", irq);
>>                        break;
>> -               }
>> +               } else if (plat->enable_irq)
>> +                       plat->enable_irq(irq);
>>        }
>>
>>        if (err) {
>>                for (i = i - 1; i >= 0; --i) {
>>                        irq = platform_get_irq(pmu_device, i);
>> -                       if (irq >= 0)
>> +                       if (irq >= 0) {
>> +                               if (plat->disable_irq)
>> +                                       plat->disable_irq(irq);
>>                                free_irq(irq, NULL);
>> +                       }
>>                }
>>                release_pmu(pmu_device);
>>                pmu_device = NULL;
>> @@ -442,11 +446,16 @@ static void
>>  armpmu_release_hardware(void)
>>  {
>>        int i, irq;
>> +       struct arm_pmu_platdata *plat =
>> +               dev_get_platdata(&pmu_device->dev);
>>
>>        for (i = pmu_device->num_resources - 1; i >= 0; --i) {
>>                irq = platform_get_irq(pmu_device, i);
>> -               if (irq >= 0)
>> +               if (irq >= 0) {
>> +                       if (plat->disable_irq)
>> +                               plat->disable_irq(irq);
>>                        free_irq(irq, NULL);
>> +               }
>>        }
>>        armpmu->stop();
>>
>> --
>> 1.7.3
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel at lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>>
>



More information about the linux-arm-kernel mailing list