Message ID | 1229717357.5496@xw6200 |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
From: "Matt Carlson" <mcarlson@broadcom.com> Date: Wed, 17 Dec 2008 16:33:37 -0800 > The first hunk of this patch inverts a flag that was accidentally > toggled as part of commit 0a459aac9d151c2e36ec65723b9b845b24c5cbc3. > > The second hunk of the patch removes the call to > device_may_wakeup() in the 5906 config detection path. At the > point of the call, the driver shouldn't be querying for WOL > capability. It should be detecting and setting it. > > Signed-off-by: Matt Carlson <mcarlson@broadcom.com> > Signed-off-by: Benjamin Li <benli@broadcom.com> > Signed-off-by: Michael Chan <mchan@broadcom.com> Applied. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c index 06bd2f4..bacbfc5 100644 --- a/drivers/net/tg3.c +++ b/drivers/net/tg3.c @@ -2237,7 +2237,7 @@ static int tg3_set_power_state(struct tg3 *tp, pci_power_t state) } } } else { - do_low_power = false; + do_low_power = true; if (tp->link_config.phy_is_low_power == 0) { tp->link_config.phy_is_low_power = 1; @@ -11544,8 +11544,7 @@ static void __devinit tg3_get_eeprom_hw_cfg(struct tg3 *tp) if (val & VCPU_CFGSHDW_ASPM_DBNC) tp->tg3_flags |= TG3_FLAG_ASPM_WORKAROUND; if ((val & VCPU_CFGSHDW_WOL_ENABLE) && - (val & VCPU_CFGSHDW_WOL_MAGPKT) && - device_may_wakeup(&tp->pdev->dev)) + (val & VCPU_CFGSHDW_WOL_MAGPKT)) tp->tg3_flags |= TG3_FLAG_WOL_ENABLE; goto done; }