[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <8A42379416420646B9BFAC9682273B6D0E44312E@limkexm3.ad.analog.com>
Date: Wed, 7 Oct 2009 14:35:06 +0100
From: "Hennerich, Michael" <Michael.Hennerich@...log.com>
To: "Mark Brown" <broonie@...nsource.wolfsonmicro.com>,
"Frysinger, Michael" <Michael.Frysinger@...log.com>
CC: "Mike Frysinger" <vapier@...too.org>,
"Samuel Ortiz" <sameo@...ux.intel.com>,
<uclinux-dist-devel@...ckfin.uclinux.org>,
<linux-kernel@...r.kernel.org>, "Bryan Wu" <cooloney@...nel.org>
Subject: RE: [PATCH v3] mfd: ADP5520 Multifunction LCD Backlight and KeypadInput Device Driver
>From: Mark Brown [mailto:broonie@...nsource.wolfsonmicro.com]
>
>On Wed, Oct 07, 2009 at 02:01:42PM +0100, Hennerich, Michael wrote:
>
>> Most architectures define NR_IRQS to exactly the number of on-chip
IRQs.
>> Therefore irq_desc pointers > NR_IRQS will fail on most
architectures.
>> This driver should be usable out of the box - but I guess this may
>> prevent this.
>
>Indeed, that's rather unfortunate. This wouldn't be the only driver
>with that dependency, though, so I'd expect to see the interesting
>architectures getting fixed.
We have an update patch that fixes the platform typo and namespace
pollution in the header file.
I think we send out this patch as final for mainline inclusion.
For the interrupt controller feature - I think I can wait until the
interesting architectures get fixed.
Thanks for your feedback and help
best regards,
Michael
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists