> -----Original Message----- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Wednesday, October 29, 2014 9:10 AM > To: Kirsher, Jeffrey T; Nelson, Shannon > Cc: Brandeburg, Jesse; Allan, Bruce W; Wyborny, Carolyn; Skidmore, > Donald C; Rose, Gregory V; Vick, Matthew; Ronciak, John; Williams, Mitch > A; Linux NICS; Joe Perches; e1000-devel@lists.sourceforge.net; kernel- > janit...@vger.kernel.org > Subject: [patch] i40e: precedence bug in i40e_get_pfc_delay() > > The ">>" operation has higher precedence than the "&" operation. Also > the casting is not needed. > > Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com> > --- > This is static checker work and I haven't tested it on real hardware. > > diff --git a/drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c > b/drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c > index 00bc0cd..db9f9d3 100644 > --- a/drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c > +++ b/drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c > @@ -40,8 +40,8 @@ static void i40e_get_pfc_delay(struct i40e_hw *hw, u16 > *delay) > u32 val; > > val = rd32(hw, I40E_PRTDCB_GENC); > - *delay = (u16)(val & I40E_PRTDCB_GENC_PFCLDA_MASK >> > - I40E_PRTDCB_GENC_PFCLDA_SHIFT); > + *delay = (val & I40E_PRTDCB_GENC_PFCLDA_MASK) >> > + I40E_PRTDCB_GENC_PFCLDA_SHIFT; > } > > /**
Thanks. We had just recently caught this internally, as well. We'll have Jeff add this to our tree. sln ------------------------------------------------------------------------------ _______________________________________________ E1000-devel mailing list E1000-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
