> -----Original Message----- > From: Aaron Conole [mailto:aconole at redhat.com] > Sent: Friday, February 26, 2016 2:49 AM > To: dev at dpdk.org > Cc: Lu, Wenzhuo <wenzhuo.lu at intel.com>; Zhang, Helin > <helin.zhang at intel.com>; Ananyev, Konstantin > <konstantin.ananyev at intel.com>; Richardson, Bruce > <bruce.richardson at intel.com> > Subject: [PATCH 7/8] drivers/net/ixgbe: Signed left shift operator > > Tell the compiler to use an unsigned constant for the config shifts. > > Signed-off-by: Aaron Conole <aconole at redhat.com> Acked-by: Helin Zhang <helin.zhang at intel.com>
> --- > drivers/net/ixgbe/ixgbe_pf.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/ixgbe/ixgbe_pf.c b/drivers/net/ixgbe/ixgbe_pf.c index > 2ffbd1f..8b5119f 100644 > --- a/drivers/net/ixgbe/ixgbe_pf.c > +++ b/drivers/net/ixgbe/ixgbe_pf.c > @@ -236,9 +236,9 @@ int ixgbe_pf_host_configure(struct rte_eth_dev > *eth_dev) > vfre_slot = (vf_num >> VFRE_SHIFT) > 0 ? 1 : 0; > > /* Enable pools reserved to PF only */ > - IXGBE_WRITE_REG(hw, IXGBE_VFRE(vfre_slot), (~0) << vfre_offset); > + IXGBE_WRITE_REG(hw, IXGBE_VFRE(vfre_slot), (~0U) << vfre_offset); > IXGBE_WRITE_REG(hw, IXGBE_VFRE(vfre_slot ^ 1), vfre_slot - 1); > - IXGBE_WRITE_REG(hw, IXGBE_VFTE(vfre_slot), (~0) << vfre_offset); > + IXGBE_WRITE_REG(hw, IXGBE_VFTE(vfre_slot), (~0U) << vfre_offset); > IXGBE_WRITE_REG(hw, IXGBE_VFTE(vfre_slot ^ 1), vfre_slot - 1); > > /* PFDMA Tx General Switch Control Enables VMDQ loopback */ > -- > 2.5.0
