From nobody Mon Apr 6 18:53:46 2026 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.16]) (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 E135C3E7155; Wed, 18 Mar 2026 15:13:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.16 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773846789; cv=none; b=hq6BpYdsQRg+U/dYbiFNiC7y6Xynsuz975lffnQkPz7DyVa0LWvLChkB8PspcbR8XnVRlQsUAmBoRKb6pbGpwYOGjD0DgpPO1qxdAiCJ6Zp0xdFbLMU8RrOvuifkNZy4Zi11RKvsCs8wE9fuke6+xX1a8lnfaDQBDSlDDAen8R0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773846789; c=relaxed/simple; bh=wF4iUZV76f7W3PXuwWxNV/7Gg0YYVO8Px9yNm4Hmdq0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=sJP/TGG1AnhqtQ56he6WUMLnzQdBIf4OIfQuNQ5FR+4Oc4xOWBqx82et/lLy5IIFVWoJ6OoRY3IfZHyDnwBLfRUk9yQxOOD68XPGvLuUlW9oaSOMSeEbjWmsxBaI/5i6qkX4SwzHnmtf+yWUPtTxtR/KuMIoU5QbEkS/tG+yQzU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=pass smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=h4qPid2+; arc=none smtp.client-ip=198.175.65.16 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="h4qPid2+" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1773846787; x=1805382787; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=wF4iUZV76f7W3PXuwWxNV/7Gg0YYVO8Px9yNm4Hmdq0=; b=h4qPid2+F/Qt8b6LLFS0xWK5cwHS4j8KW2LJs/IJZjEixEwVWhNIMuot kTrjsmrx/2rRcCUKojRHyeQ09yxT6Gj1jQ2M2aNEw33VVFre48JOMNVwx 65Sqc2tBRbInp2QwkQVKCdHh3Tg9iDFQkBNLPzJo5vWpPPZ3IPeaE8jcW /eelZw0Y5+/3QzKBcRS19bZHKcJhGPnpQd+uMDtpMqzR1pBxKbLybZ1UM skkONboYMnJFlOESmvgM9AFkg7T0595CR3BcIeSVM2KOGUe6myIq+4aSm 6A2bDuEfBzabs6lmK4KAUlPbMd8FpCq5lQVSo4HlC1B2GWZtJkQkToiXE g==; X-CSE-ConnectionGUID: VviiMuL0QV2WshtWPXKb7g== X-CSE-MsgGUID: ruKh1qALRsq85MMW022Efg== X-IronPort-AV: E=McAfee;i="6800,10657,11733"; a="75084547" X-IronPort-AV: E=Sophos;i="6.23,127,1770624000"; d="scan'208";a="75084547" Received: from fmviesa001.fm.intel.com ([10.60.135.141]) by orvoesa108.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Mar 2026 08:13:03 -0700 X-CSE-ConnectionGUID: l+khbGIXTJyimFpL2g8fCQ== X-CSE-MsgGUID: eBhFSEgOTQyilBQdH4U1qQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,127,1770624000"; d="scan'208";a="247122458" Received: from black.igk.intel.com ([10.91.253.5]) by fmviesa001.fm.intel.com with ESMTP; 18 Mar 2026 08:13:01 -0700 Received: by black.igk.intel.com (Postfix, from userid 1003) id 219E7A2; Wed, 18 Mar 2026 16:12:58 +0100 (CET) From: Andy Shevchenko To: Mika Westerberg , Andy Shevchenko , linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Andy Shevchenko , Linus Walleij Subject: [PATCH v1 5/5] pinctrl: intel: define iterator variables inside for-loop Date: Wed, 18 Mar 2026 16:10:19 +0100 Message-ID: <20260318151256.2590375-6-andriy.shevchenko@linux.intel.com> X-Mailer: git-send-email 2.50.1 In-Reply-To: <20260318151256.2590375-1-andriy.shevchenko@linux.intel.com> References: <20260318151256.2590375-1-andriy.shevchenko@linux.intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Reduce the scope of the iterator variables by defining them inside the respective for-loops. This makes code more robust against reuse of the same variable in the future, which might lead to some mistakes. Signed-off-by: Andy Shevchenko --- drivers/pinctrl/intel/pinctrl-intel.c | 44 ++++++++++++--------------- 1 file changed, 19 insertions(+), 25 deletions(-) diff --git a/drivers/pinctrl/intel/pinctrl-intel.c b/drivers/pinctrl/intel/= pinctrl-intel.c index 7311b787dfc6..c506f9f343c3 100644 --- a/drivers/pinctrl/intel/pinctrl-intel.c +++ b/drivers/pinctrl/intel/pinctrl-intel.c @@ -431,7 +431,6 @@ static int intel_pinmux_set_mux(struct pinctrl_dev *pct= ldev, { struct intel_pinctrl *pctrl =3D pinctrl_dev_get_drvdata(pctldev); const struct intel_pingroup *grp =3D &pctrl->soc->groups[group]; - int i; =20 guard(raw_spinlock_irqsave)(&pctrl->lock); =20 @@ -439,13 +438,13 @@ static int intel_pinmux_set_mux(struct pinctrl_dev *p= ctldev, * All pins in the groups needs to be accessible and writable * before we can enable the mux for this group. */ - for (i =3D 0; i < grp->grp.npins; i++) { + for (unsigned int i =3D 0; i < grp->grp.npins; i++) { if (!intel_pad_usable(pctrl, grp->grp.pins[i])) return -EBUSY; } =20 /* Now enable the mux setting for each pin in the group */ - for (i =3D 0; i < grp->grp.npins; i++) { + for (unsigned int i =3D 0; i < grp->grp.npins; i++) { void __iomem *padcfg0; u32 value, pmode; =20 @@ -909,12 +908,12 @@ static int intel_config_set(struct pinctrl_dev *pctld= ev, unsigned int pin, unsigned long *configs, unsigned int nconfigs) { struct intel_pinctrl *pctrl =3D pinctrl_dev_get_drvdata(pctldev); - int i, ret; + int ret; =20 if (!intel_pad_usable(pctrl, pin)) return -ENOTSUPP; =20 - for (i =3D 0; i < nconfigs; i++) { + for (unsigned int i =3D 0; i < nconfigs; i++) { switch (pinconf_to_config_param(configs[i])) { case PIN_CONFIG_BIAS_DISABLE: case PIN_CONFIG_BIAS_PULL_UP: @@ -1323,9 +1322,8 @@ static void intel_gpio_irq_init(struct intel_pinctrl = *pctrl) =20 for_each_intel_pin_community(pctrl, community) { void __iomem *reg, *is; - unsigned int gpp; =20 - for (gpp =3D 0; gpp < community->ngpps; gpp++) { + for (unsigned int gpp =3D 0; gpp < community->ngpps; gpp++) { reg =3D community->regs + community->ie_offset + gpp * 4; is =3D community->regs + community->is_offset + gpp * 4; =20 @@ -1436,14 +1434,14 @@ static int intel_pinctrl_add_padgroups_by_gpps(stru= ct intel_pinctrl *pctrl, struct intel_community *community) { struct intel_padgroup *gpps; + unsigned int ngpps =3D community->ngpps; unsigned int padown_num =3D 0; - size_t i, ngpps =3D community->ngpps; =20 gpps =3D devm_kcalloc(pctrl->dev, ngpps, sizeof(*gpps), GFP_KERNEL); if (!gpps) return -ENOMEM; =20 - for (i =3D 0; i < ngpps; i++) { + for (unsigned int i =3D 0; i < ngpps; i++) { gpps[i] =3D community->gpps[i]; =20 if (gpps[i].size > INTEL_PINCTRL_MAX_GPP_SIZE) @@ -1476,18 +1474,18 @@ static int intel_pinctrl_add_padgroups_by_size(stru= ct intel_pinctrl *pctrl, struct intel_community *community) { struct intel_padgroup *gpps; - unsigned int npins =3D community->npins; + unsigned int npins =3D community->npins, ngpps; unsigned int padown_num =3D 0; - size_t i, ngpps =3D DIV_ROUND_UP(npins, community->gpp_size); =20 if (community->gpp_size > INTEL_PINCTRL_MAX_GPP_SIZE) return -EINVAL; =20 + ngpps =3D DIV_ROUND_UP(npins, community->gpp_size); gpps =3D devm_kcalloc(pctrl->dev, ngpps, sizeof(*gpps), GFP_KERNEL); if (!gpps) return -ENOMEM; =20 - for (i =3D 0; i < ngpps; i++) { + for (unsigned int i =3D 0; i < ngpps; i++) { unsigned int gpp_size =3D community->gpp_size; =20 gpps[i].reg_num =3D i; @@ -1513,7 +1511,6 @@ static int intel_pinctrl_pm_init(struct intel_pinctrl= *pctrl) const struct intel_pinctrl_soc_data *soc =3D pctrl->soc; struct intel_community_context *communities; struct intel_pad_context *pads; - int i; =20 pads =3D devm_kcalloc(pctrl->dev, soc->npins, sizeof(*pads), GFP_KERNEL); if (!pads) @@ -1525,7 +1522,7 @@ static int intel_pinctrl_pm_init(struct intel_pinctrl= *pctrl) return -ENOMEM; =20 =20 - for (i =3D 0; i < pctrl->ncommunities; i++) { + for (unsigned int i =3D 0; i < pctrl->ncommunities; i++) { struct intel_community *community =3D &pctrl->communities[i]; u32 *intmask, *hostown; =20 @@ -1578,7 +1575,7 @@ int intel_pinctrl_probe(struct platform_device *pdev, { struct device *dev =3D &pdev->dev; struct intel_pinctrl *pctrl; - int i, ret, irq; + int ret, irq; =20 pctrl =3D devm_kzalloc(dev, sizeof(*pctrl), GFP_KERNEL); if (!pctrl) @@ -1598,7 +1595,7 @@ int intel_pinctrl_probe(struct platform_device *pdev, if (!pctrl->communities) return -ENOMEM; =20 - for (i =3D 0; i < pctrl->ncommunities; i++) { + for (unsigned int i =3D 0; i < pctrl->ncommunities; i++) { struct intel_community *community =3D &pctrl->communities[i]; unsigned short capability_offset[6]; void __iomem *regs; @@ -1806,10 +1803,9 @@ static int intel_pinctrl_suspend_noirq(struct device= *dev) struct intel_pinctrl *pctrl =3D dev_get_drvdata(dev); struct intel_community_context *communities; struct intel_pad_context *pads; - int i; =20 pads =3D pctrl->context.pads; - for (i =3D 0; i < pctrl->soc->npins; i++) { + for (unsigned int i =3D 0; i < pctrl->soc->npins; i++) { const struct pinctrl_pin_desc *desc =3D &pctrl->soc->pins[i]; void __iomem *padcfg; u32 val; @@ -1828,7 +1824,7 @@ static int intel_pinctrl_suspend_noirq(struct device = *dev) } =20 communities =3D pctrl->context.communities; - for (i =3D 0; i < pctrl->ncommunities; i++) { + for (unsigned int i =3D 0; i < pctrl->ncommunities; i++) { struct intel_community *community =3D &pctrl->communities[i]; void __iomem *base; unsigned int gpp; @@ -1915,13 +1911,12 @@ static int intel_pinctrl_resume_noirq(struct device= *dev) struct intel_pinctrl *pctrl =3D dev_get_drvdata(dev); const struct intel_community_context *communities; const struct intel_pad_context *pads; - int i; =20 /* Mask all interrupts */ intel_gpio_irq_init(pctrl); =20 pads =3D pctrl->context.pads; - for (i =3D 0; i < pctrl->soc->npins; i++) { + for (unsigned int i =3D 0; i < pctrl->soc->npins; i++) { const struct pinctrl_pin_desc *desc =3D &pctrl->soc->pins[i]; =20 if (!(intel_pinctrl_should_save(pctrl, desc->number) || @@ -1938,17 +1933,16 @@ static int intel_pinctrl_resume_noirq(struct device= *dev) } =20 communities =3D pctrl->context.communities; - for (i =3D 0; i < pctrl->ncommunities; i++) { + for (unsigned int i =3D 0; i < pctrl->ncommunities; i++) { struct intel_community *community =3D &pctrl->communities[i]; void __iomem *base; - unsigned int gpp; =20 base =3D community->regs + community->ie_offset; - for (gpp =3D 0; gpp < community->ngpps; gpp++) + for (unsigned int gpp =3D 0; gpp < community->ngpps; gpp++) intel_restore_intmask(pctrl, i, base, gpp, communities[i].intmask[gpp]); =20 base =3D community->regs + community->hostown_offset; - for (gpp =3D 0; gpp < community->ngpps; gpp++) + for (unsigned int gpp =3D 0; gpp < community->ngpps; gpp++) intel_restore_hostown(pctrl, i, base, gpp, communities[i].hostown[gpp]); } =20 --=20 2.50.1