linux-kernel - Re: [PATCH] IIO: Adds ACPI support for ST gyroscopes

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150326101616.GD1878@lahna.fi.intel.com>
Date:	Thu, 26 Mar 2015 12:16:16 +0200
From:	Mika Westerberg <mika.westerberg@...ux.intel.com>
To:	Octavian Purdila <octavian.purdila@...el.com>,
	Linus Walleij <linus.walleij@...aro.org>
Cc:	Lars-Peter Clausen <lars@...afoo.de>,
	Robert Dolca <robert.dolca@...il.com>,
	Robert Dolca <robert.dolca@...el.com>,
	"linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
	Jonathan Cameron <jic23@...nel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Hartmut Knaack <knaack.h@....de>,
	Peter Meerwald <pmeerw@...erw.net>,
	Denis CIOCCA <denis.ciocca@...com>
Subject: Re: [PATCH] IIO: Adds ACPI support for ST gyroscopes

On Wed, Mar 25, 2015 at 11:12:16PM +0200, Octavian Purdila wrote:
> On Wed, Mar 25, 2015 at 3:21 PM, Mika Westerberg
> <mika.westerberg@...ux.intel.com> wrote:
> > On Wed, Mar 25, 2015 at 02:25:05PM +0200, Mika Westerberg wrote:
> >> I think we can do the same for ACPI GpioInts so that we introduce
> >> acpi_gpio_irq_get() that translates from GpioInt to Linux IRQ
> >> numberspace. Then we can do something like below in I2C core:
> >>
> >>       if (client->irq <= 0) {
> >>               int irq = -ENOENT;
> >>
> >>                 if (dev->of_node)
> >>                         irq = of_irq_get(dev->of_node, 0);
> >>                 else if (ACPI_COMPANION(dev))
> >>                         irq = acpi_gpio_irq_get(ACPI_COMPANION(dev), 0);
> >>
> >>               if (irq == -EPROBE_DEFER)
> >>                         return irq;
> >>                 if (irq < 0)
> >>                         irq = 0;
> >>
> >>                 client->irq = irq;
> >>       }
> >>
> >> Now it has the drawback that the first GpioInt will not be available to
> >> the driver anymore (as a GPIO since it is locked) but if DT already does
> >> the same we should be fine.
> >
> > Below patch should take care of this.
> >
> 
> One issue we noticed is that now the gpio request and set input
> directions operations are not called anymore. Some gpio controller
> drivers (dln2, adnp, lynx_point from quickly browsing the code) do not
> explicitly enable the GPIO pin nor set direction to input when the
> interrupt is enabled. Depending on hardware this may be an issue - it
> is on dln2 for example.
> 
> Should the gpio controllers enable and set to input in irq_enable,
> irq_bus_sync_unlock, etc.? Or should this be done in gpiolib?

Good question.

In general I think that it is assumed that the boot firmware configures
the pin upfront. However, we have seen too many times that it actually
doesn't happen or it is configured wrong.

Perhaps we could do this in GPIO core, for example in
gpiochip_irq_reqres/gpiochip_irq_map or so.

Linus?
--
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/