lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 18 Nov 2013 09:58:14 +0100 From: Belisko Marek <marek.belisko@...il.com> To: Mark Rutland <mark.rutland@....com> Cc: "arnd@...db.de" <arnd@...db.de>, "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>, "neilb@...e.de" <neilb@...e.de>, "hns@...delico.com" <hns@...delico.com>, "rob.herring@...xeda.com" <rob.herring@...xeda.com>, Pawel Moll <Pawel.Moll@....com>, "swarren@...dotorg.org" <swarren@...dotorg.org>, "ijc+devicetree@...lion.org.uk" <ijc+devicetree@...lion.org.uk>, "rob@...dley.net" <rob@...dley.net>, "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>, "linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org> Subject: Re: [PATCH 2/3] misc: bmp085: Add DT bindings for EOC gpio line and direct irq. Hi Mark, On Fri, Nov 15, 2013 at 4:30 PM, Mark Rutland <mark.rutland@....com> wrote: > On Thu, Nov 14, 2013 at 09:46:48PM +0000, Marek Belisko wrote: >> Signed-off-by: Marek Belisko <marek@...delico.com> >> --- >> Documentation/devicetree/bindings/misc/bmp085.txt | 8 ++++ >> drivers/misc/bmp085.c | 53 ++++++++++++++++++++--- >> 2 files changed, 55 insertions(+), 6 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/misc/bmp085.txt b/Documentation/devicetree/bindings/misc/bmp085.txt >> index 91dfda2..c6033d5 100644 >> --- a/Documentation/devicetree/bindings/misc/bmp085.txt >> +++ b/Documentation/devicetree/bindings/misc/bmp085.txt >> @@ -8,6 +8,9 @@ Optional properties: >> - temp-measurement-period: temperature measurement period (milliseconds) >> - default-oversampling: default oversampling value to be used at startup, >> value range is 0-3 with rising sensitivity. >> +- gpio: if device is using EOC irq line gpio can be specified to setup interrupt >> + handling >> +- irq: interrupt with no gpio >> >> Example: >> >> @@ -17,4 +20,9 @@ pressure@77 { >> chip-id = <10>; >> temp-measurement-period = <100>; >> default-oversampling = <2>; >> + gpio = <&gpio0 17 0>; >> + >> + OR >> + >> + irq = <75>; > > There's some contention over the description of gpio-based IRQs in DT. > From the point of view of the device there is a logical IRQ output; the > fact that this happens to be wired up to a GPIO pin that can happen to > generate interrupts isn't anything to do with the device itself. There > are plenty of device we have now whose interrupt lines could be wired to > GPIOs. I see no reason to extend their bindings to support explicit > GPIOs for IRQs, and I see no reason the driver should have to handle > this. > > It would be far nicer for the device binding to just have the interrupts > property, and for the gpio controller to act as an interrupt-controller, > with the appropriate pin management. OK. Can you please give me some example in current bindings tree? Something like in: Documentation/devicetree/bindings/fb/mxsfb.txt ? So if I understand correctly we define only property interrupts = <xxx> which will be exact interrupt number for cpu gpio connected to bmp085 irq line. > > Thanks, > Mark. Thanks, marek -- as simple and primitive as possible ------------------------------------------------- Marek Belisko - OPEN-NANDRA Freelance Developer Ruska Nova Ves 219 | Presov, 08005 Slovak Republic Tel: +421 915 052 184 skype: marekwhite twitter: #opennandra web: http://open-nandra.com -- 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