[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