|WARNING| pw134898 [PATCH v2 4/4] net/hns3: use stdatomic API

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Dec 7 02:47:07 CET 2023


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/134898

_coding style issues_


CHECK:SPACING: No space is necessary after a cast
#83: FILE: drivers/net/hns3/hns3_cmd.c:47:
+	static RTE_ATOMIC(uint64_t) hns3_dma_memzone_id;

CHECK:SPACING: No space is necessary after a cast
#337: FILE: drivers/net/hns3/hns3_ethdev.h:402:
+	RTE_ATOMIC(uint16_t) schedule;

CHECK:SPACING: No space is necessary after a cast
#340: FILE: drivers/net/hns3/hns3_ethdev.h:404:
+	RTE_ATOMIC(uint16_t) resetting;

CHECK:SPACING: No space is necessary after a cast
#343: FILE: drivers/net/hns3/hns3_ethdev.h:406:
+	RTE_ATOMIC(uint16_t) disable_cmd;

CHECK:SPACING: No space is necessary after a cast
#348: FILE: drivers/net/hns3/hns3_ethdev.h:410:
+	RTE_ATOMIC(uint64_t) pending;

CHECK:SPACING: No space is necessary after a cast
#351: FILE: drivers/net/hns3/hns3_ethdev.h:412:
+	RTE_ATOMIC(uint64_t) request;

CHECK:SPACING: No space is necessary after a cast
#360: FILE: drivers/net/hns3/hns3_ethdev.h:500:
+	RTE_ATOMIC(unsigned int) secondary_cnt; /* Number of secondary processes init'd. */

CHECK:SPACING: No space is necessary after a cast
#369: FILE: drivers/net/hns3/hns3_ethdev.h:845:
+	RTE_ATOMIC(uint16_t) pf_push_lsc_cap;

CHECK:SPACING: No space is necessary after a cast
#378: FILE: drivers/net/hns3/hns3_ethdev.h:854:
+	RTE_ATOMIC(uint16_t) poll_job_started; /* whether poll job is started */

WARNING:FUNCTION_ARGUMENTS: function definition argument 'uint64_t *' should also have an identifier name
#804: FILE: drivers/net/hns3/hns3_intr.h:174:
+void hns3_handle_msix_error(struct hns3_adapter *hns, RTE_ATOMIC(uint64_t *)levels);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'uint64_t *' should also have an identifier name
#805: FILE: drivers/net/hns3/hns3_intr.h:175:
+void hns3_handle_ras_error(struct hns3_adapter *hns, RTE_ATOMIC(uint64_t *)levels);

total: 0 errors, 2 warnings, 9 checks, 752 lines checked


More information about the test-report mailing list