From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj2-f11.google.com (mail-pj2-f11.google.com [74.125.227.139]) (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 4A1EF30DECE for ; Sun, 2 Aug 2026 18:25:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.227.139 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785695157; cv=none; b=srQsymnC78mmFSuYi8sQpjnjC81oMBBKZWUjYJ/TnDjgo1CgdDekktdlH4/XcamrwEoW6PdNiTq78hSyZKRwctBQVppP0vEWCYQxmqkCvCY8NSkiYgAbZdvyfMlcSre0Ybttt9uh7hnPInoZgiPGBaj+pDxHhMAXztnjrL0vC0A= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785695157; c=relaxed/simple; bh=miFaafVCXNwSq+nl8Kau8j0yfw0gqzbw8XlRunQpNTg=; h=Date:From:To:CC:Subject:In-Reply-To:References:Message-ID: MIME-Version:Content-Type; b=WhJYv+tMeUCWmI3nOI+HjqOEMSluR+WsJ88Jvag46hFsa6fUcO3w1FmpcuQdP+cool73i7uFspaLJZGq5RZETIU+wEMi2BQPsjxNtdm5I8Ng2+/uNDHEKl+GPrTakRxdOx8QD6rzjkT4CtWWcu8r/yJ8IiTuLwPj1g2BhHYcmS8= 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=N/qVa73x; arc=none smtp.client-ip=74.125.227.139 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="N/qVa73x" Received: by mail-pj2-f11.google.com with SMTP id d9443c01a7336-2ceb5a39c48so8716875ad.1 for ; Sun, 02 Aug 2026 11:25:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785695155; x=1786299955; darn=vger.kernel.org; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to:content-type; bh=F7s3GZYSnT0cjOvsmd+6nT8896EOTxni+8aCua2o/oc=; b=N/qVa73xU8RNQ3mmC9UYKsojEB88PfmggNYXMEHeaxME238RDYfVWkUEk97kfb+RYU 84U04adTYm2THkZRZk9biZR9I6AXLfjfok48qNMD2f1Ob522Wvly4t6/nsUpnDElQOv3 qhIGqw8YOm7wDp4enivkP3rVXSeGLxOZEkhqmMxgxDDUX6PtOcdpFTTsdftTlD9wyxiw n7UkbGd4TUi4c9qozeCWEyl7Y4LdIZe+iIR0D5HZgXu0mEEpOsVAlF+5R1L9tJdRENd5 J4R31vcmPCee7hJw5XME4zEorEvpen/KtZzNLjCFJ5XXeekLocv2bJtLHJsg1YLHe+FD 5KbA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785695155; x=1786299955; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=F7s3GZYSnT0cjOvsmd+6nT8896EOTxni+8aCua2o/oc=; b=qnd3GxvdP+nLcBzOrXxWryGEhl2GkrZtelOzLCy8NcnZKj2iWwpQHy3IyFT/0M7gIP KXJs1WRnu4aDYtaapxHOffp15gYlw4xcAUUXajVhgvxe9kmsmRN9S2VcpoDpr8Yu3rjR kTOdxYqHQsvvmHmOBf+B4s2VOEBgdkciO2Wku+pBRdZGmqHKAS/vAnAJ2yLjr7Xxzmwy o0J/ydD634mBOEpZf4agGezyAPl6aCqd5a5cy8SgIT5NNY6XRgQ6e5alSQNlkk9ozBR1 EvtSgQFac8KO4JDmciOujNdgDW++LpbLfAaXKF0ocIJ4+7W+buMOCxe8ql+Y264thjWY jedw== X-Forwarded-Encrypted: i=1; AHgh+RryEpUnaK1NibWZwFk9y6CHK+xWTu/VgRKW2afT2JF8O6n056tRWNm4dcRyHiSHfGXw/1l65Ab7TanR4vM9ihtyIQ==@vger.kernel.org X-Gm-Message-State: AOJu0YwL8muxlMHuyKFg2ML7SAmoksJlscytSH8bFwJfX93wRrEPV8Oq d3YMWE8pBHTQkDVdNkd8gGQroTDdmkcwWCU4jkgaG0r96Sl2b2ijO1hn X-Gm-Gg: AR+sD13RWuJPamQDjBAEiNDtFph+yEqeEmsIOeuoAQiCZ1ARI+WOamKjNVMnqobDit6 QlmHjF/lJXS5sqnb1p7g2qwNsCg79eIEZj0qPPb+G/72ojz1BnddenGAOG8H12yHZ+H3KvKPW46 BHDUATJ906cT4IQrbMIfHr96w7yO1lISdzi80RzgMpHPD4Llj++fAdWF5440YrS7QOy19+UOfiV ZnvDBKu2hol0cZI2tV69k0cvQAHvSubKL43WPp9OB9uj+J46NesXo8VLuIxdMsrb6MAINejbHeG IV57ACVuaZc5B+vr1tL+e1H4EEbDbEFbZwBIbR3i19JYbmOxiX8MKLwCBuRJQ+Pni9/Fj3whi/a itPxhh1iDZOyjlpFwCJnWHnjojy4c6qN8UORA3d2OTHKuoLckdcUYQpCMOUBYYjiFSRaSBke9Kg 5YHtA/bMJwif0+zcC7wMEXH/VIGlymjYj4iVLQNYUmvfLFKM8w+zA0QOPUb3WA6i/3 X-Received: by 2002:a17:903:b4b:b0:2cc:dacc:fe27 with SMTP id d9443c01a7336-2d0522984dbmr69929465ad.30.1785695155310; Sun, 02 Aug 2026 11:25:55 -0700 (PDT) Received: from ehlo.thunderbird.net ([2401:5a0:1000:10:233::14]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2d04b0eb7b7sm27443245ad.39.2026.08.02.11.25.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 02 Aug 2026 11:25:54 -0700 (PDT) Date: Mon, 03 Aug 2026 02:25:45 +0800 From: Coia Prant To: Andrew Lunn CC: Andrew Lunn , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Vinod Koul , Maxime Chevallier , Maxime Coquelin , Alexandre Torgue , Lad Prabhakar , Romain Gantois , Heiner Kallweit , Neil Armstrong , Russell King , Shawn Lin , David Heidelberg , netdev@vger.kernel.org, linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-renesas-soc@vger.kernel.org, Jiawen Wu Subject: =?US-ASCII?Q?Re=3A_=5BPATCH_v2_06/10=5D_net=3A_pcs=3A_xpcs=3A_add_?= =?US-ASCII?Q?ANRESTART_support_for_SGMII_link_recovery?= User-Agent: K-9 Mail for Android In-Reply-To: <2886b24e-1c3d-48f7-8b22-9620fa0404f7@lunn.ch> References: <20260801142244.679437-2-coiaprant@gmail.com> <20260801142244.679437-8-coiaprant@gmail.com> <2886b24e-1c3d-48f7-8b22-9620fa0404f7@lunn.ch> Message-ID: <2E676CD9-82F3-404A-B24D-27679042783B@gmail.com> Precedence: bulk X-Mailing-List: linux-renesas-soc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On August 2, 2026 10:30:52 PM GMT+08:00, Andrew Lunn wro= te: >On Sun, Aug 02, 2026 at 11:11:40AM +0800, Coia Prant wrote: >> Hi Andrew, >>=20 >> Thanks for the careful review=2E >>=20 >> > This probably needs more wide spread testing=2E I would be afraid tha= t >> > if you kick off an AN cycle, you loose the link until it completes? >> > You would of noticed that on your hardware, but other hardware >> > obviously behaves differently=2E=2E=2E >>=20 >> The trigger condition is: link is down AND an_complete is set=2E > >Ah, i missed the "link is down" check=2E That makes it safer=2E > >I still find it interesting that this devices is different=2E Has the >silicon vendor done the integration of the licensed IP wrong? > >> Regarding testing: >> - Jiawen Wu has tested this on Wangxun NICs and reported no regression >> (Tested-by tag included in the patch)=2E > >Probably Maxime will test on the platform he has, but i don't know if >any use the XPCS=2E > > Andrew Hi Andrew, Thanks for the follow-up=2E Honestly, I'm not sure why the RK3568 XPCS behaves this way=2E I don't hav= e other DW XPCS hardware to compare against, so I can't say whether this is a silicon integration quirk or something else=2E What I can confirm is that on RK3568, the PCS does not restart AN automatically when the link drops and comes back=2E Without this ANRESTART change, the link stays dead forever=2E I agree that wider testing is needed=2E If Maxime or anyone else finds a regression on other hardware, we can revisit and condition it with a platform-specific flag=2E For now, the change is limited to the SGMII recovery path with a safe condition (link down + an_complete), so the risk should be minimal=2E Thanks, Coia From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id ECA8FC55171 for ; Sun, 2 Aug 2026 18:26:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:References: In-Reply-To:Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=XugwvvI9E/3DHBZMlk4Oa66l0DmBPUsswEWID0dfwjE=; b=g2Xg2cXGlUc+PA cteJQ0JrkUotsAeZO5K5KBfT3qZTUjrei/Qf9MGnqmp/8dFKW/SYuWsujBISSWLRUZlUgi/snsax+ iCoYAzpB+7gMqG+w2O9Lo+tCDxllnVNlKw6k1oCvZjY62Z5xgZ+rMlQ5SGisOpO8aP6cGSrEDtaqr ALmjeI9cqe1s5rCgImiSkBFUJikg9SawNl9j3ERncS1FRK1nZRTx5PuVhL92zVVFw4zoJVGNa0v60 9UNHLdOZasb8EvlqpjHccAzsY3cMMYmYWK1m7P/S3UcoewCY5ExnHk/33BQc/jHruGyiN2HtlQRZK FPiq+zqgHU9P31boEmYA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wqaso-0000000FvqN-3KPI; Sun, 02 Aug 2026 18:25:58 +0000 Received: from mail-pj2-x0b.google.com ([2607:f8b0:4864:39::b]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wqasm-0000000FvpN-2gPC for linux-rockchip@lists.infradead.org; Sun, 02 Aug 2026 18:25:57 +0000 Received: by mail-pj2-x0b.google.com with SMTP id d9443c01a7336-2ceb5a39c48so8716825ad.1 for ; Sun, 02 Aug 2026 11:25:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785695155; x=1786299955; darn=lists.infradead.org; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to:content-type; bh=F7s3GZYSnT0cjOvsmd+6nT8896EOTxni+8aCua2o/oc=; b=ZivFzCHugaAxc/wqeCsPDqBC4Tgcqmv3FcCGZ6dbz9CLD7WG5l4MwqBdiT5Uyvlugd gML3OPrHyD39PjYkMHDNq1ARR00uf8qXALk3eB047ecDjj+di+d3Tjede8svJvSOTkk6 fji8yYHltCuclWFCpZ8gp+WO2SO5BEi9CahSGuLtTg/Cv1MU5qgYzxM5Ahhfvf7ei7/V s/BcaqWAdEzlbrHY1qJDEZdDZhg5fv94OSHT2kl+C6c8qZppJ+i+RpZeLhiJl75d3jax vhY1pQEVgVkRSUqcDZYJSPS4Hhr2GvZ2CtuU3smrQmBjBUuiPpLlVg4Jlvj5oJWZ7+3O x8dA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785695155; x=1786299955; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=F7s3GZYSnT0cjOvsmd+6nT8896EOTxni+8aCua2o/oc=; b=l29GSRrxqssAZP8QEdInO44LvM/YNENnFzo3lCvceHvUWfiIclB8FSp979GhouqhVC uAr+0/H2p92cKdK2LfYMqiOb2LU+6FAS81wSNnrPgRxs2d/eX8tLB3DC4dfrMM2vSd+Q SqJ0IB7cJTKEkFjpeZU2Xh7mDHC1Ss0cg4GVzqVIyX6ZRDnJg0oPu7Wsfv6QYKt/zkE/ CIeqvbx116gmnoucero1OHUpZ5RwpE2QG0ynVpExm6orIb7AK2PtqCI5wpuniOOMHSav Ob0P9F2bctjV/eC42t4cL2NWEaUjE75ZGXmC3ESVKDSAfmTK/OWC22N78kltG7uITnLH /4FA== X-Forwarded-Encrypted: i=1; AHgh+RpO9ivlho0PvU9BK5tXBkN1dpblgNkYiZVW8DzCgX5czHiEadQ0NF/yCZaYHjRTQB1VAaUW+pd0LEsNXW3eNw==@lists.infradead.org X-Gm-Message-State: AOJu0Yzo+JLdP84EOqOFbbL3n1TDMYkHlm9MjCwC2SBvtbJmYvL8q7Ld 4DklFWD102KEa7gk4FwXn5pewMgUNI+vDvi5IKXiz+aL8HABXrB4PI1R X-Gm-Gg: AR+sD12vvzYsGjqCtVRpTL6chv70Su4ZAg76pFlSOaSCb5I6xBA4dlBTMC53ALC/XEL KgMCynLboqnvF0gsiZMlnQAQTN8NdLk5m50PjNFZ825EFbPxKw2ag8Chex1V9GgZ+RG7IRK96q6 6KBYL3ddMuPHJOKaq4GZ9vG7cGMUht/WGc/PB/dzbvWmrke2D2vgaGoOY3BNh5QXXA2tq/QR94g WHIjkDc3P0NSrmCVyVdEm8MavdjAbZCqnRYro/pcWl/ponDIgYVsFui6+2Us8lt8z4xJwxy0+Dy xBMKWuzhQL0KSeF6sfUUoH+DLkSk4PRV6zLAZR3DnCROyeOe7FzFGrOtwQGFT4ViTwOlVs0kTTN uqMb6SEK1/qsOaVh1wYykgN6u4Znqmq0xvX1F6aulfpJt1/4y1Hs0Bio+Y8QvaMAr5/uJILMDRi SRUlxRJvDHhW9f/Gr5cTMBVLPO3xiKlzay9W6WZdkxCeBN3GA5B+K08RjVsuYE78y6 X-Received: by 2002:a17:903:b4b:b0:2cc:dacc:fe27 with SMTP id d9443c01a7336-2d0522984dbmr69929465ad.30.1785695155310; Sun, 02 Aug 2026 11:25:55 -0700 (PDT) Received: from ehlo.thunderbird.net ([2401:5a0:1000:10:233::14]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2d04b0eb7b7sm27443245ad.39.2026.08.02.11.25.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 02 Aug 2026 11:25:54 -0700 (PDT) Date: Mon, 03 Aug 2026 02:25:45 +0800 From: Coia Prant To: Andrew Lunn CC: Andrew Lunn , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Vinod Koul , Maxime Chevallier , Maxime Coquelin , Alexandre Torgue , Lad Prabhakar , Romain Gantois , Heiner Kallweit , Neil Armstrong , Russell King , Shawn Lin , David Heidelberg , netdev@vger.kernel.org, linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-renesas-soc@vger.kernel.org, Jiawen Wu Subject: =?US-ASCII?Q?Re=3A_=5BPATCH_v2_06/10=5D_net=3A_pcs=3A_xpcs=3A_add_?= =?US-ASCII?Q?ANRESTART_support_for_SGMII_link_recovery?= User-Agent: K-9 Mail for Android In-Reply-To: <2886b24e-1c3d-48f7-8b22-9620fa0404f7@lunn.ch> References: <20260801142244.679437-2-coiaprant@gmail.com> <20260801142244.679437-8-coiaprant@gmail.com> <2886b24e-1c3d-48f7-8b22-9620fa0404f7@lunn.ch> Message-ID: <2E676CD9-82F3-404A-B24D-27679042783B@gmail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260802_112556_685163_8FAC69F6 X-CRM114-Status: GOOD ( 17.02 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org On August 2, 2026 10:30:52 PM GMT+08:00, Andrew Lunn wrote: >On Sun, Aug 02, 2026 at 11:11:40AM +0800, Coia Prant wrote: >> Hi Andrew, >> >> Thanks for the careful review. >> >> > This probably needs more wide spread testing. I would be afraid that >> > if you kick off an AN cycle, you loose the link until it completes? >> > You would of noticed that on your hardware, but other hardware >> > obviously behaves differently... >> >> The trigger condition is: link is down AND an_complete is set. > >Ah, i missed the "link is down" check. That makes it safer. > >I still find it interesting that this devices is different. Has the >silicon vendor done the integration of the licensed IP wrong? > >> Regarding testing: >> - Jiawen Wu has tested this on Wangxun NICs and reported no regression >> (Tested-by tag included in the patch). > >Probably Maxime will test on the platform he has, but i don't know if >any use the XPCS. > > Andrew Hi Andrew, Thanks for the follow-up. Honestly, I'm not sure why the RK3568 XPCS behaves this way. I don't have other DW XPCS hardware to compare against, so I can't say whether this is a silicon integration quirk or something else. What I can confirm is that on RK3568, the PCS does not restart AN automatically when the link drops and comes back. Without this ANRESTART change, the link stays dead forever. I agree that wider testing is needed. If Maxime or anyone else finds a regression on other hardware, we can revisit and condition it with a platform-specific flag. For now, the change is limited to the SGMII recovery path with a safe condition (link down + an_complete), so the risk should be minimal. Thanks, Coia _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 8AD74C55162 for ; Sun, 2 Aug 2026 18:26:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:References: In-Reply-To:Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=kq3nnJZlY7pLtAMWoYb+BG7mbicpI+1ULe9oOkZQ8M0=; b=bFtDjqVTYBU5de LeOKOj2lLhih4tCg112oqpcYxCfgHWLcJLIe/MxjQUpDFMwTvWox6TKA688MCdYNTfAusMRd02HFJ uBnQSkA3TwMtjEoPKV0k7o8sRStRgunV00xodARzYIEjlmWXbLm7KxdlI6LTgIKpoGVXH0KbLPVvc BCiB2oO1/dHHwAlnOZKp6Yp7mwZP8YM2vJgi1u79XjvD+laEo5oI3+ScLmiTi3D0iB3RuSJ0ZPIuc pnLmJahayFkwHctVnZYohN98yk7Rxb38pO9E+rYr1Pc+MwNIS0hg/79fLZxwZZWwcwic1jce+DpMv w9RwkdVKxI9zADkEkw9w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wqasq-0000000FvrF-0BGj; Sun, 02 Aug 2026 18:26:00 +0000 Received: from mail-pj2-x04.google.com ([2607:f8b0:4864:39::4]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wqasm-0000000FvpO-2jdF for linux-phy@lists.infradead.org; Sun, 02 Aug 2026 18:25:58 +0000 Received: by mail-pj2-x04.google.com with SMTP id d9443c01a7336-2cc80544c74so7550835ad.0 for ; Sun, 02 Aug 2026 11:25:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785695155; x=1786299955; darn=lists.infradead.org; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to:content-type; bh=F7s3GZYSnT0cjOvsmd+6nT8896EOTxni+8aCua2o/oc=; b=ZivFzCHugaAxc/wqeCsPDqBC4Tgcqmv3FcCGZ6dbz9CLD7WG5l4MwqBdiT5Uyvlugd gML3OPrHyD39PjYkMHDNq1ARR00uf8qXALk3eB047ecDjj+di+d3Tjede8svJvSOTkk6 fji8yYHltCuclWFCpZ8gp+WO2SO5BEi9CahSGuLtTg/Cv1MU5qgYzxM5Ahhfvf7ei7/V s/BcaqWAdEzlbrHY1qJDEZdDZhg5fv94OSHT2kl+C6c8qZppJ+i+RpZeLhiJl75d3jax vhY1pQEVgVkRSUqcDZYJSPS4Hhr2GvZ2CtuU3smrQmBjBUuiPpLlVg4Jlvj5oJWZ7+3O x8dA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785695155; x=1786299955; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=F7s3GZYSnT0cjOvsmd+6nT8896EOTxni+8aCua2o/oc=; b=a9qkbgW1RTmzAI5P1iFUuqlChIGLtVMU+gn1eQ8mbwOVSGHGLYY+qeiggJ6/JeC89H VgnIYYggdrxzFg5EKAYnmRcO8nnFy1JbP4cSAk0+TrR6LavAanRNuqQ9VihTLdNNql+P 1DP9RUdMQF2Mt4egmn8qjSUjZMBRFDOhCfXY7gpOJPEFOR9lTIJQa7NalnIAKcTHhhoH x8UZpz/xQPS98A++MmbHk+GnGLmGvEBkx/FAxcfMbqV9wzAnlUOHV5CIm5x1rN4sdRzD WSaM3ePUxXWZq778jgJbzqDmrBws2oOVEp5S/4rpjkaLBENQL5GJ2rl4TrzO78tCbQoh ODqA== X-Forwarded-Encrypted: i=1; AHgh+Rq8/lK+avnY/cP6g7jx5/VBvqxiA31bzB45/+QyZomQ/58Or5dQvJ2kND8l94YCtwif483WzjVyFK8=@lists.infradead.org X-Gm-Message-State: AOJu0YxLfT/frTlbNkVLLmoei7LD6ENSkPyXyPxTpva9IabukWYPVSHZ hxFedTzro+OQ10OzBiOUr4NifXgRW4JwiBUgXz0jN0AWK9rel3EXulRl X-Gm-Gg: AR+sD11oF9Ht2oisWpDlZU4gtYqlnnCEDJGHBfsHyrLroIodp4cF2vDAPnqSI4fsQk0 gVaDMd48JYK4lep8M5Vdisb5IHGl8PQgeRVVUUsA8rwxXJr5I/3XFTDFHBUrchG0q/SjmTnxWKL zEUQ+OcavSe/yiNDvD0X47Jtdd9GtNtZIiZSR3fChLR8Sm5WSYnw9e60HvTPsbYiBv34qKGS1bF E8/zmPP53BPhb/ZLovOV36yPuwDbCh9yx/lLygCfZcUjzLOOPSZ/5s+BUPhjJqBrckb9PA3fcIY g2T+g+tbpi982CHFlVB8SKHk/Qh47BjHjTj5hHTiaDXXbasJxzFcx5iGpyYx3jqlKpFbSIPn1Ru WWdMU3oQjCU1mXQjG1KM5yBBGr5WwhNH0UWoX/s+IiT+zE/Fk2ZVGlTIbDf7+vcOJmWGpuUof3L YppiM13Hpxj57Wpa8+DSerF1vpkfmQ5/TpfTUEEApNtI+CQYslsSm52+ZCYgReCjnh X-Received: by 2002:a17:903:b4b:b0:2cc:dacc:fe27 with SMTP id d9443c01a7336-2d0522984dbmr69929465ad.30.1785695155310; Sun, 02 Aug 2026 11:25:55 -0700 (PDT) Received: from ehlo.thunderbird.net ([2401:5a0:1000:10:233::14]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2d04b0eb7b7sm27443245ad.39.2026.08.02.11.25.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 02 Aug 2026 11:25:54 -0700 (PDT) Date: Mon, 03 Aug 2026 02:25:45 +0800 From: Coia Prant To: Andrew Lunn CC: Andrew Lunn , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Vinod Koul , Maxime Chevallier , Maxime Coquelin , Alexandre Torgue , Lad Prabhakar , Romain Gantois , Heiner Kallweit , Neil Armstrong , Russell King , Shawn Lin , David Heidelberg , netdev@vger.kernel.org, linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-renesas-soc@vger.kernel.org, Jiawen Wu Subject: =?US-ASCII?Q?Re=3A_=5BPATCH_v2_06/10=5D_net=3A_pcs=3A_xpcs=3A_add_?= =?US-ASCII?Q?ANRESTART_support_for_SGMII_link_recovery?= User-Agent: K-9 Mail for Android In-Reply-To: <2886b24e-1c3d-48f7-8b22-9620fa0404f7@lunn.ch> References: <20260801142244.679437-2-coiaprant@gmail.com> <20260801142244.679437-8-coiaprant@gmail.com> <2886b24e-1c3d-48f7-8b22-9620fa0404f7@lunn.ch> Message-ID: <2E676CD9-82F3-404A-B24D-27679042783B@gmail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260802_112556_686909_FF0D7AFB X-CRM114-Status: GOOD ( 16.80 ) X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org On August 2, 2026 10:30:52 PM GMT+08:00, Andrew Lunn wrote: >On Sun, Aug 02, 2026 at 11:11:40AM +0800, Coia Prant wrote: >> Hi Andrew, >> >> Thanks for the careful review. >> >> > This probably needs more wide spread testing. I would be afraid that >> > if you kick off an AN cycle, you loose the link until it completes? >> > You would of noticed that on your hardware, but other hardware >> > obviously behaves differently... >> >> The trigger condition is: link is down AND an_complete is set. > >Ah, i missed the "link is down" check. That makes it safer. > >I still find it interesting that this devices is different. Has the >silicon vendor done the integration of the licensed IP wrong? > >> Regarding testing: >> - Jiawen Wu has tested this on Wangxun NICs and reported no regression >> (Tested-by tag included in the patch). > >Probably Maxime will test on the platform he has, but i don't know if >any use the XPCS. > > Andrew Hi Andrew, Thanks for the follow-up. Honestly, I'm not sure why the RK3568 XPCS behaves this way. I don't have other DW XPCS hardware to compare against, so I can't say whether this is a silicon integration quirk or something else. What I can confirm is that on RK3568, the PCS does not restart AN automatically when the link drops and comes back. Without this ANRESTART change, the link stays dead forever. I agree that wider testing is needed. If Maxime or anyone else finds a regression on other hardware, we can revisit and condition it with a platform-specific flag. For now, the change is limited to the SGMII recovery path with a safe condition (link down + an_complete), so the risk should be minimal. Thanks, Coia -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy