On Wed, Jul 23, 2014 at 10:06 AM, Guenter Roeck <[email protected]> wrote: > On 07/23/2014 05:51 AM, Sasha Levin wrote: >> >> Revert since we're trying to return -ENXIO from a function returning >> unsigned int. Not only it causes compiler warnings it's also obviously >> incorrect. >> >> In general, watch for patches from Nick Krause since they are not even >> build tested. >> >> Signed-off-by: Sasha Levin <[email protected]> > > > Guess I wasn't fast enough with my comments :-( > > Acked-by: Guenter Roeck <[email protected]> > > >> --- >> arch/m68k/q40/q40ints.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/m68k/q40/q40ints.c b/arch/m68k/q40/q40ints.c >> index 9dfa1ea..513f9bb 100644 >> --- a/arch/m68k/q40/q40ints.c >> +++ b/arch/m68k/q40/q40ints.c >> @@ -49,7 +49,7 @@ static unsigned int q40_irq_startup(struct irq_data >> *data) >> case 1: case 2: case 8: case 9: >> case 11: case 12: case 13: >> printk("%s: ISA IRQ %d not implemented by HW\n", __func__, >> irq); >> - return -ENXIO; >> + /* FIXME return -ENXIO; */ >> } >> return 0; >> } >> >
I guess this is another bad patch :(. Nick -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
