From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 14BC13B4E9D; Tue, 26 May 2026 21:52:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779832340; cv=none; b=g8rUroiPwUTrZ+P0TXk9sklYmU/0mDXYnQh68VQj2+YPsJYd7grXjE1TlduwGUiRKbKE09cSX+zG8bY5IMrpGu60GrWV2AXbAty3UjUpL4GMDWn2eNuzSatu+7Jngvc9yv/wz7Mdy/ulan/TKLY+C2a24RHZpWn0dbUmSTpLIM0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779832340; c=relaxed/simple; bh=nUpVC0QG3MB56nqPHHF1WjBXBZiBKt28DL44cOSKh08=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=DxoK9PS7Fc5+BUKwJV00bGDX2nVkjbQiWeBEUpigkRyzpDb4NIfe+YuUX7vjfY5GnlQBn2ZPsf8GkMt2yL3otHwWP0CUNOTeaj5n49RPWmx9F2YZ5mZwlkiBsEYy1nii97AisLA6wAE0bg0/8AuVVszFc0QQqDb8ueYs6vk1sWQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=geuR/Alf; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="geuR/Alf" Received: by smtp.kernel.org (Postfix) with ESMTPS id C760EC2BCF5; Tue, 26 May 2026 21:52:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1779832339; bh=nUpVC0QG3MB56nqPHHF1WjBXBZiBKt28DL44cOSKh08=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=geuR/AlfpcdR83wBXdQT9D/he2X/pkuwsL1eBjhyLFnSFdqz/1bU85xGBViMIsnSj AwE6jyOu1Tbi1v49KN8+as/mBT+rABYtin2Nmq4DWVw4zXoDHPgwC+CuM4kh8UQDIg 4IMAyQgtI8yNylYA8wxP3CJHXVkWY6vFpPBJmbSF7j5UajGqKqlET/eYvHmM9FN39j UpupFyH0wW1vX9HTNHrdFjLstWV02i+bnWodcn45r2hW2HQt/i9qpplHNWLU8nItyD fwp5shGJReb3+nTSJ7DIy20QGqZVDPpryXRifbCcVQT1wq/8JxtSW07DYqWztwB1UE IH5ey5LvHJKfg== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id BAD84CD6E41; Tue, 26 May 2026 21:52:19 +0000 (UTC) From: Ciprian Regus via B4 Relay Date: Wed, 27 May 2026 00:51:48 +0300 Subject: [PATCH net-next v2 03/10] net: ethernet: oa_tc6: add OA_TC6_BROKEN_PHY quirk flag Precedence: bulk X-Mailing-List: linux-doc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260527-adin1140-driver-v2-3-37e5c8d4e0a0@analog.com> References: <20260527-adin1140-driver-v2-0-37e5c8d4e0a0@analog.com> In-Reply-To: <20260527-adin1140-driver-v2-0-37e5c8d4e0a0@analog.com> To: Parthiban Veerasooran , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Jonathan Corbet , Shuah Khan , Andrew Lunn , Heiner Kallweit , Russell King , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, devicetree@vger.kernel.org, Ciprian Regus X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1779832338; l=5464; i=ciprian.regus@analog.com; s=20260330; h=from:subject:message-id; bh=MPRIc8vmHW/vXPKJZwVJUFyrwrfcbow4YLTfjncEe+U=; b=NjdT2brTF2fS6tAjlgi1zzBHEIQOL5rVJWErUbQWKGoej/DXgNvIM1tsgVPwKM5MvMBSKpqhQ njgbUjCcU0SCA22SqJS5BcFcpLVfa8Meof+Li0ShZXWe4HyquckY29k X-Developer-Key: i=ciprian.regus@analog.com; a=ed25519; pk=8WoNhI0kQcQUl8YqJO5ZevROYk9HP8lOIeIgIYgjfbc= X-Endpoint-Received: by B4 Relay for ciprian.regus@analog.com/20260330 with auth_id=703 X-Original-From: Ciprian Regus Reply-To: ciprian.regus@analog.com From: Ciprian Regus Some MAC-PHY devices need custom MDIO bus access functions to work around hardware issues. Add the OA_TC6_BROKEN_PHY quirk flag so drivers can opt in to skip oa_tc6's internal PHY init and manage the PHY themselves. When the flag is set, oa_tc6 skips MDIO bus registration, PHY discovery and PHY connection, leaving these to the driver. Drivers that do not set the flag retain the existing behavior. To avoid extending the oa_tc6_init() signature for each new option, convert it to take a config struct. Update lan865x and the framework documentation accordingly. Signed-off-by: Ciprian Regus --- v2 changelog: - Added the quirk flag field in the oa_tc6_config struct and a first value entry (OA_TC6_BROKEN_PHY) instead of the mii_bus struct. --- Documentation/networking/oa-tc6-framework.rst | 3 +-- drivers/net/ethernet/microchip/lan865x/lan865x.c | 6 +++++- drivers/net/ethernet/oa_tc6.c | 25 +++++++++++++++++------- include/linux/oa_tc6.h | 12 +++++++++++- 4 files changed, 35 insertions(+), 11 deletions(-) diff --git a/Documentation/networking/oa-tc6-framework.rst b/Documentation/networking/oa-tc6-framework.rst index fe2aabde923a..eaa5b4b85b34 100644 --- a/Documentation/networking/oa-tc6-framework.rst +++ b/Documentation/networking/oa-tc6-framework.rst @@ -453,8 +453,7 @@ Device drivers API The include/linux/oa_tc6.h defines the following functions: -.. c:function:: struct oa_tc6 *oa_tc6_init(struct spi_device *spi, \ - struct net_device *netdev) +.. c:function:: struct oa_tc6 *oa_tc6_init(struct oa_tc6_config *config); Initialize OA TC6 lib. diff --git a/drivers/net/ethernet/microchip/lan865x/lan865x.c b/drivers/net/ethernet/microchip/lan865x/lan865x.c index 0277d9737369..c509c8a3e321 100644 --- a/drivers/net/ethernet/microchip/lan865x/lan865x.c +++ b/drivers/net/ethernet/microchip/lan865x/lan865x.c @@ -332,6 +332,7 @@ static const struct net_device_ops lan865x_netdev_ops = { static int lan865x_probe(struct spi_device *spi) { + struct oa_tc6_config tc6_config = {}; struct net_device *netdev; struct lan865x_priv *priv; int ret; @@ -346,7 +347,10 @@ static int lan865x_probe(struct spi_device *spi) spi_set_drvdata(spi, priv); INIT_WORK(&priv->multicast_work, lan865x_multicast_work_handler); - priv->tc6 = oa_tc6_init(spi, netdev); + tc6_config.spi = spi; + tc6_config.netdev = netdev; + + priv->tc6 = oa_tc6_init(&tc6_config); if (!priv->tc6) { ret = -ENODEV; goto free_netdev; diff --git a/drivers/net/ethernet/oa_tc6.c b/drivers/net/ethernet/oa_tc6.c index baba5aad84df..7ae3639beadd 100644 --- a/drivers/net/ethernet/oa_tc6.c +++ b/drivers/net/ethernet/oa_tc6.c @@ -134,6 +134,7 @@ struct oa_tc6 { bool rx_buf_overflow; bool int_flag; bool prot_ctrl; + enum oa_tc6_quirk_flag quirk_flags; }; enum oa_tc6_header_type { @@ -580,6 +581,9 @@ static int oa_tc6_phy_init(struct oa_tc6 *tc6) { int ret; + if (tc6->quirk_flags & OA_TC6_BROKEN_PHY) + return 0; + ret = oa_tc6_check_phy_reg_direct_access_capability(tc6); if (ret) { netdev_err(tc6->netdev, @@ -616,6 +620,9 @@ static int oa_tc6_phy_init(struct oa_tc6 *tc6) static void oa_tc6_phy_exit(struct oa_tc6 *tc6) { + if (tc6->quirk_flags & OA_TC6_BROKEN_PHY) + return; + phy_disconnect(tc6->phydev); oa_tc6_mdiobus_unregister(tc6); } @@ -1277,24 +1284,28 @@ static int oa_tc6_check_ctrl_protection(struct oa_tc6 *tc6) /** * oa_tc6_init - allocates and initializes oa_tc6 structure. - * @spi: device with which data will be exchanged. - * @netdev: network device interface structure. + * @config: pointer to a caller-filled structure describing the MACPHY + * (SPI device, net_device, and config flags). * * Return: pointer reference to the oa_tc6 structure if the MAC-PHY * initialization is successful otherwise NULL. */ -struct oa_tc6 *oa_tc6_init(struct spi_device *spi, struct net_device *netdev) +struct oa_tc6 *oa_tc6_init(struct oa_tc6_config *config) { struct oa_tc6 *tc6; int ret; - tc6 = devm_kzalloc(&spi->dev, sizeof(*tc6), GFP_KERNEL); + if (!config) + return NULL; + + tc6 = devm_kzalloc(&config->spi->dev, sizeof(*tc6), GFP_KERNEL); if (!tc6) return NULL; - tc6->spi = spi; - tc6->netdev = netdev; - SET_NETDEV_DEV(netdev, &spi->dev); + tc6->spi = config->spi; + tc6->netdev = config->netdev; + tc6->quirk_flags = config->quirk_flags; + SET_NETDEV_DEV(tc6->netdev, &tc6->spi->dev); mutex_init(&tc6->spi_ctrl_lock); spin_lock_init(&tc6->tx_skb_lock); diff --git a/include/linux/oa_tc6.h b/include/linux/oa_tc6.h index 15f58e3c56c7..6c38bf49e2a7 100644 --- a/include/linux/oa_tc6.h +++ b/include/linux/oa_tc6.h @@ -12,7 +12,17 @@ struct oa_tc6; -struct oa_tc6 *oa_tc6_init(struct spi_device *spi, struct net_device *netdev); +enum oa_tc6_quirk_flag { + OA_TC6_BROKEN_PHY = BIT(0), +}; + +struct oa_tc6_config { + struct spi_device *spi; + struct net_device *netdev; + enum oa_tc6_quirk_flag quirk_flags; +}; + +struct oa_tc6 *oa_tc6_init(struct oa_tc6_config *config); void oa_tc6_exit(struct oa_tc6 *tc6); int oa_tc6_write_register(struct oa_tc6 *tc6, u32 address, u32 value); int oa_tc6_write_registers(struct oa_tc6 *tc6, u32 address, u32 value[], -- 2.43.0