Hi Helin, Thanks for this new patchset version.
2014-06-23 20:57, Helin Zhang: > i40e supports 16 and 32 bytes RX descriptors which can be configured. > It needs to check the driver type and the configuration to determine > if 16 or 32 bytes RX descriptors is being used, for reading and > displaying the different sizes of RX descriptors. > > Signed-off-by: Helin Zhang <helin.zhang at intel.com> > Acked-by: Jing Chen <jing.d.chen at intel.com> > Acked-by: Cunming Liang <cunming.liang at intel.com> Just before applying theses patches, I checked with checkpatch.pl which warned me about this: > + ring_rxd_display_dword(rte_le_to_cpu_64(\ > + ring[desc_id].lo_dword)); > + ring_rxd_display_dword(rte_le_to_cpu_64(\ > + ring[desc_id].hi_dword)); > + ring_rxd_display_dword(rte_le_to_cpu_64(\ > + ring[desc_id].resv1)); > + ring_rxd_display_dword(rte_le_to_cpu_64(\ > + ring[desc_id].resv2)); WARNING:LINE_CONTINUATIONS: Avoid unnecessary line continuations Could you remove it please? Thanks -- Thomas
