[PATCH] ARM: tegra: consolidate GIC extension usage

Colin Cross ccross at google.com
Wed Apr 27 13:34:02 EDT 2011


On Wed, Apr 27, 2011 at 4:44 AM, Marc Zyngier <marc.zyngier at arm.com> wrote:
> On Wed, 27 Apr 2011 09:36:57 +0100
> Russell King - ARM Linux <linux at arm.linux.org.uk> wrote:
>
>> On Wed, Apr 20, 2011 at 11:58:24AM +0100, Marc Zyngier wrote:
>>
>> A good follow-on step would be...
>>
>> >  static void tegra_mask(struct irq_data *d)
>> >  {
>> > -   tegra_gic_mask_irq(d);
>> >     tegra_legacy_mask_irq(d->irq);
>> >  }
>>
>> ... arranging tegra_legacy_mask_irq() etc to take struct irq_data directly.
>> These functions could then be removed entirely as they serve no useful
>> purpose other than transforming the new API back to the old API.
>
> How about the following patch? I haven't touched the gic_arch_extn part
> as it would break Will's fastEOI patch set (which depends on this one),
> but will prepare another patch cleaning up that part.

I have a patch that fixes all of this and more as part of the Tegra PM
series, but I was sitting on it assuming Tegra PM would not go into
2.6.40.  Let me sort out the cleanups from the PM stuff, and I'll post
a patch that drops legacy_irq.c entirely.



More information about the linux-arm-kernel mailing list