Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page:
include/odp/api/spec/packet.h
line 10
@@ -83,6 +83,22 @@ typedef struct odp_packet_data_range {
} odp_packet_data_range_t;
+/**
+ * Checksum check status in packet
+ */
+typedef enum odp_packet_chksum_status_t {
+ /** Checksum was not checked. Checksum check was not attempted or
+ * the attempt failed. */
+ ODP_PACKET_CHKSUM_UNKNOWN = 0,
Comment:
Since ODP_PACKET_CHKSUM_OK is the most likely case, wouldn't it be better to
use 0 for that enum? In almost all cases HW is going to "do the right thing" so
SW should only need to initialize this to something else if that's not the case.
https://github.com/Linaro/odp/pull/167#discussion_r137341971
updated_at 2017-09-06 18:10:45