[PATCH v2 1/3] ethernet: Add new driver for Marvell Armada 375 network unit

David Miller davem at davemloft.net
Tue Jul 8 15:26:55 PDT 2014


From: Ezequiel Garcia <ezequiel.garcia at free-electrons.com>
Date: Sat,  5 Jul 2014 08:54:46 -0300

> +module_param(rxq_number, int, S_IRUGO);
> +module_param(txq_number, int, S_IRUGO);

Please don't use module parameters (undocumented ones at that) for things
like this.

Instead, use or create standard interfaces which other drivers can use as
well, so that the user interface is consistent for setting things of this
nature regardless of device type.

Thanks.



More information about the linux-arm-kernel mailing list