From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH v12 03/10] netdev: cavium: octeon: Add Octeon III BGX Ethernet Nexus Date: Fri, 29 Jun 2018 11:19:05 +0900 (KST) Message-ID: <20180629.111905.863023692523959153.davem@davemloft.net> References: <1530134719-19407-4-git-send-email-steven.hill@cavium.com> <20180628084156.GF16727@lunn.ch> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: andrew@lunn.ch, steven.hill@cavium.com, netdev@vger.kernel.org, cchavva@caviumnetworks.com To: cmunoz@cavium.com Return-path: Received: from shards.monkeyblade.net ([23.128.96.9]:36592 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030327AbeF2CTJ (ORCPT ); Thu, 28 Jun 2018 22:19:09 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: From: Carlos Munoz Date: Thu, 28 Jun 2018 14:20:05 -0700 > > > On 06/28/2018 01:41 AM, Andrew Lunn wrote: >> External Email >> >>> +static char *mix_port; >>> +module_param(mix_port, charp, 0444); >>> +MODULE_PARM_DESC(mix_port, "Specifies which ports connect to MIX interfaces."); >>> + >>> +static char *pki_port; >>> +module_param(pki_port, charp, 0444); >>> +MODULE_PARM_DESC(pki_port, "Specifies which ports connect to the PKI."); >> Module parameters are generally not liked. Can you do without them? > > These parameters change the kernel port assignment required by user > space applications. We rather keep them as they simplify the > process. This is actually a terrible user experience. Please provide a way to do this by performing operations on a device object after the driver loads. Use something like devlink or similar if you have to.