[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