On Fri, 20 Jul 2012 18:09:26 +0800, <wei_w...@realsil.com.cn> wrote: > +static int msi_en = 1; > +module_param(msi_en, int, S_IRUGO | S_IWUSR); > +MODULE_PARM_DESC(msi_en, "Enable MSI");
Should this be a bool, and a module_param(msi_en, bool, S_IRUGO | S_IWUSR);? > +static int adma_mode = 1; > +module_param(adma_mode, int, S_IRUGO | S_IWUSR); > +MODULE_PARM_DESC(adma_mode, "ADMA Mode"); This too? > + /* Data transferred in ADMA mode or not */ > + int adma_mode; And this? Cheers, Rusty. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/