From nobody Wed Apr 8 14:21:53 2026 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 E92AAECAAA1 for ; Sat, 10 Sep 2022 01:11:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229540AbiIJBL6 (ORCPT ); Fri, 9 Sep 2022 21:11:58 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60188 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230447AbiIJBLm (ORCPT ); Fri, 9 Sep 2022 21:11:42 -0400 Received: from mail-io1-xd30.google.com (mail-io1-xd30.google.com [IPv6:2607:f8b0:4864:20::d30]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0309E100429 for ; Fri, 9 Sep 2022 18:11:40 -0700 (PDT) Received: by mail-io1-xd30.google.com with SMTP id 138so1785094iou.9 for ; Fri, 09 Sep 2022 18:11:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date; bh=1UAIle1abMp4ppcBNEu/1g47bVXwskpesnhdVVUVLXg=; b=zpaLuPY5WsMrQzCIfXUuiYac/KKjPfqh7KmdrgBvrz/skLJXlrPeNIu29//gc6hV9D OGOp3SONkOGwxncAgR77g0NgkM3Q7LIZoIBUKX4IIgHrbEoeiHI+6YZnvpPDLyIUi9te D3Uew1f8cNHMcB1/KgLLM1Cqtaoy08baFDQedrDGVpOZTsgkXsuzbj7rqXi0MnpPv3L/ fjbte/WLjG7ayvnsjK0tIvzJ1ZHSoPCSvhyKMovwM+tevQlBhkXrKES4pcVJfXf65Z44 KtUILUXTbVv3fyi9V2X+LqG9us7NIIUJHglxYUKemJaEOFwulFC1Z6n5NS84R25OaCgW uUbw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date; bh=1UAIle1abMp4ppcBNEu/1g47bVXwskpesnhdVVUVLXg=; b=CiHHaiPSoZsBPJ0NdxeBZQvQoZBXyBGRaM0X9PudXtylweMJgnDsQcN7p20D4FzxYV HfdTEj50UeVqrLDzpmCfONR8LRg+o0R99QN+HV4aC1pIw/JiGMdRxYvKEl3p0P9MTgMI ZU8idcOXPmQHtiLhY+QwdBWaePiJVz284150DIiGx6849OoLuQKvyRGcHY1VEO9k9kXF 8c3lBvM6gsmGwPRYHU5c/kYuocaGynx1QV+zYJ9YYPZoz/Scg6c7vG1aRRfCmV+AMS3V whdudEFmUsm+sbPVgPTRpfFFfkjgF+Gp8Thl9KaZjRbyxWisUqzolVnLWqR7Ro04XaeX pTXA== X-Gm-Message-State: ACgBeo2Sr5sdE0oTu/TuoBHGL8zM0RKQ6sHr6YLPCSEtrA7K2YM8Xgug lAYCGNSfUoXHBeQoNJpaU9zW9Q== X-Google-Smtp-Source: AA6agR7PrXHpnbRAXTicoUKOCKAJQIBDirUPrVqFtJrSiECQSJXhcXvdP9U/UNvklq0qk62rEz1eyA== X-Received: by 2002:a05:6602:140d:b0:68b:1bd1:1c54 with SMTP id t13-20020a056602140d00b0068b1bd11c54mr7922355iov.9.1662772299060; Fri, 09 Sep 2022 18:11:39 -0700 (PDT) Received: from presto.localdomain ([98.61.227.136]) by smtp.gmail.com with ESMTPSA id u133-20020a02238b000000b00348e1a6491asm733064jau.137.2022.09.09.18.11.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 09 Sep 2022 18:11:38 -0700 (PDT) From: Alex Elder To: davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com Cc: mka@chromium.org, evgreen@chromium.org, andersson@kernel.org, quic_cpratapa@quicinc.com, quic_avuyyuru@quicinc.com, quic_jponduru@quicinc.com, quic_subashab@quicinc.com, elder@kernel.org, netdev@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next 4/6] net: ipa: don't reuse variable names Date: Fri, 9 Sep 2022 20:11:29 -0500 Message-Id: <20220910011131.1431934-5-elder@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220910011131.1431934-1-elder@linaro.org> References: <20220910011131.1431934-1-elder@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" In ipa_endpoint_init_hdr(), as well as ipa_endpoint_init_hdr_ext(), a top-level automatic variable named "offset" is used to represent the offset of a register. However, deeper within each of those functions is *another* definition of a local variable with the same name, representing something else. Scoping rules ensure the result is what was intended, but this variable name reuse is bad practice and makes the code confusing. Fix this by naming the inner variable "off". Use "off" instead of "checksum_offset" in ipa_endpoint_init_cfg() for consistency. Signed-off-by: Alex Elder --- drivers/net/ipa/ipa_endpoint.c | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/net/ipa/ipa_endpoint.c b/drivers/net/ipa/ipa_endpoint.c index 66d2bfdf9e423..eb68ce47698d0 100644 --- a/drivers/net/ipa/ipa_endpoint.c +++ b/drivers/net/ipa/ipa_endpoint.c @@ -494,12 +494,12 @@ static void ipa_endpoint_init_cfg(struct ipa_endpoint= *endpoint) enum ipa_version version =3D endpoint->ipa->version; =20 if (endpoint->toward_ipa) { - u32 checksum_offset; + u32 off; =20 /* Checksum header offset is in 4-byte units */ - checksum_offset =3D sizeof(struct rmnet_map_header); - checksum_offset /=3D sizeof(u32); - val |=3D u32_encode_bits(checksum_offset, + off =3D sizeof(struct rmnet_map_header); + off /=3D sizeof(u32); + val |=3D u32_encode_bits(off, CS_METADATA_HDR_OFFSET_FMASK); =20 enabled =3D version < IPA_VERSION_4_5 @@ -590,20 +590,20 @@ static void ipa_endpoint_init_hdr(struct ipa_endpoint= *endpoint) =20 /* Define how to fill fields in a received QMAP header */ if (!endpoint->toward_ipa) { - u32 offset; /* Field offset within header */ + u32 off; /* Field offset within header */ =20 /* Where IPA will write the metadata value */ - offset =3D offsetof(struct rmnet_map_header, mux_id); - val |=3D ipa_metadata_offset_encoded(version, offset); + off =3D offsetof(struct rmnet_map_header, mux_id); + val |=3D ipa_metadata_offset_encoded(version, off); =20 /* Where IPA will write the length */ - offset =3D offsetof(struct rmnet_map_header, pkt_len); + off =3D offsetof(struct rmnet_map_header, pkt_len); /* Upper bits are stored in HDR_EXT with IPA v4.5 */ if (version >=3D IPA_VERSION_4_5) - offset &=3D field_mask(HDR_OFST_PKT_SIZE_FMASK); + off &=3D field_mask(HDR_OFST_PKT_SIZE_FMASK); =20 val |=3D HDR_OFST_PKT_SIZE_VALID_FMASK; - val |=3D u32_encode_bits(offset, HDR_OFST_PKT_SIZE_FMASK); + val |=3D u32_encode_bits(off, HDR_OFST_PKT_SIZE_FMASK); } /* For QMAP TX, metadata offset is 0 (modem assumes this) */ val |=3D HDR_OFST_METADATA_VALID_FMASK; @@ -653,11 +653,11 @@ static void ipa_endpoint_init_hdr_ext(struct ipa_endp= oint *endpoint) if (ipa->version >=3D IPA_VERSION_4_5) { /* HDR_TOTAL_LEN_OR_PAD_OFFSET is 0, so MSB is 0 */ if (endpoint->config.qmap && !endpoint->toward_ipa) { - u32 offset; + u32 off; =20 - offset =3D offsetof(struct rmnet_map_header, pkt_len); - offset >>=3D hweight32(HDR_OFST_PKT_SIZE_FMASK); - val |=3D u32_encode_bits(offset, + off =3D offsetof(struct rmnet_map_header, pkt_len); + off >>=3D hweight32(HDR_OFST_PKT_SIZE_FMASK); + val |=3D u32_encode_bits(off, HDR_OFST_PKT_SIZE_MSB_FMASK); /* HDR_ADDITIONAL_CONST_LEN is 0 so MSB is 0 */ } --=20 2.34.1