On Mon, Sep 24, 2018 at 9:54 PM Jakub Kicinski
<jakub.kicin...@netronome.com> wrote:
>
> On Mon, 24 Sep 2018 10:46:21 +0530, Vasundhara Volam wrote:
> > This patch adds a new file to add information about configuration
> > parameters that are supported by bnxt_en driver via devlink.
> >
> > Cc: "David S. Miller" <da...@davemloft.net>
> > Cc: Jonathan Corbet <cor...@lwn.net>
> > Cc: linux-...@vger.kernel.org
> > Cc: Jiri Pirko <j...@mellanox.com>
> > Cc: Michael Chan <michael.c...@broadcom.com>
> >
> > Signed-off-by: Vasundhara Volam <vasundhara-v.vo...@broadcom.com>
> > ---
> > Documentation/networking/devlink-params-bnxt.txt | 35
> > ++++++++++++++++++++++++
> > 1 file changed, 35 insertions(+)
> > create mode 100644 Documentation/networking/devlink-params-bnxt.txt
> >
> > diff --git a/Documentation/networking/devlink-params-bnxt.txt
> > b/Documentation/networking/devlink-params-bnxt.txt
> > new file mode 100644
> > index 0000000..ca7c457
> > --- /dev/null
> > +++ b/Documentation/networking/devlink-params-bnxt.txt
> > @@ -0,0 +1,35 @@
> > +enable_sriov [DEVICE, GENERIC]
> > + Enable Single Root I/O Virtualisation (SRIOV) in
> > + the device.
> > + Type: Boolean
> > + Configuration mode: Permanent
> > +
> > +ignore_ari [DEVICE, GENERIC]
> > + Ignore Alternative Routing-ID Interpretation (ARI)
> > + capability. If enabled, adapter will ignore ARI
> > + capability even when platforms has the support
> > + enabled and creates same number of partitions when
> > + platform does not support ARI.
> > + Type: Boolean
> > + Configuration mode: Permanent
> > +
> > +msix_vec_per_pf_max [DEVICE, GENERIC]
> > + Provides the maximum number of MSIX interrupts that
> > + a device can create. Value is same across all
> > + physical functions (PFs) in the device.
> > + Type: u32
> > + Configuration mode: Permanent
> > +
> > +msix_vec_per_pf_min [DEVICE, GENERIC]
> > + Provides the minimum number of MSIX interrupts
> > required
> > + or the device initialization. Value is same across all
> > + physical functions (PFs) in the device.
> > + Type: u32
> > + Configuration mode: Permanent
>
> Why duplicate the description of the generic parameters?
Not all generic parameters are used by all drivers. So, I want to add
information about
type and configuration mode about generic parameters used by bnxt_en driver.
I can remove description part keeping type and configuration mode, if
it looks duplication.
>
> > +gre_ver_check [DEVICE, DRIVER-SPECIFIC]
> > + Generic Routing Encapsulation (GRE) version check will
> > + be enabled in the device. If disabled, device skips
> > + version checking for incoming packets.
> > + Type: Boolean
> > + Configuration mode: Permanent
>
