From patchwork Mon Dec 11 17:50:13 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mikhail Rudenko X-Patchwork-Id: 176843 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:bcd1:0:b0:403:3b70:6f57 with SMTP id r17csp7230379vqy; Mon, 11 Dec 2023 09:51:33 -0800 (PST) X-Google-Smtp-Source: AGHT+IF++xaJuKMZvc5TRbb1Xj63YTESo5u1AkQlPAdLFhUYLVXWzNev0eBNi7RKdGN4Z1Cpyvck X-Received: by 2002:a05:6a20:748f:b0:18c:f734:64e5 with SMTP id p15-20020a056a20748f00b0018cf73464e5mr2459867pzd.44.1702317093599; Mon, 11 Dec 2023 09:51:33 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1702317093; cv=none; d=google.com; s=arc-20160816; b=C+ZjJP21JmKzi/YynUTlrZ0xz6uXbBO8cY3yca8/8TM8eSnE19r6SGHJb2QarOiXxK FKIoOmE9ZXTFRdKKd01t75JMI6LTaGFAvFTFn15UUKgOS6JuN9t/id/qlE4yhQTXhBdE 7z7vXKE3O8TeII9BdnzScI1TmSfmRu/gIehcXzThBCQKQ0w8Or59Mw6xIeL76rS4dyDe JKvca58Hg52U4Zlg3emoW6jp2nJNfgHgKlbMy4LfbooWbt0UigK8m0dEsJGi2cBUG1RV rQTKMNJsoaN7e96mdAT8Z6DC+aq6se4pS0U24t8dwMD/+x95wWsuyt73ulvo7/Any4B7 EY7g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=cRYPXlKC8fxVqZOLpMO9mgZ04bcZLhSPp5xxyRwBBIg=; fh=fWwPNk1JCACPIViZY2QlWlGqVImCQC9sYxd2Gp7QPV0=; b=x63L0E5eKur3p66cWcEejLOVTw8aKhy/rFbl3P5biCZ2g0YpQIQxyaDv9+sGp+s6wp +34Qg88uscel9+tC/PMqKRjhoTsDlylCVps967HSMOIO06g4DT9mlgDJTWY8B4tVO1Rv QY/cg3WqRQKc0XjYF33cv91igjR7LaQKAc0CbkjUAwmtACyPBaaAGunQ2ITAx/yV4gS/ YyM/zF+IRPjI4Jwn+gSYGTi3v1Vu8ydg9jQqNTukwVm+KhzAC8GBTr1P94C1E5cgPBKC N+KndkJ1Lov0UGTMycIkGBbGKcKHTX2E4ggYZ5GX3ZFp9Rntlr3zsR9DJknH6FJP5xPL DT2Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=mVvyEERY; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from snail.vger.email (snail.vger.email. [2620:137:e000::3:7]) by mx.google.com with ESMTPS id b13-20020a630c0d000000b005c65eb2bc62si6155180pgl.237.2023.12.11.09.51.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 11 Dec 2023 09:51:33 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=mVvyEERY; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id CA67F80A30F4; Mon, 11 Dec 2023 09:51:31 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345147AbjLKRvU (ORCPT + 99 others); Mon, 11 Dec 2023 12:51:20 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37638 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1344952AbjLKRuw (ORCPT ); Mon, 11 Dec 2023 12:50:52 -0500 Received: from mail-lj1-x22a.google.com (mail-lj1-x22a.google.com [IPv6:2a00:1450:4864:20::22a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DE62FE3; Mon, 11 Dec 2023 09:50:57 -0800 (PST) Received: by mail-lj1-x22a.google.com with SMTP id 38308e7fff4ca-2ca0715f0faso65825371fa.0; Mon, 11 Dec 2023 09:50:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702317055; x=1702921855; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=cRYPXlKC8fxVqZOLpMO9mgZ04bcZLhSPp5xxyRwBBIg=; b=mVvyEERYOK8hWlNpLlOjY/CYVkWA/3QEhiZ6MVvCCqbmKnLt/te6GRyhDCQegdNqkI cl1SsPQoQG1kwjXevalIZCQeAROePdlt3ctnO+Bz5wmjjSW9eMHMXku+iR3uxa8NEiuW aR9iKoZEH7yzG7JznKbrNVv6+dIME7mvWVqaAuGq3S0GMljLoouocApUkgG4qNKdcbnI Lawml4fC1z1izWtLQzLCTTlALDMNl6XlXzX8euy6CbOu4RxeOcfi05EfDE1oDt+VSCai /oJa+d+VSw+sJSQCCtPy66K/0/0ERL8zFb7eOnUAiwyrnKyYeY0DKfivaBZqDoyeSzKZ NS8Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702317055; x=1702921855; 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:message-id:reply-to; bh=cRYPXlKC8fxVqZOLpMO9mgZ04bcZLhSPp5xxyRwBBIg=; b=W02CPBSrcnA6TlCkYez8B1z2OlguZS961ezdH8ioBDSAA1/G5NuXVvI2xIzwF4P6/0 89yrv1LJbqYo5JABCT9SY3uLPfziq+X4QOHSmCMf+i8ePkH1BAkI0HLGfJXIkpLgHJMQ RQ6WseiJYsk46vE/dKa4jUcAFLorOss5LNPGdaKi70fubcb6KZHV+RhF1Ktutk60RnZU J71uZsr8ZFTotWMsdTnHfpPzFA8XOhaJ6+hiOgasvnLhGKUslBuUzCHx5wQNAahagt3C bYU/2Y86cEaId+rrkbMAtA64qlt+EiVMGVS6kQdnWxv1D7q+66hVhG+XT8s7/C/pWBXu VoOw== X-Gm-Message-State: AOJu0Yx5JxLxThJGz3Ar4o4zoEfBDcIm+HKJnLvGo/cwM/4ZfYfWVytA OGbtWu2I/RsgtJugPvB/rhzqhOTMcw6k+LCY X-Received: by 2002:a2e:3909:0:b0:2cc:1fc1:e209 with SMTP id g9-20020a2e3909000000b002cc1fc1e209mr993981lja.88.1702317055474; Mon, 11 Dec 2023 09:50:55 -0800 (PST) Received: from localhost ([83.149.246.185]) by smtp.gmail.com with ESMTPSA id f7-20020a05651c03c700b002ca0192dad7sm1310294ljp.44.2023.12.11.09.50.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 11 Dec 2023 09:50:55 -0800 (PST) From: Mikhail Rudenko To: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Sakari Ailus , Laurent Pinchart , Jacopo Mondi , Tommaso Merciai , Christophe JAILLET , Dave Stevenson , Mauro Carvalho Chehab , Mikhail Rudenko Subject: [PATCH 10/19] media: i2c: ov4689: Make horizontal blanking configurable Date: Mon, 11 Dec 2023 20:50:13 +0300 Message-ID: <20231211175023.1680247-11-mike.rudenko@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231211175023.1680247-1-mike.rudenko@gmail.com> References: <20231211175023.1680247-1-mike.rudenko@gmail.com> MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Mon, 11 Dec 2023 09:51:31 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1785008848683957697 X-GMAIL-MSGID: 1785008848683957697 Make horizontal blanking configurable. To do so, set HTS register according to the requested horizontal blanking in ov4689_set_ctrl instead of the register table. Default HTS value is not changed by this patch. Minimal HTS value is found experimentally and corresponds to 90 fps framerate at minimum vertical blanking. Real HTS value is the register value multiplied by 4. Signed-off-by: Mikhail Rudenko Reviewed-by: Laurent Pinchart --- drivers/media/i2c/ov4689.c | 33 +++++++++++++++++++++------------ 1 file changed, 21 insertions(+), 12 deletions(-) diff --git a/drivers/media/i2c/ov4689.c b/drivers/media/i2c/ov4689.c index 9fa06941a0e5..67d4004bdcfb 100644 --- a/drivers/media/i2c/ov4689.c +++ b/drivers/media/i2c/ov4689.c @@ -30,7 +30,6 @@ #define OV4689_REG_EXPOSURE CCI_REG24(0x3500) #define OV4689_EXPOSURE_MIN 4 #define OV4689_EXPOSURE_STEP 1 -#define OV4689_VTS_MAX 0x7fff #define OV4689_REG_GAIN CCI_REG16(0x3508) #define OV4689_GAIN_STEP 1 @@ -41,6 +40,11 @@ #define OV4689_TEST_PATTERN_DISABLE 0x0 #define OV4689_REG_VTS CCI_REG16(0x380e) +#define OV4689_VTS_MAX 0x7fff + +#define OV4689_REG_HTS CCI_REG16(0x380c) +#define OV4689_HTS_DIVIDER 4 +#define OV4689_HTS_MAX 0x7fff #define OV4689_LANES 4 #define OV4689_XVCLK_FREQ 24000000 @@ -61,6 +65,7 @@ struct ov4689_mode { u32 width; u32 height; u32 hts_def; + u32 hts_min; u32 vts_def; u32 exp_def; u32 pixel_rate; @@ -104,7 +109,7 @@ struct ov4689_gain_range { /* * Xclk 24Mhz - * max_framerate 30fps + * max_framerate 90fps * mipi_datarate per lane 1008Mbps */ static const struct cci_reg_sequence ov4689_2688x1520_regs[] = { @@ -175,8 +180,6 @@ static const struct cci_reg_sequence ov4689_2688x1520_regs[] = { /* Timing control */ {CCI_REG8(0x3801), 0x08}, /* H_CROP_START_L h_crop_start[7:0] = 0x08 */ {CCI_REG8(0x3805), 0x97}, /* H_CROP_END_L h_crop_end[7:0] = 0x97 */ - {CCI_REG8(0x380c), 0x0a}, /* TIMING_HTS_H hts[14:8] = 0x0a */ - {CCI_REG8(0x380d), 0x0e}, /* TIMING_HTS_L hts[7:0] = 0x0e */ {CCI_REG8(0x3811), 0x08}, /* H_WIN_OFF_L h_win_off[7:0] = 0x08*/ {CCI_REG8(0x3813), 0x04}, /* V_WIN_OFF_L v_win_off[7:0] = 0x04 */ {CCI_REG8(0x3819), 0x01}, /* VSYNC_END_L vsync_end_point[7:0] = 0x01 */ @@ -237,7 +240,8 @@ static const struct ov4689_mode supported_modes[] = { .crop_top = 8, .crop_left = 16, .exp_def = 1536, - .hts_def = 4 * 2574, + .hts_def = 10296, + .hts_min = 3432, .vts_def = 1554, .pixel_rate = 480000000, .reg_list = ov4689_2688x1520_regs, @@ -596,6 +600,11 @@ static int ov4689_set_ctrl(struct v4l2_ctrl *ctrl) case V4L2_CID_TEST_PATTERN: ret = ov4689_enable_test_pattern(ov4689, val); break; + case V4L2_CID_HBLANK: + cci_write(regmap, OV4689_REG_HTS, + (val + ov4689->cur_mode->width) / + OV4689_HTS_DIVIDER, &ret); + break; default: dev_warn(dev, "%s Unhandled id:0x%x, val:0x%x\n", __func__, ctrl->id, val); @@ -618,13 +627,13 @@ static int ov4689_initialize_controls(struct ov4689 *ov4689) struct v4l2_ctrl_handler *handler; const struct ov4689_mode *mode; s64 exposure_max, vblank_def; + s64 hblank_def, hblank_min; struct v4l2_ctrl *ctrl; - s64 h_blank_def; int ret; handler = &ov4689->ctrl_handler; mode = ov4689->cur_mode; - ret = v4l2_ctrl_handler_init(handler, 10); + ret = v4l2_ctrl_handler_init(handler, 11); if (ret) return ret; @@ -636,11 +645,11 @@ static int ov4689_initialize_controls(struct ov4689 *ov4689) v4l2_ctrl_new_std(handler, NULL, V4L2_CID_PIXEL_RATE, 0, mode->pixel_rate, 1, mode->pixel_rate); - h_blank_def = mode->hts_def - mode->width; - ctrl = v4l2_ctrl_new_std(handler, NULL, V4L2_CID_HBLANK, h_blank_def, - h_blank_def, 1, h_blank_def); - if (ctrl) - ctrl->flags |= V4L2_CTRL_FLAG_READ_ONLY; + hblank_def = mode->hts_def - mode->width; + hblank_min = mode->hts_min - mode->width; + v4l2_ctrl_new_std(handler, &ov4689_ctrl_ops, V4L2_CID_HBLANK, + hblank_min, OV4689_HTS_MAX - mode->width, + OV4689_HTS_DIVIDER, hblank_def); vblank_def = mode->vts_def - mode->height; v4l2_ctrl_new_std(handler, &ov4689_ctrl_ops, V4L2_CID_VBLANK,