| Date | Fri, 11 Feb 2011 12:42:29 +0100 (CET) | From | Thomas Gleixner <> | Subject | Re: [patch 34/75] arm: ep93xx: Kill another instance of broken irq_desc fiddling |
| |
On Fri, 11 Feb 2011, Ryan Mallon wrote:
> On 02/11/2011 12:37 PM, Thomas Gleixner wrote: > > 1. This is a copy of the borked code in gpiolib > > 2. If you need information about irq state which is not exposed, then talk > > to the maintainer of that code instead of adding totaly horrible open > > coded access. > > This code got added simply because it is sometimes helpful to be able to > see how various gpio/irq pins are configured. I'm happy to drop the > functionality (see below), but is there a better way to get this > information? Is it already available somewhere else (proc, sys)?
No, but we can add that if it's required in a sane form.
I don't have objections to expose debug informations in general, but I have objections that random code does this especially, when it's duplicated random code :)
Thanks,
tglx
|