On Tue, 2014-01-07 at 18:00 +0100, Julia Lawall wrote: > From: Julia Lawall <[email protected]> > > This driver does not need to adjust the power state on suspend, so the > call to pci_set_power_state in the resume function is a no-op. Drop it, to > make the code more understandable. > > Signed-off-by: Julia Lawall <[email protected]> Tested-by: Phil Schmitt <[email protected]> Signed-off-by: Aaron Brown <[email protected]>
> --- > drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c > b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c > index a5d3167..5709fb0 100644 > --- a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c > +++ b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c > @@ -3226,7 +3226,6 @@ static int ixgbevf_resume(struct pci_dev *pdev) > struct net_device *netdev = adapter->netdev; > u32 err; > > - pci_set_power_state(pdev, PCI_D0); > pci_restore_state(pdev); > /* > * pci_restore_state clears dev->state_saved so call > > -- > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html ------------------------------------------------------------------------------ CenturyLink Cloud: The Leader in Enterprise Cloud Services. Learn Why More Businesses Are Choosing CenturyLink Cloud For Critical Workloads, Development Environments & Everything In Between. Get a Quote or Start a Free Trial Today. http://pubads.g.doubleclick.net/gampad/clk?id=119420431&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
