Skip to content

Commit 0c5e52c

Browse files
committed
Merge: tg3: driver update to v6.12
MR: https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9/-/merge_requests/5757 JIRA: https://issues.redhat.com/browse/RHEL-52671 JIRA: https://issues.redhat.com/browse/RHEL-52757 No real changes. Just code cleanups. Note that commit 0644646 ("tg3: Remove setting of RX software timestamp") is not included here, but in !5657. Signed-off-by: Michal Schmidt <mschmidt@redhat.com> Approved-by: José Ignacio Tornos Martínez <jtornosm@redhat.com> Approved-by: Chris von Recklinghausen <crecklin@redhat.com> Approved-by: CKI KWF Bot <cki-ci-bot+kwf-gitlab-com@redhat.com> Merged-by: Rado Vrbovsky <rvrbovsk@redhat.com>
2 parents f4903a9 + 23e4dd1 commit 0c5e52c

File tree

1 file changed

+8
-39
lines changed
  • drivers/net/ethernet/broadcom

1 file changed

+8
-39
lines changed

drivers/net/ethernet/broadcom/tg3.c

Lines changed: 8 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -4019,7 +4019,7 @@ static int tg3_power_up(struct tg3 *tp)
40194019

40204020
static int tg3_setup_phy(struct tg3 *, bool);
40214021

4022-
static int tg3_power_down_prepare(struct tg3 *tp)
4022+
static void tg3_power_down_prepare(struct tg3 *tp)
40234023
{
40244024
u32 misc_host_ctrl;
40254025
bool device_should_wake, do_low_power;
@@ -4263,7 +4263,7 @@ static int tg3_power_down_prepare(struct tg3 *tp)
42634263

42644264
tg3_ape_driver_state_change(tp, RESET_KIND_SHUTDOWN);
42654265

4266-
return 0;
4266+
return;
42674267
}
42684268

42694269
static void tg3_power_down(struct tg3 *tp)
@@ -4354,21 +4354,12 @@ static int tg3_phy_autoneg_cfg(struct tg3 *tp, u32 advertise, u32 flowctrl)
43544354
if (!err) {
43554355
u32 err2;
43564356

4357-
val = 0;
4358-
/* Advertise 100-BaseTX EEE ability */
4359-
if (advertise & ADVERTISED_100baseT_Full)
4360-
val |= MDIO_AN_EEE_ADV_100TX;
4361-
/* Advertise 1000-BaseT EEE ability */
4362-
if (advertise & ADVERTISED_1000baseT_Full)
4363-
val |= MDIO_AN_EEE_ADV_1000T;
4364-
4365-
if (!tp->eee.eee_enabled) {
4357+
if (!tp->eee.eee_enabled)
43664358
val = 0;
4367-
linkmode_zero(tp->eee.advertised);
4368-
} else {
4369-
mii_eee_cap1_mod_linkmode_t(tp->eee.advertised, val);
4370-
}
4359+
else
4360+
val = ethtool_adv_to_mmd_eee_adv_t(advertise);
43714361

4362+
mii_eee_cap1_mod_linkmode_t(tp->eee.advertised, val);
43724363
err = tg3_phy_cl45_write(tp, MDIO_MMD_AN, MDIO_AN_EEE_ADV, val);
43734364
if (err)
43744365
val = 0;
@@ -18089,7 +18080,6 @@ static int tg3_suspend(struct device *device)
1808918080
{
1809018081
struct net_device *dev = dev_get_drvdata(device);
1809118082
struct tg3 *tp = netdev_priv(dev);
18092-
int err = 0;
1809318083

1809418084
rtnl_lock();
1809518085

@@ -18113,32 +18103,11 @@ static int tg3_suspend(struct device *device)
1811318103
tg3_flag_clear(tp, INIT_COMPLETE);
1811418104
tg3_full_unlock(tp);
1811518105

18116-
err = tg3_power_down_prepare(tp);
18117-
if (err) {
18118-
int err2;
18119-
18120-
tg3_full_lock(tp, 0);
18121-
18122-
tg3_flag_set(tp, INIT_COMPLETE);
18123-
err2 = tg3_restart_hw(tp, true);
18124-
if (err2)
18125-
goto out;
18126-
18127-
tg3_timer_start(tp);
18128-
18129-
netif_device_attach(dev);
18130-
tg3_netif_start(tp);
18131-
18132-
out:
18133-
tg3_full_unlock(tp);
18134-
18135-
if (!err2)
18136-
tg3_phy_start(tp);
18137-
}
18106+
tg3_power_down_prepare(tp);
1813818107

1813918108
unlock:
1814018109
rtnl_unlock();
18141-
return err;
18110+
return 0;
1814218111
}
1814318112

1814418113
static int tg3_resume(struct device *device)

0 commit comments

Comments
 (0)