Received: by 2002:a05:6358:16cc:b0:ea:6187:17c9 with SMTP id r12csp1289411rwl; Thu, 5 Jan 2023 11:17:29 -0800 (PST) X-Google-Smtp-Source: AMrXdXuKsz9lQz4YeXWYletUffAQrrU+AorZRfY6a5ritKDsHUrI7FqIcAwU6uyhAcyHj31ZCnij X-Received: by 2002:a05:6a20:1715:b0:b0:25ba:1764 with SMTP id bn21-20020a056a20171500b000b025ba1764mr55821759pzb.26.1672946248942; Thu, 05 Jan 2023 11:17:28 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1672946248; cv=none; d=google.com; s=arc-20160816; b=fWMw0cu485eUq9/7x91ENvUvl6Dd0TvSTNwvYwZV+y1j6Z+eywWhmlirx3y2XYXAs3 JuBOBJH8aL63fljfXm49juQ1+kwfcIDyxPzfDC/1STsH03zfkDau9s1VOZUb7hY3BmyD lyA4rTxNnqhRJ3AvoZDFzc6uVeSkxaBMvf5mP7USr0L2IWXi8LN2EuQnuj/aTZrGHK/q aYG/v1vdUHBaHVBhiuJcYLdGS6AIoEwGFIgSLAh0p8LD5skyfDrPoxXxY4MG0pYzTqfK WyfoRpLndiuBN1VRcPpPAzrC9GvgsYEIV5lHBYWUffIAGUR3zVA7V3MjfUS2rbQQKKoV oqLQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=QrdF2fu5/3Y/xqo4XD4CaE8xCI0Co+92085uN0QaYrM=; b=HP0dhFD8KXosLcUFijMMzdQqx7KIQUbWskopOJ18CIxHBSdaftSet+fw+CjDX3Xea1 nc6U8++xPRTJ0uraG9z9KzjOcaJz77NRLfAeQh9fVEn3oHDAv0xjTurKmMU3Wne0Fcpg n/vNynWC4KGCzpQT2U9UH4bYNuKOufrlv1DFqL1voWoEvWaob7BE4OPlBEdXkvw41S20 U1TK2zyL/TqOMbMEe76SD+AyoBifC7mqBEHnRhGENHxoe+fkpdq9uMbkgCvRZ5sjjqKV 9woE6Y4bEc1kXjBcfJ0et7qtXcvvRdGLVhqlTHMp5+vFUIyXA7aeXd4afwGs6SfJBEb1 qgog== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail (test mode) header.i=@armlinux.org.uk header.s=pandora-2019 header.b=SYSGv+Zq; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=armlinux.org.uk Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id x4-20020a63db44000000b00476e6407413si41056336pgi.15.2023.01.05.11.17.21; Thu, 05 Jan 2023 11:17:28 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=fail (test mode) header.i=@armlinux.org.uk header.s=pandora-2019 header.b=SYSGv+Zq; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=armlinux.org.uk Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235157AbjAES6o (ORCPT + 55 others); Thu, 5 Jan 2023 13:58:44 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35568 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231683AbjAES6m (ORCPT ); Thu, 5 Jan 2023 13:58:42 -0500 Received: from pandora.armlinux.org.uk (pandora.armlinux.org.uk [IPv6:2001:4d48:ad52:32c8:5054:ff:fe00:142]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6E2665F497; Thu, 5 Jan 2023 10:58:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=QrdF2fu5/3Y/xqo4XD4CaE8xCI0Co+92085uN0QaYrM=; b=SYSGv+Zq3DeIgfB6IXKAwdXiGr CBrorpbrwtNmJ28nVzmCqVSM6HUNH1XeG7vIux6csC3JBlqYipnI1KVUJyXrIr4E1e8VLoIGhdgJJ xGSqIn7yoOS1QNvYFYl3rHI+8WEZtfQSOsVgtzkjwzIDWT/2lTThUuZZLMhZK78NWNdxKFh6wlZVZ l8SJrcc3xksaF3fpeANGChMrGrF8mNQ+w8W4i6fZ8Y3bsFmAPs5NMFQ08gv2NToKCjGWAQluLZfLF gycW+i14LshrioZTTcV/cCuq5FnlRa7TI6uVK2r3wZXjo+1N5taSITEDWqbpegzlQZKTFWiJwRSPK uszpTWzA==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:35988) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1pDVRm-0007SC-Tb; Thu, 05 Jan 2023 18:58:38 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1pDVRk-0004D2-PO; Thu, 05 Jan 2023 18:58:37 +0000 Date: Thu, 5 Jan 2023 18:58:36 +0000 From: "Russell King (Oracle)" To: Sean Anderson Cc: Vladimir Oltean , Andrew Lunn , Heiner Kallweit , netdev@vger.kernel.org, "David S . Miller" , Paolo Abeni , linux-kernel@vger.kernel.org, Jakub Kicinski , Eric Dumazet , Tim Harvey Subject: Re: [PATCH net-next v5 4/4] phy: aquantia: Determine rate adaptation support from registers Message-ID: References: <20230103220511.3378316-1-sean.anderson@seco.com> <20230103220511.3378316-5-sean.anderson@seco.com> <20230105140421.bqd2aed6du5mtxn4@skbuf> <6ffe6719-648c-36aa-74be-467c8db40531@seco.com> <20230105173445.72rvdt4etvteageq@skbuf> <3919acb9-04bb-0ca0-07b9-45e96c4dad10@seco.com> <20230105175206.h3nmvccnzml2xa5d@skbuf> <20230105175542.ozqn67o3qmadnaph@skbuf> <39660d10-69b9-fa52-5a49-67d5f7e1acaf@seco.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <39660d10-69b9-fa52-5a49-67d5f7e1acaf@seco.com> Sender: Russell King (Oracle) X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jan 05, 2023 at 01:03:49PM -0500, Sean Anderson wrote: > On 1/5/23 12:55, Vladimir Oltean wrote: > > On Thu, Jan 05, 2023 at 07:52:06PM +0200, Vladimir Oltean wrote: > >> On Thu, Jan 05, 2023 at 12:43:47PM -0500, Sean Anderson wrote: > >> > Again, this is to comply with the existing API assumptions. The current > >> > code is buggy. Of course, another way around this is to modify the API. > >> > I have chosen this route because I don't have a situation like you > >> > described. But if support for that is important to you, I encourage you > >> > to refactor things. > >> > >> I don't think I'm aware of a practical situation like that either. > >> I remember seeing some S32G boards with Aquantia PHYs which use 2500BASE-X > >> for 2.5G and SGMII for <=1G, but that's about it in terms of protocol switching. > >> As for Layerscape boards, SERDES protocol switching is a very new concept there, > >> so they're all going to be provisioned for PAUSE all the way down > >> (or USXGMII, where that is available). > >> > >> I just pointed this out because it jumped out to me. I don't have > >> something against this patch getting accepted as it is. > > > > A real-life (albeit niche) scenario where someone might have an Aquantia > > firmware provisioned like this would be a 10G capable port that also > > wants to support half duplex at 10/100 speeds. Although I'm not quite > > sure who cares about half duplex all that much these days. > > IMO if we really want to support this, the easier way would be to teach > the phy driver how to change the rate adaptation mode. That way we could > always advertise rate adaptation, but if someone came along and > requested 10HD we could reconfigure the phy to support it. However, this > was deemed too risky in the discussion for v1, since we don't really > know how the firmware interacts with the registers. I'm not sure about "someone came along and requested 10HD". Don't you mean "if someone plugged the RJ45 into a 10bT hub which only supports 10HD" ? Or are you suggesting that we shouldn't advertise 10HD and 100HD along with everything else, and then switch into this special mode if someone wants to advertise these and disable all other link modes? -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!