From patchwork Sat Feb 17 18:08:21 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Lunn X-Patchwork-Id: 20542 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:693c:2685:b0:108:e6aa:91d0 with SMTP id mn5csp446484dyc; Sat, 17 Feb 2024 10:10:22 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCWh69f8kvS82WKp0e5g+93Wvh7cUM/znN86/bODvV6B1JdEeDeiodMWjDGmcTXp0kze0wT8X6s3PJd1P9lR+BfzN1UYzg== X-Google-Smtp-Source: AGHT+IEyowxigaCZbAJY68aBijkhWfInbOKtaF5i6NspY4dhJySiZu2nWdwTVHinoGoFpqnDt6hF X-Received: by 2002:aa7:d148:0:b0:563:c54e:f7 with SMTP id r8-20020aa7d148000000b00563c54e00f7mr4820180edo.17.1708193422693; Sat, 17 Feb 2024 10:10:22 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708193422; cv=pass; d=google.com; s=arc-20160816; b=N56Br2R/g13iTKyABUMZJByMypsWnf5fdhI7q1KXIWN7wPbkjmbmmpxvf3ioyaohK1 EgwbJm2esSWkGEhvplcRbSUwRSOwESYLE+glT/YLKCgNgLrAVq9BZTfP2HN0+oyW+NaU ZeRStA3gxkvaiglZM4igAMYnanTuzjgAAPQcmRkVxRzv79NQe7xiJXgPNCJZ2xaZwK9J 3iIeVfUBToCGEnNg/GWg0wClmVgU2Oj1CZkhJRnrg7clqP2mEjKSz2k96+zb4w6jXYuF Is6I4kEFe44GKw0f48DfLGs0T6e6QPs4iZx3amLw3wIw0lfcwcrXJ+I8yKskfE15gsJD 4NLw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=cc:to:content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:from :dkim-signature; bh=D5hka2iRwHIJ0nHqjCwQ9odeFeQKeB0FEoZhwjpQjRw=; fh=P3MkbJTmYKEBx+Jla3V8R5fCd1H1lmomdQcQsX8Xqvo=; b=Q7zvb0zziiW5UxZ+mJb58SXt+T3Px6lWi153+pbiAHNVxiL1AIuYSPqz58Y1T3dwzG lN+Wlv03WP6omDoQDdSpEdoOrDXs7kvaDsDS3nLdYLFNE0L3+bLXylzFfH8u08WyIxxO rOs4l3Cp5uLPODkSJNj9o+uJao/b2qPTOOH+KOBXfj44BoB3bExOChvlYAnoqv2v6020 r2WVUeWAElNoUOnJQXua2M9fLvXe6jmZMV1/QfU4CNIaGHHqCmuSL/9pdK1efIwM8G5s 74Hz8OX9O6irPFreRcKmVulFrOcCdkOzRoloCbwFfAsjWPGn6x4iRBM0Jjta1wj46tL9 gN1A==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@lunn.ch header.s=20171124 header.b="m9m/FrVr"; arc=pass (i=1 spf=pass spfdomain=lunn.ch dkim=pass dkdomain=lunn.ch dmarc=pass fromdomain=lunn.ch); spf=pass (google.com: domain of linux-kernel+bounces-69999-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-69999-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=lunn.ch Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id c1-20020aa7c981000000b005615b041027si986124edt.248.2024.02.17.10.10.22 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 17 Feb 2024 10:10:22 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-69999-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3; Authentication-Results: mx.google.com; dkim=pass header.i=@lunn.ch header.s=20171124 header.b="m9m/FrVr"; arc=pass (i=1 spf=pass spfdomain=lunn.ch dkim=pass dkdomain=lunn.ch dmarc=pass fromdomain=lunn.ch); spf=pass (google.com: domain of linux-kernel+bounces-69999-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-69999-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=lunn.ch Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 4AA081F21F12 for ; Sat, 17 Feb 2024 18:10:22 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id C5DBD7E786; Sat, 17 Feb 2024 18:09:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=lunn.ch header.i=@lunn.ch header.b="m9m/FrVr" Received: from vps0.lunn.ch (vps0.lunn.ch [156.67.10.101]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4AE3F41A91; Sat, 17 Feb 2024 18:09:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=156.67.10.101 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708193366; cv=none; b=jlx+nbGDpSSc0aHr7yuOh/MbJpRFAN1cC0fzfGt7+ZeBY0fa5Fl+gC2qF3DBEuPd0OzUxr+FXaSK0PMZ7i1R2aD82pOg5fOMjileSeB0r+f4Iwjroy1y6yanutRmURVHShFgbP65w9+ky4AENKwyuz4ynUT34TVMTY8ZVaT0nBs= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708193366; c=relaxed/simple; bh=3Kx8F3a38kE44wmpjeYUBBglZy2Yhk30yOTvCVmccZI=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=fi7znEdgx4MMbhdNMvvBILI96TismjdqnQGgThdtelK2PK/1Tbh43ATsmPJs/+9LXwqUQ6AHPVaq/JvxR97/iLUvnvp8DkSu9VwlP+1LyVaxsX55zX7+gF1q+nu7Md3zpeIAyLiVzWBk5iKUJU+VOV3s5wtDOa7lRrZy5HCTg5o= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=lunn.ch; spf=pass smtp.mailfrom=lunn.ch; dkim=pass (1024-bit key) header.d=lunn.ch header.i=@lunn.ch header.b=m9m/FrVr; arc=none smtp.client-ip=156.67.10.101 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=lunn.ch Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=lunn.ch DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=Cc:To:Content-Transfer-Encoding:Content-Type:MIME-Version: Message-Id:Date:Subject:From:From:Sender:Reply-To:Subject:Date:Message-ID:To: Cc:MIME-Version:Content-Type:Content-Transfer-Encoding:Content-ID: Content-Description:Content-Disposition:In-Reply-To:References; bh=D5hka2iRwHIJ0nHqjCwQ9odeFeQKeB0FEoZhwjpQjRw=; b=m9m/FrVrlmVdBho+MjDwoKimDQ 4YCAGru8ScJ94PBR98SbEponUntGWYn1vr8JvHwIxreBkWoEXd5+iXHF/lioNymGGxDbESD/a2QS3 e017KJsdCwG2iRKQBz2sfTzkLJqQ5dDNn87CJvGay6tO2Bo1KzN9+roRyemC280/O5OY=; Received: from c-76-156-36-110.hsd1.mn.comcast.net ([76.156.36.110] helo=thinkpad.home.lunn.ch) by vps0.lunn.ch with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rbP7m-0084hf-J4; Sat, 17 Feb 2024 19:09:18 +0100 From: Andrew Lunn Subject: [PATCH net-next v3 0/8] drivers: net: Convert EEE handling to use linkmode bitmaps Date: Sat, 17 Feb 2024 12:08:21 -0600 Message-Id: <20240217-keee-u32-cleanup-v3-0-fcf6b62a0c7f@lunn.ch> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIABX20GUC/3WNQQ6DIBQFr2JY9zcISLCr3qPpAuFbSQ0aUGJjv HsJm3bRLicvM28nEYPDSC7VTgImF93kM/BTRcyg/QPB2cyEUSYoowKeiAgrZ2BG1H6doVPSSiU aZRUlWZsD9m4ryRvxuIDHbSH3vAwuLlN4la9Ul/1/NtVAoe8kUsVZa1t9HVfvz2YoqcS+9PqXz rIutGm46BS3Un704zjegA7HWPcAAAA= To: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Ariel Elior , Manish Chopra , Jesse Brandeburg , Tony Nguyen Cc: linux-usb@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, intel-wired-lan@lists.osuosl.org, Andrew Lunn , Jacob Keller X-Mailer: b4 0.12.4 X-Developer-Signature: v=1; a=openpgp-sha256; l=2843; i=andrew@lunn.ch; h=from:subject:message-id; bh=3Kx8F3a38kE44wmpjeYUBBglZy2Yhk30yOTvCVmccZI=; b=owEBbQKS/ZANAwAKAea/DcumaUyEAcsmYgBl0PYwMe5qMhz9ZsflDQuNQ76sJnltsYIR9Y+6p C8sq2oUf0GJAjMEAAEKAB0WIQRh+xAly1MmORb54bfmvw3LpmlMhAUCZdD2MAAKCRDmvw3LpmlM hPCfD/0althDffrS4QaG2R2pNdI9MNwkgg+ZcCNO6lHixTc1h2GNTmg69uwk6c3yeMyImS9oVJ0 kOQkic5svqF87Q5PKgM7YIPGy2eOWV6kxPD5pFAN+Gkt5T9rBSffcXdK1OoALxNP9v+VMaHHlsZ 9J74ReHexGordDXaXiKafqoXz9r+k9gHF6f+rM4+goCjp2/nIyG6R9DBeCiDTyWffPaK2UUMyZu 1l8hMFOyE8Ku1Ahrnxl2m9lIL/8Ic5oE2GJZqc/Dd0J7kAPlvSKExDsInmU6BiCheEk2gARapnc oGxyO+0aDxu4VD3eIVfk9mVwjoYaP79C4JPM8dvi0FIGf3iuHMrvBcdj0k3JFd/3fAMeuC4mlSv mwnKRxLL3CtCOwwQwkOl5g8d5jUKmTHew4bT8ywiLF31U+gfAkIvDyBB6QnhTp85VblhaTnOv1L A4xhJk0e/52YSWPtqoUw0T1prP2azBvV4yJBmPRjKjAiCwaTZhlU+4rZ5xyNO6akiEEvyrBd9io zzToa3m60gbdSRDkhnnhSP596k/NODTkR2a2DTaXInYS5T447zgJlSbLmjQ39MzJdN7BJoeOdZu 0lO+BCg8rxtQt1Gq5RgecbOPDtG6DSPZ0LogwHoc2M4rlX8I8ZamnZvIqEGwNKytLcauEs90HSH cb3SERRwMogNeOA== X-Developer-Key: i=andrew@lunn.ch; a=openpgp; fpr=61FB1025CB53263916F9E1B7E6BF0DCBA6694C84 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1791170626038836928 X-GMAIL-MSGID: 1791170626038836928 EEE has until recently been limited to lower speeds due to the use of the legacy u32 for link speeds. This restriction has been lifted, with the use of linkmode bitmaps, added in the following patches: 1f069de63602 ethtool: add linkmode bitmap support to struct ethtool_keee 1d756ff13da6 ethtool: add suffix _u32 to legacy bitmap members of struct ethtool_keee 285cc15cc555 ethtool: adjust struct ethtool_keee to kernel needs 0b3100bc8fa7 ethtool: switch back from ethtool_keee to ethtool_eee for ioctl d80a52335374 ethtool: replace struct ethtool_eee with a new struct ethtool_keee on kernel side This patchset converts some MAC drivers still using the old _u32 to link modes, with the aim of soon being able to remove the legacy _u32 members in the keee structure. A couple of Intel drivers do odd things with EEE, setting the autoneg bit. It is unclear why, no other driver does, ethtool does not display it, and EEE is always negotiated. One patch in this series deletes this code. Comments on why its actually useful and should be kept are gratefully received. Signed-off-by: Andrew Lunn --- Changes in v3: - Add list of commits adding linkmodes to EEE to cover letter - Fix grammar error in cover letter. - Add Reviewed-by from Jacob Keller - Link to v2: https://lore.kernel.org/r/20240214-keee-u32-cleanup-v2-0-4ac534b83d66@lunn.ch Changes in v2: - igb: Fix type 100BaseT to 1000BaseT. - Link to v1: https://lore.kernel.org/r/20240204-keee-u32-cleanup-v1-0-fb6e08329d9a@lunn.ch --- Andrew Lunn (8): net: usb: r8152: Use linkmode helpers for EEE net: usb: ax88179_178a: Use linkmode helpers for EEE net: qlogic: qede: Use linkmode helpers for EEE net: ethernet: ixgbe: Convert EEE to use linkmodes net: intel: i40e/igc: Remove setting Autoneg in EEE capabilities net: intel: e1000e: Use linkmode helpers for EEE net: intel: igb: Use linkmode helpers for EEE net: intel: igc: Use linkmode helpers for EEE drivers/net/ethernet/intel/e1000e/ethtool.c | 17 +++++-- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 7 +-- drivers/net/ethernet/intel/igb/igb_ethtool.c | 33 ++++++++----- drivers/net/ethernet/intel/igc/igc_ethtool.c | 13 ++--- drivers/net/ethernet/intel/ixgbe/ixgbe_ethtool.c | 48 ++++++++++--------- drivers/net/ethernet/qlogic/qede/qede_ethtool.c | 60 +++++++++++++++--------- drivers/net/usb/Kconfig | 1 + drivers/net/usb/ax88179_178a.c | 9 ++-- drivers/net/usb/r8152.c | 31 ++++++------ 9 files changed, 123 insertions(+), 96 deletions(-) --- base-commit: dfe7f12788f5428d8bc16e86eedcfa889a3e3b29 change-id: 20240204-keee-u32-cleanup-b86d68458d80 Best regards,