From nobody Wed Apr 8 04:39:33 2026 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F16ED3D47C3 for ; Tue, 10 Mar 2026 17:38:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.43 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773164345; cv=none; b=G5RK353gp5Y+iZnqNgKCtHZeVlFDJanDJhY+JVkXik5azIaDKLcdV9zBdgTWMZYMIISpzXR/zlPy+7Nw6rTqSKuk9TemejGhC+tyXAbqcrwfy5eFPuQLhqHtipBqnf+ArCW2SfWrDA5o/tnvcs/yRZ8lzlFLYAoGnf91GKleGDo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773164345; c=relaxed/simple; bh=6nh8hser+whvA1PfR1iGtoLj1pfTFR51/unCHyuL39U=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=X2lMziZdeSmwo0JSYug2/5699t1TAcxKl4j5vSXsPpCEof0pKAwCN+WzoMPFFs/aEjKWiHP1ZngDiLn7dLExVCzZaIpZNRlGLeQ2OONoVaKDQNyOZcxZjVdfvvZntz/aSYWETlqNnuruTd22pD8/a8T7oiZeqnhWvTNToETq25M= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=jJphveOm; arc=none smtp.client-ip=209.85.128.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="jJphveOm" Received: by mail-wm1-f43.google.com with SMTP id 5b1f17b1804b1-4852ff06541so34595465e9.2 for ; Tue, 10 Mar 2026 10:38:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1773164338; x=1773769138; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=K5kecdfbWVFCpvWANmeDGj+7c6bf15MwYHY69kbw3bw=; b=jJphveOmcZXg3YcEj4YNym7kkl38U68mgse3zolN6zCvkmgjDD4elBa2DJqQP2P3uO xZ3EKSrHNiYZ7HAAxwwnFLpK7U82WCJ4SEHyzmzeNEJmTKO3Z5O6FPBET8tA7C5AAkp3 ZZJPcQgjaAL0gHgZoHx94fss9jCzHRFvb7umouqiZje78H1vh5B4qQvfgRXEJ4kkWBOJ rlTracDFR+WK5SugtrFUaWuPKUDMf+yGUFeIQ2EFmjD52+JpgxLu5zwa2Fve6T2QPJYM zIImulIvla0noQvAWD01mCAKDLu5CVWvkGRcRFvhJfuag2GHP7PmmrTUE6Wuj9yi8aYL DnBQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1773164338; x=1773769138; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=K5kecdfbWVFCpvWANmeDGj+7c6bf15MwYHY69kbw3bw=; b=BUoMAUzQ55+QNwP+RarFU9yjdvBwUKBWRNk1+KR02LjJ5kDx3A+TMCWYcj+q7EGDhF zpBJPOtTP86CWxAhLr1s3GYD1jf+2tBgSj87l78LDVhc4sEtwQdiD9O12OwoJZwwOKbB oTWw5g/cHBQMgh44t3cStCotyqBzH7GpadoY5Y/TJbFsIwQuq7d2lkb42VV2b1vDul+5 c5zYo4jdqKraoA6FyhS2WLDu/g/h/JfRDzya7PXyjcO+bF4JlvORf3SDJlfaU+SYO0jN HVW1C1sAUIGLYnZfCyZX/t5YmTiqtgimBikQHzL4TXJ42BkKEfgPle2dcvSLOkSqsZnN vmXQ== X-Forwarded-Encrypted: i=1; AJvYcCVtKVD7XvtdyTryBSKwoNtauNF+iH5EsqfJOjx9pXC2qQGykXR2N67hTjOIFJKVE/oaFsRQ5/1mSvi4Mp0=@vger.kernel.org X-Gm-Message-State: AOJu0Yytz5kAYLICcHtoO0gBfkM/9si1uc11ybuM7xnqJ9zxJnZhS7fT lRRbIv8WizV/FxZkYRL9xu26di0gfiiLOxN8VK1L3sA+NPQBV8rBzeMo X-Gm-Gg: ATEYQzwF6OkznAzHdfZt1F8qccho6HwojedWKyg46gOGF4sCdYn1bmTcF5DNgpXxax8 vjtabFLRrBrZTB+FoPI1MzExImLEPCO+aZuF1BIyId1Pg5W86XnQlIqYGckUIpUgXkdwNesqyzg YV1gBUToCf8ynG8ydjvsN/YnHOAt/QWmelXpCLknbbJZ3AsL6S9LaP/s9kdJFTOmD4brMTGHQtO PLtu5e8eHj86cuimJYZs1UN1A7XNfEgtuwf1kKfVvqIwS9w6T6hasAy9nHwsl27AvRx32rXDqnN gSsIziJ5KiyPtYkQvNtnieXG/1Kb2O/Mw66ZtvwA9vdScc3RwgceW06lVB+r7j0jkAJ26FR6faR 9/Pd3qIChUvD+9J/Rxdg5VhseqbOI25U/zJNHo1s0RBh9svvry8nSamvsFgrj769z2ng5TVER8F UILbTaVX5nzAsre9SiTlCYgdg= X-Received: by 2002:a05:600c:1e2a:b0:47e:e57d:404 with SMTP id 5b1f17b1804b1-485269582efmr255465235e9.16.1773164338375; Tue, 10 Mar 2026 10:38:58 -0700 (PDT) Received: from nas.local ([2001:912:1ac0:1e00:c662:37ff:fe09:93df]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-48541a900easm103239295e9.9.2026.03.10.10.38.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Mar 2026 10:38:58 -0700 (PDT) From: Damien Dejean To: andrew@lunn.ch, krzk+dt@kernel.org, robh@kernel.org, kuba@kernel.org, maxime.chevallier@bootlin.com, pabeni@redhat.com Cc: netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, edumazet@google.com, davem@davemloft.net, hkallweit1@gmail.com, Damien Dejean Subject: [PATCH net-next v7 4/4] net: phy: realtek: add RTL8224 polarity support Date: Tue, 10 Mar 2026 18:38:46 +0100 Message-ID: <20260310173846.230923-5-dam.dejean@gmail.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260310173846.230923-1-dam.dejean@gmail.com> References: <20260310173846.230923-1-dam.dejean@gmail.com> 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 Content-Type: text/plain; charset="utf-8" The RTL8224 has a register to configure the polarity of every pair of each port. It provides device designers more flexbility when wiring the chip. Unfortunately, the register is left in an unknown state after a reset. Thus on devices where the bootloader don't initialize it, the driver has to do it to detect and use a link. The MDI polarity swap can be set in the device tree using the property enet-phy-pair-polarity. The u32 value is a bitfield where bit[0..3] control the polarity of pairs A..D. Signed-off-by: Damien Dejean --- drivers/net/phy/realtek/realtek_main.c | 77 ++++++++++++++++++++------ 1 file changed, 59 insertions(+), 18 deletions(-) diff --git a/drivers/net/phy/realtek/realtek_main.c b/drivers/net/phy/realt= ek/realtek_main.c index ee3f7318e933..a3691f9c19a5 100644 --- a/drivers/net/phy/realtek/realtek_main.c +++ b/drivers/net/phy/realtek/realtek_main.c @@ -172,6 +172,7 @@ #define RTL8224_SRAM_RTCT_LEN(pair) (0x8028 + (pair) * 4) =20 #define RTL8224_VND1_MDI_PAIR_SWAP 0xa90 +#define RTL8224_VND1_MDI_POLARITY_SWAP 0xa94 =20 #define RTL8366RB_POWER_SAVE 0x15 #define RTL8366RB_POWER_SAVE_ON BIT(12) @@ -1822,12 +1823,35 @@ static int rtl8224_cable_test_get_status(struct phy= _device *phydev, bool *finish return rtl8224_cable_test_report(phydev, finished); } =20 +static int rtl8224_package_modify_mmd(struct phy_device *phydev, int devad, + u32 regnum, u16 mask, u16 set) +{ + int val, ret; + + phy_lock_mdio_bus(phydev); + + val =3D __phy_package_read_mmd(phydev, 0, devad, regnum); + if (val < 0) { + ret =3D val; + goto exit; + } + + val &=3D ~mask; + val |=3D set; + + ret =3D __phy_package_write_mmd(phydev, 0, devad, regnum, val); + +exit: + phy_unlock_mdio_bus(phydev); + return ret; +} + static int rtl8224_mdi_config_order(struct phy_device *phydev) { struct device_node *np =3D phydev->mdio.dev.of_node; u8 port_offset =3D phydev->mdio.addr & 3; u32 order =3D 0; - int ret, val; + int ret; =20 ret =3D of_property_read_u32(np, "enet-phy-pair-order", &order); =20 @@ -1841,29 +1865,46 @@ static int rtl8224_mdi_config_order(struct phy_devi= ce *phydev) if (order & ~1) return -EINVAL; =20 - phy_lock_mdio_bus(phydev); - val =3D __phy_package_read_mmd(phydev, 0, MDIO_MMD_VEND1, - RTL8224_VND1_MDI_PAIR_SWAP); - if (val < 0) { - ret =3D val; - goto exit; - } + return rtl8224_package_modify_mmd(phydev, MDIO_MMD_VEND1, + RTL8224_VND1_MDI_PAIR_SWAP, + BIT(port_offset), + order ? BIT(port_offset) : 0); +} =20 - if (order) - val |=3D (1 << port_offset); - else - val &=3D ~(1 << port_offset); +static int rtl8224_mdi_config_polarity(struct phy_device *phydev) +{ + struct device_node *np =3D phydev->mdio.dev.of_node; + u8 offset =3D (phydev->mdio.addr & 3) * 4; + u32 polarity =3D 0; + int ret; =20 - ret =3D __phy_package_write_mmd(phydev, 0, MDIO_MMD_VEND1, - RTL8224_VND1_MDI_PAIR_SWAP, val); -exit: - phy_unlock_mdio_bus(phydev); - return ret; + ret =3D of_property_read_u32(np, "enet-phy-pair-polarity", &polarity); + + /* Do nothing if the property is not present */ + if (ret =3D=3D -EINVAL) + return 0; + + if (ret) + return ret; + + if (polarity & ~0xf) + return -EINVAL; + + return rtl8224_package_modify_mmd(phydev, MDIO_MMD_VEND1, + RTL8224_VND1_MDI_POLARITY_SWAP, + 0xf << offset, + polarity << offset); } =20 static int rtl8224_config_init(struct phy_device *phydev) { - return rtl8224_mdi_config_order(phydev); + int ret; + + ret =3D rtl8224_mdi_config_order(phydev); + if (ret) + return ret; + + return rtl8224_mdi_config_polarity(phydev); } =20 static int rtl8224_probe(struct phy_device *phydev) --=20 2.47.3