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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 88EE7C3F6B0 for ; Wed, 24 Aug 2022 23:33:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230481AbiHXXdw (ORCPT ); Wed, 24 Aug 2022 19:33:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44244 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232024AbiHXXdf (ORCPT ); Wed, 24 Aug 2022 19:33:35 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7BD7680F61 for ; Wed, 24 Aug 2022 16:32:36 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id B8673B826C1 for ; Wed, 24 Aug 2022 23:32:34 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5DD04C433D6; Wed, 24 Aug 2022 23:32:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1661383953; bh=pKapiAD4TnK7Pah/51fnwPcS2nBWoF3jOA8I1sk51kQ=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=j5bE0Q8dTthKiOQHc9y+cmRcTUeGc1rYynIIv4oyXuINiYGMisTm4Bh3UYDasQqfz jrJOqKuNeL99BceMYYae1dE6K0bnDjYX58vAjShT0qanIM87XnEaw7Hwrn1EaHzpaw f+3cgLVMyn20e/SaTTWOUAcxm3z2oXFnucprqNnYgYFtSLLs2luJO3G4jPoi4O1dC5 cKcWbvrwnTK0V5Vv4vdEvrUxqKYbMBxXDN5HPAkZj3Ww98AHXTZEquF63E1eXO4+SD tDpIEkKZWhOeSiQYxXrgnWGI3KuS/mmcmlltOEzXVOFSfOw2jKFhiDVoF5XN2DieGS 4mIs3YjsycFbg== Date: Wed, 24 Aug 2022 16:32:32 -0700 From: Jakub Kicinski To: "Keller, Jacob E" Cc: "netdev@vger.kernel.org" , "Greenwalt, Paul" Subject: Re: [PATCH net-next 2/2] ice: add support for Auto FEC with FEC disabled via ETHTOOL_SFECPARAM Message-ID: <20220824163232.3c3e7b28@kernel.org> In-Reply-To: References: <20220823150438.3613327-1-jacob.e.keller@intel.com> <20220823150438.3613327-3-jacob.e.keller@intel.com> <20220823151745.3b6b67cb@kernel.org> <20220824154727.450ff9d9@kernel.org> <20220824160230.3c2f06b2@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Wed, 24 Aug 2022 23:13:47 +0000 Keller, Jacob E wrote: > > > According to the folks I talked to what we have here, we didn't > > > understand this as being from a standard, but if it is I'd love to > > > read on it. =20 > >=20 > > Table 110C=E2=80=931=E2=80=94Host and cable assembly combinations > > in IEEE 802.3 2018, that's what I was thinking of. =20 >=20 > Ah. I am not sure if the state machine in firmware uses this table or > not, but my guess is that it does. Yeah, I thought that's what AUTO means, otherwise if everyone did their own thing we wouldn't get link up without real autoneg enabled, right? Now the table only specifies minimum FEC requirements, in your case (and perhaps for others) No FEC does not get used, presumably the selection is only done between R and RS?