<p>So I am. Great spot Dmitry.</p>
<p>I'll fix and resubmit next week.</p>
<p>Sent from my mobile Linux device.</p>
<div class="gmail_quote">On Jul 6, 2012 6:40 PM, "Dmitry Torokhov" <<a href="mailto:dmitry.torokhov@gmail.com">dmitry.torokhov@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Lee,<br>
<br>
On Fri, Jul 06, 2012 at 08:52:43AM +0100, Lee Jones wrote:<br>
> Sorry Dmitry, looks like I missed you off of the CC list on this one.<br>
><br>
> Here it is again:<br>
><br>
> Before we can use any domain allocated IRQ, we need to first create a<br>
> map between the Hardware IRQ (hwirq) and the Linux Virtual IRQ (virq).<br>
> We do this with a helper function provided by the AB8500 IRQ domain<br>
> controller called ab8500_irq_get_virq(). We need to do this for both<br>
> IRQs which the Power-On-Key driver uses; one for button press, the other<br>
> for button depress.<br>
><br>
> Signed-off-by: Lee Jones <<a href="mailto:lee.jones@linaro.org">lee.jones@linaro.org</a>><br>
> ---<br>
> drivers/input/misc/ab8500-ponkey.c |    2 ++<br>
> 1 file changed, 2 insertions(+)<br>
><br>
> diff --git a/drivers/input/misc/ab8500-ponkey.c b/drivers/input/misc/ab8500-ponkey.c<br>
> index 5ceb23e..ae4b683 100644<br>
> --- a/drivers/input/misc/ab8500-ponkey.c<br>
> +++ b/drivers/input/misc/ab8500-ponkey.c<br>
> @@ -82,6 +82,7 @@ static int __devinit ab8500_ponkey_probe(struct platform_device *pdev)<br>
>       input_set_capability(input, EV_KEY, KEY_POWER);<br>
> +     ponkey->irq_dbf = ab8500_irq_get_virq(ab8500, ponkey->irq_dbf);<br>
>       error = request_any_context_irq(ponkey->irq_dbf, ab8500_ponkey_handler,<br>
>                                       0, "ab8500-ponkey-dbf", ponkey);<br>
>       if (error < 0) {<br>
> @@ -90,6 +91,7 @@ static int __devinit ab8500_ponkey_probe(struct platform_device *pdev)<br>
>               goto err_free_mem;<br>
>       }<br>
> +     ponkey->irq_dbf = ab8500_irq_get_virq(ab8500, ponkey->irq_dbr);<br>
<br>
I do not think this is right - you are storing irq_dbr in irq_dbf. Also,<br>
I think you want to do this earlier, like this:<br>
<br>
-       ponkey->irq_dbf = irq_dbf;<br>
-       ponkey->irq_dbr = irq_dbr;<br>
+       ponkey->irq_dbf = ab8500_irq_get_virq(ab8500, irq_dbf);<br>
+       ponkey->irq_dbr = ab8500_irq_get_virq(ab8500, irq_dbr);<br>
<br>
Thanks.<br>
<br>
--<br>
Dmitry<br>
</blockquote></div>