lkml.org 
[lkml]   [2015]   [Jan]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 8/8] gpio/mpc8xxx: Use of_mm_gpiochip_remove
Date
>>>>> "Ricardo" == Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> writes:

> Hello Peter
>>
>> > - hwirq = irq_of_parse_and_map(np, 0);
>> > - if (hwirq == NO_IRQ)
>> > + mpc8xxx_gc->irqn = irq_of_parse_and_map(np, 0);
>> > + if (mpc8xxx_gc->irqn == NO_IRQ)
>> > return 0;
>>
>>
>> With this return 0 converted to do of_mm_gpiochip_remove():

> Are you sure? The driver can still work as a normal gpio without the
> irq domain part and the remove function consider this option. The
> original code did also continue....

Ahh yes, you are right. Sorry, it's been a while since I wrote that code
(2008).

> If you still want to abort if no irq I can of course make the change.

No, it is fine.

--
Bye, Peter Korsgaard


\
 
 \ /
  Last update: 2015-01-20 12:01    [W:0.074 / U:0.564 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site