"Brandeburg, Jesse" <[email protected]> writes: > On Mon, 18 Oct 2010, Julian Stecklina wrote: > >> Hello, >> >> the attached patch removes 3 pointless lines in the igbvf driver. There >> is no functional change. > > thanks for your patch, but the list strips non-text attachments, can you > please post inline or as a text attachment?
It strips text/x-diff attachments... :-/ Here is it inline: commit 469008ac8cf33374c6f2a56bc92b9540b50d40cf Author: Julian Stecklina <[email protected]> Date: Mon Oct 18 16:05:42 2010 +0200 Removed unused variable in igbvf. Signed-off-by: Julian Stecklina <[email protected]> diff --git a/drivers/net/igbvf/netdev.c b/drivers/net/igbvf/netdev.c index 098f966..faf5a16 100644 --- a/drivers/net/igbvf/netdev.c +++ b/drivers/net/igbvf/netdev.c @@ -1853,8 +1853,6 @@ static void igbvf_watchdog_task(struct work_struct *work) if (link) { if (!netif_carrier_ok(netdev)) { - bool txb2b = 1; - mac->ops.get_link_up_info(&adapter->hw, &adapter->link_speed, &adapter->link_duplex); @@ -1864,11 +1862,9 @@ static void igbvf_watchdog_task(struct work_struct *work) adapter->tx_timeout_factor = 1; switch (adapter->link_speed) { case SPEED_10: - txb2b = 0; adapter->tx_timeout_factor = 16; break; case SPEED_100: - txb2b = 0; /* maybe add some timeout factor ? */ break; } ------------------------------------------------------------------------------ Download new Adobe(R) Flash(R) Builder(TM) 4 The new Adobe(R) Flex(R) 4 and Flash(R) Builder(TM) 4 (formerly Flex(R) Builder(TM)) enable the development of rich applications that run across multiple browsers and platforms. Download your free trials today! http://p.sf.net/sfu/adobe-dev2dev _______________________________________________ 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
