On 07.04.2017 13:00, Nikhil Agarwal wrote: > +typedef union odp_ipsec_protocols_t { > + /** Cipher algorithms */ > + struct { > + /** ODP_IPSEC_ESP */ > + uint32_t esp : 1; > + > + /** ODP_IPSEC_AH */ > + uint32_t ah : 1;
Small suggestion: it might be reasonable to split esp and ah to fully identify ESP_IPv4/ESP_IPv6 and AH_IPv4/AH_IPv6. -- With best wishes Dmitry