Received: by 2002:a05:6358:16cc:b0:ea:6187:17c9 with SMTP id r12csp3073346rwl; Fri, 6 Jan 2023 15:20:54 -0800 (PST) X-Google-Smtp-Source: AMrXdXuHBhR6z5J8c3tAycfSPrRwLZshatU/r1ZKOjXiuExO4zYVY9D/BC/6sFqoM+q5blnEZ8II X-Received: by 2002:a17:902:b60d:b0:192:6464:f433 with SMTP id b13-20020a170902b60d00b001926464f433mr48071156pls.1.1673047254101; Fri, 06 Jan 2023 15:20:54 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1673047254; cv=none; d=google.com; s=arc-20160816; b=tuorIOHOAsC/vXrgAVYwJFwNUOtkWRIt40yyeMuDNvOJwU5Ma1dCNJc05t65y7Gexk N9m4e24XNjMFidTS7MPu9tZBmyEQNqdBJbcy1kQINnlNKvS60/bYSQBOr+DsgNn323on r9VcyUi7UhFIczQa2o5oBzDS028NYxG9MQwAph6IOoaWtKFvBUCO5gCmHeemTbBEb03N RDPGgG6DDn+4jG1CoIxeu1uZwJyTQpWa9WNWCzozFeDB/ifE9xURnvd0uTUM4gIkqQzV HXREv7yvAEujdjqIpv6pA0+vTfDWoYkRsBtUrJutahG6gvgRlItlzrtb7l/Qbd3WF0O+ 09SA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=laPbFbryd1xlQcDIQ7snFfBwO8xqn9PB54qQPKLgqQQ=; b=e40rqS7NOD3MPuCiV244VHJS7tKbddLsdVPE7LMHPMpV0gYn0EunTz9GeAKBFJU+tz tBD8QFRvafHA3xLd7enpl1MxxgfI8AFmznhNRqTjypSfgTTeY9qHLYS63Bayybtb4uVh eXiV9Q3COTmRE4JtlzdHag774RFz+zcgWCZwYH5uclG6iCTkco+X7sUzpRLuoJ8grK76 K2xAiocmsMeTt+aZFsh8ZV4TYc1qcANZhaQaSB7GIODu1vW0nL4oxSxptIL3chqg9F6h bjq+syJAYj9HSTVmRkQpoWbvV2zvnWa8QAwksNJKd/fa11/LNGt5Fyif51DeKz6aMyBY eFZw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=HiasRf1V; 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=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id b14-20020a170902e94e00b001781e393237si2170292pll.443.2023.01.06.15.20.43; Fri, 06 Jan 2023 15:20:54 -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=pass header.i=@gmail.com header.s=20210112 header.b=HiasRf1V; 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=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230326AbjAFXDv (ORCPT + 55 others); Fri, 6 Jan 2023 18:03:51 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37606 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229726AbjAFXDs (ORCPT ); Fri, 6 Jan 2023 18:03:48 -0500 Received: from mail-ej1-x635.google.com (mail-ej1-x635.google.com [IPv6:2a00:1450:4864:20::635]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 40FE1E9B; Fri, 6 Jan 2023 15:03:47 -0800 (PST) Received: by mail-ej1-x635.google.com with SMTP id u9so6838360ejo.0; Fri, 06 Jan 2023 15:03:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=laPbFbryd1xlQcDIQ7snFfBwO8xqn9PB54qQPKLgqQQ=; b=HiasRf1VmxesIeh8c+yINW7g5Vt0WCU9vX05gzdGTBie90jee4tVzIbTHFrZGZ+Kmm 21TziMmtmzkw90eooU5uu8QV0XBPwk8jJr1DuRjvL6wAXuvLjNdf7fSFwtrG6PvswTPh dxZoh7j/ZvXkuxwXHFwkocF0lvnx4DfRgH0Db1BSRdIbeELjDtxi3dSP5BTp/4Yk85Jt iFuPiSHRCHA6k3TxBEoZ+LZpnBxsZ+1E4dLRfeSUNo8u5E5E1z4CafsDgwCTgCK4xelS /VrH5jxJnEBXRJ9WuxiHHn+1DXXvl1nLKwVM+tAIDaQMMT1Bh2+arW3zUOvF5Si26sWd 0o7A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=laPbFbryd1xlQcDIQ7snFfBwO8xqn9PB54qQPKLgqQQ=; b=UMYpSzWTL3qmkjJsbg/xakqgU5j/vR6MhXmkrUuxRHnZGkRLI8ZERrHU6HfGJftgX5 EPgwjDKEagOgLzUDQMDhcOONUba5h/KQvJwpqurJ8o3wnUUNKeRRgyiltJhzJvSAM7yS 9ioRFYyx+yJ1D7xEaa9kA5+yZA6Io+K3fQmtyI4Xe8dQjggZoEDbkhnw1g/ZujIEb66M UxyjcKwu4VZ8o6fPErXP8Sel27sGehL1nlgUmDlZNq0EQQgpbYZD3lgaH8DgDof1JuL8 bdu3eUBUJZRn6eoHy77jF02l7ZJTYwGNLzuB5sk+4I/NBSeT7MC1xTWdKqD/r+CrHxfL ChwA== X-Gm-Message-State: AFqh2komSGJ3i/8CgCe5g3KnDzmYW3Gx2HoDy9bpICNhXUHWLnsuzomW OwoRRR5Yd6GFz+0sb/LYvxY= X-Received: by 2002:a17:907:8748:b0:7c1:ad6:7331 with SMTP id qo8-20020a170907874800b007c10ad67331mr54570641ejc.27.1673046225466; Fri, 06 Jan 2023 15:03:45 -0800 (PST) Received: from skbuf ([188.26.184.223]) by smtp.gmail.com with ESMTPSA id vj22-20020a170907131600b0078c213ad441sm840681ejb.101.2023.01.06.15.03.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 06 Jan 2023 15:03:45 -0800 (PST) Date: Sat, 7 Jan 2023 01:03:43 +0200 From: Vladimir Oltean To: "Russell King (Oracle)" Cc: Sean Anderson , 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: <20230106230343.2noq2hxr4quqbtk4@skbuf> 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> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS 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 05:46:48PM +0000, Russell King (Oracle) wrote: > On Thu, Jan 05, 2023 at 07:34:45PM +0200, Vladimir Oltean wrote: > > So we lose the advertisement of 5G and 2.5G, even if the firmware is > > provisioned for them via 10GBASE-R rate adaptation, right? Because when > > asked "What kind of rate matching is supported for 10GBASE-R?", the > > Aquantia driver will respond "None". > > The code doesn't have the ability to do any better right now - since > we don't know what sets of interface modes _could_ be used by the PHY > and whether each interface mode may result in rate adaption. > > To achieve that would mean reworking yet again all the phylink > validation from scratch, and probably reworking phylib and most of > the PHY drivers too so that they provide a lot more information > about their host interface behaviour. > > I don't think there is an easy way to have a "perfect" solution > immediately - it's going to take a while to evolve - and probably > painfully evolve due to the slowness involved in updating all the > drivers that make use of phylink in some way. Serious question. What do we gain in practical terms with this patch set applied? With certain firmware provisioning, some unsupported link modes won't be advertised anymore. But also, with other firmware, some supported link modes won't be advertised anymore. IIUC, Tim Harvey's firmware ultimately had incorrect provisioning, it's not like the existing code prevents his use case from working.