Hey Bart,

+void irq_poll_init_am(struct irq_poll *iop, unsigned int nr_events,
+        unsigned short nr_levels, unsigned short start_level, irq_poll_am_fn 
*amfn)
+{
+       iop->amfn = amfn;
+       irq_am_init(&iop->am, nr_events, nr_levels, start_level, irq_poll_am);
+}


This function has a large number of parameters and most of them are passed 
verbatim to
irq_am_init(). Please consider to introduce a structure for these parameters 
such that
the number of function arguments stays reasonable.

I can definitely change that.

Reply via email to