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 AA720C55171 for ; Sun, 2 Aug 2026 15:00:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=L4NN96LF4St9BOce9EbqXSjeEKLi906TafrDSHuyFzE=; b=ZNXO+22k1yGoKOLA1Ltctd4pjQ XtHhFAe3Cjuxf84crGKXcEtCkuRvnIzN0S8A7h/bptvh3PijwbvY6oKlWrAVqNqEC4StJh+0CjWHf B4a4iR9Dgyjh2gbfNSad2BtDgna9DdfSrVC/perNdzMdyIajbXDcHwOgCwYbvMWDtMs47vOeGv9Tn eQWhIHyXw8pMUVkNJJy15LOsYgLz5Is/283j+EE3LFJfCRo1fAtryxsDDQqJTKctpEBCKiLIYbW2p +6M2OqJ5M/Jw1qCnp9brjrruq9H4UTGvKKBUaWyYJ03JYC86+KTk3+H0HyLUebvY7vrWCCFfuPTUn tk27zNHA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wqXfS-0000000FlQ5-3a4k; Sun, 02 Aug 2026 14:59:58 +0000 Received: from smtpout-03.galae.net ([185.246.85.4]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wqXfO-0000000FlO0-351a; Sun, 02 Aug 2026 14:59:57 +0000 Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-03.galae.net (Postfix) with ESMTPS id 7EE004E410A0; Sun, 2 Aug 2026 14:59:51 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 2D8FD6029B; Sun, 2 Aug 2026 14:59:51 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 954BB11C17641; Sun, 2 Aug 2026 16:59:29 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1785682785; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:content-language:in-reply-to:references; bh=L4NN96LF4St9BOce9EbqXSjeEKLi906TafrDSHuyFzE=; b=xW7qyKOyUBRdLJ9lR5d2R8ZCztLSdd0LHQKI4xxvys3kxNB2rrR5hTvjy1awyaGRKayqlh nyl+NkZd7qdP8uDJHyTNtzhgVpd/6RUYdtH3A2Yk6JarrXEQT2oPnQIvUeWWqdhsC2kvny SJxokonb3HmmuOqwuZ6AVC26MMfsm3aAZegcQhmPfgbUhBFHUEha3gh18AaIgOKZtvFB12 PtYZ4wk6A+qklUtdyGID2NbeBZI6l4YUxhuyLEOp6w+KbcHvC69LiOU5T3SOy9C1218GBf aJcNsR3ESNKUqyy57rYP7BB3V6SvQYlY4KZsp8ZK8Beyr86iSV07meKDgKpjHg== Message-ID: <82df688c-80c7-4536-afd8-1332005b0736@bootlin.com> Date: Sun, 2 Aug 2026 16:59:28 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 06/10] net: pcs: xpcs: add ANRESTART support for SGMII link recovery To: Andrew Lunn , Coia Prant Cc: Andrew Lunn , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Vinod Koul , 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 References: <20260801142244.679437-2-coiaprant@gmail.com> <20260801142244.679437-8-coiaprant@gmail.com> <2886b24e-1c3d-48f7-8b22-9620fa0404f7@lunn.ch> From: Maxime Chevallier Content-Language: en-US In-Reply-To: <2886b24e-1c3d-48f7-8b22-9620fa0404f7@lunn.ch> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Last-TLS-Session-Version: TLSv1.3 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260802_075954_908119_2E0A586B X-CRM114-Status: GOOD ( 14.74 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, On 8/2/26 16:30, 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. Indeed :( The only XPCS HW I have right now is the ksz9477 switch, and it uses an old version of the IP with its own quirks. I'll test it indeed, but I've yet to find some affordable HW with an XPCS and that I can easily source :( Maxime > > Andrew