Received: by 2002:a05:6a10:f3d0:0:0:0:0 with SMTP id a16csp427794pxv; Thu, 8 Jul 2021 05:55:47 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy756vM/GwQ62lLixiwhnXnMtRondhGXox7+4qXATdCTsUQXBcFjWsndCn3VF4HIBprN0E3 X-Received: by 2002:a17:907:1ddb:: with SMTP id og27mr262277ejc.540.1625748947083; Thu, 08 Jul 2021 05:55:47 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1625748947; cv=pass; d=google.com; s=arc-20160816; b=pv8Ca+yeFRgd/lAtMk8YyDPM4vIYaVJ3VAigaKmiIikvg7mVdA4Dv9lqSS47/D80iT hOgHY8UlTSlYdCuSKmprSkv9B8M0G2L/wCkKgW8EF9raRmS1H48bDj7zShqLrhz2ATYa QklPIoZacICdoAo8IRIQVESXNkujDsLD+crvWnKAnFKwRMvbZkzJWvXyFoR6c1OkjZPN 8x032F4oubT2ueErXzIGtsBYAtA1hsLaJ8ivIU++yx2hGTxh01m6Csl//acCMdOKODVC f/ffuChp9iVT26AZSJchTe3P4GkOMrostgCJ+VjipHBowcIgCiO1GvOrqKqvOnYiNXoS JtHg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=eyU4MWyhDXvpOi4mNOhJCefcoLfFxwpLY0qXOIyKWTY=; b=i0XugSOAXucEWdmxhTanSgS/xT9KNlfffwWp4SqgKhTOKopCzcwk2xfa/aiSNinh/D FPIqA38A3KOD2KpXQbuRXTcYvyRpk096Ht4qRgMG6rDnKBcWcD8Nl4sN0uc3jeUu0SXo /x39EMFSPvUQe8+ieAlW+HbZlQvJDRoMd6ozu1pkSncbtTBq8WHjwMv1Pj8msHiY2+b8 ZOoGt+xP0zqQjQzJOZynTmx2i/tHgmS0zdmnMo/b/uIr0bSTB3wYw1dH4/dkLQDIY8TU 0zsH90JSK8xNa/lT2QUH3wth9JYEDlALxGG8meNKPhXQyiFCIdm6BmwTLLWmkqn6xKD6 KLOA== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@iki.fi header.s=lahtoruutu header.b=lTYgeaD2; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id mp10si3072916ejc.391.2021.07.08.05.55.24; Thu, 08 Jul 2021 05:55:47 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@iki.fi header.s=lahtoruutu header.b=lTYgeaD2; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231747AbhGHM5N (ORCPT + 99 others); Thu, 8 Jul 2021 08:57:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56030 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230433AbhGHM5M (ORCPT ); Thu, 8 Jul 2021 08:57:12 -0400 Received: from lahtoruutu.iki.fi (lahtoruutu.iki.fi [IPv6:2a0b:5c81:1c1::37]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0B9E4C061574; Thu, 8 Jul 2021 05:54:29 -0700 (PDT) Received: from hillosipuli.retiisi.eu (dbdx-ftd5cf51mv6xr7xy-3.rev.dnainternet.fi [IPv6:2001:14ba:8e7:f240:7911:3355:4753:76e6]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: sailus) by lahtoruutu.iki.fi (Postfix) with ESMTPSA id 43AC91B00046; Thu, 8 Jul 2021 15:54:25 +0300 (EEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iki.fi; s=lahtoruutu; t=1625748865; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=eyU4MWyhDXvpOi4mNOhJCefcoLfFxwpLY0qXOIyKWTY=; b=lTYgeaD2TRsAWMRr4uY1OQe2TxQo+KPXNUCDP2AXddk6VkwYLH657fMVC+BDB6W7W5sywC Qy5g5JlJIi3jS1j496XVd3H6lB5rLYF+EhU8thygv4vzj7BOiLnLW7AT2y/awi/YgEpa+4 7sOFNdx+BwgGgtfjnsBclLxNvWpbt93FkzzZe+m+vg41yIys8dP9qid6RezgNMs9bA14uy E/SMcqHa6jbVm8wYOnLpTVy9YSqhpfGzq4GUO7gk1lFFmjrOR+Ai/+AoBE24zhNzQGxWL1 o5J5LuvqPglPopZlTuvp2PshihRlg4+zXfbQmKqauuPAYTBSsOQgQDNbEAYojg== Received: from valkosipuli.localdomain (valkosipuli.localdomain [IPv6:fd35:1bc8:1a6:d3d5::80:2]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by hillosipuli.retiisi.eu (Postfix) with ESMTPS id E02A7634C87; Thu, 8 Jul 2021 15:53:45 +0300 (EEST) Received: from localhost ([127.0.0.1] helo=valkosipuli.retiisi.eu) by valkosipuli.localdomain with esmtp (Exim 4.92) (envelope-from ) id 1m1TXs-0001zR-5D; Thu, 08 Jul 2021 15:54:24 +0300 Date: Thu, 8 Jul 2021 15:54:24 +0300 From: Sakari Ailus To: Martin Kepplinger Cc: krzysztof.kozlowski@canonical.com, laurent.pinchart@ideasonboard.com, devicetree@vger.kernel.org, kernel@puri.sm, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, mchehab@kernel.org, paul.kocialkowski@bootlin.com, pavel@ucw.cz, phone-devel@vger.kernel.org, robh@kernel.org, shawnx.tu@intel.com Subject: Re: [PATCH v6 3/5] media: i2c: add driver for the SK Hynix Hi-846 8M pixel camera Message-ID: <20210708125424.GY3@valkosipuli.retiisi.eu> References: <20210628101054.828579-1-martin.kepplinger@puri.sm> <20210628101054.828579-4-martin.kepplinger@puri.sm> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210628101054.828579-4-martin.kepplinger@puri.sm> User-Agent: Mutt/1.10.1 (2018-07-13) ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=iki.fi; s=lahtoruutu; t=1625748865; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=eyU4MWyhDXvpOi4mNOhJCefcoLfFxwpLY0qXOIyKWTY=; b=Qy9aF4bMRosJSNmlacH0CjZtgZZf+QGGan9VUq30SCAz6CHXT8dH5XmLyR3TWkh2I15hth bARz93eTjSSccXAuziqVuexc2cp0z39bUap/d5ZYrfKegvpQODj0Og+JXmXbxZqqpNRWxf 0Wq4Cg1cTXSXkoBvqMnFAiZqo3rDhP2mJzAofrPj0QoIlJVETSo7iYY8tEv2cIlQ/o2Ha5 lKKJHkTZdbm3wOJgORr0ZUXyqui/2ZNhrudL47W36FJiZsvM7rG6gRZ3kbk5UNqkxY+oTH rflrf8j7iZFAHs2OG8AeCzea//sktmtwALK/ZiyKCATpDn6sZePmhMdqzbV8Og== ARC-Authentication-Results: i=1; ORIGINATING; auth=pass smtp.auth=sailus smtp.mailfrom=sakari.ailus@iki.fi ARC-Seal: i=1; s=lahtoruutu; d=iki.fi; t=1625748865; a=rsa-sha256; cv=none; b=YR+uS7Guy5wOEmG6z6IFdxUfNCaKTFqXIYpOauWESumIKIsg3hkthd+AQ/gukvyKLGvhRz YbIT6nmPXkxuIkhGf5fGMBwhK0y1+t+GiZeWHkjKPi0Z+k1pFrMPUaXxOD7ra83YmjNQ+Y i+PgR4J6PFYiabWcycurZ3LkqB+xhhyubo1hc9il7dROtMAVA5uHvHWGCA0ySjjXgbKo8p CN+h46YsJ3iKYq4DDVO1sJ11hiTZ2yeEUzIK6OnwzwFWscnPv2/yWWp+8yZOLg1ORkg9OQ y2EF3NzzEIFOo6cDZ2HVsDubH1TaTNqZ/gXhmSESlTqtiFVo8Wqd9OfkrsMaVg== Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Martin, A few more comments... On Mon, Jun 28, 2021 at 12:10:52PM +0200, Martin Kepplinger wrote: > The SK Hynix Hi-846 is a 1/4" 8M Pixel CMOS Image Sensor. It supports > usual features like I2C control, CSI-2 for frame data, digital/analog > gain control or test patterns. > > This driver supports the 640x480, 1280x720 and 1632x1224 resolution > modes. It supports runtime PM in order not to draw any unnecessary power. > > The part is also called YACG4D0C9SHC and a datasheet can be found at > https://product.skhynix.com/products/cis/cis.go > > The large sets of partly undocumented register values are for example > found when searching for the hi846_mipi_raw_Sensor.c Android driver. > > Signed-off-by: Martin Kepplinger > --- > MAINTAINERS | 6 + > drivers/media/i2c/Kconfig | 13 + > drivers/media/i2c/Makefile | 1 + > drivers/media/i2c/hi846.c | 2128 ++++++++++++++++++++++++++++++++++++ > 4 files changed, 2148 insertions(+) > create mode 100644 drivers/media/i2c/hi846.c > > diff --git a/MAINTAINERS b/MAINTAINERS > index 2c5dc7d20d81..e10783523636 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -8579,6 +8579,12 @@ S: Maintained > T: git git://linuxtv.org/media_tree.git > F: drivers/media/i2c/hi556.c > > +HYNIX HI846 SENSOR DRIVER > +M: Martin Kepplinger > +L: linux-media@vger.kernel.org > +S: Maintained > +F: drivers/media/i2c/hi846.c > + > Hyper-V/Azure CORE AND DRIVERS > M: "K. Y. Srinivasan" > M: Haiyang Zhang > diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig > index 588f8eb95984..5b020d05960c 100644 > --- a/drivers/media/i2c/Kconfig > +++ b/drivers/media/i2c/Kconfig > @@ -753,6 +753,19 @@ config VIDEO_IMX208 > To compile this driver as a module, choose M here: the > module will be called imx208. > > +config VIDEO_HI846 > + tristate "Hynix Hi-846 sensor support" > + depends on I2C && VIDEO_V4L2 > + select MEDIA_CONTROLLER > + select VIDEO_V4L2_SUBDEV_API > + select V4L2_FWNODE > + help > + This is a Video4Linux2 sensor driver for the Hynix > + Hi-846 camera. > + > + To compile this driver as a module, choose M here: the > + module will be called hi846. > + > config VIDEO_IMX214 > tristate "Sony IMX214 sensor support" > depends on GPIOLIB && I2C && VIDEO_V4L2 > diff --git a/drivers/media/i2c/Makefile b/drivers/media/i2c/Makefile > index 1168fa6b84ed..303cd7bc4e66 100644 > --- a/drivers/media/i2c/Makefile > +++ b/drivers/media/i2c/Makefile > @@ -116,6 +116,7 @@ obj-$(CONFIG_VIDEO_ML86V7667) += ml86v7667.o > obj-$(CONFIG_VIDEO_OV2659) += ov2659.o > obj-$(CONFIG_VIDEO_TC358743) += tc358743.o > obj-$(CONFIG_VIDEO_HI556) += hi556.o > +obj-$(CONFIG_VIDEO_HI846) += hi846.o > obj-$(CONFIG_VIDEO_IMX208) += imx208.o > obj-$(CONFIG_VIDEO_IMX214) += imx214.o > obj-$(CONFIG_VIDEO_IMX219) += imx219.o > diff --git a/drivers/media/i2c/hi846.c b/drivers/media/i2c/hi846.c > new file mode 100644 > index 000000000000..4b660ef9284e > --- /dev/null > +++ b/drivers/media/i2c/hi846.c > @@ -0,0 +1,2128 @@ > +// SPDX-License-Identifier: GPL-2.0 > +// Copyright (c) 2021 Purism SPC > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +#define HI846_MEDIA_BUS_FORMAT MEDIA_BUS_FMT_SGBRG10_1X10 > +#define HI846_RGB_DEPTH 10 > + > +/* Frame length lines / Vertical timings */ > +#define HI846_REG_FLL 0x0006 > +#define HI846_FLL_MAX 0xffff > + > +/* Horizontal timing */ > +#define HI846_REG_LLP 0x0008 > +#define HI846_LINE_LENGTH 3800 > + > +#define HI846_REG_BINNING_MODE 0x000c > + > +#define HI846_REG_IMAGE_ORIENTATION 0x000e > + > +#define HI846_REG_UNKNOWN_0022 0x0022 > + > +#define HI846_REG_Y_ADDR_START_VACT_H 0x0026 > +#define HI846_REG_Y_ADDR_START_VACT_L 0x0027 > +#define HI846_REG_UNKNOWN_0028 0x0028 > + > +#define HI846_REG_Y_ADDR_END_VACT_H 0x002c > +#define HI846_REG_Y_ADDR_END_VACT_L 0x002d > + > +#define HI846_REG_Y_ODD_INC_FOBP 0x002e > +#define HI846_REG_Y_EVEN_INC_FOBP 0x002f > + > +#define HI846_REG_Y_ODD_INC_VACT 0x0032 > +#define HI846_REG_Y_EVEN_INC_VACT 0x0033 > + > +#define HI846_REG_GROUPED_PARA_HOLD 0x0046 > + > +#define HI846_REG_TG_ENABLE 0x004c > + > +#define HI846_REG_UNKNOWN_005C 0x005c > + > +#define HI846_REG_UNKNOWN_006A 0x006a > + > +/* > + * Long exposure time. Actually, exposure is a 20 bit value that > + * includes the lower 4 bits of 0x0073 too. only 16 bit are used > + * right now > + */ > +#define HI846_REG_EXPOSURE 0x0074 > +#define HI846_EXPOSURE_MIN 6 > +#define HI846_EXPOSURE_MAX_MARGIN 2 > +#define HI846_EXPOSURE_STEP 1 > + > +/* Analog gain controls from sensor */ > +#define HI846_REG_ANALOG_GAIN 0x0077 > +#define HI846_ANAL_GAIN_MIN 0 > +#define HI846_ANAL_GAIN_MAX 240 > +#define HI846_ANAL_GAIN_STEP 8 > + > +/* Digital gain controls from sensor */ > +#define HI846_REG_MWB_GR_GAIN_H 0x0078 > +#define HI846_REG_MWB_GR_GAIN_L 0x0079 > +#define HI846_REG_MWB_GB_GAIN_H 0x007a > +#define HI846_REG_MWB_GB_GAIN_L 0x007b > +#define HI846_REG_MWB_R_GAIN_H 0x007c > +#define HI846_REG_MWB_R_GAIN_L 0x007d > +#define HI846_REG_MWB_B_GAIN_H 0x007e > +#define HI846_REG_MWB_B_GAIN_L 0x007f > +#define HI846_DGTL_GAIN_MIN 512 > +#define HI846_DGTL_GAIN_MAX 8191 > +#define HI846_DGTL_GAIN_STEP 1 > +#define HI846_DGTL_GAIN_DEFAULT 512 > + > +#define HI846_REG_X_ADDR_START_HACT_H 0x0120 > +#define HI846_REG_X_ADDR_END_HACT_H 0x0122 > + > +#define HI846_REG_UNKNOWN_012A 0x012a > + > +#define HI846_REG_UNKNOWN_0200 0x0200 > + > +#define HI846_REG_UNKNOWN_021C 0x021c > +#define HI846_REG_UNKNOWN_021E 0x021e > + > +#define HI846_REG_UNKNOWN_0402 0x0402 > +#define HI846_REG_UNKNOWN_0404 0x0404 > +#define HI846_REG_UNKNOWN_0408 0x0408 > +#define HI846_REG_UNKNOWN_0410 0x0410 > +#define HI846_REG_UNKNOWN_0412 0x0412 > +#define HI846_REG_UNKNOWN_0414 0x0414 > + > +#define HI846_REG_UNKNOWN_0418 0x0418 > + > +#define HI846_REG_UNKNOWN_051E 0x051e > + > +/* Formatter */ > +#define HI846_REG_X_START_H 0x0804 > +#define HI846_REG_X_START_L 0x0805 > + > +/* MIPI */ > +#define HI846_REG_UNKNOWN_0900 0x0900 > +#define HI846_REG_MIPI_TX_OP_EN 0x0901 > +#define HI846_REG_MIPI_TX_OP_MODE 0x0902 > +#define HI846_RAW8 BIT(5) > + > +#define HI846_REG_UNKNOWN_090C 0x090c > +#define HI846_REG_UNKNOWN_090E 0x090e > + > +#define HI846_REG_UNKNOWN_0914 0x0914 > +#define HI846_REG_TLPX 0x0915 > +#define HI846_REG_TCLK_PREPARE 0x0916 > +#define HI846_REG_TCLK_ZERO 0x0917 > +#define HI846_REG_UNKNOWN_0918 0x0918 > +#define HI846_REG_THS_PREPARE 0x0919 > +#define HI846_REG_THS_ZERO 0x091a > +#define HI846_REG_THS_TRAIL 0x091b > +#define HI846_REG_TCLK_POST 0x091c > +#define HI846_REG_TCLK_TRAIL_MIN 0x091d > +#define HI846_REG_UNKNOWN_091E 0x091e > + > +#define HI846_REG_UNKNOWN_0954 0x0954 > +#define HI846_REG_UNKNOWN_0956 0x0956 > +#define HI846_REG_UNKNOWN_0958 0x0958 > +#define HI846_REG_UNKNOWN_095A 0x095a > + > +/* ISP Common */ > +#define HI846_REG_MODE_SELECT 0x0a00 > +#define HI846_MODE_STANDBY 0x00 > +#define HI846_MODE_STREAMING 0x01 > +#define HI846_REG_FAST_STANDBY_MODE 0x0a02 > +#define HI846_REG_ISP_EN_H 0x0a04 > + > +/* Test Pattern Control */ > +#define HI846_REG_ISP 0x0a05 > +#define HI846_REG_ISP_TPG_EN 0x01 > +#define HI846_REG_TEST_PATTERN 0x020a /* 1-9 */ > + > +#define HI846_REG_UNKNOWN_0A0C 0x0a0c > + > +/* Windowing */ > +#define HI846_REG_X_OUTPUT_SIZE_H 0x0a12 > +#define HI846_REG_X_OUTPUT_SIZE_L 0x0a13 > +#define HI846_REG_Y_OUTPUT_SIZE_H 0x0a14 > +#define HI846_REG_Y_OUTPUT_SIZE_L 0x0a15 > + > +/* ISP Common */ > +#define HI846_REG_PEDESTAL_EN 0x0a1a > + > +#define HI846_REG_UNKNOWN_0A1E 0x0a1e > + > +/* Horizontal Binning Mode */ > +#define HI846_REG_HBIN_MODE 0x0a22 > + > +#define HI846_REG_UNKNOWN_0A24 0x0a24 > +#define HI846_REG_UNKNOWN_0B02 0x0b02 > +#define HI846_REG_UNKNOWN_0B10 0x0b10 > +#define HI846_REG_UNKNOWN_0B12 0x0b12 > +#define HI846_REG_UNKNOWN_0B14 0x0b14 > + > +/* BLC (Black Level Calibration) */ > +#define HI846_REG_BLC_CTL0 0x0c00 > + > +#define HI846_REG_UNKNOWN_0C06 0x0c06 > +#define HI846_REG_UNKNOWN_0C10 0x0c10 > +#define HI846_REG_UNKNOWN_0C12 0x0c12 > +#define HI846_REG_UNKNOWN_0C14 0x0c14 > +#define HI846_REG_UNKNOWN_0C16 0x0c16 > + > +#define HI846_REG_UNKNOWN_0E04 0x0e04 > + > +#define HI846_REG_CHIP_ID_L 0x0f16 > +#define HI846_REG_CHIP_ID_H 0x0f17 > +#define HI846_CHIP_ID_L 0x46 > +#define HI846_CHIP_ID_H 0x08 > + > +#define HI846_REG_UNKNOWN_0F04 0x0f04 > +#define HI846_REG_UNKNOWN_0F08 0x0f08 > + > +/* PLL */ > +#define HI846_REG_PLL_CFG_MIPI2_H 0x0f2a > +#define HI846_REG_PLL_CFG_MIPI2_L 0x0f2b > + > +#define HI846_REG_UNKNOWN_0F30 0x0f30 > +#define HI846_REG_PLL_CFG_RAMP1_H 0x0f32 > +#define HI846_REG_UNKNOWN_0F36 0x0f36 > +#define HI846_REG_PLL_CFG_MIPI1_H 0x0f38 > + > +#define HI846_REG_UNKNOWN_2008 0x2008 > +#define HI846_REG_UNKNOWN_326E 0x326e > + > +struct hi846_reg { > + u16 address; > + u16 val; > +}; > + > +struct hi846_reg_list { > + u32 num_of_regs; > + const struct hi846_reg *regs; > +}; > + > +struct hi846_mode { > + /* Frame width in pixels */ > + u32 width; > + > + /* Frame height in pixels */ > + u32 height; > + > + /* Horizontal timing size */ > + u32 llp; > + > + /* Link frequency needed for this resolution */ > + u8 link_freq_index; > + > + u16 fps; > + > + /* vertical timining size */ > + u16 frame_len; > + > + const struct hi846_reg_list reg_list_config; > + const struct hi846_reg_list reg_list_2lane; > + const struct hi846_reg_list reg_list_4lane; > + > + /* position inside of the 3264x2448 pixel array */ > + struct v4l2_rect crop; > +}; > + > +static const struct hi846_reg hi846_init_2lane[] = { > + {HI846_REG_MODE_SELECT, 0x0000}, > + /* regs below are unknown */ > + {0x2000, 0x100a}, > + {0x2002, 0x00ff}, > + {0x2004, 0x0007}, > + {0x2006, 0x3fff}, > + {0x2008, 0x3fff}, > + {0x200a, 0xc216}, > + {0x200c, 0x1292}, > + {0x200e, 0xc01a}, > + {0x2010, 0x403d}, > + {0x2012, 0x000e}, > + {0x2014, 0x403e}, > + {0x2016, 0x0b80}, > + {0x2018, 0x403f}, > + {0x201a, 0x82ae}, > + {0x201c, 0x1292}, > + {0x201e, 0xc00c}, > + {0x2020, 0x4130}, > + {0x2022, 0x43e2}, > + {0x2024, 0x0180}, > + {0x2026, 0x4130}, > + {0x2028, 0x7400}, > + {0x202a, 0x5000}, > + {0x202c, 0x0253}, > + {0x202e, 0x0ad1}, > + {0x2030, 0x2360}, > + {0x2032, 0x0009}, > + {0x2034, 0x5020}, > + {0x2036, 0x000b}, > + {0x2038, 0x0002}, > + {0x203a, 0x0044}, > + {0x203c, 0x0016}, > + {0x203e, 0x1792}, > + {0x2040, 0x7002}, > + {0x2042, 0x154f}, > + {0x2044, 0x00d5}, > + {0x2046, 0x000b}, > + {0x2048, 0x0019}, > + {0x204a, 0x1698}, > + {0x204c, 0x000e}, > + {0x204e, 0x099a}, > + {0x2050, 0x0058}, > + {0x2052, 0x7000}, > + {0x2054, 0x1799}, > + {0x2056, 0x0310}, > + {0x2058, 0x03c3}, > + {0x205a, 0x004c}, > + {0x205c, 0x064a}, > + {0x205e, 0x0001}, > + {0x2060, 0x0007}, > + {0x2062, 0x0bc7}, > + {0x2064, 0x0055}, > + {0x2066, 0x7000}, > + {0x2068, 0x1550}, > + {0x206a, 0x158a}, > + {0x206c, 0x0004}, > + {0x206e, 0x1488}, > + {0x2070, 0x7010}, > + {0x2072, 0x1508}, > + {0x2074, 0x0004}, > + {0x2076, 0x0016}, > + {0x2078, 0x03d5}, > + {0x207a, 0x0055}, > + {0x207c, 0x08ca}, > + {0x207e, 0x2019}, > + {0x2080, 0x0007}, > + {0x2082, 0x7057}, > + {0x2084, 0x0fc7}, > + {0x2086, 0x5041}, > + {0x2088, 0x12c8}, > + {0x208a, 0x5060}, > + {0x208c, 0x5080}, > + {0x208e, 0x2084}, > + {0x2090, 0x12c8}, > + {0x2092, 0x7800}, > + {0x2094, 0x0802}, > + {0x2096, 0x040f}, > + {0x2098, 0x1007}, > + {0x209a, 0x0803}, > + {0x209c, 0x080b}, > + {0x209e, 0x3803}, > + {0x20a0, 0x0807}, > + {0x20a2, 0x0404}, > + {0x20a4, 0x0400}, > + {0x20a6, 0xffff}, > + {0x20a8, 0xf0b2}, > + {0x20aa, 0xffef}, > + {0x20ac, 0x0a84}, > + {0x20ae, 0x1292}, > + {0x20b0, 0xc02e}, > + {0x20b2, 0x4130}, > + {0x23fe, 0xc056}, > + {0x3232, 0xfc0c}, > + {0x3236, 0xfc22}, > + {0x3248, 0xfca8}, > + {0x326a, 0x8302}, > + {0x326c, 0x830a}, > + {0x326e, 0x0000}, > + {0x32ca, 0xfc28}, > + {0x32cc, 0xc3bc}, > + {0x32ce, 0xc34c}, > + {0x32d0, 0xc35a}, > + {0x32d2, 0xc368}, > + {0x32d4, 0xc376}, > + {0x32d6, 0xc3c2}, > + {0x32d8, 0xc3e6}, > + {0x32da, 0x0003}, > + {0x32dc, 0x0003}, > + {0x32de, 0x00c7}, > + {0x32e0, 0x0031}, > + {0x32e2, 0x0031}, > + {0x32e4, 0x0031}, > + {0x32e6, 0xfc28}, > + {0x32e8, 0xc3bc}, > + {0x32ea, 0xc384}, > + {0x32ec, 0xc392}, > + {0x32ee, 0xc3a0}, > + {0x32f0, 0xc3ae}, > + {0x32f2, 0xc3c4}, > + {0x32f4, 0xc3e6}, > + {0x32f6, 0x0003}, > + {0x32f8, 0x0003}, > + {0x32fa, 0x00c7}, > + {0x32fc, 0x0031}, > + {0x32fe, 0x0031}, > + {0x3300, 0x0031}, > + {0x3302, 0x82ca}, > + {0x3304, 0xc164}, > + {0x3306, 0x82e6}, > + {0x3308, 0xc19c}, > + {0x330a, 0x001f}, > + {0x330c, 0x001a}, > + {0x330e, 0x0034}, > + {0x3310, 0x0000}, > + {0x3312, 0x0000}, > + {0x3314, 0xfc94}, > + {0x3316, 0xc3d8}, > + /* regs above are unknown */ > + {HI846_REG_MODE_SELECT, 0x0000}, > + {HI846_REG_UNKNOWN_0E04, 0x0012}, > + {HI846_REG_Y_ODD_INC_FOBP, 0x1111}, > + {HI846_REG_Y_ODD_INC_VACT, 0x1111}, > + {HI846_REG_UNKNOWN_0022, 0x0008}, > + {HI846_REG_Y_ADDR_START_VACT_H, 0x0040}, > + {HI846_REG_UNKNOWN_0028, 0x0017}, > + {HI846_REG_Y_ADDR_END_VACT_H, 0x09cf}, > + {HI846_REG_UNKNOWN_005C, 0x2101}, > + {HI846_REG_FLL, 0x09de}, > + {HI846_REG_LLP, 0x0ed8}, > + {HI846_REG_IMAGE_ORIENTATION, 0x0100}, > + {HI846_REG_BINNING_MODE, 0x0022}, > + {HI846_REG_HBIN_MODE, 0x0000}, > + {HI846_REG_UNKNOWN_0A24, 0x0000}, > + {HI846_REG_X_START_H, 0x0000}, > + {HI846_REG_X_OUTPUT_SIZE_H, 0x0cc0}, > + {HI846_REG_Y_OUTPUT_SIZE_H, 0x0990}, > + {HI846_REG_EXPOSURE, 0x09d8}, > + {HI846_REG_ANALOG_GAIN, 0x0000}, > + {HI846_REG_GROUPED_PARA_HOLD, 0x0000}, > + {HI846_REG_UNKNOWN_051E, 0x0000}, > + {HI846_REG_UNKNOWN_0200, 0x0400}, > + {HI846_REG_PEDESTAL_EN, 0x0c00}, > + {HI846_REG_UNKNOWN_0A0C, 0x0010}, > + {HI846_REG_UNKNOWN_0A1E, 0x0ccf}, > + {HI846_REG_UNKNOWN_0402, 0x0110}, > + {HI846_REG_UNKNOWN_0404, 0x00f4}, > + {HI846_REG_UNKNOWN_0408, 0x0000}, > + {HI846_REG_UNKNOWN_0410, 0x008d}, > + {HI846_REG_UNKNOWN_0412, 0x011a}, > + {HI846_REG_UNKNOWN_0414, 0x864c}, > + {HI846_REG_UNKNOWN_021C, 0x0003}, > + {HI846_REG_UNKNOWN_021E, 0x0235}, > + {HI846_REG_BLC_CTL0, 0x9150}, > + {HI846_REG_UNKNOWN_0C06, 0x0021}, > + {HI846_REG_UNKNOWN_0C10, 0x0040}, > + {HI846_REG_UNKNOWN_0C12, 0x0040}, > + {HI846_REG_UNKNOWN_0C14, 0x0040}, > + {HI846_REG_UNKNOWN_0C16, 0x0040}, > + {HI846_REG_FAST_STANDBY_MODE, 0x0100}, > + {HI846_REG_ISP_EN_H, 0x014a}, > + {HI846_REG_UNKNOWN_0418, 0x0000}, > + {HI846_REG_UNKNOWN_012A, 0x03b4}, > + {HI846_REG_X_ADDR_START_HACT_H, 0x0046}, > + {HI846_REG_X_ADDR_END_HACT_H, 0x0376}, > + {HI846_REG_UNKNOWN_0B02, 0xe04d}, > + {HI846_REG_UNKNOWN_0B10, 0x6821}, > + {HI846_REG_UNKNOWN_0B12, 0x0120}, > + {HI846_REG_UNKNOWN_0B14, 0x0001}, > + {HI846_REG_UNKNOWN_2008, 0x38fd}, > + {HI846_REG_UNKNOWN_326E, 0x0000}, > + {HI846_REG_UNKNOWN_0900, 0x0320}, > + {HI846_REG_MIPI_TX_OP_MODE, 0xc31a}, > + {HI846_REG_UNKNOWN_0914, 0xc109}, > + {HI846_REG_TCLK_PREPARE, 0x061a}, > + {HI846_REG_UNKNOWN_0918, 0x0306}, > + {HI846_REG_THS_ZERO, 0x0b09}, > + {HI846_REG_TCLK_POST, 0x0c07}, > + {HI846_REG_UNKNOWN_091E, 0x0a00}, > + {HI846_REG_UNKNOWN_090C, 0x042a}, > + {HI846_REG_UNKNOWN_090E, 0x006b}, > + {HI846_REG_UNKNOWN_0954, 0x0089}, > + {HI846_REG_UNKNOWN_0956, 0x0000}, > + {HI846_REG_UNKNOWN_0958, 0xca00}, > + {HI846_REG_UNKNOWN_095A, 0x9240}, > + {HI846_REG_UNKNOWN_0F08, 0x2f04}, > + {HI846_REG_UNKNOWN_0F30, 0x001f}, > + {HI846_REG_UNKNOWN_0F36, 0x001f}, > + {HI846_REG_UNKNOWN_0F04, 0x3a00}, > + {HI846_REG_PLL_CFG_RAMP1_H, 0x025a}, > + {HI846_REG_PLL_CFG_MIPI1_H, 0x025a}, > + {HI846_REG_PLL_CFG_MIPI2_H, 0x0024}, > + {HI846_REG_UNKNOWN_006A, 0x0100}, > + {HI846_REG_TG_ENABLE, 0x0100}, > +}; > + > +static const struct hi846_reg hi846_init_4lane[] = { > + {0x2000, 0x987a}, > + {0x2002, 0x00ff}, > + {0x2004, 0x0047}, > + {0x2006, 0x3fff}, > + {0x2008, 0x3fff}, > + {0x200a, 0xc216}, > + {0x200c, 0x1292}, > + {0x200e, 0xc01a}, > + {0x2010, 0x403d}, > + {0x2012, 0x000e}, > + {0x2014, 0x403e}, > + {0x2016, 0x0b80}, > + {0x2018, 0x403f}, > + {0x201a, 0x82ae}, > + {0x201c, 0x1292}, > + {0x201e, 0xc00c}, > + {0x2020, 0x4130}, > + {0x2022, 0x43e2}, > + {0x2024, 0x0180}, > + {0x2026, 0x4130}, > + {0x2028, 0x7400}, > + {0x202a, 0x5000}, > + {0x202c, 0x0253}, > + {0x202e, 0x0ad1}, > + {0x2030, 0x2360}, > + {0x2032, 0x0009}, > + {0x2034, 0x5020}, > + {0x2036, 0x000b}, > + {0x2038, 0x0002}, > + {0x203a, 0x0044}, > + {0x203c, 0x0016}, > + {0x203e, 0x1792}, > + {0x2040, 0x7002}, > + {0x2042, 0x154f}, > + {0x2044, 0x00d5}, > + {0x2046, 0x000b}, > + {0x2048, 0x0019}, > + {0x204a, 0x1698}, > + {0x204c, 0x000e}, > + {0x204e, 0x099a}, > + {0x2050, 0x0058}, > + {0x2052, 0x7000}, > + {0x2054, 0x1799}, > + {0x2056, 0x0310}, > + {0x2058, 0x03c3}, > + {0x205a, 0x004c}, > + {0x205c, 0x064a}, > + {0x205e, 0x0001}, > + {0x2060, 0x0007}, > + {0x2062, 0x0bc7}, > + {0x2064, 0x0055}, > + {0x2066, 0x7000}, > + {0x2068, 0x1550}, > + {0x206a, 0x158a}, > + {0x206c, 0x0004}, > + {0x206e, 0x1488}, > + {0x2070, 0x7010}, > + {0x2072, 0x1508}, > + {0x2074, 0x0004}, > + {0x2076, 0x0016}, > + {0x2078, 0x03d5}, > + {0x207a, 0x0055}, > + {0x207c, 0x08ca}, > + {0x207e, 0x2019}, > + {0x2080, 0x0007}, > + {0x2082, 0x7057}, > + {0x2084, 0x0fc7}, > + {0x2086, 0x5041}, > + {0x2088, 0x12c8}, > + {0x208a, 0x5060}, > + {0x208c, 0x5080}, > + {0x208e, 0x2084}, > + {0x2090, 0x12c8}, > + {0x2092, 0x7800}, > + {0x2094, 0x0802}, > + {0x2096, 0x040f}, > + {0x2098, 0x1007}, > + {0x209a, 0x0803}, > + {0x209c, 0x080b}, > + {0x209e, 0x3803}, > + {0x20a0, 0x0807}, > + {0x20a2, 0x0404}, > + {0x20a4, 0x0400}, > + {0x20a6, 0xffff}, > + {0x20a8, 0xf0b2}, > + {0x20aa, 0xffef}, > + {0x20ac, 0x0a84}, > + {0x20ae, 0x1292}, > + {0x20b0, 0xc02e}, > + {0x20b2, 0x4130}, > + {0x20b4, 0xf0b2}, > + {0x20b6, 0xffbf}, > + {0x20b8, 0x2004}, > + {0x20ba, 0x403f}, > + {0x20bc, 0x00c3}, > + {0x20be, 0x4fe2}, > + {0x20c0, 0x8318}, > + {0x20c2, 0x43cf}, > + {0x20c4, 0x0000}, > + {0x20c6, 0x9382}, > + {0x20c8, 0xc314}, > + {0x20ca, 0x2003}, > + {0x20cc, 0x12b0}, > + {0x20ce, 0xcab0}, > + {0x20d0, 0x4130}, > + {0x20d2, 0x12b0}, > + {0x20d4, 0xc90a}, > + {0x20d6, 0x4130}, > + {0x20d8, 0x42d2}, > + {0x20da, 0x8318}, > + {0x20dc, 0x00c3}, > + {0x20de, 0x9382}, > + {0x20e0, 0xc314}, > + {0x20e2, 0x2009}, > + {0x20e4, 0x120b}, > + {0x20e6, 0x120a}, > + {0x20e8, 0x1209}, > + {0x20ea, 0x1208}, > + {0x20ec, 0x1207}, > + {0x20ee, 0x1206}, > + {0x20f0, 0x4030}, > + {0x20f2, 0xc15e}, > + {0x20f4, 0x4130}, > + {0x20f6, 0x1292}, > + {0x20f8, 0xc008}, > + {0x20fa, 0x4130}, > + {0x20fc, 0x42d2}, > + {0x20fe, 0x82a1}, > + {0x2100, 0x00c2}, > + {0x2102, 0x1292}, > + {0x2104, 0xc040}, > + {0x2106, 0x4130}, > + {0x2108, 0x1292}, > + {0x210a, 0xc006}, > + {0x210c, 0x42a2}, > + {0x210e, 0x7324}, > + {0x2110, 0x9382}, > + {0x2112, 0xc314}, > + {0x2114, 0x2011}, > + {0x2116, 0x425f}, > + {0x2118, 0x82a1}, > + {0x211a, 0xf25f}, > + {0x211c, 0x00c1}, > + {0x211e, 0xf35f}, > + {0x2120, 0x2406}, > + {0x2122, 0x425f}, > + {0x2124, 0x00c0}, > + {0x2126, 0xf37f}, > + {0x2128, 0x522f}, > + {0x212a, 0x4f82}, > + {0x212c, 0x7324}, > + {0x212e, 0x425f}, > + {0x2130, 0x82d4}, > + {0x2132, 0xf35f}, > + {0x2134, 0x4fc2}, > + {0x2136, 0x01b3}, > + {0x2138, 0x93c2}, > + {0x213a, 0x829f}, > + {0x213c, 0x2421}, > + {0x213e, 0x403e}, > + {0x2140, 0xfffe}, > + {0x2142, 0x40b2}, > + {0x2144, 0xec78}, > + {0x2146, 0x831c}, > + {0x2148, 0x40b2}, > + {0x214a, 0xec78}, > + {0x214c, 0x831e}, > + {0x214e, 0x40b2}, > + {0x2150, 0xec78}, > + {0x2152, 0x8320}, > + {0x2154, 0xb3d2}, > + {0x2156, 0x008c}, > + {0x2158, 0x2405}, > + {0x215a, 0x4e0f}, > + {0x215c, 0x503f}, > + {0x215e, 0xffd8}, > + {0x2160, 0x4f82}, > + {0x2162, 0x831c}, > + {0x2164, 0x90f2}, > + {0x2166, 0x0003}, > + {0x2168, 0x008c}, > + {0x216a, 0x2401}, > + {0x216c, 0x4130}, > + {0x216e, 0x421f}, > + {0x2170, 0x831c}, > + {0x2172, 0x5e0f}, > + {0x2174, 0x4f82}, > + {0x2176, 0x831e}, > + {0x2178, 0x5e0f}, > + {0x217a, 0x4f82}, > + {0x217c, 0x8320}, > + {0x217e, 0x3ff6}, > + {0x2180, 0x432e}, > + {0x2182, 0x3fdf}, > + {0x2184, 0x421f}, > + {0x2186, 0x7100}, > + {0x2188, 0x4f0e}, > + {0x218a, 0x503e}, > + {0x218c, 0xffd8}, > + {0x218e, 0x4e82}, > + {0x2190, 0x7a04}, > + {0x2192, 0x421e}, > + {0x2194, 0x831c}, > + {0x2196, 0x5f0e}, > + {0x2198, 0x4e82}, > + {0x219a, 0x7a06}, > + {0x219c, 0x0b00}, > + {0x219e, 0x7304}, > + {0x21a0, 0x0050}, > + {0x21a2, 0x40b2}, > + {0x21a4, 0xd081}, > + {0x21a6, 0x0b88}, > + {0x21a8, 0x421e}, > + {0x21aa, 0x831e}, > + {0x21ac, 0x5f0e}, > + {0x21ae, 0x4e82}, > + {0x21b0, 0x7a0e}, > + {0x21b2, 0x521f}, > + {0x21b4, 0x8320}, > + {0x21b6, 0x4f82}, > + {0x21b8, 0x7a10}, > + {0x21ba, 0x0b00}, > + {0x21bc, 0x7304}, > + {0x21be, 0x007a}, > + {0x21c0, 0x40b2}, > + {0x21c2, 0x0081}, > + {0x21c4, 0x0b88}, > + {0x21c6, 0x4392}, > + {0x21c8, 0x7a0a}, > + {0x21ca, 0x0800}, > + {0x21cc, 0x7a0c}, > + {0x21ce, 0x0b00}, > + {0x21d0, 0x7304}, > + {0x21d2, 0x022b}, > + {0x21d4, 0x40b2}, > + {0x21d6, 0xd081}, > + {0x21d8, 0x0b88}, > + {0x21da, 0x0b00}, > + {0x21dc, 0x7304}, > + {0x21de, 0x0255}, > + {0x21e0, 0x40b2}, > + {0x21e2, 0x0081}, > + {0x21e4, 0x0b88}, > + {0x21e6, 0x4130}, > + {0x23fe, 0xc056}, > + {0x3232, 0xfc0c}, > + {0x3236, 0xfc22}, > + {0x3238, 0xfcfc}, > + {0x323a, 0xfd84}, > + {0x323c, 0xfd08}, > + {0x3246, 0xfcd8}, > + {0x3248, 0xfca8}, > + {0x324e, 0xfcb4}, > + {0x326a, 0x8302}, > + {0x326c, 0x830a}, > + {0x326e, 0x0000}, > + {0x32ca, 0xfc28}, > + {0x32cc, 0xc3bc}, > + {0x32ce, 0xc34c}, > + {0x32d0, 0xc35a}, > + {0x32d2, 0xc368}, > + {0x32d4, 0xc376}, > + {0x32d6, 0xc3c2}, > + {0x32d8, 0xc3e6}, > + {0x32da, 0x0003}, > + {0x32dc, 0x0003}, > + {0x32de, 0x00c7}, > + {0x32e0, 0x0031}, > + {0x32e2, 0x0031}, > + {0x32e4, 0x0031}, > + {0x32e6, 0xfc28}, > + {0x32e8, 0xc3bc}, > + {0x32ea, 0xc384}, > + {0x32ec, 0xc392}, > + {0x32ee, 0xc3a0}, > + {0x32f0, 0xc3ae}, > + {0x32f2, 0xc3c4}, > + {0x32f4, 0xc3e6}, > + {0x32f6, 0x0003}, > + {0x32f8, 0x0003}, > + {0x32fa, 0x00c7}, > + {0x32fc, 0x0031}, > + {0x32fe, 0x0031}, > + {0x3300, 0x0031}, > + {0x3302, 0x82ca}, > + {0x3304, 0xc164}, > + {0x3306, 0x82e6}, > + {0x3308, 0xc19c}, > + {0x330a, 0x001f}, > + {0x330c, 0x001a}, > + {0x330e, 0x0034}, > + {0x3310, 0x0000}, > + {0x3312, 0x0000}, > + {0x3314, 0xfc94}, > + {0x3316, 0xc3d8}, > + > + {0x0a00, 0x0000}, > + {0x0e04, 0x0012}, > + {0x002e, 0x1111}, > + {0x0032, 0x1111}, > + {0x0022, 0x0008}, > + {0x0026, 0x0040}, > + {0x0028, 0x0017}, > + {0x002c, 0x09cf}, > + {0x005c, 0x2101}, > + {0x0006, 0x09de}, > + {0x0008, 0x0ed8}, > + {0x000e, 0x0100}, > + {0x000c, 0x0022}, > + {0x0a22, 0x0000}, > + {0x0a24, 0x0000}, > + {0x0804, 0x0000}, > + {0x0a12, 0x0cc0}, > + {0x0a14, 0x0990}, > + {0x0074, 0x09d8}, > + {0x0076, 0x0000}, > + {0x051e, 0x0000}, > + {0x0200, 0x0400}, > + {0x0a1a, 0x0c00}, > + {0x0a0c, 0x0010}, > + {0x0a1e, 0x0ccf}, > + {0x0402, 0x0110}, > + {0x0404, 0x00f4}, > + {0x0408, 0x0000}, > + {0x0410, 0x008d}, > + {0x0412, 0x011a}, > + {0x0414, 0x864c}, > + /* for OTP */ > + {0x021c, 0x0003}, > + {0x021e, 0x0235}, > + /* for OTP */ > + {0x0c00, 0x9950}, > + {0x0c06, 0x0021}, > + {0x0c10, 0x0040}, > + {0x0c12, 0x0040}, > + {0x0c14, 0x0040}, > + {0x0c16, 0x0040}, > + {0x0a02, 0x0100}, > + {0x0a04, 0x015a}, > + {0x0418, 0x0000}, > + {0x0128, 0x0028}, > + {0x012a, 0xffff}, > + {0x0120, 0x0046}, > + {0x0122, 0x0376}, > + {0x012c, 0x0020}, > + {0x012e, 0xffff}, > + {0x0124, 0x0040}, > + {0x0126, 0x0378}, > + {0x0746, 0x0050}, > + {0x0748, 0x01d5}, > + {0x074a, 0x022b}, > + {0x074c, 0x03b0}, > + {0x0756, 0x043f}, > + {0x0758, 0x3f1d}, > + {0x0b02, 0xe04d}, > + {0x0b10, 0x6821}, > + {0x0b12, 0x0120}, > + {0x0b14, 0x0001}, > + {0x2008, 0x38fd}, > + {0x326e, 0x0000}, > + {0x0900, 0x0300}, > + {0x0902, 0xc319}, > + {0x0914, 0xc109}, > + {0x0916, 0x061a}, > + {0x0918, 0x0407}, > + {0x091a, 0x0a0b}, > + {0x091c, 0x0e08}, > + {0x091e, 0x0a00}, > + {0x090c, 0x0427}, > + {0x090e, 0x0059}, > + {0x0954, 0x0089}, > + {0x0956, 0x0000}, > + {0x0958, 0xca80}, > + {0x095a, 0x9240}, > + {0x0f08, 0x2f04}, > + {0x0f30, 0x001f}, > + {0x0f36, 0x001f}, > + {0x0f04, 0x3a00}, > + {0x0f32, 0x025a}, > + {0x0f38, 0x025a}, > + {0x0f2a, 0x4124}, > + {0x006a, 0x0100}, > + {0x004c, 0x0100}, > + {0x0044, 0x0001}, > +}; > + > +static const struct hi846_reg mode_640x480_config[] = { > + {HI846_REG_MODE_SELECT, 0x0000}, > + {HI846_REG_Y_ODD_INC_FOBP, 0x7711}, > + {HI846_REG_Y_ODD_INC_VACT, 0x7711}, > + {HI846_REG_Y_ADDR_START_VACT_H, 0x0148}, > + {HI846_REG_Y_ADDR_END_VACT_H, 0x08c7}, > + {HI846_REG_UNKNOWN_005C, 0x4404}, > + {HI846_REG_FLL, 0x0277}, > + {HI846_REG_LLP, 0x0ed8}, > + {HI846_REG_BINNING_MODE, 0x0322}, > + {HI846_REG_HBIN_MODE, 0x0200}, > + {HI846_REG_UNKNOWN_0A24, 0x0000}, > + {HI846_REG_X_START_H, 0x0058}, > + {HI846_REG_X_OUTPUT_SIZE_H, 0x0280}, > + {HI846_REG_Y_OUTPUT_SIZE_H, 0x01e0}, > + > + /* For OTP */ > + {HI846_REG_UNKNOWN_021C, 0x0003}, > + {HI846_REG_UNKNOWN_021E, 0x0235}, > + > + {HI846_REG_ISP_EN_H, 0x016a}, > + {HI846_REG_UNKNOWN_0418, 0x0210}, > + {HI846_REG_UNKNOWN_0B02, 0xe04d}, > + {HI846_REG_UNKNOWN_0B10, 0x7021}, > + {HI846_REG_UNKNOWN_0B12, 0x0120}, > + {HI846_REG_UNKNOWN_0B14, 0x0001}, > + {HI846_REG_UNKNOWN_2008, 0x38fd}, > + {HI846_REG_UNKNOWN_326E, 0x0000}, > +}; > + > +static const struct hi846_reg mode_640x480_mipi_2lane[] = { > + {HI846_REG_UNKNOWN_0900, 0x0300}, > + {HI846_REG_MIPI_TX_OP_MODE, 0x4319}, > + {HI846_REG_UNKNOWN_0914, 0xc105}, > + {HI846_REG_TCLK_PREPARE, 0x030c}, > + {HI846_REG_UNKNOWN_0918, 0x0304}, > + {HI846_REG_THS_ZERO, 0x0708}, > + {HI846_REG_TCLK_POST, 0x0b04}, > + {HI846_REG_UNKNOWN_091E, 0x0500}, > + {HI846_REG_UNKNOWN_090C, 0x0208}, > + {HI846_REG_UNKNOWN_090E, 0x009a}, > + {HI846_REG_UNKNOWN_0954, 0x0089}, > + {HI846_REG_UNKNOWN_0956, 0x0000}, > + {HI846_REG_UNKNOWN_0958, 0xca80}, > + {HI846_REG_UNKNOWN_095A, 0x9240}, > + {HI846_REG_PLL_CFG_MIPI2_H, 0x4924}, > + {HI846_REG_TG_ENABLE, 0x0100}, > +}; > + > +static const struct hi846_reg mode_1280x720_config[] = { > + {HI846_REG_MODE_SELECT, 0x0000}, > + {HI846_REG_Y_ODD_INC_FOBP, 0x3311}, > + {HI846_REG_Y_ODD_INC_VACT, 0x3311}, > + {HI846_REG_Y_ADDR_START_VACT_H, 0x0238}, > + {HI846_REG_Y_ADDR_END_VACT_H, 0x07d7}, > + {HI846_REG_UNKNOWN_005C, 0x4202}, > + {HI846_REG_FLL, 0x034a}, > + {HI846_REG_LLP, 0x0ed8}, > + {HI846_REG_BINNING_MODE, 0x0122}, > + {HI846_REG_HBIN_MODE, 0x0100}, > + {HI846_REG_UNKNOWN_0A24, 0x0000}, > + {HI846_REG_X_START_H, 0x00b0}, > + {HI846_REG_X_OUTPUT_SIZE_H, 0x0500}, > + {HI846_REG_Y_OUTPUT_SIZE_H, 0x02d0}, > + {HI846_REG_EXPOSURE, 0x0344}, > + > + /* For OTP */ > + {HI846_REG_UNKNOWN_021C, 0x0003}, > + {HI846_REG_UNKNOWN_021E, 0x0235}, > + > + {HI846_REG_ISP_EN_H, 0x016a}, > + {HI846_REG_UNKNOWN_0418, 0x0410}, > + {HI846_REG_UNKNOWN_0B02, 0xe04d}, > + {HI846_REG_UNKNOWN_0B10, 0x6c21}, > + {HI846_REG_UNKNOWN_0B12, 0x0120}, > + {HI846_REG_UNKNOWN_0B14, 0x0005}, > + {HI846_REG_UNKNOWN_2008, 0x38fd}, > + {HI846_REG_UNKNOWN_326E, 0x0000}, > +}; > + > +static const struct hi846_reg mode_1280x720_mipi_2lane[] = { > + {HI846_REG_UNKNOWN_0900, 0x0300}, > + {HI846_REG_MIPI_TX_OP_MODE, 0x4319}, > + {HI846_REG_UNKNOWN_0914, 0xc109}, > + {HI846_REG_TCLK_PREPARE, 0x061a}, > + {HI846_REG_UNKNOWN_0918, 0x0407}, > + {HI846_REG_THS_ZERO, 0x0a0b}, > + {HI846_REG_TCLK_POST, 0x0e08}, > + {HI846_REG_UNKNOWN_091E, 0x0a00}, > + {HI846_REG_UNKNOWN_090C, 0x0427}, > + {HI846_REG_UNKNOWN_090E, 0x0145}, > + {HI846_REG_UNKNOWN_0954, 0x0089}, > + {HI846_REG_UNKNOWN_0956, 0x0000}, > + {HI846_REG_UNKNOWN_0958, 0xca80}, > + {HI846_REG_UNKNOWN_095A, 0x9240}, > + {HI846_REG_PLL_CFG_MIPI2_H, 0x4124}, > + {HI846_REG_TG_ENABLE, 0x0100}, > +}; > + > +static const struct hi846_reg mode_1280x720_mipi_4lane[] = { > + /* 360Mbps */ > + {HI846_REG_UNKNOWN_0900, 0x0300}, > + {HI846_REG_MIPI_TX_OP_MODE, 0xc319}, > + {HI846_REG_UNKNOWN_0914, 0xc105}, > + {HI846_REG_TCLK_PREPARE, 0x030c}, > + {HI846_REG_UNKNOWN_0918, 0x0304}, > + {HI846_REG_THS_ZERO, 0x0708}, > + {HI846_REG_TCLK_POST, 0x0b04}, > + {HI846_REG_UNKNOWN_091E, 0x0500}, > + {HI846_REG_UNKNOWN_090C, 0x0208}, > + {HI846_REG_UNKNOWN_090E, 0x008a}, > + {HI846_REG_UNKNOWN_0954, 0x0089}, > + {HI846_REG_UNKNOWN_0956, 0x0000}, > + {HI846_REG_UNKNOWN_0958, 0xca80}, > + {HI846_REG_UNKNOWN_095A, 0x9240}, > + {HI846_REG_PLL_CFG_MIPI2_H, 0x4924}, > + {HI846_REG_TG_ENABLE, 0x0100}, > +}; > + > +static const struct hi846_reg mode_1632x1224_config[] = { > + {HI846_REG_MODE_SELECT, 0x0000}, > + {HI846_REG_Y_ODD_INC_FOBP, 0x3311}, > + {HI846_REG_Y_ODD_INC_VACT, 0x3311}, > + {HI846_REG_Y_ADDR_START_VACT_H, 0x0040}, > + {HI846_REG_Y_ADDR_END_VACT_H, 0x09cf}, > + {HI846_REG_UNKNOWN_005C, 0x4202}, > + {HI846_REG_FLL, 0x09de}, > + {HI846_REG_LLP, 0x0ed8}, > + {HI846_REG_BINNING_MODE, 0x0122}, > + {HI846_REG_HBIN_MODE, 0x0100}, > + {HI846_REG_UNKNOWN_0A24, 0x0000}, > + {HI846_REG_X_START_H, 0x0000}, > + {HI846_REG_X_OUTPUT_SIZE_H, 0x0660}, > + {HI846_REG_Y_OUTPUT_SIZE_H, 0x04c8}, > + {HI846_REG_EXPOSURE, 0x09d8}, > + > + /* For OTP */ > + {HI846_REG_UNKNOWN_021C, 0x0003}, > + {HI846_REG_UNKNOWN_021E, 0x0235}, > + > + {HI846_REG_ISP_EN_H, 0x016a}, > + {HI846_REG_UNKNOWN_0418, 0x0000}, > + {HI846_REG_UNKNOWN_0B02, 0xe04d}, > + {HI846_REG_UNKNOWN_0B10, 0x6c21}, > + {HI846_REG_UNKNOWN_0B12, 0x0120}, > + {HI846_REG_UNKNOWN_0B14, 0x0005}, > + {HI846_REG_UNKNOWN_2008, 0x38fd}, > + {HI846_REG_UNKNOWN_326E, 0x0000}, > +}; > + > +static const struct hi846_reg mode_1632x1224_mipi_2lane[] = { > + {HI846_REG_UNKNOWN_0900, 0x0300}, > + {HI846_REG_MIPI_TX_OP_MODE, 0x4319}, > + {HI846_REG_UNKNOWN_0914, 0xc109}, > + {HI846_REG_TCLK_PREPARE, 0x061a}, > + {HI846_REG_UNKNOWN_0918, 0x0407}, > + {HI846_REG_THS_ZERO, 0x0a0b}, > + {HI846_REG_TCLK_POST, 0x0e08}, > + {HI846_REG_UNKNOWN_091E, 0x0a00}, > + {HI846_REG_UNKNOWN_090C, 0x0427}, > + {HI846_REG_UNKNOWN_090E, 0x0069}, > + {HI846_REG_UNKNOWN_0954, 0x0089}, > + {HI846_REG_UNKNOWN_0956, 0x0000}, > + {HI846_REG_UNKNOWN_0958, 0xca80}, > + {HI846_REG_UNKNOWN_095A, 0x9240}, > + {HI846_REG_PLL_CFG_MIPI2_H, 0x4124}, > + {HI846_REG_TG_ENABLE, 0x0100}, > +}; > + > +static const struct hi846_reg mode_1632x1224_mipi_4lane[] = { > + {HI846_REG_UNKNOWN_0900, 0x0300}, > + {HI846_REG_MIPI_TX_OP_MODE, 0xc319}, > + {HI846_REG_UNKNOWN_0914, 0xc105}, > + {HI846_REG_TCLK_PREPARE, 0x030c}, > + {HI846_REG_UNKNOWN_0918, 0x0304}, > + {HI846_REG_THS_ZERO, 0x0708}, > + {HI846_REG_TCLK_POST, 0x0b04}, > + {HI846_REG_UNKNOWN_091E, 0x0500}, > + {HI846_REG_UNKNOWN_090C, 0x0208}, > + {HI846_REG_UNKNOWN_090E, 0x001c}, > + {HI846_REG_UNKNOWN_0954, 0x0089}, > + {HI846_REG_UNKNOWN_0956, 0x0000}, > + {HI846_REG_UNKNOWN_0958, 0xca80}, > + {HI846_REG_UNKNOWN_095A, 0x9240}, > + {HI846_REG_PLL_CFG_MIPI2_H, 0x4924}, > + {HI846_REG_TG_ENABLE, 0x0100}, > +}; > + > +static const char * const hi846_test_pattern_menu[] = { > + "Disabled", > + "Solid Colour", > + "100% Colour Bars", > + "Fade To Grey Colour Bars", > + "PN9", > + "Gradient Horizontal", > + "Gradient Vertical", > + "Check Board", > + "Slant Pattern", > + "Resolution Pattern", > +}; > + > +#define FREQ_INDEX_640 0 > +#define FREQ_INDEX_1280 1 > +static const s64 hi846_link_freqs[] = { > + [FREQ_INDEX_640] = 80000000, > + [FREQ_INDEX_1280] = 200000000, > +}; > + > +static const struct hi846_reg_list hi846_init_regs_list_2lane = { > + .num_of_regs = ARRAY_SIZE(hi846_init_2lane), > + .regs = hi846_init_2lane, > +}; > + > +static const struct hi846_reg_list hi846_init_regs_list_4lane = { > + .num_of_regs = ARRAY_SIZE(hi846_init_4lane), > + .regs = hi846_init_4lane, > +}; > + > +static const struct hi846_mode supported_modes[] = { > + { > + .width = 640, > + .height = 480, > + .link_freq_index = FREQ_INDEX_640, > + .fps = 120, > + .frame_len = 631, > + .llp = HI846_LINE_LENGTH, > + .reg_list_config = { > + .num_of_regs = ARRAY_SIZE(mode_640x480_config), > + .regs = mode_640x480_config, > + }, > + .reg_list_2lane = { > + .num_of_regs = ARRAY_SIZE(mode_640x480_mipi_2lane), > + .regs = mode_640x480_mipi_2lane, > + }, > + .reg_list_4lane = { > + .num_of_regs = 0, > + }, > + .crop = { > + .left = 0x58, > + .top = 0x148, > + .width = 640 * 4, > + .height = 480 * 4, > + }, > + }, > + { > + .width = 1280, > + .height = 720, > + .link_freq_index = FREQ_INDEX_1280, > + .fps = 90, > + .frame_len = 842, > + .llp = HI846_LINE_LENGTH, > + .reg_list_config = { > + .num_of_regs = ARRAY_SIZE(mode_1280x720_config), > + .regs = mode_1280x720_config, > + }, > + .reg_list_2lane = { > + .num_of_regs = ARRAY_SIZE(mode_1280x720_mipi_2lane), > + .regs = mode_1280x720_mipi_2lane, > + }, > + .reg_list_4lane = { > + .num_of_regs = ARRAY_SIZE(mode_1280x720_mipi_4lane), > + .regs = mode_1280x720_mipi_4lane, > + }, > + .crop = { > + .left = 0xb0, > + .top = 0x238, > + .width = 1280 * 2, > + .height = 720 * 2, > + }, > + }, > + { > + .width = 1632, > + .height = 1224, > + .link_freq_index = FREQ_INDEX_1280, > + .fps = 30, > + .frame_len = 2526, > + .llp = HI846_LINE_LENGTH, > + .reg_list_config = { > + .num_of_regs = ARRAY_SIZE(mode_1632x1224_config), > + .regs = mode_1632x1224_config, > + }, > + .reg_list_2lane = { > + .num_of_regs = ARRAY_SIZE(mode_1632x1224_mipi_2lane), > + .regs = mode_1632x1224_mipi_2lane, > + }, > + .reg_list_4lane = { > + .num_of_regs = ARRAY_SIZE(mode_1632x1224_mipi_4lane), > + .regs = mode_1632x1224_mipi_4lane, > + }, > + .crop = { > + .left = 0x0, > + .top = 0x0, > + .width = 1632 * 2, > + .height = 1224 * 2, > + }, > + } > +}; > + > +struct hi846_datafmt { > + u32 code; > + enum v4l2_colorspace colorspace; > +}; > + > +static const char * const hi846_supply_names[] = { > + "vddio", /* Digital I/O (1.8V or 2.8V) */ > + "vdda", /* Analog (2.8V) */ > + "vddd", /* Digital Core (1.2V) */ > +}; > + > +#define HI846_NUM_SUPPLIES ARRAY_SIZE(hi846_supply_names) > + > +struct hi846 { > + struct gpio_desc *rst_gpio; > + struct gpio_desc *shutdown_gpio; > + struct regulator_bulk_data supplies[HI846_NUM_SUPPLIES]; > + struct clk *clock; > + const struct hi846_datafmt *fmt; > + struct v4l2_subdev sd; > + struct media_pad pad; > + struct v4l2_ctrl_handler ctrl_handler; > + u8 nr_lanes; > + > + struct v4l2_ctrl *link_freq; > + struct v4l2_ctrl *pixel_rate; > + struct v4l2_ctrl *vblank; > + struct v4l2_ctrl *hblank; > + struct v4l2_ctrl *exposure; > + > + struct mutex mutex; /* protect cur_mode, streaming and chip access */ > + const struct hi846_mode *cur_mode; > + bool streaming; > +}; > + > +static inline struct hi846 *to_hi846(struct v4l2_subdev *sd) > +{ > + return container_of(sd, struct hi846, sd); > +} > + > +static const struct hi846_datafmt hi846_colour_fmts[] = { > + { HI846_MEDIA_BUS_FORMAT, V4L2_COLORSPACE_RAW }, > +}; > + > +static const struct hi846_datafmt *hi846_find_datafmt(u32 code) > +{ > + unsigned int i; > + > + for (i = 0; i < ARRAY_SIZE(hi846_colour_fmts); i++) > + if (hi846_colour_fmts[i].code == code) > + return &hi846_colour_fmts[i]; > + > + return NULL; > +} > + > +static inline u8 hi846_get_link_freq_index(struct hi846 *hi846) > +{ > + return hi846->cur_mode->link_freq_index; > +} > + > +static u64 hi846_get_link_freq(struct hi846 *hi846) > +{ > + u8 index = hi846_get_link_freq_index(hi846); > + > + return hi846_link_freqs[index]; > +} > + > +static u64 hi846_calc_pixel_rate(struct hi846 *hi846) > +{ > + u64 link_freq = hi846_get_link_freq(hi846); > + u64 pixel_rate = link_freq * 2 * hi846->nr_lanes; > + > + do_div(pixel_rate, HI846_RGB_DEPTH); > + > + return pixel_rate; > +} > + > +static int hi846_read_reg(struct hi846 *hi846, u16 reg, u8 *val) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + struct i2c_msg msgs[2]; > + u8 addr_buf[2]; > + u8 data_buf[1] = {0}; > + int ret; > + > + put_unaligned_be16(reg, addr_buf); > + msgs[0].addr = client->addr; > + msgs[0].flags = 0; > + msgs[0].len = sizeof(addr_buf); > + msgs[0].buf = addr_buf; > + msgs[1].addr = client->addr; > + msgs[1].flags = I2C_M_RD; > + msgs[1].len = 1; > + msgs[1].buf = &data_buf[0]; > + > + ret = i2c_transfer(client->adapter, msgs, ARRAY_SIZE(msgs)); > + if (ret != ARRAY_SIZE(msgs)) { > + dev_err(&client->dev, "i2c read error: %d\n", ret); > + return -EIO; > + } > + > + *val = data_buf[0]; > + > + return 0; > +} > + > +static int hi846_write_reg(struct hi846 *hi846, u16 reg, u8 val) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + u8 buf[3] = { reg >> 8, reg & 0xff, val }; > + struct i2c_msg msg[] = { > + { .addr = client->addr, .flags = 0, > + .len = ARRAY_SIZE(buf), .buf = buf }, > + }; > + int ret; > + > + ret = i2c_transfer(client->adapter, msg, ARRAY_SIZE(msg)); > + if (ret != ARRAY_SIZE(msg)) { > + dev_err(&client->dev, "i2c write error\n"); > + return -EIO; > + } > + > + return 0; > +} > + > +static void hi846_write_reg_16(struct hi846 *hi846, u16 reg, u16 val, int *err) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + u8 buf[6]; > + int ret; > + > + if (*err < 0) > + return; > + > + put_unaligned_be16(reg, buf); > + put_unaligned_be32(val << 8 * 2, buf + 2); > + ret = i2c_master_send(client, buf, 4); > + if (ret != 4) { > + dev_err(&client->dev, "i2c_master_send != 4: %d\n", ret); > + *err = -EIO; > + } > +} > + > +static int hi846_write_reg_list(struct hi846 *hi846, > + const struct hi846_reg_list *r_list) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + unsigned int i; > + int ret = 0; > + > + for (i = 0; i < r_list->num_of_regs; i++) { > + hi846_write_reg_16(hi846, r_list->regs[i].address, > + r_list->regs[i].val, &ret); > + if (ret) { > + dev_err_ratelimited(&client->dev, > + "failed to write reg 0x%4.4x. error = %d", > + r_list->regs[i].address, ret); > + return ret; > + } > + } > + > + return 0; > +} > + > +static int hi846_update_digital_gain(struct hi846 *hi846, u16 d_gain) > +{ > + int ret = 0; > + > + hi846_write_reg_16(hi846, HI846_REG_MWB_GR_GAIN_H, d_gain, &ret); > + hi846_write_reg_16(hi846, HI846_REG_MWB_GB_GAIN_H, d_gain, &ret); > + hi846_write_reg_16(hi846, HI846_REG_MWB_R_GAIN_H, d_gain, &ret); > + hi846_write_reg_16(hi846, HI846_REG_MWB_B_GAIN_H, d_gain, &ret); > + > + return ret; > +} > + > +static int hi846_test_pattern(struct hi846 *hi846, u32 pattern) > +{ > + int ret; > + u8 val; > + > + if (pattern) { > + ret = hi846_read_reg(hi846, HI846_REG_ISP, &val); > + if (ret) > + return ret; > + > + ret = hi846_write_reg(hi846, HI846_REG_ISP, > + val | HI846_REG_ISP_TPG_EN); > + if (ret) > + return ret; > + } > + > + return hi846_write_reg(hi846, HI846_REG_TEST_PATTERN, pattern); > +} > + > +static int hi846_set_ctrl(struct v4l2_ctrl *ctrl) > +{ > + struct hi846 *hi846 = container_of(ctrl->handler, > + struct hi846, ctrl_handler); > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + s64 exposure_max; > + int ret = 0; > + u32 shutter, frame_len; > + > + /* Propagate change of current control to all related controls */ > + if (ctrl->id == V4L2_CID_VBLANK) { > + /* Update max exposure while meeting expected vblanking */ > + exposure_max = hi846->cur_mode->height + ctrl->val - > + HI846_EXPOSURE_MAX_MARGIN; > + __v4l2_ctrl_modify_range(hi846->exposure, > + hi846->exposure->minimum, > + exposure_max, hi846->exposure->step, > + exposure_max); > + } > + > + if (!pm_runtime_get_if_in_use(&client->dev)) > + return 0; > + > + switch (ctrl->id) { > + case V4L2_CID_ANALOGUE_GAIN: > + ret = hi846_write_reg(hi846, HI846_REG_ANALOG_GAIN, ctrl->val); > + break; > + > + case V4L2_CID_DIGITAL_GAIN: > + ret = hi846_update_digital_gain(hi846, ctrl->val); > + break; > + > + case V4L2_CID_EXPOSURE: > + shutter = ctrl->val; > + frame_len = hi846->cur_mode->frame_len; > + > + if (shutter > frame_len - 6) { /* margin */ > + frame_len = shutter + 6; > + if (frame_len > 0xffff) { /* max frame len */ > + frame_len = 0xffff; > + } > + } > + > + if (shutter < 6) > + shutter = 6; > + if (shutter > (0xffff - 6)) > + shutter = 0xffff - 6; > + > + hi846_write_reg_16(hi846, HI846_REG_FLL, frame_len, &ret); > + hi846_write_reg_16(hi846, HI846_REG_EXPOSURE, shutter, &ret); > + break; > + > + case V4L2_CID_VBLANK: > + /* Update FLL that meets expected vertical blanking */ > + hi846_write_reg_16(hi846, HI846_REG_FLL, > + hi846->cur_mode->height + ctrl->val, &ret); > + break; > + case V4L2_CID_TEST_PATTERN: > + ret = hi846_test_pattern(hi846, ctrl->val); > + break; > + > + default: > + ret = -EINVAL; > + break; > + } > + > + pm_runtime_put(&client->dev); > + > + return ret; > +} > + > +static const struct v4l2_ctrl_ops hi846_ctrl_ops = { > + .s_ctrl = hi846_set_ctrl, > +}; > + > +static int hi846_init_controls(struct hi846 *hi846) > +{ > + struct v4l2_ctrl_handler *ctrl_hdlr; > + s64 exposure_max, h_blank; > + int ret; > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + > + ctrl_hdlr = &hi846->ctrl_handler; > + ret = v4l2_ctrl_handler_init(ctrl_hdlr, 8); > + if (ret) > + return ret; > + > + ctrl_hdlr->lock = &hi846->mutex; > + > + hi846->link_freq = > + v4l2_ctrl_new_int_menu(ctrl_hdlr, &hi846_ctrl_ops, > + V4L2_CID_LINK_FREQ, > + ARRAY_SIZE(hi846_link_freqs) - 1, > + 0, hi846_link_freqs); > + if (hi846->link_freq) > + hi846->link_freq->flags |= V4L2_CTRL_FLAG_READ_ONLY; > + > + hi846->pixel_rate = > + v4l2_ctrl_new_std(ctrl_hdlr, &hi846_ctrl_ops, > + V4L2_CID_PIXEL_RATE, 0, > + hi846_calc_pixel_rate(hi846), 1, > + hi846_calc_pixel_rate(hi846)); > + hi846->vblank = v4l2_ctrl_new_std(ctrl_hdlr, &hi846_ctrl_ops, > + V4L2_CID_VBLANK, > + hi846->cur_mode->frame_len - > + hi846->cur_mode->height, > + HI846_FLL_MAX - > + hi846->cur_mode->height, 1, > + hi846->cur_mode->frame_len - > + hi846->cur_mode->height); > + > + h_blank = hi846->cur_mode->llp - hi846->cur_mode->width; > + > + hi846->hblank = v4l2_ctrl_new_std(ctrl_hdlr, &hi846_ctrl_ops, > + V4L2_CID_HBLANK, h_blank, h_blank, 1, > + h_blank); > + if (hi846->hblank) > + hi846->hblank->flags |= V4L2_CTRL_FLAG_READ_ONLY; > + > + v4l2_ctrl_new_std(ctrl_hdlr, &hi846_ctrl_ops, V4L2_CID_ANALOGUE_GAIN, > + HI846_ANAL_GAIN_MIN, HI846_ANAL_GAIN_MAX, > + HI846_ANAL_GAIN_STEP, HI846_ANAL_GAIN_MIN); > + v4l2_ctrl_new_std(ctrl_hdlr, &hi846_ctrl_ops, V4L2_CID_DIGITAL_GAIN, > + HI846_DGTL_GAIN_MIN, HI846_DGTL_GAIN_MAX, > + HI846_DGTL_GAIN_STEP, HI846_DGTL_GAIN_DEFAULT); > + exposure_max = hi846->cur_mode->frame_len - HI846_EXPOSURE_MAX_MARGIN; > + hi846->exposure = v4l2_ctrl_new_std(ctrl_hdlr, &hi846_ctrl_ops, > + V4L2_CID_EXPOSURE, > + HI846_EXPOSURE_MIN, exposure_max, > + HI846_EXPOSURE_STEP, > + exposure_max); > + v4l2_ctrl_new_std_menu_items(ctrl_hdlr, &hi846_ctrl_ops, > + V4L2_CID_TEST_PATTERN, > + ARRAY_SIZE(hi846_test_pattern_menu) - 1, > + 0, 0, hi846_test_pattern_menu); > + if (ctrl_hdlr->error) { > + dev_err(&client->dev, "v4l ctrl handler error: %d\n", > + ctrl_hdlr->error); > + return ctrl_hdlr->error; > + } > + > + hi846->sd.ctrl_handler = ctrl_hdlr; > + > + return 0; > +} > + > +static int hi846_set_video_mode(struct hi846 *hi846, int fps) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + u64 frame_length; > + int ret = 0; > + int dummy_lines; > + u64 link_freq = hi846_get_link_freq(hi846); > + > + dev_dbg(&client->dev, "%s: link freq: %llu\n", __func__, > + hi846_get_link_freq(hi846)); > + > + do_div(link_freq, fps); > + frame_length = link_freq; > + do_div(frame_length, HI846_LINE_LENGTH); > + > + dummy_lines = (frame_length > hi846->cur_mode->frame_len) ? > + (frame_length - hi846->cur_mode->frame_len) : 0; > + > + frame_length = hi846->cur_mode->frame_len + dummy_lines; > + > + dev_dbg(&client->dev, "%s: frame length calculated: %llu\n", __func__, > + frame_length); > + > + hi846_write_reg_16(hi846, HI846_REG_FLL, frame_length & 0xFFFF, &ret); > + hi846_write_reg_16(hi846, HI846_REG_LLP, HI846_LINE_LENGTH & 0xFFFF, &ret); Could you wrap the lines over 80, unless you have a sound reason to keep them longer, please? > + > + return ret; > +} > + > +static int hi846_start_streaming(struct hi846 *hi846) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + int ret = 0; > + u8 val; > + > + if (hi846->nr_lanes == 2) > + ret = hi846_write_reg_list(hi846, &hi846_init_regs_list_2lane); > + else > + ret = hi846_write_reg_list(hi846, &hi846_init_regs_list_4lane); > + if (ret) { > + dev_err(&client->dev, "failed to set plls: %d\n", ret); > + return ret; > + } > + > + ret = hi846_write_reg_list(hi846, &hi846->cur_mode->reg_list_config); > + if (ret) { > + dev_err(&client->dev, "failed to set mode: %d\n", ret); > + return ret; > + } > + > + if (hi846->nr_lanes == 2) { > + if (!hi846->cur_mode->reg_list_2lane.num_of_regs) { > + dev_err(&client->dev, "2 lanes unsupported for this mode\n"); This needs to be taken into account when the mode is selected, not at the time when streaming is about to start. > + return -EINVAL; > + } > + ret = hi846_write_reg_list(hi846, &hi846->cur_mode->reg_list_2lane); > + } else { > + if (!hi846->cur_mode->reg_list_4lane.num_of_regs) { > + dev_err(&client->dev, "4 lanes unsupported for this mode\n"); > + return -EINVAL; > + } > + ret = hi846_write_reg_list(hi846, &hi846->cur_mode->reg_list_4lane); > + } > + if (ret) { > + dev_err(&client->dev, "failed to set mipi mode: %d\n", ret); > + return ret; > + } > + > + hi846_set_video_mode(hi846, hi846->cur_mode->fps); > + > + ret = __v4l2_ctrl_handler_setup(hi846->sd.ctrl_handler); > + if (ret) > + return ret; > + > + /* > + * Reading 0x0034 is purely done for debugging reasons: It is not > + * documented in the DS but only mentioned once: > + * "If 0x0034[2] bit is disabled , Visible pixel width and height is 0." > + * So even though that sounds like we won't see anything, we don't > + * know more about this, so in that case only inform the user but do > + * nothing more. > + */ > + ret = hi846_read_reg(hi846, 0x0034, &val); > + if (ret) > + return ret; > + if (!(val & BIT(2))) > + dev_info(&client->dev, "visible pixel width and height is 0\n"); > + > + ret = hi846_write_reg(hi846, HI846_REG_MODE_SELECT, > + HI846_MODE_STREAMING); > + if (ret) { > + dev_err(&client->dev, "failed to start stream"); > + return ret; > + } > + > + hi846->streaming = 1; > + > + dev_dbg(&client->dev, "%s: started streaming successfully\n", __func__); > + > + return ret; > +} > + > +static void hi846_stop_streaming(struct hi846 *hi846) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + > + if (hi846_write_reg(hi846, HI846_REG_MODE_SELECT, HI846_MODE_STANDBY)) > + dev_err(&client->dev, "failed to stop stream"); > + > + hi846->streaming = 0; > +} > + > +static int hi846_set_stream(struct v4l2_subdev *sd, int enable) > +{ > + struct hi846 *hi846 = to_hi846(sd); > + struct i2c_client *client = v4l2_get_subdevdata(sd); > + int ret = 0; > + > + if (hi846->streaming == enable) > + return 0; > + > + mutex_lock(&hi846->mutex); > + > + if (enable) { > + ret = pm_runtime_get_sync(&client->dev); > + if (ret < 0) { > + pm_runtime_put_noidle(&client->dev); > + goto out; > + } > + > + ret = hi846_start_streaming(hi846); > + if (ret) { > + enable = 0; > + hi846_stop_streaming(hi846); > + pm_runtime_put(&client->dev); > + } > + } else { > + hi846_stop_streaming(hi846); > + pm_runtime_put(&client->dev); > + } > + > +out: > + mutex_unlock(&hi846->mutex); > + > + return ret; > +} > + > +static int hi846_power_on(struct hi846 *hi846) > +{ > + int ret; > + > + ret = regulator_bulk_enable(HI846_NUM_SUPPLIES, hi846->supplies); > + if (ret < 0) > + return ret; > + > + ret = clk_prepare_enable(hi846->clock); > + if (ret < 0) > + goto err_reg; > + > + if (hi846->shutdown_gpio) > + gpiod_set_value_cansleep(hi846->shutdown_gpio, 0); > + > + /* 30us = 2400 cycles at 80Mhz */ > + usleep_range(30, 60); > + if (hi846->rst_gpio) > + gpiod_set_value_cansleep(hi846->rst_gpio, 0); > + usleep_range(30, 60); > + > + return 0; > + > +err_reg: > + regulator_bulk_disable(HI846_NUM_SUPPLIES, hi846->supplies); > + > + return ret; > +} > + > +static void hi846_power_off(struct hi846 *hi846) > +{ > + if (hi846->rst_gpio) > + gpiod_set_value_cansleep(hi846->rst_gpio, 1); > + > + if (hi846->shutdown_gpio) > + gpiod_set_value_cansleep(hi846->shutdown_gpio, 1); > + > + clk_disable_unprepare(hi846->clock); > + regulator_bulk_disable(HI846_NUM_SUPPLIES, hi846->supplies); > +} > + > +static int __maybe_unused hi846_suspend(struct device *dev) > +{ > + struct i2c_client *client = to_i2c_client(dev); > + struct v4l2_subdev *sd = i2c_get_clientdata(client); > + struct hi846 *hi846 = to_hi846(sd); > + > + if (hi846->streaming) > + hi846_stop_streaming(hi846); > + > + hi846_power_off(hi846); > + > + return 0; > +} > + > +static int __maybe_unused hi846_resume(struct device *dev) > +{ > + struct i2c_client *client = to_i2c_client(dev); > + struct v4l2_subdev *sd = i2c_get_clientdata(client); > + struct hi846 *hi846 = to_hi846(sd); > + int ret; > + > + ret = hi846_power_on(hi846); > + if (ret) > + return ret; > + > + if (hi846->streaming) { > + ret = hi846_start_streaming(hi846); > + if (ret) { > + dev_err(dev, "%s: start streaming failed: %d\n", > + __func__, ret); > + goto error; > + } > + } > + > + return 0; > + > +error: > + hi846_power_off(hi846); > + return ret; > +} > + > +static int hi846_set_format(struct v4l2_subdev *sd, > + struct v4l2_subdev_state *sd_state, > + struct v4l2_subdev_format *format) > +{ > + struct hi846 *hi846 = to_hi846(sd); > + struct v4l2_mbus_framefmt *mf = &format->format; > + struct i2c_client *client = v4l2_get_subdevdata(sd); > + const struct hi846_datafmt *fmt = hi846_find_datafmt(mf->code); > + u32 tgt_fps; > + s32 vblank_def, h_blank; > + > + if (!fmt) { > + mf->code = hi846_colour_fmts[0].code; > + mf->colorspace = hi846_colour_fmts[0].colorspace; > + fmt = &hi846_colour_fmts[0]; > + } > + > + if (format->which == V4L2_SUBDEV_FORMAT_TRY) { > + *v4l2_subdev_get_try_format(sd, sd_state, format->pad) = *mf; > + return 0; > + } > + > + mutex_lock(&hi846->mutex); > + > + if (hi846->streaming) { > + mutex_unlock(&hi846->mutex); > + return -EBUSY; > + } > + > + hi846->fmt = fmt; > + > + hi846->cur_mode = > + v4l2_find_nearest_size(supported_modes, > + ARRAY_SIZE(supported_modes), > + width, height, mf->width, mf->height); > + dev_dbg(&client->dev, "%s: found mode: %dx%d\n", __func__, > + hi846->cur_mode->width, hi846->cur_mode->height); > + > + tgt_fps = hi846->cur_mode->fps; > + dev_dbg(&client->dev, "%s: target fps: %d\n", __func__, tgt_fps); > + > + mf->width = hi846->cur_mode->width; > + mf->height = hi846->cur_mode->height; > + mf->code = HI846_MEDIA_BUS_FORMAT; > + mf->field = V4L2_FIELD_NONE; > + > + __v4l2_ctrl_s_ctrl(hi846->link_freq, hi846_get_link_freq_index(hi846)); > + __v4l2_ctrl_s_ctrl_int64(hi846->pixel_rate, > + hi846_calc_pixel_rate(hi846)); > + > + /* Update limits and set FPS to default */ > + vblank_def = hi846->cur_mode->frame_len - hi846->cur_mode->height; > + __v4l2_ctrl_modify_range(hi846->vblank, > + hi846->cur_mode->frame_len - hi846->cur_mode->height, > + HI846_FLL_MAX - hi846->cur_mode->height, 1, > + vblank_def); > + __v4l2_ctrl_s_ctrl(hi846->vblank, vblank_def); > + > + h_blank = hi846->cur_mode->llp - hi846->cur_mode->width; > + > + __v4l2_ctrl_modify_range(hi846->hblank, h_blank, h_blank, 1, > + h_blank); > + > + dev_dbg(&client->dev, "Set fmt w=%d h=%d code=0x%x colorspace=0x%x\n", > + mf->width, mf->height, > + fmt->code, fmt->colorspace); > + > + mutex_unlock(&hi846->mutex); > + > + return 0; > +} > + > +static int hi846_get_format(struct v4l2_subdev *sd, > + struct v4l2_subdev_state *sd_state, > + struct v4l2_subdev_format *format) > +{ > + struct hi846 *hi846 = to_hi846(sd); > + struct v4l2_mbus_framefmt *mf = &format->format; > + struct i2c_client *client = v4l2_get_subdevdata(sd); > + > + if (format->which == V4L2_SUBDEV_FORMAT_TRY) { > + format->format = *v4l2_subdev_get_try_format(&hi846->sd, > + sd_state, > + format->pad); > + return 0; > + } > + > + mutex_lock(&hi846->mutex); > + mf->code = HI846_MEDIA_BUS_FORMAT; > + mf->colorspace = V4L2_COLORSPACE_RAW; > + mf->field = V4L2_FIELD_NONE; > + mf->width = hi846->cur_mode->width; > + mf->height = hi846->cur_mode->height; > + mutex_unlock(&hi846->mutex); > + dev_dbg(&client->dev, > + "Get format w=%d h=%d code=0x%x colorspace=0x%x\n", > + mf->width, mf->height, mf->code, mf->colorspace); > + > + return 0; > +} > + > +static int hi846_enum_mbus_code(struct v4l2_subdev *sd, > + struct v4l2_subdev_state *sd_state, > + struct v4l2_subdev_mbus_code_enum *code) > +{ > + if (code->pad || code->index > 0) > + return -EINVAL; > + > + code->code = HI846_MEDIA_BUS_FORMAT; > + > + return 0; > +} > + > +static int hi846_enum_frame_size(struct v4l2_subdev *sd, > + struct v4l2_subdev_state *sd_state, > + struct v4l2_subdev_frame_size_enum *fse) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(sd); > + > + if (fse->pad || fse->index >= ARRAY_SIZE(supported_modes)) > + return -EINVAL; > + > + if (fse->code != HI846_MEDIA_BUS_FORMAT) { > + dev_err(&client->dev, "frame size enum not matching\n"); > + return -EINVAL; > + } > + > + fse->min_width = supported_modes[fse->index].width; > + fse->max_width = supported_modes[fse->index].width; > + fse->min_height = supported_modes[fse->index].height; > + fse->max_height = supported_modes[fse->index].height; > + > + dev_dbg(&client->dev, "%s: max width: %d max height: %d\n", __func__, > + fse->max_width, fse->max_height); > + > + return 0; > +} > + > +static int hi846_get_selection(struct v4l2_subdev *sd, > + struct v4l2_subdev_state *sd_state, > + struct v4l2_subdev_selection *sel) > +{ > + struct hi846 *hi846 = to_hi846(sd); > + > + switch (sel->target) { > + case V4L2_SEL_TGT_CROP: > + mutex_lock(&hi846->mutex); > + switch (sel->which) { > + case V4L2_SUBDEV_FORMAT_TRY: > + v4l2_subdev_get_try_crop(sd, sd_state, sel->pad); > + break; > + case V4L2_SUBDEV_FORMAT_ACTIVE: > + sel->r = hi846->cur_mode->crop; > + break; > + } > + mutex_unlock(&hi846->mutex); > + return 0; > + case V4L2_SEL_TGT_CROP_BOUNDS: > + case V4L2_SEL_TGT_CROP_DEFAULT: > + sel->r.top = 0; > + sel->r.left = 0; > + sel->r.width = 3264; > + sel->r.height = 2448; If you can't change it, then the values related to a given mode should be returned for CROP and CROP_DEFAULT, and these values here for BOUNDS. Please also add NATIVE_SIZE rectangle, same values as BOUNDS here. > + return 0; > + default: > + return -EINVAL; > + } > +} > + > +static int hi846_init_cfg(struct v4l2_subdev *sd, > + struct v4l2_subdev_state *sd_state) > +{ > + struct hi846 *hi846 = to_hi846(sd); > + struct v4l2_mbus_framefmt *mf; > + > + mf = v4l2_subdev_get_try_format(sd, sd_state, 0); > + > + mutex_lock(&hi846->mutex); > + mf->code = HI846_MEDIA_BUS_FORMAT; > + mf->colorspace = V4L2_COLORSPACE_RAW; > + mf->field = V4L2_FIELD_NONE; > + mf->width = hi846->cur_mode->width; > + mf->height = hi846->cur_mode->height; > + mutex_unlock(&hi846->mutex); > + > + return 0; > +} > + > +static const struct v4l2_subdev_video_ops hi846_video_ops = { > + .s_stream = hi846_set_stream, > +}; > + > +static const struct v4l2_subdev_pad_ops hi846_pad_ops = { > + .init_cfg = hi846_init_cfg, > + .enum_frame_size = hi846_enum_frame_size, > + .enum_mbus_code = hi846_enum_mbus_code, > + .set_fmt = hi846_set_format, > + .get_fmt = hi846_get_format, > + .get_selection = hi846_get_selection, > +}; > + > +static const struct v4l2_subdev_ops hi846_subdev_ops = { > + .video = &hi846_video_ops, > + .pad = &hi846_pad_ops, > +}; > + > +static const struct media_entity_operations hi846_subdev_entity_ops = { > + .link_validate = v4l2_subdev_link_validate, > +}; > + > +static int hi846_identify_module(struct hi846 *hi846) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(&hi846->sd); > + int ret; > + u8 hi, lo; > + > + ret = hi846_read_reg(hi846, HI846_REG_CHIP_ID_L, &lo); > + if (ret) > + return ret; > + > + if (lo != HI846_CHIP_ID_L) { > + dev_err(&client->dev, "wrong chip id low byte: %x", lo); > + return -ENXIO; > + } > + > + ret = hi846_read_reg(hi846, HI846_REG_CHIP_ID_H, &hi); > + if (ret) > + return ret; > + > + if (hi != HI846_CHIP_ID_H) { > + dev_err(&client->dev, "wrong chip id high byte: %x", hi); > + return -ENXIO; > + } > + > + dev_info(&client->dev, "chip id %02X %02X using %d mipi lanes\n", hi, lo, > + hi846->nr_lanes); > + > + return 0; > +} > + > +static int hi846_parse_dt(struct hi846 *hi846, struct device *dev) > +{ > + struct fwnode_handle *ep; > + struct fwnode_handle *fwnode = dev_fwnode(dev); > + struct v4l2_fwnode_endpoint bus_cfg = { > + .bus_type = V4L2_MBUS_CSI2_DPHY > + }; > + int ret; > + > + ep = fwnode_graph_get_next_endpoint(fwnode, NULL); > + if (!ep) { > + dev_err(dev, "unable to find endpoint node\n"); > + return -ENXIO; > + } > + > + ret = v4l2_fwnode_endpoint_alloc_parse(ep, &bus_cfg); > + fwnode_handle_put(ep); > + if (ret) { > + dev_err(dev, "failed to parse endpoint node: %d\n", ret); > + return ret; > + } > + > + if (bus_cfg.bus.mipi_csi2.num_data_lanes != 2 && > + bus_cfg.bus.mipi_csi2.num_data_lanes != 4) { > + dev_err(dev, "number of CSI2 data lanes %d is not supported", > + bus_cfg.bus.mipi_csi2.num_data_lanes); > + v4l2_fwnode_endpoint_free(&bus_cfg); > + return -EINVAL; > + } > + Could you add a check here for the supported link frequencies? This is generally a property of the board, not the sensor. See e.g. imx290 driver for an example. > + hi846->nr_lanes = bus_cfg.bus.mipi_csi2.num_data_lanes; > + v4l2_fwnode_endpoint_free(&bus_cfg); > + > + hi846->rst_gpio = devm_gpiod_get_optional(dev, "reset", GPIOD_OUT_LOW); > + if (IS_ERR(hi846->rst_gpio)) { > + dev_err(dev, "failed to get reset gpio: %pe\n", hi846->rst_gpio); > + return PTR_ERR(hi846->rst_gpio); > + } > + > + hi846->shutdown_gpio = devm_gpiod_get_optional(dev, "shutdown", GPIOD_OUT_LOW); > + if (IS_ERR(hi846->shutdown_gpio)) { > + dev_err(dev, "failed to get shutdown gpio: %pe\n", hi846->shutdown_gpio); > + return PTR_ERR(hi846->shutdown_gpio); > + } > + > + return 0; > +} > + > +static int hi846_probe(struct i2c_client *client) > +{ > + struct hi846 *hi846; > + int ret; > + int i; > + u32 mclk_freq; > + > + hi846 = devm_kzalloc(&client->dev, sizeof(*hi846), GFP_KERNEL); > + if (!hi846) > + return -ENOMEM; > + > + ret = hi846_parse_dt(hi846, &client->dev); > + if (ret) { > + dev_err(&client->dev, "failed to check HW configuration: %d", ret); > + return ret; > + } > + > + hi846->clock = devm_clk_get(&client->dev, NULL); > + if (IS_ERR(hi846->clock)) { > + dev_err(&client->dev, "failed to get clock: %pe\n", hi846->clock); > + return PTR_ERR(hi846->clock); > + } > + > + mclk_freq = clk_get_rate(hi846->clock); > + dev_dbg(&client->dev, "clk rate: %u\n", mclk_freq); > + > + for (i = 0; i < HI846_NUM_SUPPLIES; i++) > + hi846->supplies[i].supply = hi846_supply_names[i]; > + > + ret = devm_regulator_bulk_get(&client->dev, HI846_NUM_SUPPLIES, > + hi846->supplies); > + if (ret < 0) > + return ret; > + > + mutex_init(&hi846->mutex); > + > + hi846->cur_mode = &supported_modes[0]; > + ret = hi846_init_controls(hi846); > + if (ret) { > + dev_err(&client->dev, "failed to init controls: %d", ret); > + return ret; > + } > + > + v4l2_i2c_subdev_init(&hi846->sd, client, &hi846_subdev_ops); > + hi846->sd.flags |= V4L2_SUBDEV_FL_HAS_DEVNODE; > + hi846->sd.entity.ops = &hi846_subdev_entity_ops; > + hi846->sd.entity.function = MEDIA_ENT_F_CAM_SENSOR; > + hi846->pad.flags = MEDIA_PAD_FL_SOURCE; > + ret = media_entity_pads_init(&hi846->sd.entity, 1, &hi846->pad); > + if (ret) { > + dev_err(&client->dev, "failed to init entity pads: %d", ret); > + goto probe_error_v4l2_ctrl_handler_free; > + } > + > + ret = hi846_power_on(hi846); > + if (ret) > + goto probe_error_media_entity_cleanup; > + > + ret = hi846_identify_module(hi846); > + if (ret) > + goto probe_error_power_off; > + > + ret = v4l2_async_register_subdev_sensor(&hi846->sd); > + if (ret < 0) { > + dev_err(&client->dev, "failed to register V4L2 subdev: %d", > + ret); > + goto probe_error_power_off; > + } > + > + pm_runtime_set_active(&client->dev); > + pm_runtime_enable(&client->dev); > + pm_runtime_idle(&client->dev); > + > + return 0; > + > +probe_error_power_off: > + hi846_power_off(hi846); > + > +probe_error_media_entity_cleanup: > + media_entity_cleanup(&hi846->sd.entity); > + > +probe_error_v4l2_ctrl_handler_free: > + v4l2_ctrl_handler_free(hi846->sd.ctrl_handler); > + mutex_destroy(&hi846->mutex); > + > + return ret; > +} > + > +static int hi846_remove(struct i2c_client *client) > +{ > + struct v4l2_subdev *sd = i2c_get_clientdata(client); > + struct hi846 *hi846 = to_hi846(sd); > + > + v4l2_async_unregister_subdev(sd); > + media_entity_cleanup(&sd->entity); > + v4l2_ctrl_handler_free(sd->ctrl_handler); > + > + pm_runtime_disable(&client->dev); > + if (!pm_runtime_status_suspended(&client->dev)) > + hi846_suspend(&client->dev); > + pm_runtime_set_suspended(&client->dev); > + > + mutex_destroy(&hi846->mutex); > + > + return 0; > +} > + > +static UNIVERSAL_DEV_PM_OPS(hi846_pm_ops, hi846_suspend, hi846_resume, NULL); > + > +static const struct of_device_id hi846_of_match[] = { > + { .compatible = "hynix,hi846", }, > + {}, > +}; > +MODULE_DEVICE_TABLE(of, hi846_of_match); > + > +static struct i2c_driver hi846_i2c_driver = { > + .driver = { > + .name = "hi846", > + .pm = &hi846_pm_ops, > + .of_match_table = of_match_ptr(hi846_of_match), > + }, > + .probe_new = hi846_probe, > + .remove = hi846_remove, > +}; > + > +module_i2c_driver(hi846_i2c_driver); > + > +MODULE_AUTHOR("Angus Ainslie "); > +MODULE_AUTHOR("Martin Kepplinger "); > +MODULE_DESCRIPTION("Hynix HI846 sensor driver"); > +MODULE_LICENSE("GPL v2"); -- Kind regards, Sakari Ailus