mlxsw: Move "used_kvd_sizes" check to mlxsw_pci_config_profile
authorJiri Pirko <jiri@mellanox.com>
Sun, 1 Apr 2018 14:34:56 +0000 (17:34 +0300)
committerDavid S. Miller <davem@davemloft.net>
Sun, 1 Apr 2018 14:56:43 +0000 (10:56 -0400)
The check should be done directly in mlxsw_pci_config_profile, as for
other profile items. Also, be consistent in naming with the rest and
rename to "used_kvd_sizes".

Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlxsw/core.h
drivers/net/ethernet/mellanox/mlxsw/pci.c
drivers/net/ethernet/mellanox/mlxsw/spectrum.c

index 0d64526993643df6f095198c75eb6d4ffd758812..ff9daa09341d6f32121984ed58ca2b85435cd29c 100644 (file)
@@ -235,8 +235,7 @@ struct mlxsw_config_profile {
                used_max_pkey:1,
                used_ar_sec:1,
                used_adaptive_routing_group_cap:1,
-               used_kvd_split_data:1; /* indicate for the kvd's values */
-
+               used_kvd_sizes:1;
        u8      max_vepa_channels;
        u16     max_mid;
        u16     max_pgt;
index e30c6ce3dcb422b32dce48421324c2e88d600c92..5ab068aec033d6663d844644a389057d31131a25 100644 (file)
@@ -1164,7 +1164,7 @@ static int mlxsw_pci_config_profile(struct mlxsw_pci *mlxsw_pci, char *mbox,
                mlxsw_cmd_mbox_config_profile_adaptive_routing_group_cap_set(
                        mbox, profile->adaptive_routing_group_cap);
        }
-       if (MLXSW_RES_VALID(res, KVD_SIZE)) {
+       if (profile->used_kvd_sizes && MLXSW_RES_VALID(res, KVD_SIZE)) {
                err = mlxsw_pci_profile_get_kvd_sizes(mlxsw_pci, profile, res);
                if (err)
                        return err;
index d503cdbeae297ed7fabebc75016c3f3ee98bfd30..12062aab13c553580fed120ea210ae5df071fcab 100644 (file)
@@ -3793,7 +3793,7 @@ static const struct mlxsw_config_profile mlxsw_sp_config_profile = {
        .max_ib_mc                      = 0,
        .used_max_pkey                  = 1,
        .max_pkey                       = 0,
-       .used_kvd_split_data            = 1,
+       .used_kvd_sizes                 = 1,
        .kvd_hash_single_parts          = 59,
        .kvd_hash_double_parts          = 41,
        .kvd_linear_size                = MLXSW_SP_KVD_LINEAR_SIZE,
@@ -3934,8 +3934,7 @@ static int mlxsw_sp_kvd_sizes_get(struct mlxsw_core *mlxsw_core,
        int err;
 
        if (!MLXSW_CORE_RES_VALID(mlxsw_core, KVD_SINGLE_MIN_SIZE) ||
-           !MLXSW_CORE_RES_VALID(mlxsw_core, KVD_DOUBLE_MIN_SIZE) ||
-           !profile->used_kvd_split_data)
+           !MLXSW_CORE_RES_VALID(mlxsw_core, KVD_DOUBLE_MIN_SIZE))
                return -EIO;
 
        /* The hash part is what left of the kvd without the