Hi Tony, thanks for the quick turnaround,

On Sat, Nov 17 2012, Tony Prisk wrote:
> +     regular_irq = irq_of_parse_and_map(np, 0);
> +     dma_irq = irq_of_parse_and_map(np, 1);
> +
> +     if ((!regular_irq) | (!dma_irq)) {
> +             dev_err(&pdev->dev, "Getting IRQs failed!\n");

I think you meant ||, not |, and you could remove the redundant parens.

Other than that, I think this is ready -- if you want to ACK the above
change I'll fold it in without having you resend the patch.

Thanks,

- Chris.
-- 
Chris Ball   <c...@laptop.org>   <http://printf.net/>
One Laptop Per Child
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to