From nobody Mon Apr 6 10:32:30 2026 Received: from smtpout-04.galae.net (smtpout-04.galae.net [185.171.202.116]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 25A733F786B for ; Thu, 19 Mar 2026 18:17:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.171.202.116 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773944245; cv=none; b=Q0ud7AWbYKJmK1Lh8yqJz6enkPxM9SuwqMm4V32syiUjj0vOYVp1KHaFH57oMotnYBFIMjBnQOptb1ZkKnEr1uxrWQqq0xms0XCtdF+DaJIa2isPW55tdT1L+nPNwMijqd58xCj+auzidxckOWiQstgC3EOkqSbeexB65SLKc/Q= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773944245; c=relaxed/simple; bh=VmoCxf/LKpudsCGR8vlmAvnLPDMFMQ+wNAJlA+M6mf0=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=ew7nNptFiwmSyEXD9qPCUL3mNwQrZkbwFbf32PcOz7Y823GvHIMR00Op6Q6rLCe47gbuZYqFlodkcr+nuPIg1HNMOiwrTuwoqlAywycgqxADX9WQUVwTadlofzSfId9GnSXFxQBw5XB7yFBOdZlQ2NV7JaazVdMUjV/8xBxXdbI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=XPencwRH; arc=none smtp.client-ip=185.171.202.116 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="XPencwRH" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-04.galae.net (Postfix) with ESMTPS id 7C213C5507F; Thu, 19 Mar 2026 18:17:45 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 5D53760050; Thu, 19 Mar 2026 18:17:20 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 792D110450A96; Thu, 19 Mar 2026 19:17:15 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1773944239; h=from:subject:date:message-id:to:cc:mime-version: content-transfer-encoding; bh=GhDTKiA+AQMLigmNF3JlnBZbT990XvOcqXq4KBKI+Vs=; b=XPencwRHT23THIBY9L55xww8D260XMTawse+FF94oFRQo6piA73Ps13S5GSsDCW6gV+RUz JLep0pmxd8U1XMwluUjvb1SZVHGfw5ENwJkyLQJZ0KJ7Ow1SXDHLTTU9aHtOxNGvHXYfBO k+4YlK01s7HlQWmJOURAptAItd54RsA9tq3L5GAIyQsfSEDaLqSNqc2BdBsphV5nZvX0xa 8CYRzSLsPZ5QBVzZg804urnwPkTU0gJ3gEO5evGsmhBjU/YGwVJe4wrVaszNy/EHWW5MKT LXCmkGWnx0FfigV0oXqYRaJLKf0G4lV/Tph1ehi3kaIZoYPp5+LH2ZwRs/davQ== From: Maxime Chevallier To: Andrew Lunn , Jakub Kicinski , davem@davemloft.net, Eric Dumazet , Paolo Abeni , Simon Horman , Woojung Huh , Vladimir Oltean , UNGLinuxDriver@microchip.com Cc: Maxime Chevallier , thomas.petazzoni@bootlin.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next] net: dsa: microchip: Don't embed struct phy_device to maintain the port state Date: Thu, 19 Mar 2026 19:17:04 +0100 Message-ID: <20260319181705.1576679-1-maxime.chevallier@bootlin.com> X-Mailer: git-send-email 2.49.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Last-TLS-Session-Version: TLSv1.3 Content-Type: text/plain; charset="utf-8" The KSZ9477 maintains the SGMII port's state for speed, duplex and link status to be able to fixup the accesses to its internal older version of the Designware XPCS. However, it does so by embedding a full instance of struct phy_device, only to use the 'speed', 'link' and 'duplex' fields. This is also only used for the SGMII port, it's otherwise unused for all other regular ports. Replace that with simple int/bool values. Signed-off-by: Maxime Chevallier Reviewed-by: Simon Horman --- drivers/net/dsa/microchip/ksz9477.c | 17 ++++++++--------- drivers/net/dsa/microchip/ksz_common.c | 2 +- drivers/net/dsa/microchip/ksz_common.h | 4 +++- 3 files changed, 12 insertions(+), 11 deletions(-) diff --git a/drivers/net/dsa/microchip/ksz9477.c b/drivers/net/dsa/microchi= p/ksz9477.c index 5facffbb9c9a..5416016b33e0 100644 --- a/drivers/net/dsa/microchip/ksz9477.c +++ b/drivers/net/dsa/microchip/ksz9477.c @@ -224,14 +224,13 @@ static int ksz9477_pcs_read(struct mii_bus *bus, int = phy, int mmd, int reg) else duplex =3D DUPLEX_HALF; =20 - if (!p->phydev.link || - p->phydev.speed !=3D speed || - p->phydev.duplex !=3D duplex) { + if (!p->link || p->speed !=3D speed || + p->duplex !=3D duplex) { u16 ctrl; =20 - p->phydev.link =3D 1; - p->phydev.speed =3D speed; - p->phydev.duplex =3D duplex; + p->link =3D true; + p->speed =3D speed; + p->duplex =3D duplex; port_sgmii_r(dev, port, mmd, MII_BMCR, &ctrl); ctrl &=3D BMCR_ANENABLE; @@ -241,10 +240,10 @@ static int ksz9477_pcs_read(struct mii_bus *bus, int = phy, int mmd, int reg) ctrl); } } else { - p->phydev.link =3D 0; + p->link =3D false; } } else if (reg =3D=3D MII_BMSR) { - p->phydev.link =3D !!(val & BMSR_LSTATUS); + p->link =3D !!(val & BMSR_LSTATUS); } } =20 @@ -557,7 +556,7 @@ int ksz9477_r_phy(struct ksz_device *dev, u16 addr, u16= reg, u16 *data) val =3D 0x0700; break; case MII_STAT1000: - if (p->phydev.speed =3D=3D SPEED_1000) + if (p->speed =3D=3D SPEED_1000) val =3D 0x3800; else val =3D 0; diff --git a/drivers/net/dsa/microchip/ksz_common.c b/drivers/net/dsa/micro= chip/ksz_common.c index c517478cc476..144373e13bea 100644 --- a/drivers/net/dsa/microchip/ksz_common.c +++ b/drivers/net/dsa/microchip/ksz_common.c @@ -3983,7 +3983,7 @@ static void ksz9477_phylink_mac_link_up(struct phylin= k_config *config, if (dev->info->internal_phy[port]) return; =20 - p->phydev.speed =3D speed; + p->speed =3D speed; =20 ksz_port_set_xmii_speed(dev, port, speed); =20 diff --git a/drivers/net/dsa/microchip/ksz_common.h b/drivers/net/dsa/micro= chip/ksz_common.h index 929aff4c55de..18f13ee9c7b6 100644 --- a/drivers/net/dsa/microchip/ksz_common.h +++ b/drivers/net/dsa/microchip/ksz_common.h @@ -129,7 +129,9 @@ struct ksz_port { bool learning; bool isolated; int stp_state; - struct phy_device phydev; + int speed; + int duplex; + bool link; =20 u32 fiber:1; /* port is fiber */ u32 force:1; --=20 2.49.0