[ofa-general][PATCH] mlx4_core: Multi Protocol support

[Roland Dreier]

 > +static int mod_param_if_eth = 1;
 > +module_param_named(if_eth, mod_param_if_eth, bool, 0444);
 > +MODULE_PARM_DESC(if_eth, "Enable ETH interface be loaded (0/1, default 1)");
 > +
 > +static int mod_param_if_fc = 1;
 > +module_param_named(if_fc, mod_param_if_fc, bool, 0444);
 > +MODULE_PARM_DESC(if_fc, "Enable FC interface be loaded (0/1, default 1)");

I don't see any place where these values are checked.  And I don't quite
know why they would be necessary anyway.  Why would someone want to set
one of these to 0?  Couldn't they get the same effect by just not
loading the module in question?

 - R.