Mon, Oct 01, 2018 at 06:04:08AM CEST, [email protected] wrote: >On Sat, Sep 29, 2018 at 6:27 PM Jiri Pirko <[email protected]> wrote: >> >> Fri, Sep 28, 2018 at 08:28:23AM CEST, [email protected] 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" <[email protected]> >> >Cc: Jonathan Corbet <[email protected]> >> >Cc: [email protected] >> >Cc: Jiri Pirko <[email protected]> >> >Cc: Michael Chan <[email protected]> >> >Signed-off-by: Vasundhara Volam <[email protected]> >> >--- >> > Documentation/networking/devlink-params-bnxt.txt | 22 >> > ++++++++++++++++++++++ >> > 1 file changed, 22 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..c7bc9d8 >> >--- /dev/null >> >+++ b/Documentation/networking/devlink-params-bnxt.txt >> >@@ -0,0 +1,22 @@ >> >+enable_sriov [DEVICE, GENERIC] >> >+ Type: Boolean >> >> No need to list "Type" here. You have it in devlink-params.txt for >> generic params already. >Ok, I thought type will give clarity when configuration mode is listed and >when user checks out only driver specific readme file. >I can remove it in next version of patchset.
Yes please. The thing is, once this info is on 2 places, I'm pretty sure what eventually there would be different types on different places. Leaving the type only on one place reduces need for sync. Thanks. >> >> >> >+ Configuration mode: Permanent >> >+ >> >+ignore_ari [DEVICE, GENERIC] >> >+ Type: Boolean >> >+ Configuration mode: Permanent >> >+ >> >+msix_vec_per_pf_max [DEVICE, GENERIC] >> >+ Type: u32 >> >+ Configuration mode: Permanent >> >+ >> >+msix_vec_per_pf_min [DEVICE, GENERIC] >> >+ Type: u32 >> >+ Configuration mode: Permanent >> >+ >> >+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 >> >-- >> >1.8.3.1 >> >
