> -----Original Message----- > From: Josh Triplett [mailto:[email protected]] > Sent: Monday, December 16, 2013 9:43 AM > To: Wyborny, Carolyn > Cc: Rashika Kheria; [email protected]; Kirsher, Jeffrey T; > Brandeburg, > Jesse; Allan, Bruce W; Skidmore, Donald C; Rose, Gregory V; Waskiewicz Jr, > Peter P; Duyck, Alexander H; Ronciak, John; Dave, Tushar N; Abodunrin, Akeem > G; Vick, Matthew; [email protected]; [email protected] > Subject: Re: [PATCH v2 1/9] drivers: net: Remove unused function > igb_get_eee_status_i354() in e1000_82575.c > > On Mon, Dec 16, 2013 at 05:11:25PM +0000, Wyborny, Carolyn wrote: > > > -----Original Message----- > > > From: Rashika Kheria [mailto:[email protected]] > > > Sent: Saturday, December 14, 2013 4:15 AM > > > To: [email protected] > > > Cc: Kirsher, Jeffrey T; Brandeburg, Jesse; Allan, Bruce W; Wyborny, > > > Carolyn; Skidmore, Donald C; Rose, Gregory V; Waskiewicz Jr, Peter > > > P; Duyck, Alexander H; Ronciak, John; Dave, Tushar N; Abodunrin, > > > Akeem G; Vick, Matthew; e1000- [email protected]; > > > [email protected]; [email protected] > > > Subject: [PATCH v2 1/9] drivers: net: Remove unused function > > > igb_get_eee_status_i354() in e1000_82575.c > > > > > > This patch removes the function igb_get_eee_status_i354() in > > > e1000_82575.c because it is unused. > > > > > > It thus eliminates the following warning in > > > ethernet/intel/igb/e1000_82575.c: > > > drivers/net/ethernet/intel/igb/e1000_82575.c:2591:5: warning: no > > > previous prototype for ‘igb_get_eee_status_i354’ > > > [-Wmissing-prototypes] > > > > > > Signed-off-by: Rashika Kheria <[email protected]> > > > Reviewed-by: Josh Triplett <[email protected]> > > > --- > > > drivers/net/ethernet/intel/igb/e1000_82575.c | 32 > > > -------------------------- > > > 1 file changed, 32 deletions(-) > > > > > > diff --git a/drivers/net/ethernet/intel/igb/e1000_82575.c > > > b/drivers/net/ethernet/intel/igb/e1000_82575.c > > > index 47c2d10..18e5200 100644 > > > --- a/drivers/net/ethernet/intel/igb/e1000_82575.c > > > +++ b/drivers/net/ethernet/intel/igb/e1000_82575.c > > > @@ -2580,38 +2580,6 @@ out: > > > return ret_val; > > > } > > > > > > -/** > > > - * igb_get_eee_status_i354 - Get EEE status > > > - * @hw: pointer to the HW structure > > > - * @status: EEE status > > > - * > > > - * Get EEE status by guessing based on whether Tx or Rx LPI > > > indications have > > > - * been received. > > > - **/ > > > -s32 igb_get_eee_status_i354(struct e1000_hw *hw, bool *status) -{ > > > - struct e1000_phy_info *phy = &hw->phy; > > > - s32 ret_val = 0; > > > - u16 phy_data; > > > - > > > - /* Check if EEE is supported on this device. */ > > > - if ((hw->phy.media_type != e1000_media_type_copper) || > > > - (phy->id != M88E1543_E_PHY_ID)) > > > - goto out; > > > - > > > - ret_val = igb_read_xmdio_reg(hw, E1000_PCS_STATUS_ADDR_I354, > > > - E1000_PCS_STATUS_DEV_I354, > > > - &phy_data); > > > - if (ret_val) > > > - goto out; > > > - > > > - *status = phy_data & (E1000_PCS_STATUS_TX_LPI_RCVD | > > > - E1000_PCS_STATUS_RX_LPI_RCVD) ? true : false; > > > - > > > -out: > > > - return ret_val; > > > -} > > > - > > > static const u8 e1000_emc_temp_data[4] = { > > > E1000_EMC_INTERNAL_DATA, > > > E1000_EMC_DIODE1_DATA, > > > -- > > > 1.7.9.5 > > > > NACK. > > > > Thanks for the patch Rashika, but this is the incorrect fix for this > > warning The function is called in the igb_probe function, so you > > cannot remove it and I see the prototype in the e1000_82575.h file. > > Can you double check your source pull? > > From top-of-tree linux.git: > > ~/src/linux$ git grep igb_get_eee_status_i354 > drivers/net/ethernet/intel/igb/e1000_82575.c: * igb_get_eee_status_i354 - Get > EEE status > drivers/net/ethernet/intel/igb/e1000_82575.c:s32jjj > igb_get_eee_status_i354(struct e1000_hw *hw, bool *status) > > A comment and the function itself; no other references. In what tree are you > seeing a reference to igb_get_eee_status_i354? > > - Josh Triplett
Thanks Josh, My mistake. I was looking at the "get" not "set" function. However, this is still not the right fix for the driver as the actual problem is the missing code that is supposed to be calling that function in igb_get_eee in igb_ethtool.c. The function needs to stay. I will submit the correct fix ASAP. Thanks, Carolyn ------------------------------------------------------------------------------ Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk _______________________________________________ E1000-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
