net: microchip: sparx5: kunit test: Fix compile warnings.
authorHoratiu Vultur <horatiu.vultur@microchip.com>
Thu, 17 Nov 2022 13:28:12 +0000 (14:28 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 21 Nov 2022 09:47:18 +0000 (09:47 +0000)
When VCAP_KUNIT_TEST is enabled the following warnings are generated:

drivers/net/ethernet/microchip/vcap/vcap_api_kunit.c:257:34: warning: Using plain integer as NULL pointer
drivers/net/ethernet/microchip/vcap/vcap_api_kunit.c:258:41: warning: Using plain integer as NULL pointer
drivers/net/ethernet/microchip/vcap/vcap_api_kunit.c:342:23: warning: Using plain integer as NULL pointer
drivers/net/ethernet/microchip/vcap/vcap_api_kunit.c:359:23: warning: Using plain integer as NULL pointer
drivers/net/ethernet/microchip/vcap/vcap_api_kunit.c:1327:34: warning: Using plain integer as NULL pointer
drivers/net/ethernet/microchip/vcap/vcap_api_kunit.c:1328:41: warning: Using plain integer as NULL pointer

Therefore fix this.

Fixes: dccc30cc4906 ("net: microchip: sparx5: Add KUNIT test of counters and sorted rules")
Fixes: c956b9b318d9 ("net: microchip: sparx5: Adding KUNIT tests of key/action values in VCAP API")
Fixes: 67d637516fa9 ("net: microchip: sparx5: Adding KUNIT test for the VCAP API")
Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/microchip/vcap/vcap_api_kunit.c

index 6858e44..194734c 100644 (file)
@@ -254,8 +254,8 @@ test_vcap_xn_rule_creator(struct kunit *test, int cid, enum vcap_user user,
                          u16 priority,
                          int id, int size, int expected_addr)
 {
-       struct vcap_rule *rule = 0;
-       struct vcap_rule_internal *ri = 0;
+       struct vcap_rule *rule;
+       struct vcap_rule_internal *ri;
        enum vcap_keyfield_set keyset = VCAP_KFS_NO_VALUE;
        enum vcap_actionfield_set actionset = VCAP_AFS_NO_VALUE;
        int ret;
@@ -339,7 +339,7 @@ static void vcap_api_set_bit_1_test(struct kunit *test)
                .sw_width = 52,
                .reg_idx = 1,
                .reg_bitpos = 20,
-               .tg = 0
+               .tg = NULL,
        };
        u32 stream[2] = {0};
 
@@ -356,7 +356,7 @@ static void vcap_api_set_bit_0_test(struct kunit *test)
                .sw_width = 52,
                .reg_idx = 2,
                .reg_bitpos = 11,
-               .tg = 0
+               .tg = NULL,
        };
        u32 stream[3] = {~0, ~0, ~0};
 
@@ -1324,8 +1324,8 @@ static void vcap_api_encode_rule_test(struct kunit *test)
                        .actionstream = actdata,
                },
        };
-       struct vcap_rule *rule = 0;
-       struct vcap_rule_internal *ri = 0;
+       struct vcap_rule *rule;
+       struct vcap_rule_internal *ri;
        int vcap_chain_id = 10005;
        enum vcap_user user = VCAP_USER_VCAP_UTIL;
        u16 priority = 10;