Hi,
On Sun, Nov 18 2012, Tony Prisk wrote:
>> 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.
>
> Oops - Good catch.
> Acked-by: Tony Prisk <[email protected]>
Thanks, pushed to mmc-next for 3.8.
- Chris.
--
Chris Ball <[email protected]> <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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
