On Sat, 26 Jun 2010 01:51:13 am Randy Dunlap wrote: > On 06/25/10 05:50, Rusty Russell wrote: > > On Fri, 25 Jun 2010 07:51:22 am Randy Dunlap wrote: > >> From: Randy Dunlap <randy.dunlap@xxxxxxxxxx> > >> > >> The <arg> parameter in module_param_cb() should be a pointer, > > > > Well, I don't think there's anything wrong with it being a function. > > Ah, yes, I see. That makes it a sparse error, right? Well, it's a weird corner case, but yes. > > Meanwhile, we can do this instead. Does it solve it as well? > > Yes, that also fixes it. > Thanks. Thanks, I've added your tested-by. Cheers, Rusty. -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html