On Wed, 21 Jan 2015, Robert Jarzmik wrote: > Lee Jones <lee.jones@xxxxxxxxxx> writes: > > > On Wed, 21 Jan 2015, Robert Jarzmik wrote: > >> > > > platform_get_irq()? > >> > > No. I need the flags. > >> > Where are they used? > >> A couple of lines below, using local "irqflags" variable : > >> ret = devm_request_irq(&pdev->dev, cot->irq, lubbock_irq_handler, > >> irqflags, dev_name(&pdev->dev), cot); > > > > No, I mean where are they _used_? > I don't get your point. > > For you, isn't it "using" irqflags by passing them as an argument to devm_request_irq() No, this is where the argument is being _passed_. > ??? And if you think ahead, then there are used all around in > kernel/irq/manage.c, as you probably know. Perhaps some context would help. Looking at one of the other patches in the series it appears the flag you're trying to capture is IORESOURCE_IRQ_LOWEDGE. When I grep for where this is being _used_ (think 'consumed, rather than passed. I only see a single entry in drivers/pnp/interface.c. That's what got me thinking... are you sure you're a) making use of this flag and b) assuming the answer to 'a' is "no" and if everyting's working correctly in the code's current state, are you sure you need them at all? I think to set an edge trigger on an IRQ, you should instead do so via irq_set_irq_type(), or have a missed a line or two? -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
