net/ixgbe: fix VF VLAN insert API
authorE. Scott Daniels <daniels@research.att.com>
Wed, 19 Oct 2016 14:47:33 +0000 (15:47 +0100)
committerBruce Richardson <bruce.richardson@intel.com>
Wed, 26 Oct 2016 17:42:23 +0000 (19:42 +0200)
commitaaffc740ec2cfdb304523c67020f09ad343cfacf
tree23a32d18dbf356a8de5af3e22cc36da824d59cbb
parentf9e0e7ab78232c03e431de51fd50d6854ce03938
net/ixgbe: fix VF VLAN insert API

The final parameter to rte_pmd_ixgbe_set_vf_vlan_insert is uint8_t
and treated as a binary flag when it needs to be a uint16_t
and treated as a VLAN id.  The data sheet (sect 8.2.3.27.13) describes
the right most 16 bits as the VLAN id that is to be inserted; the
16.11  code is accepting only a 1 or 0 thus effectively only
allowing the VLAN id 1 to be inserted (0 disables the insertion
setting).

This patch changes the final parm name to represent the data that
is being accepted (vlan_id), changes the type to permit all valid
VLAN ids, and validates the parameter based on the range of 0 to
4095. Corresponding changes to prototype and documentation in the
.h file.

Fixes: 49e248223e9f71 ("net/ixgbe: add API for VF management")

Signed-off-by: E. Scott Daniels <daniels@research.att.com>
Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
Acked-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
app/test-pmd/cmdline.c
doc/guides/testpmd_app_ug/testpmd_funcs.rst
drivers/net/ixgbe/ixgbe_ethdev.c
drivers/net/ixgbe/rte_pmd_ixgbe.h