From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f172.google.com (mail-pf1-f172.google.com [209.85.210.172]) (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 7EF141917CD for ; Thu, 25 Jun 2026 23:51:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782431462; cv=none; b=VevmrYi3FKfcAwCEEXIb+A2UU1bax7HA+wqDWAAnUEokxORY9KhNdzUQariGIydqWarhP0ATRW0UjYEQ7yQco+zgyTURyZH+MQ0qqAgs1F4hLNEXka40xR/p/H+57qWCxNEv7HDWShOyrptzRXnzjGkO2TPWp4mvlsxgKYloXlA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782431462; c=relaxed/simple; bh=he76//56YME9jSWTiGsiGo7sY90s/qVXcUe5rVo2hP4=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=j5pugLeg/NWOxZuYWh1WkWoa/QsfQvDQJGC+VnNKCUdhw99v61orGlVm/LLntIZmWLLq5BgS0gd98q0zfZMUW1ZCjTC02VWl66kzfybeG2JjkQLojQhCVhafzYM0mkukOa0umoPict+mWX19igD+gXTYKGwYpqnlyolmzvkPKSo= 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=GwKW40ja; arc=none smtp.client-ip=209.85.210.172 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="GwKW40ja" Received: by mail-pf1-f172.google.com with SMTP id d2e1a72fcca58-84538597e1fso156730b3a.1 for ; Thu, 25 Jun 2026 16:51:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1782431461; x=1783036261; 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; bh=vB7wM8cMcHCRy2B7PVI6rXtb1cm+oM7TlYvGymfkF+4=; b=GwKW40jaLjJpOjauCsF4LBFvAgzXEdyY30V2U44B8hfhfc052htbENDDRry3LkZ14j 6XDSPX4g11db4v4DRtzdl2Xp1HyPZlXx8YK0NQKn7J61ll0hwamoF9F00nuCvuLJ/093 W3V3FAoPLteECbb3gWhrRWE4LAAaDIaWWyfgQIFDqqnBYPOo8MciQkjO7+Iz5L95QXMe MmXOi12HYFfRWNfJ9L4AwCyXxY+EbuvNbxxJQ82NqcYzv3XRjQ4SpmUDR25hZYWYY0qd QpvAgiM5oyKgFX1qHWAp90eNIwplzIwNLlN64/5/6tmYu0DIfU+nstaM0qXbBM9KUCj3 85Iw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782431461; x=1783036261; 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; bh=vB7wM8cMcHCRy2B7PVI6rXtb1cm+oM7TlYvGymfkF+4=; b=kS4VSfi4YvEbLBgrwpd/VUjNtoEqeBKgpIxEYdy0F8ssNDpS+i/q6GgRiRblTeyA34 ZQV/BLZdogL17RxrtYY2SGnHyTdhKFIiWPVok9I4tTzqkqkfS37araPd/7NLaFJYFM2N D/PmmXoionMmSvs0oFeHtTyh2DAg5eXvOID9eP+7QdgkXEyCMNQlefylpSVZZ2LBsuAv +rEkeNZ6t0UOxb2k0y8bmiTYnvMlX+zKZws7Ynn3a19cVosTpwvrBfiC2hykrmEdxDSM U2Ma9JdAP/Uwb3nRdq63stZoxGP8Tike26VXod2TGXALca943wlIiiwDiJvbjSHQNYjT ETfQ== X-Forwarded-Encrypted: i=1; AHgh+RorXC6eRLHSAQDAu1jsrkmnt7TIgUu8oehTJV1UJqCLiyFLA2sR4cyhLS465z5UqIPBkb9sqvJFx7t8vqg=@vger.kernel.org X-Gm-Message-State: AOJu0YzjJyvnKlgcRWv3FssleZuYRCSs2rFDCbMyDD/3WEYMtUp+DOp4 3GDsLM+mZzKFCNGXCT1C6pQBBPudRalJXtbZe23fJQSyyTOzwep15XBU X-Gm-Gg: AfdE7cmOl3Dd4Au5ztRlN65677JiQFQWwdYIHpQcNrNOQWqt6ZVl9bhQy+j5sKJNASC F4BpwKNOSInC/R306WyNwZHPrv5X137/JUg/SvuOoWqcykMMJ4JrLPs2bfQWlBN74ZRQAXzVheb f981CKVGXP8X/VVF47NLylBNcZXFfGKljuWGOJ0CZN7mdJY1XI/4EVJFTjLUo85T3WjdvkOvIJB fOYHJIb7G4cfG9iEic5nJLpE4Svg59CTHLhsvtXGnW/TUcaZ8cUKdqXhB1ELlCN35ifc/Tw1s1p ODSZR3lnmECtnJy6CxOo87mXL3GonDTtgxhZ17ZDWOB4t4bR56F1RLE3ejfg2+nsaap8jYo6bd9 2ZmHxW7t1WpSEfJRvrkJwj3pWt1VWwNW3feisYmI3ExBqA9JtmIHjWQcG+I5hTbEOuZEzkCHkLm Ed+yTaFqM3pRKRUTpYPZDe8lSnziOBVNQEpJtY X-Received: by 2002:a05:6a00:390b:b0:845:6252:f69b with SMTP id d2e1a72fcca58-845b3b09722mr5604830b3a.46.1782431460796; Thu, 25 Jun 2026 16:51:00 -0700 (PDT) Received: from x1c ([2405:9800:b670:b64b:acd1:19e:37f9:15e4]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-845c7c935f6sm159448b3a.19.2026.06.25.16.50.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 25 Jun 2026 16:51:00 -0700 (PDT) From: Tharit Tangkijwanichakul To: Sakari Ailus , Dave Stevenson , Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, skhan@linuxfoundation.org, linux-kernel-mentees@lists.linux.dev, Tharit Tangkijwanichakul Subject: [PATCH v2] media: imx219: fix test pattern ordering and add patterns Date: Fri, 26 Jun 2026 06:50:51 +0700 Message-ID: <20260625235051.5784-1-tharitt97@gmail.com> X-Mailer: git-send-email 2.53.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit The test pattern menu currently maps 1 to color bars and 2 to a solid color, whereas the imx219 datasheet specifies the reverse layout. fix the ordering to align with the hardware specification. Additionally, add 5 missing test patterns to complete the available hardware test patterns. The changes were validated on a raspberry pi 5 with an imx219 sensor using v4l2-ctl and rpicam-still. Signed-off-by: Tharit Tangkijwanichakul --- Changes in v2: - Fix typo in commit message: correct current color bars index to 1 and solid color to 2 - No code changes from v1 drivers/media/i2c/imx219.c | 31 +++++++++++++++++++++++-------- 1 file changed, 23 insertions(+), 8 deletions(-) diff --git a/drivers/media/i2c/imx219.c b/drivers/media/i2c/imx219.c index 223d3753c..8f0c04afd 100644 --- a/drivers/media/i2c/imx219.c +++ b/drivers/media/i2c/imx219.c @@ -110,11 +110,16 @@ /* Test Pattern Control */ #define IMX219_REG_TEST_PATTERN CCI_REG16(0x0600) -#define IMX219_TEST_PATTERN_DISABLE 0 -#define IMX219_TEST_PATTERN_SOLID_COLOR 1 -#define IMX219_TEST_PATTERN_COLOR_BARS 2 -#define IMX219_TEST_PATTERN_GREY_COLOR 3 -#define IMX219_TEST_PATTERN_PN9 4 +#define IMX219_TEST_PATTERN_DISABLE 0 +#define IMX219_TEST_PATTERN_SOLID_COLOR 1 +#define IMX219_TEST_PATTERN_COLOR_BARS 2 +#define IMX219_TEST_PATTERN_GREY_COLOR 3 +#define IMX219_TEST_PATTERN_PN9 4 +#define IMX219_TEST_PATTERN_16SPLIT_COLOR_BARS 5 +#define IMX219_TEST_PATTERN_16SPLIT_INV_COLOR_BARS 6 +#define IMX219_TEST_PATTERN_COLUMN_COUNTER 7 +#define IMX219_TEST_PATTERN_INV_COLUMN_COUNTER 8 +#define IMX219_TEST_PATTERN_PN31 9 /* Test pattern colour components */ #define IMX219_REG_TESTP_RED CCI_REG16(0x0602) @@ -235,18 +240,28 @@ static const s64 imx219_link_freq_4lane_menu[] = { static const char * const imx219_test_pattern_menu[] = { "Disabled", - "Color Bars", "Solid Color", + "Color Bars", "Grey Color Bars", - "PN9" + "PN9", + "16 Split Color Bars", + "16 Split Inverted Color Bars", + "Column Counter", + "Inverted Column Counter", + "PN31" }; static const int imx219_test_pattern_val[] = { IMX219_TEST_PATTERN_DISABLE, - IMX219_TEST_PATTERN_COLOR_BARS, IMX219_TEST_PATTERN_SOLID_COLOR, + IMX219_TEST_PATTERN_COLOR_BARS, IMX219_TEST_PATTERN_GREY_COLOR, IMX219_TEST_PATTERN_PN9, + IMX219_TEST_PATTERN_16SPLIT_COLOR_BARS, + IMX219_TEST_PATTERN_16SPLIT_INV_COLOR_BARS, + IMX219_TEST_PATTERN_COLUMN_COUNTER, + IMX219_TEST_PATTERN_INV_COLUMN_COUNTER, + IMX219_TEST_PATTERN_PN31 }; /* regulator supplies */ -- 2.53.0