From nobody Thu Oct 2 23:47:55 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 BEE98258CE5; Tue, 9 Sep 2025 21:08:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757452093; cv=none; b=H9GZV8/gqMJAWOAvGKfvbK85GxqfIo5eDHQm+Rcd2Y8vlagvmiOmRFDFJj335NpdHve9FvqpNVDZrUxjWvuDNxzGIoO3tX1JaYpdDKIKqLBUVO1yeBKF6tWdLxy4sY3p5HZLlDL/2Y7987D/N+rUQrodlOe/T/WTV6Vb/SXkXcg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757452093; c=relaxed/simple; bh=pxzpuyqnlW4OnsRKv8aNxFfdEU8dvfaFVpoBzcAkfXE=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=kNRDu/9bhriZUEex0do97JbWIGuMYc0SlS+Hf/0J3bqaGGuEBDSGF2cU0ZHN/4dY+seRARNzLCeO4+761QJLAIWtPcfb0cl7YemVzDgsqxV4Z24+FHtmcGmdCmJFNuA+4KJKTJeT+xEurfxx33dPCWcMFAaYVhzCNnLz68ONcM8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=ToQFk8Ko; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="ToQFk8Ko" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 09B36C4CEF7; Tue, 9 Sep 2025 21:08:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1757452093; bh=pxzpuyqnlW4OnsRKv8aNxFfdEU8dvfaFVpoBzcAkfXE=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=ToQFk8KoJ/uTHwTEdfRzBHXauI2eG89U1hG7B2G9deKlxv6ZT/rQPfmTPaMqpDEGt OSlbcmqUqLDIcbSsqvusAgvDfOtkI4QOP2hmoRPyYeIy6B1ArEJdhZz6Ygdn4JLTWB jeXWGD0v3zVWtUmLYiH81f2hPil5KBFD/OOfti1tXXQLfOz/jFxsADa5kZqLYAiCH6 CGytdHYsgr3/gXgR4e/5gUFlvWWsbE8ioGmD8G18Ydiw7HGBDNq598w3ENUGOYxy7u 8ST6RyAas4RDGNAPJDW2MG2U0QC2LDs5ECgFngYyEhOMtAtDzWaga1kRsG9RLhROnU CPqDkCJyqQFVg== From: "Matthieu Baerts (NGI0)" Date: Tue, 09 Sep 2025 23:07:49 +0200 Subject: [PATCH net-next 3/8] tools: ynl: remove assigned but never used variable Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250909-net-next-ynl-ruff-v1-3-238c2bccdd99@kernel.org> References: <20250909-net-next-ynl-ruff-v1-0-238c2bccdd99@kernel.org> In-Reply-To: <20250909-net-next-ynl-ruff-v1-0-238c2bccdd99@kernel.org> To: Donald Hunter , Jakub Kicinski , "David S. Miller" , Eric Dumazet , Paolo Abeni , Simon Horman Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, "Matthieu Baerts (NGI0)" X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=1826; i=matttbe@kernel.org; h=from:subject:message-id; bh=pxzpuyqnlW4OnsRKv8aNxFfdEU8dvfaFVpoBzcAkfXE=; b=owGbwMvMwCVWo/Th0Gd3rumMp9WSGDIOTDe4u3SpcPycadxXnT/bPfx8d9GVp/l7GC4cEWSaE db9VWJnT0cpC4MYF4OsmCKLdFtk/sznVbwlXn4WMHNYmUCGMHBxCsBEul8wMnSc2NkZdTVi+qw/ X22XCO2caXVmyg/jnJ0J8x70zfsmnyXC8D9s5i1fRtkt+rO0bP/lbV/7b9o1IekICauHgRu+C5p rzGYHAA== X-Developer-Key: i=matttbe@kernel.org; a=openpgp; fpr=E8CB85F76877057A6E27F77AF6B7824F4269A073 These variables are assigned but never used according to Ruff. They can then be safely removed. This is linked to Ruff error F841 [1]: A variable that is defined but not used is likely a mistake, and should be removed to avoid confusion. Link: https://docs.astral.sh/ruff/rules/unused-variable/ [1] Signed-off-by: Matthieu Baerts (NGI0) --- tools/net/ynl/pyynl/ethtool.py | 1 - tools/net/ynl/pyynl/ynl_gen_c.py | 2 -- 2 files changed, 3 deletions(-) diff --git a/tools/net/ynl/pyynl/ethtool.py b/tools/net/ynl/pyynl/ethtool.py index 44440beab62f52f240e10f0678a6564f449d26d4..ef2cbad41f9bdd4e22c1be95632= 6417c9ee23109 100755 --- a/tools/net/ynl/pyynl/ethtool.py +++ b/tools/net/ynl/pyynl/ethtool.py @@ -156,7 +156,6 @@ def main(): global args args =3D parser.parse_args() =20 - script_abs_dir =3D os.path.dirname(os.path.abspath(sys.argv[0])) spec =3D os.path.join(spec_dir(), 'ethtool.yaml') schema =3D os.path.join(schema_dir(), 'genetlink-legacy.yaml') =20 diff --git a/tools/net/ynl/pyynl/ynl_gen_c.py b/tools/net/ynl/pyynl/ynl_gen= _c.py index fb7e03805a1130115ccf70d1e3fbdd6ac485374a..957fae8e27ede6fcd51fb2a98d3= 56a6d67d0352e 100755 --- a/tools/net/ynl/pyynl/ynl_gen_c.py +++ b/tools/net/ynl/pyynl/ynl_gen_c.py @@ -2803,8 +2803,6 @@ def print_kernel_policy_sparse_enum_validates(family,= cw): cw.p('/* Sparse enums validation callbacks */') first =3D False =20 - sign =3D '' if attr.type[0] =3D=3D 'u' else '_signed' - suffix =3D 'ULL' if attr.type[0] =3D=3D 'u' else 'LL' cw.write_func_prot('static int', f'{c_lower(attr.enum_name)}_v= alidate', ['const struct nlattr *attr', 'struct netli= nk_ext_ack *extack']) cw.block_start() --=20 2.51.0