From nobody Mon Sep 28 22:31:01 2026 Received: from mail-pg1-f171.google.com (mail-pg1-f171.google.com [209.85.215.171]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6D43F28852E for ; Sun, 16 Aug 2026 01:03:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.171 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786842214; cv=none; b=M5qbyDSMVWUQTjxM1/JH1md7krBuo0mG6CmK0Ld41k0UXsBzCpZxPYuh95OigKf5yOQ0Bn2p+Ssk4SdBKVfp7Arj955jSWC4UkqfOkUvmX0TrgxL51TTvXXFSRxDkyTbIWUNIxY4C27H0CxQvxqn7EWUz5iwW3SqYcLMyJ5kq4I= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786842214; c=relaxed/simple; bh=FifGW5MG7JbFILeAVwa1jNog8QfeQwpsY+9kcqd4X7s=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=kT5bgADzOAoPMXxfDhmbNbn3/8GxpHmrYqqzaHjfYl3PLYlD2UeSYnq88n15bV3XwbxjwT3XD2nToLjzuJ524goDtLwAE3LkgjGu76ocEYDMfMby6GQCapVIYdZ72iLoc89ADWfYrp/yAP36TjMMRVqRUlaZ3Wyd7tlmqlmzmBY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=F9pIVOdf; arc=none smtp.client-ip=209.85.215.171 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="F9pIVOdf" Received: by mail-pg1-f171.google.com with SMTP id 41be03b00d2f7-c9eefcf9175so2086937a12.3 for ; Sat, 15 Aug 2026 18:03:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786842213; x=1787447013; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=lsCwMPKOG1ojLz18oQtlTHtiNUW+J/Ma5sW3249cgm8=; b=F9pIVOdf7Ro7nyG3/zXv6WgRdaJVEClNVvusAiJvCVxj2bZTSNdu6tZKy8YRYCzDSM zc6bjwgm2TTq0Fn4rWxc4VlpA/EW4SHezSyaG1lyRNHYmf5R3anmIIDx/j3uF5BptSUI EvfCufi9LtqSJuh3EMPh5JQrp0C4MItEt4yXSw75Fsn4+xOEv+QbrTNSUiZBRzJbZT44 CyG3GIxO7Me34/FYtiGWg1MKA6pGaMoOGrjn2XYU6LHQrW/0C+GgpyjuI0fcJ6HlrgSF 2v4oOtSnPVXtwX7Jb8ESBsmWbXpgj5GF2ldj11LPtpFOAXr7A+/pIGrurr5uvSEBkdsi pKxw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786842213; x=1787447013; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=lsCwMPKOG1ojLz18oQtlTHtiNUW+J/Ma5sW3249cgm8=; b=Psk2kVSI15ujRSJD3lz9TwyqyBCkHal3WCwOhBp/m7+b/v4XSD2jeJ+AWpAaxdDOyz ubYzc7ToGO72+Ush+M61I4HERpMTTlizGNvZFNN2joyGKhLo50G+UEsof72F0nDg2g5U r3zCPADSUUvHrDYcaxurPXmgMJjtsdJpT193/vqpNKPut7rupW6DY3nQiXiQIziqtRZd 63LwhUpwBgf8b89fI1+RVsvukXITjEk2nf6nzAL2BHA3o6FI9qe0xhtCDi6aYiUkkjYv D5hnSo5U51KiPGX3Ksp5XLjtxv0h6Cmp7DEi7UN/J/jRh56uYLNjsZjcECM2UfnkCQhw DlwA== X-Forwarded-Encrypted: i=1; AHgh+Rr4BD30ULsfjdt3sRoi7H1yo3zqpzbTf1bhAT/bHXGBwm0aMaK1HKpUHbW7N1dc9tdLz0+4zfPt7WowDHw=@vger.kernel.org X-Gm-Message-State: AOJu0YzN9HFUnOBxoMvENFf0WNxsRiuPVtRtIMGfTovzNIBFIaEqI5wo 0EI0w8B9t9EL2SakyK8fdYyNiAeqCjHb8kdlj2FT6GlNRaa7NIeTvkP2 X-Gm-Gg: AR+sD10c2SvwUHlh3kv+1872m5cbgdfSl7gD6h45Gj2k+mK0+f+JsKbSpKiCNwrbTCT 646byKiC5t1dO/qR5MOLvkeXNynsPI5K35jxDMVXfh0kEb6/hFJINOKEdO+brIyv/oxjSKAryOG U8qV/BNjx4VeAu4OBChU/rs3f/6tSeU9b+n2oPwuh3wxrYosXEv3Y2FH5oB57hPmAt4V46JXWBS 8jqGARfP4q82e178DMS9VIe/9CdCECaXK8CFyfiPsMlQqswGPc41vzVJxhgIS9KH2QUnNC7eweP KV/Sh8h1u2ghQbmcFioJe57QBonnH1NP/687pGhOBLH2OhfgHcpVtkeRlaL6Yf21yeaPRxO0HEd c0QMVVHSPSWPHcvDv8rdO1x1TMDXtKXw4lpXrrSUrQx49zLprR5OnMoP0739GSvYEIJQpaqH/1z Clgk4SKMMYS1jiNIM+xMqvN60ByO24YgNAqCeiaFc/tYz7RkTj+AIWEfuspOeYoaWcDeMA4tffk t9VZ9kGe18rSadnPLwQ5HneQBLAGJSyWVeDBdEwYLd0 X-Received: by 2002:a05:6a20:b598:b0:3bf:b1d3:393e with SMTP id adf61e73a8af0-3cc719c7b62mr19336093637.2.1786842212702; Sat, 15 Aug 2026 18:03:32 -0700 (PDT) Received: from ubuntu-vm.. (c-73-189-122-63.hsd1.ca.comcast.net. [73.189.122.63]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-141387c6f95sm32113463c88.6.2026.08.15.18.03.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 15 Aug 2026 18:03:32 -0700 (PDT) From: Rishab Madhugiri To: hansg@kernel.org, mchehab@kernel.org, gregkh@linuxfoundation.org Cc: andy@kernel.org, sakari.ailus@linux.intel.com, linux-staging@lists.linux.dev, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, error27@gmail.com, Rishab Madhugiri Subject: [PATCH v3] staging: media: atomisp: Use ARRAY_SIZE macro Date: Sun, 16 Aug 2026 01:01:26 +0000 Message-ID: <20260816010126.159514-1-rishab.madhugiri@gmail.com> X-Mailer: git-send-email 2.43.0 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" Clean up loop bound calculation by replacing sizeof(bds_factors) / sizeof(struct bayer_ds_factor) with the ARRAY_SIZE() helper macro to improve readability and maintainability. Move loop counter increment to the same line as the for loop and declare the loop counter locally. Signed-off-by: Rishab Madhugiri Reviewed-by: Andy Shevchenko --- v3: Declare loop counter variable locally in the for loop and remove previous function-scope declaration as suggested by Andy Shevchenko. v2: Move loop increment to the same line as the for statement as suggested by Dan Carpenter. drivers/staging/media/atomisp/pci/atomisp_compat_css20.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/media/atomisp/pci/atomisp_compat_css20.c b/dri= vers/staging/media/atomisp/pci/atomisp_compat_css20.c index 0ee52637e..eacd7ee5c 100644 --- a/drivers/staging/media/atomisp/pci/atomisp_compat_css20.c +++ b/drivers/staging/media/atomisp/pci/atomisp_compat_css20.c @@ -2118,7 +2118,6 @@ static void __configure_video_pp_input(struct atomisp= _sub_device *asd, static const struct bayer_ds_factor bds_factors[] =3D { {8, 1}, {6, 1}, {4, 1}, {3, 1}, {2, 1}, {3, 2} }; - unsigned int i; =20 if (width =3D=3D 0 && height =3D=3D 0) return; @@ -2160,8 +2159,7 @@ static void __configure_video_pp_input(struct atomisp= _sub_device *asd, bayer_ds_out_res->width =3D effective_res->width; bayer_ds_out_res->height =3D effective_res->height; =20 - for (i =3D 0; i < sizeof(bds_factors) / sizeof(struct bayer_ds_factor); - i++) { + for (unsigned int i =3D 0; i < ARRAY_SIZE(bds_factors); i++) { if (effective_res->width >=3D out_width * bds_factors[i].numerator / bds_factors[i].denominator && effective_res->height >=3D out_height * --=20 2.43.0