Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1675370

Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe functions in irq_chip"

Show all headers | View raw

* Brian Norris <briannorris@chromium.org> [170627 00:07]:
> On Mon, Jun 26, 2017 at 11:24:09PM -0700, Tony Lindgren wrote:
> > Hmm so how come drivers/bluetooth/btusb.c can't use the generic
> > dev_pm_set_dedicated_wake_irq()? Can you please take a look?
> 
> I took a look previously, and last time I did, there were too many bugs
> for it to be useful. You may have fixed the ones I reported w.r.t.
> assumptions about runtime PM.

Yes several issues got fixed over past few years. If you find issues
please let me know, otherwise I can't help.

> I also recall there being some difficulty with supporting
> level-triggered interrupts that way. (This signal has no device-level
> mask, and it triggers for all sorts of BT activity. There may not be a
> relevant "edge".)

Well typically the wakeirq needs to be disabled during runtime like we
do for the generic wakeirqs. It might be connected to the RX line for
example so it's just noise during runtime. If you actually need it during
runtime then it's a separate story but I doubt that's the case here.

Talking of GPIO interrupt triggering, I wonder if something like below
might help? It seems we're missing the setting of triggering, see below.

> > If there are issues remaining let's rather fix them so we can get rid
> > of the custom tinkering of wake-up events in the drivers.
> 
> That's nice, but that doesn't answer my questions. Perhaps that's a side
> project. The point is that we're clearly violating the documented APIs.

Certainly all these issues need to be fixed if we intent to use it. Funny
how I have not run into these with my test cases. Do you have a GPIO
irqchip on I2C for handling the wake-up interrupts?

Regards,

Tony

8< -------
diff --git a/drivers/base/power/wakeirq.c b/drivers/base/power/wakeirq.c
--- a/drivers/base/power/wakeirq.c
+++ b/drivers/base/power/wakeirq.c
@@ -198,7 +198,8 @@ int dev_pm_set_dedicated_wake_irq(struct device *dev, int irq)
 	 * so we use a threaded irq.
 	 */
 	err = request_threaded_irq(irq, NULL, handle_threaded_wake_irq,
-				   IRQF_ONESHOT, dev_name(dev), wirq);
+				   irq_get_trigger_type(irq) | IRQF_ONESHOT,
+				   dev_name(dev), wirq);
 	if (err)
 		goto err_free;
 

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Brian Norris <briannorris@chromium.org> - 2017-06-23 23:00 +0200
  Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe functions in irq_chip" Heiko Stuebner <heiko@sntech.de> - 2017-06-23 23:20 +0200
  Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Thomas Gleixner <tglx@linutronix.de> - 2017-06-24 00:20 +0200
    Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2017-06-24 00:50 +0200
      Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Thomas Gleixner <tglx@linutronix.de> - 2017-06-24 11:30 +0200
    Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Brian Norris <briannorris@chromium.org> - 2017-06-27 02:10 +0200
      Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Tony Lindgren <tony@atomide.com> - 2017-06-27 08:30 +0200
        Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Brian Norris <briannorris@chromium.org> - 2017-06-27 09:10 +0200
          Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Tony Lindgren <tony@atomide.com> - 2017-06-27 09:40 +0200
      Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Thomas Gleixner <tglx@linutronix.de> - 2017-06-27 15:10 +0200
        Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Brian Norris <briannorris@chromium.org> - 2017-06-27 19:30 +0200
          Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Thomas Gleixner <tglx@linutronix.de> - 2017-06-27 20:10 +0200
      Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Thomas Gleixner <tglx@linutronix.de> - 2017-06-27 15:10 +0200
        Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe functions in irq_chip" Heiko Stübner <heiko@sntech.de> - 2017-06-27 15:30 +0200
          Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Thomas Gleixner <tglx@linutronix.de> - 2017-06-27 18:40 +0200
  Re: [PATCH for 4.12] Revert "pinctrl: rockchip: avoid hardirq-unsafe  functions in irq_chip" Linus Walleij <linus.walleij@linaro.org> - 2017-06-29 15:10 +0200

csiph-web