On Mon, Jul 01, 2013 at 05:46:48PM +1000, Finn Thain wrote:
On Sun, 30 Jun 2013, Geert Uytterhoeven wrote:- -/* - * Register the interrupt handler for the IOPs. - * TODO: might be wrong for non-OSS machines. Anyone? - */ - -void __init iop_register_interrupts(void) -{ - if (iop_ism_present) { + /* + * Register the interrupt handler for the IOPs. + * TODO: might be wrong for non-OSS machines. Anyone?My testing with a non-OSS machine (Quadra 950) indicates that the TODO is obsolete. And the comment "register the interrupt handler for the IOPs" is a bit silly. It merely re-phrases the code. Otherwise, this patch looks good to me. Thanks.
The code was all originally written and tested only on the IIfx. That's all JMT had, and at the time it was all I had as well. I have since acquired a Quadra 950. Lots of these comments are just out of date. I know it got tested on either a Quadra 900 or Quadra 950 after that comment was put in the source. Brad Boyer flar@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-m68k" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
