* [PATCH v9 1/2] media:imx274 device tree binding file
@ 2017-10-26 6:51 Leon Luo
2017-10-26 6:51 ` [PATCH v9 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor Leon Luo
0 siblings, 1 reply; 4+ messages in thread
From: Leon Luo @ 2017-10-26 6:51 UTC (permalink / raw)
To: mchehab-DgEjT+Ai2ygdnm+yROfE0A, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
mark.rutland-5wv7dgnIgG8, hans.verkuil-FYB4Gu1CFyUAvxtiuMwx3w,
sakari.ailus-VuQAYsv1563Yd54FQh9/CA
Cc: linux-media-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
leonl-+iDsysdzPxZTt5vwKEO50dBPR1lH4CV8,
soren.brinkmann-gjFFaj9aHVfQT0dZR+AlfA
The binding file for imx274 CMOS sensor V4l2 driver
Signed-off-by: Leon Luo <leonl-+iDsysdzPxZTt5vwKEO50dBPR1lH4CV8@public.gmane.org>
Acked-by: Sören Brinkmann <soren.brinkmann-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
---
v9:
- add an entry in MAINTAINERS
v8:
- no changes
v7:
- no changes
v6:
- no changes
v5:
- add 'port' and 'endpoint' information
v4:
- no changes
v3:
- remove redundant properties and references
- document 'reg' property
v2:
- no changes
---
.../devicetree/bindings/media/i2c/imx274.txt | 33 ++++++++++++++++++++++
MAINTAINERS | 8 ++++++
2 files changed, 41 insertions(+)
create mode 100644 Documentation/devicetree/bindings/media/i2c/imx274.txt
diff --git a/Documentation/devicetree/bindings/media/i2c/imx274.txt b/Documentation/devicetree/bindings/media/i2c/imx274.txt
new file mode 100644
index 000000000000..80f2e89568e1
--- /dev/null
+++ b/Documentation/devicetree/bindings/media/i2c/imx274.txt
@@ -0,0 +1,33 @@
+* Sony 1/2.5-Inch 8.51Mp CMOS Digital Image Sensor
+
+The Sony imx274 is a 1/2.5-inch CMOS active pixel digital image sensor with
+an active array size of 3864H x 2202V. It is programmable through I2C
+interface. The I2C address is fixed to 0x1a as per sensor data sheet.
+Image data is sent through MIPI CSI-2, which is configured as 4 lanes
+at 1440 Mbps.
+
+
+Required Properties:
+- compatible: value should be "sony,imx274" for imx274 sensor
+- reg: I2C bus address of the device
+
+Optional Properties:
+- reset-gpios: Sensor reset GPIO
+
+The imx274 device node should contain one 'port' child node with
+an 'endpoint' subnode. For further reading on port node refer to
+Documentation/devicetree/bindings/media/video-interfaces.txt.
+
+Example:
+ sensor@1a {
+ compatible = "sony,imx274";
+ reg = <0x1a>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ reset-gpios = <&gpio_sensor 0 0>;
+ port {
+ sensor_out: endpoint {
+ remote-endpoint = <&csiss_in>;
+ };
+ };
+ };
diff --git a/MAINTAINERS b/MAINTAINERS
index 90230fe020f3..b5927bd4fe1e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12486,6 +12486,14 @@ S: Maintained
F: drivers/ssb/
F: include/linux/ssb/
+SONY IMX274 SENSOR DRIVER
+M: Leon Luo <leonl-+iDsysdzPxZTt5vwKEO50dBPR1lH4CV8@public.gmane.org>
+L: linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
+T: git git://linuxtv.org/media_tree.git
+S: Maintained
+F: drivers/media/i2c/imx274.c
+F: Documentation/devicetree/bindings/media/i2c/imx274.txt
+
SONY MEMORYSTICK CARD SUPPORT
M: Alex Dubov <oakad-/E1597aS9LQAvxtiuMwx3w@public.gmane.org>
W: http://tifmxx.berlios.de/
--
2.14.0.rc1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v9 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor
2017-10-26 6:51 [PATCH v9 1/2] media:imx274 device tree binding file Leon Luo
@ 2017-10-26 6:51 ` Leon Luo
2017-11-03 4:36 ` Vishal Sagar
0 siblings, 1 reply; 4+ messages in thread
From: Leon Luo @ 2017-10-26 6:51 UTC (permalink / raw)
To: mchehab, robh+dt, mark.rutland, hans.verkuil, sakari.ailus
Cc: linux-media, devicetree, linux-kernel, leonl, soren.brinkmann
The imx274 is a Sony CMOS image sensor that has 1/2.5 image size.
It supports up to 3840x2160 (4K) 60fps, 1080p 120fps. The interface
is 4-lane MIPI CSI-2 running at 1.44Gbps each.
This driver has been tested on Xilinx ZCU102 platform with a Leopard
LI-IMX274MIPI-FMC camera board.
Support for the following features:
-Resolutions: 3840x2160, 1920x1080, 1280x720
-Frame rate: 3840x2160 : 5 – 60fps
1920x1080 : 5 – 120fps
1280x720 : 5 – 120fps
-Exposure time: 16 – (frame interval) micro-seconds
-Gain: 1x - 180x
-VFLIP: enable/disabledrivers/media/i2c/imx274.c
-Test pattern: 12 test patterns
Signed-off-by: Leon Luo <leonl@leopardimaging.com>
Tested-by: Sören Brinkmann <soren.brinkmann@xilinx.com>
Acked-by: Sakari Ailus <sakari.ailus@iki.fi>
Acked-by: Chris Kohn <ckohn@xilinx.com>
---
v9:
- remove v4l2_async_unregister_subdev from probe error
v8:
- check division by zero error
v7:
- use __v4l2_ctrl_s_ctrl instead of v4l2_ctrl_s_ctrl to have
clean mutex_lock/mutex_unlock in imx274_s_stream()
- define imx274_tp_regs[] as static, move the test pattern reg
out of imx274_tp_regs[], and define it as a macro
IMX274_TEST_PATTERN_REG
v6:
- remove media/v4l2-image-sizes.h from include header
- make the header file alphabetical order
- remove fmt->pad check in imx274_get_fmt,
the V4L2 subdev framework does it already
- change 'struct reg_8 *regs' to 'struct reg_8 regs[n]',
where n is the exact numbers needed for this function
- move MODULE_DEVICE_TABLE(of, imx274_of_id_table); closer
to imx274_of_id_table definition
- remove return check of imx274_write_table in imx274_remove,
because it should remove all resources even i2c fails here
- move imx274_load_default before v4l2_async_register_subdev
v5:
- no changes
v4:
- use 32-bit data type to avoid __divdi3 compile error for i386
- clean up OR together error codesdrivers/media/i2c/imx274.c
v3:
- clean up header files
- use struct directly instead of alias #define
- use v4l2_ctrl_s_ctrl instead of v4l2_s_ctrl
- revise debug output
- move static helpers closer to their call site
- don't OR toegether error codes
- use closest valid gain value instead of erroring out
- assigne lock to the control handler and omit explicit locking
- acquire mutex lock for imx274_get_fmt
- remove format->pad check in imx274_set_fmt since the pad is always 0
- pass struct v4l2_ctrl pointer in gain/exposure/vlip/test pattern controls
- remove priv->ctrls.vflip->val = val in imx274_set_vflip()
- remove priv->ctrls.test_pattern->val = val in imx274_set_test_pattern()
- remove empty open/close callbacks
- remove empty core ops
- add more error labels in probe
- use v4l2_async_unregister_subdev instead of v4l2_device_unregister_subdev
- use dynamic debug
- split start_stream to two steps: imx274_mode_regs() and imx274_start_stream()
frame rate & exposure can be updated
between imx274_mode_regs() & imx274_start_stream()
v2:
- Fix Kconfig to not remove existing options
---
drivers/media/i2c/Kconfig | 7 +
drivers/media/i2c/Makefile | 1 +
drivers/media/i2c/imx274.c | 1810 ++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 1818 insertions(+)
create mode 100644 drivers/media/i2c/imx274.c
diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig
index 47113774a297..9659849e33a0 100644
--- a/drivers/media/i2c/Kconfig
+++ b/drivers/media/i2c/Kconfig
@@ -555,6 +555,13 @@ config VIDEO_APTINA_PLL
config VIDEO_SMIAPP_PLL
tristate
+config VIDEO_IMX274
+ tristate "Sony IMX274 sensor support"
+ depends on I2C && VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API
+ ---help---
+ This is a V4L2 sensor-level driver for the Sony IMX274
+ CMOS image sensor.
+
config VIDEO_OV2640
tristate "OmniVision OV2640 sensor support"
depends on VIDEO_V4L2 && I2C
diff --git a/drivers/media/i2c/Makefile b/drivers/media/i2c/Makefile
index c843c181dfb9..f8d57e453936 100644
--- a/drivers/media/i2c/Makefile
+++ b/drivers/media/i2c/Makefile
@@ -92,5 +92,6 @@ obj-$(CONFIG_VIDEO_IR_I2C) += ir-kbd-i2c.o
obj-$(CONFIG_VIDEO_ML86V7667) += ml86v7667.o
obj-$(CONFIG_VIDEO_OV2659) += ov2659.o
obj-$(CONFIG_VIDEO_TC358743) += tc358743.o
+obj-$(CONFIG_VIDEO_IMX274) += imx274.o
obj-$(CONFIG_SDR_MAX2175) += max2175.o
diff --git a/drivers/media/i2c/imx274.c b/drivers/media/i2c/imx274.c
new file mode 100644
index 000000000000..737dbf59a0d2
--- /dev/null
+++ b/drivers/media/i2c/imx274.c
@@ -0,0 +1,1810 @@
+/*
+ * imx274.c - IMX274 CMOS Image Sensor driver
+ *
+ * Copyright (C) 2017, Leopard Imaging, Inc.
+ *
+ * Leon Luo <leonl@leopardimaging.com>
+ * Edwin Zou <edwinz@leopardimaging.com>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms and conditions of the GNU General Public License,
+ * version 2, as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
+ * more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <linux/clk.h>
+#include <linux/delay.h>
+#include <linux/gpio.h>
+#include <linux/gpio/consumer.h>
+#include <linux/i2c.h>
+#include <linux/init.h>
+#include <linux/module.h>
+#include <linux/of_gpio.h>
+#include <linux/regmap.h>
+#include <linux/slab.h>
+#include <linux/v4l2-mediabus.h>
+#include <linux/videodev2.h>
+
+#include <media/v4l2-ctrls.h>
+#include <media/v4l2-device.h>
+#include <media/v4l2-subdev.h>
+
+/*
+ * See "SHR, SVR Setting" in datasheet
+ */
+#define IMX274_DEFAULT_FRAME_LENGTH (4550)
+#define IMX274_MAX_FRAME_LENGTH (0x000fffff)
+
+/*
+ * See "Frame Rate Adjustment" in datasheet
+ */
+#define IMX274_PIXCLK_CONST1 (72000000)
+#define IMX274_PIXCLK_CONST2 (1000000)
+
+/*
+ * The input gain is shifted by IMX274_GAIN_SHIFT to get
+ * decimal number. The real gain is
+ * (float)input_gain_value / (1 << IMX274_GAIN_SHIFT)
+ */
+#define IMX274_GAIN_SHIFT (8)
+#define IMX274_GAIN_SHIFT_MASK ((1 << IMX274_GAIN_SHIFT) - 1)
+
+/*
+ * See "Analog Gain" and "Digital Gain" in datasheet
+ * min gain is 1X
+ * max gain is calculated based on IMX274_GAIN_REG_MAX
+ */
+#define IMX274_GAIN_REG_MAX (1957)
+#define IMX274_MIN_GAIN (0x01 << IMX274_GAIN_SHIFT)
+#define IMX274_MAX_ANALOG_GAIN ((2048 << IMX274_GAIN_SHIFT)\
+ / (2048 - IMX274_GAIN_REG_MAX))
+#define IMX274_MAX_DIGITAL_GAIN (8)
+#define IMX274_DEF_GAIN (20 << IMX274_GAIN_SHIFT)
+#define IMX274_GAIN_CONST (2048) /* for gain formula */
+
+/*
+ * 1 line time in us = (HMAX / 72), minimal is 4 lines
+ */
+#define IMX274_MIN_EXPOSURE_TIME (4 * 260 / 72)
+
+#define IMX274_DEFAULT_MODE IMX274_MODE_3840X2160
+#define IMX274_MAX_WIDTH (3840)
+#define IMX274_MAX_HEIGHT (2160)
+#define IMX274_MAX_FRAME_RATE (120)
+#define IMX274_MIN_FRAME_RATE (5)
+#define IMX274_DEF_FRAME_RATE (60)
+
+/*
+ * register SHR is limited to (SVR value + 1) x VMAX value - 4
+ */
+#define IMX274_SHR_LIMIT_CONST (4)
+
+/*
+ * Constants for sensor reset delay
+ */
+#define IMX274_RESET_DELAY1 (2000)
+#define IMX274_RESET_DELAY2 (2200)
+
+/*
+ * shift and mask constants
+ */
+#define IMX274_SHIFT_8_BITS (8)
+#define IMX274_SHIFT_16_BITS (16)
+#define IMX274_MASK_LSB_2_BITS (0x03)
+#define IMX274_MASK_LSB_3_BITS (0x07)
+#define IMX274_MASK_LSB_4_BITS (0x0f)
+#define IMX274_MASK_LSB_8_BITS (0x00ff)
+
+#define DRIVER_NAME "IMX274"
+
+/*
+ * IMX274 register definitions
+ */
+#define IMX274_FRAME_LENGTH_ADDR_1 0x30FA /* VMAX, MSB */
+#define IMX274_FRAME_LENGTH_ADDR_2 0x30F9 /* VMAX */
+#define IMX274_FRAME_LENGTH_ADDR_3 0x30F8 /* VMAX, LSB */
+#define IMX274_SVR_REG_MSB 0x300F /* SVR */
+#define IMX274_SVR_REG_LSB 0x300E /* SVR */
+#define IMX274_HMAX_REG_MSB 0x30F7 /* HMAX */
+#define IMX274_HMAX_REG_LSB 0x30F6 /* HMAX */
+#define IMX274_COARSE_TIME_ADDR_MSB 0x300D /* SHR */
+#define IMX274_COARSE_TIME_ADDR_LSB 0x300C /* SHR */
+#define IMX274_ANALOG_GAIN_ADDR_LSB 0x300A /* ANALOG GAIN LSB */
+#define IMX274_ANALOG_GAIN_ADDR_MSB 0x300B /* ANALOG GAIN MSB */
+#define IMX274_DIGITAL_GAIN_REG 0x3012 /* Digital Gain */
+#define IMX274_VFLIP_REG 0x301A /* VERTICAL FLIP */
+#define IMX274_TEST_PATTERN_REG 0x303D /* TEST PATTERN */
+#define IMX274_STANDBY_REG 0x3000 /* STANDBY */
+
+#define IMX274_TABLE_WAIT_MS 0
+#define IMX274_TABLE_END 1
+
+/*
+ * imx274 I2C operation related structure
+ */
+struct reg_8 {
+ u16 addr;
+ u8 val;
+};
+
+static const struct regmap_config imx274_regmap_config = {
+ .reg_bits = 16,
+ .val_bits = 8,
+ .cache_type = REGCACHE_RBTREE,
+};
+
+enum imx274_mode {
+ IMX274_MODE_3840X2160,
+ IMX274_MODE_1920X1080,
+ IMX274_MODE_1280X720,
+
+ IMX274_MODE_START_STREAM_1,
+ IMX274_MODE_START_STREAM_2,
+ IMX274_MODE_START_STREAM_3,
+ IMX274_MODE_START_STREAM_4,
+ IMX274_MODE_STOP_STREAM
+};
+
+/*
+ * imx274 format related structure
+ */
+struct imx274_frmfmt {
+ u32 mbus_code;
+ enum v4l2_colorspace colorspace;
+ struct v4l2_frmsize_discrete size;
+ enum imx274_mode mode;
+};
+
+/*
+ * imx274 test pattern related structure
+ */
+enum {
+ TEST_PATTERN_DISABLED = 0,
+ TEST_PATTERN_ALL_000H,
+ TEST_PATTERN_ALL_FFFH,
+ TEST_PATTERN_ALL_555H,
+ TEST_PATTERN_ALL_AAAH,
+ TEST_PATTERN_VSP_5AH, /* VERTICAL STRIPE PATTERN 555H/AAAH */
+ TEST_PATTERN_VSP_A5H, /* VERTICAL STRIPE PATTERN AAAH/555H */
+ TEST_PATTERN_VSP_05H, /* VERTICAL STRIPE PATTERN 000H/555H */
+ TEST_PATTERN_VSP_50H, /* VERTICAL STRIPE PATTERN 555H/000H */
+ TEST_PATTERN_VSP_0FH, /* VERTICAL STRIPE PATTERN 000H/FFFH */
+ TEST_PATTERN_VSP_F0H, /* VERTICAL STRIPE PATTERN FFFH/000H */
+ TEST_PATTERN_H_COLOR_BARS,
+ TEST_PATTERN_V_COLOR_BARS,
+};
+
+static const char * const tp_qmenu[] = {
+ "Disabled",
+ "All 000h Pattern",
+ "All FFFh Pattern",
+ "All 555h Pattern",
+ "All AAAh Pattern",
+ "Vertical Stripe (555h / AAAh)",
+ "Vertical Stripe (AAAh / 555h)",
+ "Vertical Stripe (000h / 555h)",
+ "Vertical Stripe (555h / 000h)",
+ "Vertical Stripe (000h / FFFh)",
+ "Vertical Stripe (FFFh / 000h)",
+ "Horizontal Color Bars",
+ "Vertical Color Bars",
+};
+
+/*
+ * All-pixel scan mode (10-bit)
+ * imx274 mode1(refer to datasheet) register configuration with
+ * 3840x2160 resolution, raw10 data and mipi four lane output
+ */
+static const struct reg_8 imx274_mode1_3840x2160_raw10[] = {
+ {0x3004, 0x01},
+ {0x3005, 0x01},
+ {0x3006, 0x00},
+ {0x3007, 0x02},
+
+ {0x3018, 0xA2}, /* output XVS, HVS */
+
+ {0x306B, 0x05},
+ {0x30E2, 0x01},
+ {0x30F6, 0x07}, /* HMAX, 263 */
+ {0x30F7, 0x01}, /* HMAX */
+
+ {0x30dd, 0x01}, /* crop to 2160 */
+ {0x30de, 0x06},
+ {0x30df, 0x00},
+ {0x30e0, 0x12},
+ {0x30e1, 0x00},
+ {0x3037, 0x01}, /* to crop to 3840 */
+ {0x3038, 0x0c},
+ {0x3039, 0x00},
+ {0x303a, 0x0c},
+ {0x303b, 0x0f},
+
+ {0x30EE, 0x01},
+ {0x3130, 0x86},
+ {0x3131, 0x08},
+ {0x3132, 0x7E},
+ {0x3133, 0x08},
+ {0x3342, 0x0A},
+ {0x3343, 0x00},
+ {0x3344, 0x16},
+ {0x3345, 0x00},
+ {0x33A6, 0x01},
+ {0x3528, 0x0E},
+ {0x3554, 0x1F},
+ {0x3555, 0x01},
+ {0x3556, 0x01},
+ {0x3557, 0x01},
+ {0x3558, 0x01},
+ {0x3559, 0x00},
+ {0x355A, 0x00},
+ {0x35BA, 0x0E},
+ {0x366A, 0x1B},
+ {0x366B, 0x1A},
+ {0x366C, 0x19},
+ {0x366D, 0x17},
+ {0x3A41, 0x08},
+
+ {IMX274_TABLE_END, 0x00}
+};
+
+/*
+ * Horizontal/vertical 2/2-line binning
+ * (Horizontal and vertical weightedbinning, 10-bit)
+ * imx274 mode3(refer to datasheet) register configuration with
+ * 1920x1080 resolution, raw10 data and mipi four lane output
+ */
+static const struct reg_8 imx274_mode3_1920x1080_raw10[] = {
+ {0x3004, 0x02},
+ {0x3005, 0x21},
+ {0x3006, 0x00},
+ {0x3007, 0x11},
+
+ {0x3018, 0xA2}, /* output XVS, HVS */
+
+ {0x306B, 0x05},
+ {0x30E2, 0x02},
+
+ {0x30F6, 0x04}, /* HMAX, 260 */
+ {0x30F7, 0x01}, /* HMAX */
+
+ {0x30dd, 0x01}, /* to crop to 1920x1080 */
+ {0x30de, 0x05},
+ {0x30df, 0x00},
+ {0x30e0, 0x04},
+ {0x30e1, 0x00},
+ {0x3037, 0x01},
+ {0x3038, 0x0c},
+ {0x3039, 0x00},
+ {0x303a, 0x0c},
+ {0x303b, 0x0f},
+
+ {0x30EE, 0x01},
+ {0x3130, 0x4E},
+ {0x3131, 0x04},
+ {0x3132, 0x46},
+ {0x3133, 0x04},
+ {0x3342, 0x0A},
+ {0x3343, 0x00},
+ {0x3344, 0x1A},
+ {0x3345, 0x00},
+ {0x33A6, 0x01},
+ {0x3528, 0x0E},
+ {0x3554, 0x00},
+ {0x3555, 0x01},
+ {0x3556, 0x01},
+ {0x3557, 0x01},
+ {0x3558, 0x01},
+ {0x3559, 0x00},
+ {0x355A, 0x00},
+ {0x35BA, 0x0E},
+ {0x366A, 0x1B},
+ {0x366B, 0x1A},
+ {0x366C, 0x19},
+ {0x366D, 0x17},
+ {0x3A41, 0x08},
+
+ {IMX274_TABLE_END, 0x00}
+};
+
+/*
+ * Vertical 2/3 subsampling binning horizontal 3 binning
+ * imx274 mode5(refer to datasheet) register configuration with
+ * 1280x720 resolution, raw10 data and mipi four lane output
+ */
+static const struct reg_8 imx274_mode5_1280x720_raw10[] = {
+ {0x3004, 0x03},
+ {0x3005, 0x31},
+ {0x3006, 0x00},
+ {0x3007, 0x09},
+
+ {0x3018, 0xA2}, /* output XVS, HVS */
+
+ {0x306B, 0x05},
+ {0x30E2, 0x03},
+
+ {0x30F6, 0x04}, /* HMAX, 260 */
+ {0x30F7, 0x01}, /* HMAX */
+
+ {0x30DD, 0x01},
+ {0x30DE, 0x07},
+ {0x30DF, 0x00},
+ {0x40E0, 0x04},
+ {0x30E1, 0x00},
+ {0x3030, 0xD4},
+ {0x3031, 0x02},
+ {0x3032, 0xD0},
+ {0x3033, 0x02},
+
+ {0x30EE, 0x01},
+ {0x3130, 0xE2},
+ {0x3131, 0x02},
+ {0x3132, 0xDE},
+ {0x3133, 0x02},
+ {0x3342, 0x0A},
+ {0x3343, 0x00},
+ {0x3344, 0x1B},
+ {0x3345, 0x00},
+ {0x33A6, 0x01},
+ {0x3528, 0x0E},
+ {0x3554, 0x00},
+ {0x3555, 0x01},
+ {0x3556, 0x01},
+ {0x3557, 0x01},
+ {0x3558, 0x01},
+ {0x3559, 0x00},
+ {0x355A, 0x00},
+ {0x35BA, 0x0E},
+ {0x366A, 0x1B},
+ {0x366B, 0x19},
+ {0x366C, 0x17},
+ {0x366D, 0x17},
+ {0x3A41, 0x04},
+
+ {IMX274_TABLE_END, 0x00}
+};
+
+/*
+ * imx274 first step register configuration for
+ * starting stream
+ */
+static const struct reg_8 imx274_start_1[] = {
+ {IMX274_STANDBY_REG, 0x12},
+ {IMX274_TABLE_END, 0x00}
+};
+
+/*
+ * imx274 second step register configuration for
+ * starting stream
+ */
+static const struct reg_8 imx274_start_2[] = {
+ {0x3120, 0xF0}, /* clock settings */
+ {0x3121, 0x00}, /* clock settings */
+ {0x3122, 0x02}, /* clock settings */
+ {0x3129, 0x9C}, /* clock settings */
+ {0x312A, 0x02}, /* clock settings */
+ {0x312D, 0x02}, /* clock settings */
+
+ {0x310B, 0x00},
+
+ /* PLSTMG */
+ {0x304C, 0x00}, /* PLSTMG01 */
+ {0x304D, 0x03},
+ {0x331C, 0x1A},
+ {0x331D, 0x00},
+ {0x3502, 0x02},
+ {0x3529, 0x0E},
+ {0x352A, 0x0E},
+ {0x352B, 0x0E},
+ {0x3538, 0x0E},
+ {0x3539, 0x0E},
+ {0x3553, 0x00},
+ {0x357D, 0x05},
+ {0x357F, 0x05},
+ {0x3581, 0x04},
+ {0x3583, 0x76},
+ {0x3587, 0x01},
+ {0x35BB, 0x0E},
+ {0x35BC, 0x0E},
+ {0x35BD, 0x0E},
+ {0x35BE, 0x0E},
+ {0x35BF, 0x0E},
+ {0x366E, 0x00},
+ {0x366F, 0x00},
+ {0x3670, 0x00},
+ {0x3671, 0x00},
+
+ /* PSMIPI */
+ {0x3304, 0x32}, /* PSMIPI1 */
+ {0x3305, 0x00},
+ {0x3306, 0x32},
+ {0x3307, 0x00},
+ {0x3590, 0x32},
+ {0x3591, 0x00},
+ {0x3686, 0x32},
+ {0x3687, 0x00},
+
+ {IMX274_TABLE_END, 0x00}
+};
+
+/*
+ * imx274 third step register configuration for
+ * starting stream
+ */
+static const struct reg_8 imx274_start_3[] = {
+ {IMX274_STANDBY_REG, 0x00},
+ {0x303E, 0x02}, /* SYS_MODE = 2 */
+ {IMX274_TABLE_END, 0x00}
+};
+
+/*
+ * imx274 forth step register configuration for
+ * starting stream
+ */
+static const struct reg_8 imx274_start_4[] = {
+ {0x30F4, 0x00},
+ {0x3018, 0xA2}, /* XHS VHS OUTUPT */
+ {IMX274_TABLE_END, 0x00}
+};
+
+/*
+ * imx274 register configuration for stoping stream
+ */
+static const struct reg_8 imx274_stop[] = {
+ {IMX274_STANDBY_REG, 0x01},
+ {IMX274_TABLE_END, 0x00}
+};
+
+/*
+ * imx274 disable test pattern register configuration
+ */
+static const struct reg_8 imx274_tp_disabled[] = {
+ {0x303C, 0x00},
+ {0x377F, 0x00},
+ {0x3781, 0x00},
+ {0x370B, 0x00},
+ {IMX274_TABLE_END, 0x00}
+};
+
+/*
+ * imx274 test pattern register configuration
+ * reg 0x303D defines the test pattern modes
+ */
+static const struct reg_8 imx274_tp_regs[] = {
+ {0x303C, 0x11},
+ {0x370E, 0x01},
+ {0x377F, 0x01},
+ {0x3781, 0x01},
+ {0x370B, 0x11},
+ {IMX274_TABLE_END, 0x00}
+};
+
+static const struct reg_8 *mode_table[] = {
+ [IMX274_MODE_3840X2160] = imx274_mode1_3840x2160_raw10,
+ [IMX274_MODE_1920X1080] = imx274_mode3_1920x1080_raw10,
+ [IMX274_MODE_1280X720] = imx274_mode5_1280x720_raw10,
+
+ [IMX274_MODE_START_STREAM_1] = imx274_start_1,
+ [IMX274_MODE_START_STREAM_2] = imx274_start_2,
+ [IMX274_MODE_START_STREAM_3] = imx274_start_3,
+ [IMX274_MODE_START_STREAM_4] = imx274_start_4,
+ [IMX274_MODE_STOP_STREAM] = imx274_stop,
+};
+
+/*
+ * imx274 format related structure
+ */
+static const struct imx274_frmfmt imx274_formats[] = {
+ {MEDIA_BUS_FMT_SRGGB10_1X10, V4L2_COLORSPACE_SRGB, {3840, 2160},
+ IMX274_MODE_3840X2160},
+ {MEDIA_BUS_FMT_SRGGB10_1X10, V4L2_COLORSPACE_SRGB, {1920, 1080},
+ IMX274_MODE_1920X1080},
+ {MEDIA_BUS_FMT_SRGGB10_1X10, V4L2_COLORSPACE_SRGB, {1280, 720},
+ IMX274_MODE_1280X720},
+};
+
+/*
+ * minimal frame length for each mode
+ * refer to datasheet section "Frame Rate Adjustment (CSI-2)"
+ */
+static const int min_frame_len[] = {
+ 4550, /* mode 1, 4K */
+ 2310, /* mode 3, 1080p */
+ 2310 /* mode 5, 720p */
+};
+
+/*
+ * minimal numbers of SHR register
+ * refer to datasheet table "Shutter Setting (CSI-2)"
+ */
+static const int min_SHR[] = {
+ 12, /* mode 1, 4K */
+ 8, /* mode 3, 1080p */
+ 8 /* mode 5, 720p */
+};
+
+static const int max_frame_rate[] = {
+ 60, /* mode 1 , 4K */
+ 120, /* mode 3, 1080p */
+ 120 /* mode 5, 720p */
+};
+
+/*
+ * Number of clocks per internal offset period
+ * a constant based on mode
+ * refer to section "Integration Time in Each Readout Drive Mode (CSI-2)"
+ * in the datasheet
+ * for the implemented 3 modes, it happens to be the same number
+ */
+static const int nocpiop[] = {
+ 112, /* mode 1 , 4K */
+ 112, /* mode 3, 1080p */
+ 112 /* mode 5, 720p */
+};
+
+/*
+ * struct imx274_ctrls - imx274 ctrl structure
+ * @handler: V4L2 ctrl handler structure
+ * @exposure: Pointer to expsure ctrl structure
+ * @gain: Pointer to gain ctrl structure
+ * @vflip: Pointer to vflip ctrl structure
+ * @test_pattern: Pointer to test pattern ctrl structure
+ */
+struct imx274_ctrls {
+ struct v4l2_ctrl_handler handler;
+ struct v4l2_ctrl *exposure;
+ struct v4l2_ctrl *gain;
+ struct v4l2_ctrl *vflip;
+ struct v4l2_ctrl *test_pattern;
+};
+
+/*
+ * struct stim274 - imx274 device structure
+ * @sd: V4L2 subdevice structure
+ * @pd: Media pad structure
+ * @client: Pointer to I2C client
+ * @ctrls: imx274 control structure
+ * @format: V4L2 media bus frame format structure
+ * @frame_rate: V4L2 frame rate structure
+ * @regmap: Pointer to regmap structure
+ * @reset_gpio: Pointer to reset gpio
+ * @lock: Mutex structure
+ * @mode_index: Resolution mode index
+ */
+struct stimx274 {
+ struct v4l2_subdev sd;
+ struct media_pad pad;
+ struct i2c_client *client;
+ struct imx274_ctrls ctrls;
+ struct v4l2_mbus_framefmt format;
+ struct v4l2_fract frame_interval;
+ struct regmap *regmap;
+ struct gpio_desc *reset_gpio;
+ struct mutex lock; /* mutex lock for operations */
+ u32 mode_index;
+};
+
+/*
+ * Function declaration
+ */
+static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl);
+static int imx274_set_exposure(struct stimx274 *priv, int val);
+static int imx274_set_vflip(struct stimx274 *priv, int val);
+static int imx274_set_test_pattern(struct stimx274 *priv, int val);
+static int imx274_set_frame_interval(struct stimx274 *priv,
+ struct v4l2_fract frame_interval);
+
+static inline void msleep_range(unsigned int delay_base)
+{
+ usleep_range(delay_base * 1000, delay_base * 1000 + 500);
+}
+
+/*
+ * v4l2_ctrl and v4l2_subdev related operations
+ */
+static inline struct v4l2_subdev *ctrl_to_sd(struct v4l2_ctrl *ctrl)
+{
+ return &container_of(ctrl->handler,
+ struct stimx274, ctrls.handler)->sd;
+}
+
+static inline struct stimx274 *to_imx274(struct v4l2_subdev *sd)
+{
+ return container_of(sd, struct stimx274, sd);
+}
+
+/*
+ * imx274_regmap_util_write_table_8 - Function for writing register table
+ * @regmap: Pointer to device reg map structure
+ * @table: Table containing register values
+ * @wait_ms_addr: Flag for performing delay
+ * @end_addr: Flag for incating end of table
+ *
+ * This is used to write register table into sensor's reg map.
+ *
+ * Return: 0 on success, errors otherwise
+ */
+static int imx274_regmap_util_write_table_8(struct regmap *regmap,
+ const struct reg_8 table[],
+ u16 wait_ms_addr, u16 end_addr)
+{
+ int err;
+ const struct reg_8 *next;
+ u8 val;
+
+ int range_start = -1;
+ int range_count = 0;
+ u8 range_vals[16];
+ int max_range_vals = ARRAY_SIZE(range_vals);
+
+ for (next = table;; next++) {
+ if ((next->addr != range_start + range_count) ||
+ (next->addr == end_addr) ||
+ (next->addr == wait_ms_addr) ||
+ (range_count == max_range_vals)) {
+ if (range_count == 1)
+ err = regmap_write(regmap,
+ range_start, range_vals[0]);
+ else if (range_count > 1)
+ err = regmap_bulk_write(regmap, range_start,
+ &range_vals[0],
+ range_count);
+
+ if (err)
+ return err;
+
+ range_start = -1;
+ range_count = 0;
+
+ /* Handle special address values */
+ if (next->addr == end_addr)
+ break;
+
+ if (next->addr == wait_ms_addr) {
+ msleep_range(next->val);
+ continue;
+ }
+ }
+
+ val = next->val;
+
+ if (range_start == -1)
+ range_start = next->addr;
+
+ range_vals[range_count++] = val;
+ }
+ return 0;
+}
+
+static inline int imx274_read_reg(struct stimx274 *priv, u16 addr, u8 *val)
+{
+ int err;
+
+ err = regmap_read(priv->regmap, addr, (unsigned int *)val);
+ if (err)
+ dev_err(&priv->client->dev,
+ "%s : i2c read failed, addr = %x\n", __func__, addr);
+ else
+ dev_dbg(&priv->client->dev,
+ "%s : addr 0x%x, val=0x%x\n", __func__,
+ addr, *val);
+ return err;
+}
+
+static inline int imx274_write_reg(struct stimx274 *priv, u16 addr, u8 val)
+{
+ int err;
+
+ err = regmap_write(priv->regmap, addr, val);
+ if (err)
+ dev_err(&priv->client->dev,
+ "%s : i2c write failed, %x = %x\n", __func__,
+ addr, val);
+ else
+ dev_dbg(&priv->client->dev,
+ "%s : addr 0x%x, val=0x%x\n", __func__,
+ addr, val);
+ return err;
+}
+
+static int imx274_write_table(struct stimx274 *priv, const struct reg_8 table[])
+{
+ return imx274_regmap_util_write_table_8(priv->regmap,
+ table, IMX274_TABLE_WAIT_MS, IMX274_TABLE_END);
+}
+
+/*
+ * imx274_mode_regs - Function for set mode registers per mode index
+ * @priv: Pointer to device structure
+ * @mode: Mode index value
+ *
+ * This is used to start steam per mode index.
+ * mode = 0, start stream for sensor Mode 1: 4K/raw10
+ * mode = 1, start stream for sensor Mode 3: 1080p/raw10
+ * mode = 2, start stream for sensor Mode 5: 720p/raw10
+ *
+ * Return: 0 on success, errors otherwise
+ */
+static int imx274_mode_regs(struct stimx274 *priv, int mode)
+{
+ int err = 0;
+
+ err = imx274_write_table(priv, mode_table[IMX274_MODE_START_STREAM_1]);
+ if (err)
+ return err;
+
+ err = imx274_write_table(priv, mode_table[IMX274_MODE_START_STREAM_2]);
+ if (err)
+ return err;
+
+ err = imx274_write_table(priv, mode_table[mode]);
+
+ return err;
+}
+
+/*
+ * imx274_start_stream - Function for starting stream per mode index
+ * @priv: Pointer to device structure
+ *
+ * Return: 0 on success, errors otherwise
+ */
+static int imx274_start_stream(struct stimx274 *priv)
+{
+ int err = 0;
+
+ /*
+ * Refer to "Standby Cancel Sequence when using CSI-2" in
+ * imx274 datasheet, it should wait 10ms or more here.
+ * give it 1 extra ms for margin
+ */
+ msleep_range(11);
+ err = imx274_write_table(priv, mode_table[IMX274_MODE_START_STREAM_3]);
+ if (err)
+ return err;
+
+ /*
+ * Refer to "Standby Cancel Sequence when using CSI-2" in
+ * imx274 datasheet, it should wait 7ms or more here.
+ * give it 1 extra ms for margin
+ */
+ msleep_range(8);
+ err = imx274_write_table(priv, mode_table[IMX274_MODE_START_STREAM_4]);
+ if (err)
+ return err;
+
+ return 0;
+}
+
+/*
+ * imx274_reset - Function called to reset the sensor
+ * @priv: Pointer to device structure
+ * @rst: Input value for determining the sensor's end state after reset
+ *
+ * Set the senor in reset and then
+ * if rst = 0, keep it in reset;
+ * if rst = 1, bring it out of reset.
+ *
+ */
+static void imx274_reset(struct stimx274 *priv, int rst)
+{
+ gpiod_set_value_cansleep(priv->reset_gpio, 0);
+ usleep_range(IMX274_RESET_DELAY1, IMX274_RESET_DELAY2);
+ gpiod_set_value_cansleep(priv->reset_gpio, !!rst);
+ usleep_range(IMX274_RESET_DELAY1, IMX274_RESET_DELAY2);
+}
+
+/**
+ * imx274_s_ctrl - This is used to set the imx274 V4L2 controls
+ * @ctrl: V4L2 control to be set
+ *
+ * This function is used to set the V4L2 controls for the imx274 sensor.
+ *
+ * Return: 0 on success, errors otherwise
+ */
+static int imx274_s_ctrl(struct v4l2_ctrl *ctrl)
+{
+ struct v4l2_subdev *sd = ctrl_to_sd(ctrl);
+ struct stimx274 *imx274 = to_imx274(sd);
+ int ret = -EINVAL;
+
+ dev_dbg(&imx274->client->dev,
+ "%s : s_ctrl: %s, value: %d\n", __func__,
+ ctrl->name, ctrl->val);
+
+ switch (ctrl->id) {
+ case V4L2_CID_EXPOSURE:
+ dev_dbg(&imx274->client->dev,
+ "%s : set V4L2_CID_EXPOSURE\n", __func__);
+ ret = imx274_set_exposure(imx274, ctrl->val);
+ break;
+
+ case V4L2_CID_GAIN:
+ dev_dbg(&imx274->client->dev,
+ "%s : set V4L2_CID_GAIN\n", __func__);
+ ret = imx274_set_gain(imx274, ctrl);
+ break;
+
+ case V4L2_CID_VFLIP:
+ dev_dbg(&imx274->client->dev,
+ "%s : set V4L2_CID_VFLIP\n", __func__);
+ ret = imx274_set_vflip(imx274, ctrl->val);
+ break;
+
+ case V4L2_CID_TEST_PATTERN:
+ dev_dbg(&imx274->client->dev,
+ "%s : set V4L2_CID_TEST_PATTERN\n", __func__);
+ ret = imx274_set_test_pattern(imx274, ctrl->val);
+ break;
+ }
+
+ return ret;
+}
+
+/**
+ * imx274_get_fmt - Get the pad format
+ * @sd: Pointer to V4L2 Sub device structure
+ * @cfg: Pointer to sub device pad information structure
+ * @fmt: Pointer to pad level media bus format
+ *
+ * This function is used to get the pad format information.
+ *
+ * Return: 0 on success
+ */
+static int imx274_get_fmt(struct v4l2_subdev *sd,
+ struct v4l2_subdev_pad_config *cfg,
+ struct v4l2_subdev_format *fmt)
+{
+ struct stimx274 *imx274 = to_imx274(sd);
+
+ mutex_lock(&imx274->lock);
+ fmt->format = imx274->format;
+ mutex_unlock(&imx274->lock);
+ return 0;
+}
+
+/**
+ * imx274_set_fmt - This is used to set the pad format
+ * @sd: Pointer to V4L2 Sub device structure
+ * @cfg: Pointer to sub device pad information structure
+ * @format: Pointer to pad level media bus format
+ *
+ * This function is used to set the pad format.
+ *
+ * Return: 0 on success
+ */
+static int imx274_set_fmt(struct v4l2_subdev *sd,
+ struct v4l2_subdev_pad_config *cfg,
+ struct v4l2_subdev_format *format)
+{
+ struct v4l2_mbus_framefmt *fmt = &format->format;
+ struct stimx274 *imx274 = to_imx274(sd);
+ struct i2c_client *client = imx274->client;
+ int index;
+
+ dev_dbg(&client->dev,
+ "%s: width = %d height = %d code = %d mbus_code = %d\n",
+ __func__, fmt->width, fmt->height, fmt->code,
+ imx274_formats[imx274->mode_index].mbus_code);
+
+ mutex_lock(&imx274->lock);
+
+ for (index = 0; index < ARRAY_SIZE(imx274_formats); index++) {
+ if (imx274_formats[index].size.width == fmt->width &&
+ imx274_formats[index].size.height == fmt->height)
+ break;
+ }
+
+ if (index >= ARRAY_SIZE(imx274_formats)) {
+ /* default to first format */
+ index = 0;
+ }
+
+ imx274->mode_index = index;
+
+ if (fmt->width > IMX274_MAX_WIDTH)
+ fmt->width = IMX274_MAX_WIDTH;
+ if (fmt->height > IMX274_MAX_HEIGHT)
+ fmt->height = IMX274_MAX_HEIGHT;
+ fmt->width = fmt->width & (~IMX274_MASK_LSB_2_BITS);
+ fmt->height = fmt->height & (~IMX274_MASK_LSB_2_BITS);
+ fmt->field = V4L2_FIELD_NONE;
+
+ if (format->which == V4L2_SUBDEV_FORMAT_TRY)
+ cfg->try_fmt = *fmt;
+ else
+ imx274->format = *fmt;
+
+ mutex_unlock(&imx274->lock);
+ return 0;
+}
+
+/**
+ * imx274_g_frame_interval - Get the frame interval
+ * @sd: Pointer to V4L2 Sub device structure
+ * @fi: Pointer to V4l2 Sub device frame interval structure
+ *
+ * This function is used to get the frame interval.
+ *
+ * Return: 0 on success
+ */
+static int imx274_g_frame_interval(struct v4l2_subdev *sd,
+ struct v4l2_subdev_frame_interval *fi)
+{
+ struct stimx274 *imx274 = to_imx274(sd);
+
+ fi->interval = imx274->frame_interval;
+ dev_dbg(&imx274->client->dev, "%s frame rate = %d / %d\n",
+ __func__, imx274->frame_interval.numerator,
+ imx274->frame_interval.denominator);
+
+ return 0;
+}
+
+/**
+ * imx274_s_frame_interval - Set the frame interval
+ * @sd: Pointer to V4L2 Sub device structure
+ * @fi: Pointer to V4l2 Sub device frame interval structure
+ *
+ * This function is used to set the frame intervavl.
+ *
+ * Return: 0 on success
+ */
+static int imx274_s_frame_interval(struct v4l2_subdev *sd,
+ struct v4l2_subdev_frame_interval *fi)
+{
+ struct stimx274 *imx274 = to_imx274(sd);
+ struct v4l2_ctrl *ctrl = imx274->ctrls.exposure;
+ int min, max, def;
+ int ret;
+
+ mutex_lock(&imx274->lock);
+ ret = imx274_set_frame_interval(imx274, fi->interval);
+
+ if (!ret) {
+ /*
+ * exposure time range is decided by frame interval
+ * need to update it after frame interal changes
+ */
+ min = IMX274_MIN_EXPOSURE_TIME;
+ max = fi->interval.numerator * 1000000
+ / fi->interval.denominator;
+ def = max;
+ if (__v4l2_ctrl_modify_range(ctrl, min, max, 1, def)) {
+ dev_err(&imx274->client->dev,
+ "Exposure ctrl range update failed\n");
+ goto unlock;
+ }
+
+ /* update exposure time accordingly */
+ imx274_set_exposure(imx274, imx274->ctrls.exposure->val);
+
+ dev_dbg(&imx274->client->dev, "set frame interval to %uus\n",
+ fi->interval.numerator * 1000000
+ / fi->interval.denominator);
+ }
+
+unlock:
+ mutex_unlock(&imx274->lock);
+
+ return ret;
+}
+
+/**
+ * imx274_load_default - load default control values
+ * @priv: Pointer to device structure
+ *
+ * Return: 0 on success, errors otherwise
+ */
+static int imx274_load_default(struct stimx274 *priv)
+{
+ int ret;
+
+ /* load default control values */
+ priv->frame_interval.numerator = 1;
+ priv->frame_interval.denominator = IMX274_DEF_FRAME_RATE;
+ priv->ctrls.exposure->val = 1000000 / IMX274_DEF_FRAME_RATE;
+ priv->ctrls.gain->val = IMX274_DEF_GAIN;
+ priv->ctrls.vflip->val = 0;
+ priv->ctrls.test_pattern->val = TEST_PATTERN_DISABLED;
+
+ /* update frame rate */
+ ret = imx274_set_frame_interval(priv,
+ priv->frame_interval);
+ if (ret)
+ return ret;
+
+ /* update exposure time */
+ ret = v4l2_ctrl_s_ctrl(priv->ctrls.exposure, priv->ctrls.exposure->val);
+ if (ret)
+ return ret;
+
+ /* update gain */
+ ret = v4l2_ctrl_s_ctrl(priv->ctrls.gain, priv->ctrls.gain->val);
+ if (ret)
+ return ret;
+
+ /* update vflip */
+ ret = v4l2_ctrl_s_ctrl(priv->ctrls.vflip, priv->ctrls.vflip->val);
+ if (ret)
+ return ret;
+
+ return 0;
+}
+
+/**
+ * imx274_s_stream - It is used to start/stop the streaming.
+ * @sd: V4L2 Sub device
+ * @on: Flag (True / False)
+ *
+ * This function controls the start or stop of streaming for the
+ * imx274 sensor.
+ *
+ * Return: 0 on success, errors otherwise
+ */
+static int imx274_s_stream(struct v4l2_subdev *sd, int on)
+{
+ struct stimx274 *imx274 = to_imx274(sd);
+ int ret = 0;
+
+ dev_dbg(&imx274->client->dev, "%s : %s, mode index = %d\n", __func__,
+ on ? "Stream Start" : "Stream Stop", imx274->mode_index);
+
+ mutex_lock(&imx274->lock);
+
+ if (on) {
+ /* load mode registers */
+ imx274_mode_regs(imx274, imx274->mode_index);
+ if (ret)
+ goto fail;
+
+ /*
+ * update frame rate & expsoure. if the last mode is different,
+ * HMAX could be changed. As the result, frame rate & exposure
+ * are changed.
+ * gain is not affected.
+ */
+ ret = imx274_set_frame_interval(imx274,
+ imx274->frame_interval);
+ if (ret)
+ goto fail;
+
+ /* update exposure time */
+ ret = __v4l2_ctrl_s_ctrl(imx274->ctrls.exposure,
+ imx274->ctrls.exposure->val);
+ if (ret)
+ goto fail;
+
+ /* start stream */
+ ret = imx274_start_stream(imx274);
+ if (ret)
+ goto fail;
+ } else {
+ /* stop stream */
+ ret = imx274_write_table(imx274,
+ mode_table[IMX274_MODE_STOP_STREAM]);
+ if (ret)
+ goto fail;
+ }
+
+ mutex_unlock(&imx274->lock);
+ dev_dbg(&imx274->client->dev,
+ "%s : Done: mode = %d\n", __func__, imx274->mode_index);
+ return 0;
+
+fail:
+ mutex_unlock(&imx274->lock);
+ dev_err(&imx274->client->dev, "s_stream failed\n");
+ return ret;
+}
+
+/*
+ * imx274_get_frame_length - Function for obtaining current frame length
+ * @priv: Pointer to device structure
+ * @val: Pointer to obainted value
+ *
+ * frame_length = vmax x (svr + 1), in unit of hmax.
+ *
+ * Return: 0 on success
+ */
+static int imx274_get_frame_length(struct stimx274 *priv, u32 *val)
+{
+ int err;
+ u16 svr;
+ u32 vmax;
+ u8 reg_val[3];
+
+ /* svr */
+ err = imx274_read_reg(priv, IMX274_SVR_REG_LSB, ®_val[0]);
+ if (err)
+ goto fail;
+
+ err = imx274_read_reg(priv, IMX274_SVR_REG_MSB, ®_val[1]);
+ if (err)
+ goto fail;
+
+ svr = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
+
+ /* vmax */
+ err = imx274_read_reg(priv, IMX274_FRAME_LENGTH_ADDR_3, ®_val[0]);
+ if (err)
+ goto fail;
+
+ err = imx274_read_reg(priv, IMX274_FRAME_LENGTH_ADDR_2, ®_val[1]);
+ if (err)
+ goto fail;
+
+ err = imx274_read_reg(priv, IMX274_FRAME_LENGTH_ADDR_1, ®_val[2]);
+ if (err)
+ goto fail;
+
+ vmax = ((reg_val[2] & IMX274_MASK_LSB_3_BITS) << IMX274_SHIFT_16_BITS)
+ + (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
+
+ *val = vmax * (svr + 1);
+
+ return 0;
+
+fail:
+ dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
+ return err;
+}
+
+static int imx274_clamp_coarse_time(struct stimx274 *priv, u32 *val,
+ u32 *frame_length)
+{
+ int err;
+
+ err = imx274_get_frame_length(priv, frame_length);
+ if (err)
+ return err;
+
+ if (*frame_length < min_frame_len[priv->mode_index])
+ *frame_length = min_frame_len[priv->mode_index];
+
+ *val = *frame_length - *val; /* convert to raw shr */
+ if (*val > *frame_length - IMX274_SHR_LIMIT_CONST)
+ *val = *frame_length - IMX274_SHR_LIMIT_CONST;
+ else if (*val < min_SHR[priv->mode_index])
+ *val = min_SHR[priv->mode_index];
+
+ return 0;
+}
+
+/*
+ * imx274_set_digital gain - Function called when setting digital gain
+ * @priv: Pointer to device structure
+ * @dgain: Value of digital gain.
+ *
+ * Digital gain has only 4 steps: 1x, 2x, 4x, and 8x
+ *
+ * Return: 0 on success
+ */
+static int imx274_set_digital_gain(struct stimx274 *priv, u32 dgain)
+{
+ u8 reg_val;
+
+ reg_val = ffs(dgain);
+
+ if (reg_val)
+ reg_val--;
+
+ reg_val = clamp(reg_val, (u8)0, (u8)3);
+
+ return imx274_write_reg(priv, IMX274_DIGITAL_GAIN_REG,
+ reg_val & IMX274_MASK_LSB_4_BITS);
+}
+
+static inline void imx274_calculate_gain_regs(struct reg_8 regs[2], u16 gain)
+{
+ regs->addr = IMX274_ANALOG_GAIN_ADDR_MSB;
+ regs->val = (gain >> IMX274_SHIFT_8_BITS) & IMX274_MASK_LSB_3_BITS;
+
+ (regs + 1)->addr = IMX274_ANALOG_GAIN_ADDR_LSB;
+ (regs + 1)->val = (gain) & IMX274_MASK_LSB_8_BITS;
+}
+
+/*
+ * imx274_set_gain - Function called when setting gain
+ * @priv: Pointer to device structure
+ * @val: Value of gain. the real value = val << IMX274_GAIN_SHIFT;
+ * @ctrl: v4l2 control pointer
+ *
+ * Set the gain based on input value.
+ * The caller should hold the mutex lock imx274->lock if necessary
+ *
+ * Return: 0 on success
+ */
+static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl)
+{
+ struct reg_8 reg_list[2];
+ int err;
+ u32 gain, analog_gain, digital_gain, gain_reg;
+ int i;
+
+ gain = (u32)(ctrl->val);
+
+ dev_dbg(&priv->client->dev,
+ "%s : input gain = %d.%d\n", __func__,
+ gain >> IMX274_GAIN_SHIFT,
+ ((gain & IMX274_GAIN_SHIFT_MASK) * 100) >> IMX274_GAIN_SHIFT);
+
+ if (gain > IMX274_MAX_DIGITAL_GAIN * IMX274_MAX_ANALOG_GAIN)
+ gain = IMX274_MAX_DIGITAL_GAIN * IMX274_MAX_ANALOG_GAIN;
+ else if (gain < IMX274_MIN_GAIN)
+ gain = IMX274_MIN_GAIN;
+
+ if (gain <= IMX274_MAX_ANALOG_GAIN)
+ digital_gain = 1;
+ else if (gain <= IMX274_MAX_ANALOG_GAIN * 2)
+ digital_gain = 2;
+ else if (gain <= IMX274_MAX_ANALOG_GAIN * 4)
+ digital_gain = 4;
+ else
+ digital_gain = IMX274_MAX_DIGITAL_GAIN;
+
+ analog_gain = gain / digital_gain;
+
+ dev_dbg(&priv->client->dev,
+ "%s : digital gain = %d, analog gain = %d.%d\n",
+ __func__, digital_gain, analog_gain >> IMX274_GAIN_SHIFT,
+ ((analog_gain & IMX274_GAIN_SHIFT_MASK) * 100)
+ >> IMX274_GAIN_SHIFT);
+
+ err = imx274_set_digital_gain(priv, digital_gain);
+ if (err)
+ goto fail;
+
+ /* convert to register value, refer to imx274 datasheet */
+ gain_reg = (u32)IMX274_GAIN_CONST -
+ (IMX274_GAIN_CONST << IMX274_GAIN_SHIFT) / analog_gain;
+ if (gain_reg > IMX274_GAIN_REG_MAX)
+ gain_reg = IMX274_GAIN_REG_MAX;
+
+ imx274_calculate_gain_regs(reg_list, (u16)gain_reg);
+
+ for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
+ err = imx274_write_reg(priv, reg_list[i].addr,
+ reg_list[i].val);
+ if (err)
+ goto fail;
+ }
+
+ if (IMX274_GAIN_CONST - gain_reg == 0) {
+ err = -EINVAL;
+ goto fail;
+ }
+
+ /* convert register value back to gain value */
+ ctrl->val = (IMX274_GAIN_CONST << IMX274_GAIN_SHIFT)
+ / (IMX274_GAIN_CONST - gain_reg) * digital_gain;
+
+ dev_dbg(&priv->client->dev,
+ "%s : GAIN control success, gain_reg = %d, new gain = %d\n",
+ __func__, gain_reg, ctrl->val);
+
+ return 0;
+
+fail:
+ dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
+ return err;
+}
+
+static inline void imx274_calculate_coarse_time_regs(struct reg_8 regs[2],
+ u32 coarse_time)
+{
+ regs->addr = IMX274_COARSE_TIME_ADDR_MSB;
+ regs->val = (coarse_time >> IMX274_SHIFT_8_BITS)
+ & IMX274_MASK_LSB_8_BITS;
+ (regs + 1)->addr = IMX274_COARSE_TIME_ADDR_LSB;
+ (regs + 1)->val = (coarse_time) & IMX274_MASK_LSB_8_BITS;
+}
+
+/*
+ * imx274_set_coarse_time - Function called when setting SHR value
+ * @priv: Pointer to device structure
+ * @val: Value for exposure time in number of line_length, or [HMAX]
+ *
+ * Set SHR value based on input value.
+ *
+ * Return: 0 on success
+ */
+static int imx274_set_coarse_time(struct stimx274 *priv, u32 *val)
+{
+ struct reg_8 reg_list[2];
+ int err;
+ u32 coarse_time, frame_length;
+ int i;
+
+ coarse_time = *val;
+
+ /* convert exposure_time to appropriate SHR value */
+ err = imx274_clamp_coarse_time(priv, &coarse_time, &frame_length);
+ if (err)
+ goto fail;
+
+ /* prepare SHR registers */
+ imx274_calculate_coarse_time_regs(reg_list, coarse_time);
+
+ /* write to SHR registers */
+ for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
+ err = imx274_write_reg(priv, reg_list[i].addr,
+ reg_list[i].val);
+ if (err)
+ goto fail;
+ }
+
+ *val = frame_length - coarse_time;
+ return 0;
+
+fail:
+ dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
+ return err;
+}
+
+/*
+ * imx274_set_exposure - Function called when setting exposure time
+ * @priv: Pointer to device structure
+ * @val: Variable for exposure time, in the unit of micro-second
+ *
+ * Set exposure time based on input value.
+ * The caller should hold the mutex lock imx274->lock if necessary
+ *
+ * Return: 0 on success
+ */
+static int imx274_set_exposure(struct stimx274 *priv, int val)
+{
+ int err;
+ u16 hmax;
+ u8 reg_val[2];
+ u32 coarse_time; /* exposure time in unit of line (HMAX)*/
+
+ dev_dbg(&priv->client->dev,
+ "%s : EXPOSURE control input = %d\n", __func__, val);
+
+ /* step 1: convert input exposure_time (val) into number of 1[HMAX] */
+
+ /* obtain HMAX value */
+ err = imx274_read_reg(priv, IMX274_HMAX_REG_LSB, ®_val[0]);
+ if (err)
+ goto fail;
+ err = imx274_read_reg(priv, IMX274_HMAX_REG_MSB, ®_val[1]);
+ if (err)
+ goto fail;
+ hmax = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
+ if (hmax == 0) {
+ err = -EINVAL;
+ goto fail;
+ }
+
+ coarse_time = (IMX274_PIXCLK_CONST1 / IMX274_PIXCLK_CONST2 * val
+ - nocpiop[priv->mode_index]) / hmax;
+
+ /* step 2: convert exposure_time into SHR value */
+
+ /* set SHR */
+ err = imx274_set_coarse_time(priv, &coarse_time);
+ if (err)
+ goto fail;
+
+ priv->ctrls.exposure->val =
+ (coarse_time * hmax + nocpiop[priv->mode_index])
+ / (IMX274_PIXCLK_CONST1 / IMX274_PIXCLK_CONST2);
+
+ dev_dbg(&priv->client->dev,
+ "%s : EXPOSURE control success\n", __func__);
+ return 0;
+
+fail:
+ dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
+
+ return err;
+}
+
+/*
+ * imx274_set_vflip - Function called when setting vertical flip
+ * @priv: Pointer to device structure
+ * @val: Value for vflip setting
+ *
+ * Set vertical flip based on input value.
+ * val = 0: normal, no vertical flip
+ * val = 1: vertical flip enabled
+ * The caller should hold the mutex lock imx274->lock if necessary
+ *
+ * Return: 0 on success
+ */
+static int imx274_set_vflip(struct stimx274 *priv, int val)
+{
+ int err;
+
+ err = imx274_write_reg(priv, IMX274_VFLIP_REG, val);
+ if (err) {
+ dev_err(&priv->client->dev, "VFILP control error\n");
+ return err;
+ }
+
+ dev_dbg(&priv->client->dev,
+ "%s : VFLIP control success\n", __func__);
+
+ return 0;
+}
+
+/*
+ * imx274_set_test_pattern - Function called when setting test pattern
+ * @priv: Pointer to device structure
+ * @val: Variable for test pattern
+ *
+ * Set to different test patterns based on input value.
+ *
+ * Return: 0 on success
+ */
+static int imx274_set_test_pattern(struct stimx274 *priv, int val)
+{
+ int err = 0;
+
+ if (val == TEST_PATTERN_DISABLED) {
+ err = imx274_write_table(priv, imx274_tp_disabled);
+ } else if (val <= TEST_PATTERN_V_COLOR_BARS) {
+ err = imx274_write_reg(priv, IMX274_TEST_PATTERN_REG, val - 1);
+ if (!err)
+ err = imx274_write_table(priv, imx274_tp_regs);
+ } else {
+ err = -EINVAL;
+ }
+
+ if (!err)
+ dev_dbg(&priv->client->dev,
+ "%s : TEST PATTERN control success\n", __func__);
+ else
+ dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
+
+ return err;
+}
+
+static inline void imx274_calculate_frame_length_regs(struct reg_8 regs[3],
+ u32 frame_length)
+{
+ regs->addr = IMX274_FRAME_LENGTH_ADDR_1;
+ regs->val = (frame_length >> IMX274_SHIFT_16_BITS)
+ & IMX274_MASK_LSB_4_BITS;
+ (regs + 1)->addr = IMX274_FRAME_LENGTH_ADDR_2;
+ (regs + 1)->val = (frame_length >> IMX274_SHIFT_8_BITS)
+ & IMX274_MASK_LSB_8_BITS;
+ (regs + 2)->addr = IMX274_FRAME_LENGTH_ADDR_3;
+ (regs + 2)->val = (frame_length) & IMX274_MASK_LSB_8_BITS;
+}
+
+/*
+ * imx274_set_frame_length - Function called when setting frame length
+ * @priv: Pointer to device structure
+ * @val: Variable for frame length (= VMAX, i.e. vertical drive period length)
+ *
+ * Set frame length based on input value.
+ *
+ * Return: 0 on success
+ */
+static int imx274_set_frame_length(struct stimx274 *priv, u32 val)
+{
+ struct reg_8 reg_list[3];
+ int err;
+ u32 frame_length;
+ int i;
+
+ dev_dbg(&priv->client->dev, "%s : input length = %d\n",
+ __func__, val);
+
+ frame_length = (u32)val;
+
+ imx274_calculate_frame_length_regs(reg_list, frame_length);
+ for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
+ err = imx274_write_reg(priv, reg_list[i].addr,
+ reg_list[i].val);
+ if (err)
+ goto fail;
+ }
+
+ return 0;
+
+fail:
+ dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
+ return err;
+}
+
+/*
+ * imx274_set_frame_interval - Function called when setting frame interval
+ * @priv: Pointer to device structure
+ * @frame_interval: Variable for frame interval
+ *
+ * Change frame interval by updating VMAX value
+ * The caller should hold the mutex lock imx274->lock if necessary
+ *
+ * Return: 0 on success
+ */
+static int imx274_set_frame_interval(struct stimx274 *priv,
+ struct v4l2_fract frame_interval)
+{
+ int err;
+ u32 frame_length, req_frame_rate;
+ u16 svr;
+ u16 hmax;
+ u8 reg_val[2];
+
+ dev_dbg(&priv->client->dev, "%s: input frame interval = %d / %d",
+ __func__, frame_interval.numerator,
+ frame_interval.denominator);
+
+ if (frame_interval.numerator == 0) {
+ err = -EINVAL;
+ goto fail;
+ }
+
+ req_frame_rate = (u32)(frame_interval.denominator
+ / frame_interval.numerator);
+
+ /* boundary check */
+ if (req_frame_rate > max_frame_rate[priv->mode_index]) {
+ frame_interval.numerator = 1;
+ frame_interval.denominator =
+ max_frame_rate[priv->mode_index];
+ } else if (req_frame_rate < IMX274_MIN_FRAME_RATE) {
+ frame_interval.numerator = 1;
+ frame_interval.denominator = IMX274_MIN_FRAME_RATE;
+ }
+
+ /*
+ * VMAX = 1/frame_rate x 72M / (SVR+1) / HMAX
+ * frame_length (i.e. VMAX) = (frame_interval) x 72M /(SVR+1) / HMAX
+ */
+
+ /* SVR */
+ err = imx274_read_reg(priv, IMX274_SVR_REG_LSB, ®_val[0]);
+ if (err)
+ goto fail;
+ err = imx274_read_reg(priv, IMX274_SVR_REG_MSB, ®_val[1]);
+ if (err)
+ goto fail;
+ svr = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
+ dev_dbg(&priv->client->dev,
+ "%s : register SVR = %d\n", __func__, svr);
+
+ /* HMAX */
+ err = imx274_read_reg(priv, IMX274_HMAX_REG_LSB, ®_val[0]);
+ if (err)
+ goto fail;
+ err = imx274_read_reg(priv, IMX274_HMAX_REG_MSB, ®_val[1]);
+ if (err)
+ goto fail;
+ hmax = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
+ dev_dbg(&priv->client->dev,
+ "%s : register HMAX = %d\n", __func__, hmax);
+
+ if (hmax == 0 || frame_interval.denominator == 0) {
+ err = -EINVAL;
+ goto fail;
+ }
+
+ frame_length = IMX274_PIXCLK_CONST1 / (svr + 1) / hmax
+ * frame_interval.numerator
+ / frame_interval.denominator;
+
+ err = imx274_set_frame_length(priv, frame_length);
+ if (err)
+ goto fail;
+
+ priv->frame_interval = frame_interval;
+ return 0;
+
+fail:
+ dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
+ return err;
+}
+
+static const struct v4l2_subdev_pad_ops imx274_pad_ops = {
+ .get_fmt = imx274_get_fmt,
+ .set_fmt = imx274_set_fmt,
+};
+
+static const struct v4l2_subdev_video_ops imx274_video_ops = {
+ .g_frame_interval = imx274_g_frame_interval,
+ .s_frame_interval = imx274_s_frame_interval,
+ .s_stream = imx274_s_stream,
+};
+
+static const struct v4l2_subdev_ops imx274_subdev_ops = {
+ .pad = &imx274_pad_ops,
+ .video = &imx274_video_ops,
+};
+
+static const struct v4l2_ctrl_ops imx274_ctrl_ops = {
+ .s_ctrl = imx274_s_ctrl,
+};
+
+static const struct of_device_id imx274_of_id_table[] = {
+ { .compatible = "sony,imx274" },
+ { }
+};
+MODULE_DEVICE_TABLE(of, imx274_of_id_table);
+
+static const struct i2c_device_id imx274_id[] = {
+ { "IMX274", 0 },
+ { }
+};
+MODULE_DEVICE_TABLE(i2c, imx274_id);
+
+static int imx274_probe(struct i2c_client *client,
+ const struct i2c_device_id *id)
+{
+ struct v4l2_subdev *sd;
+ struct stimx274 *imx274;
+ int ret;
+
+ /* initialize imx274 */
+ imx274 = devm_kzalloc(&client->dev, sizeof(*imx274), GFP_KERNEL);
+ if (!imx274)
+ return -ENOMEM;
+
+ mutex_init(&imx274->lock);
+
+ /* initialize regmap */
+ imx274->regmap = devm_regmap_init_i2c(client, &imx274_regmap_config);
+ if (IS_ERR(imx274->regmap)) {
+ dev_err(&client->dev,
+ "regmap init failed: %ld\n", PTR_ERR(imx274->regmap));
+ ret = -ENODEV;
+ goto err_regmap;
+ }
+
+ /* initialize subdevice */
+ imx274->client = client;
+ sd = &imx274->sd;
+ v4l2_i2c_subdev_init(sd, client, &imx274_subdev_ops);
+ strlcpy(sd->name, DRIVER_NAME, sizeof(sd->name));
+ sd->flags |= V4L2_SUBDEV_FL_HAS_DEVNODE | V4L2_SUBDEV_FL_HAS_EVENTS;
+
+ /* initialize subdev media pad */
+ imx274->pad.flags = MEDIA_PAD_FL_SOURCE;
+ sd->entity.function = MEDIA_ENT_F_CAM_SENSOR;
+ ret = media_entity_pads_init(&sd->entity, 1, &imx274->pad);
+ if (ret < 0) {
+ dev_err(&client->dev,
+ "%s : media entity init Failed %d\n", __func__, ret);
+ goto err_regmap;
+ }
+
+ /* initialize sensor reset gpio */
+ imx274->reset_gpio = devm_gpiod_get_optional(&client->dev, "reset",
+ GPIOD_OUT_HIGH);
+ if (IS_ERR(imx274->reset_gpio)) {
+ if (PTR_ERR(imx274->reset_gpio) != -EPROBE_DEFER)
+ dev_err(&client->dev, "Reset GPIO not setup in DT");
+ ret = PTR_ERR(imx274->reset_gpio);
+ goto err_me;
+ }
+
+ /* pull sensor out of reset */
+ imx274_reset(imx274, 1);
+
+ /* initialize controls */
+ ret = v4l2_ctrl_handler_init(&imx274->ctrls.handler, 2);
+ if (ret < 0) {
+ dev_err(&client->dev,
+ "%s : ctrl handler init Failed\n", __func__);
+ goto err_me;
+ }
+
+ imx274->ctrls.handler.lock = &imx274->lock;
+
+ /* add new controls */
+ imx274->ctrls.test_pattern = v4l2_ctrl_new_std_menu_items(
+ &imx274->ctrls.handler, &imx274_ctrl_ops,
+ V4L2_CID_TEST_PATTERN,
+ ARRAY_SIZE(tp_qmenu) - 1, 0, 0, tp_qmenu);
+
+ imx274->ctrls.gain = v4l2_ctrl_new_std(
+ &imx274->ctrls.handler,
+ &imx274_ctrl_ops,
+ V4L2_CID_GAIN, IMX274_MIN_GAIN,
+ IMX274_MAX_DIGITAL_GAIN * IMX274_MAX_ANALOG_GAIN, 1,
+ IMX274_DEF_GAIN);
+
+ imx274->ctrls.exposure = v4l2_ctrl_new_std(
+ &imx274->ctrls.handler,
+ &imx274_ctrl_ops,
+ V4L2_CID_EXPOSURE, IMX274_MIN_EXPOSURE_TIME,
+ 1000000 / IMX274_DEF_FRAME_RATE, 1,
+ IMX274_MIN_EXPOSURE_TIME);
+
+ imx274->ctrls.vflip = v4l2_ctrl_new_std(
+ &imx274->ctrls.handler,
+ &imx274_ctrl_ops,
+ V4L2_CID_VFLIP, 0, 1, 1, 0);
+
+ imx274->sd.ctrl_handler = &imx274->ctrls.handler;
+ if (imx274->ctrls.handler.error) {
+ ret = imx274->ctrls.handler.error;
+ goto err_ctrls;
+ }
+
+ /* setup default controls */
+ ret = v4l2_ctrl_handler_setup(&imx274->ctrls.handler);
+ if (ret) {
+ dev_err(&client->dev,
+ "Error %d setup default controls\n", ret);
+ goto err_ctrls;
+ }
+
+ /* initialize format */
+ imx274->mode_index = IMX274_MODE_3840X2160;
+ imx274->format.width = imx274_formats[0].size.width;
+ imx274->format.height = imx274_formats[0].size.height;
+ imx274->format.field = V4L2_FIELD_NONE;
+ imx274->format.code = MEDIA_BUS_FMT_SRGGB10_1X10;
+ imx274->format.colorspace = V4L2_COLORSPACE_SRGB;
+ imx274->frame_interval.numerator = 1;
+ imx274->frame_interval.denominator = IMX274_DEF_FRAME_RATE;
+
+ /* load default control values */
+ ret = imx274_load_default(imx274);
+ if (ret) {
+ dev_err(&client->dev,
+ "%s : imx274_load_default failed %d\n",
+ __func__, ret);
+ goto err_ctrls;
+ }
+
+ /* register subdevice */
+ ret = v4l2_async_register_subdev(sd);
+ if (ret < 0) {
+ dev_err(&client->dev,
+ "%s : v4l2_async_register_subdev failed %d\n",
+ __func__, ret);
+ goto err_ctrls;
+ }
+
+ dev_info(&client->dev, "imx274 : imx274 probe success !\n");
+ return 0;
+
+err_ctrls:
+ v4l2_ctrl_handler_free(&imx274->ctrls.handler);
+err_me:
+ media_entity_cleanup(&sd->entity);
+err_regmap:
+ mutex_destroy(&imx274->lock);
+ return ret;
+}
+
+static int imx274_remove(struct i2c_client *client)
+{
+ struct v4l2_subdev *sd = i2c_get_clientdata(client);
+ struct stimx274 *imx274 = to_imx274(sd);
+
+ /* stop stream */
+ imx274_write_table(imx274, mode_table[IMX274_MODE_STOP_STREAM]);
+
+ v4l2_async_unregister_subdev(sd);
+ v4l2_ctrl_handler_free(&imx274->ctrls.handler);
+ media_entity_cleanup(&sd->entity);
+ mutex_destroy(&imx274->lock);
+ return 0;
+}
+
+static struct i2c_driver imx274_i2c_driver = {
+ .driver = {
+ .name = DRIVER_NAME,
+ .of_match_table = imx274_of_id_table,
+ },
+ .probe = imx274_probe,
+ .remove = imx274_remove,
+ .id_table = imx274_id,
+};
+
+module_i2c_driver(imx274_i2c_driver);
+
+MODULE_AUTHOR("Leon Luo <leonl@leopardimaging.com>");
+MODULE_DESCRIPTION("IMX274 CMOS Image Sensor driver");
+MODULE_LICENSE("GPL v2");
--
2.14.0.rc1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* RE: [PATCH v9 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor
2017-10-26 6:51 ` [PATCH v9 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor Leon Luo
@ 2017-11-03 4:36 ` Vishal Sagar
[not found] ` <DM2PR0201MB0685547EBAD0AF883C4BADD9A75D0-mGsVAZgi3o+bjIxu8aGtFxrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
0 siblings, 1 reply; 4+ messages in thread
From: Vishal Sagar @ 2017-11-03 4:36 UTC (permalink / raw)
To: Leon Luo, mchehab@kernel.org, robh+dt@kernel.org,
mark.rutland@arm.com, hans.verkuil@cisco.com,
sakari.ailus@linux.intel.com
Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, Soren Brinkmann
Hi Leon,
I understand this fixes correctly freeing the v4l control handlers in probe().
But if there is a scenario where the sensor is mounted on a removable daughter card,
shouldn't the probe fail if the daughter card is not connected?
A sample read/write to an IMX274 register should be sufficient to confirm this in the probe() and fail.
Does it make sense to add this in the probe()?
Regards
Vishal Sagar
> -----Original Message-----
> From: linux-media-owner@vger.kernel.org [mailto:linux-media-
> owner@vger.kernel.org] On Behalf Of Leon Luo
> Sent: Thursday, October 26, 2017 12:21 PM
> To: mchehab@kernel.org; robh+dt@kernel.org; mark.rutland@arm.com;
> hans.verkuil@cisco.com; sakari.ailus@linux.intel.com
> Cc: linux-media@vger.kernel.org; devicetree@vger.kernel.org; linux-
> kernel@vger.kernel.org; leonl@leopardimaging.com; Soren Brinkmann
> <sorenb@xlnx.xilinx.com>
> Subject: [PATCH v9 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor
>
> The imx274 is a Sony CMOS image sensor that has 1/2.5 image size.
> It supports up to 3840x2160 (4K) 60fps, 1080p 120fps. The interface
> is 4-lane MIPI CSI-2 running at 1.44Gbps each.
>
> This driver has been tested on Xilinx ZCU102 platform with a Leopard
> LI-IMX274MIPI-FMC camera board.
>
> Support for the following features:
> -Resolutions: 3840x2160, 1920x1080, 1280x720
> -Frame rate: 3840x2160 : 5 – 60fps
> 1920x1080 : 5 – 120fps
> 1280x720 : 5 – 120fps
> -Exposure time: 16 – (frame interval) micro-seconds
> -Gain: 1x - 180x
> -VFLIP: enable/disabledrivers/media/i2c/imx274.c
> -Test pattern: 12 test patterns
>
> Signed-off-by: Leon Luo <leonl@leopardimaging.com>
> Tested-by: Sören Brinkmann <soren.brinkmann@xilinx.com>
> Acked-by: Sakari Ailus <sakari.ailus@iki.fi>
> Acked-by: Chris Kohn <ckohn@xilinx.com>
> ---
> v9:
> - remove v4l2_async_unregister_subdev from probe error
> v8:
> - check division by zero error
> v7:
> - use __v4l2_ctrl_s_ctrl instead of v4l2_ctrl_s_ctrl to have
> clean mutex_lock/mutex_unlock in imx274_s_stream()
> - define imx274_tp_regs[] as static, move the test pattern reg
> out of imx274_tp_regs[], and define it as a macro
> IMX274_TEST_PATTERN_REG
> v6:
> - remove media/v4l2-image-sizes.h from include header
> - make the header file alphabetical order
> - remove fmt->pad check in imx274_get_fmt,
> the V4L2 subdev framework does it already
> - change 'struct reg_8 *regs' to 'struct reg_8 regs[n]',
> where n is the exact numbers needed for this function
> - move MODULE_DEVICE_TABLE(of, imx274_of_id_table); closer
> to imx274_of_id_table definition
> - remove return check of imx274_write_table in imx274_remove,
> because it should remove all resources even i2c fails here
> - move imx274_load_default before v4l2_async_register_subdev
> v5:
> - no changes
> v4:
> - use 32-bit data type to avoid __divdi3 compile error for i386
> - clean up OR together error codesdrivers/media/i2c/imx274.c
> v3:
> - clean up header files
> - use struct directly instead of alias #define
> - use v4l2_ctrl_s_ctrl instead of v4l2_s_ctrl
> - revise debug output
> - move static helpers closer to their call site
> - don't OR toegether error codes
> - use closest valid gain value instead of erroring out
> - assigne lock to the control handler and omit explicit locking
> - acquire mutex lock for imx274_get_fmt
> - remove format->pad check in imx274_set_fmt since the pad is always 0
> - pass struct v4l2_ctrl pointer in gain/exposure/vlip/test pattern controls
> - remove priv->ctrls.vflip->val = val in imx274_set_vflip()
> - remove priv->ctrls.test_pattern->val = val in imx274_set_test_pattern()
> - remove empty open/close callbacks
> - remove empty core ops
> - add more error labels in probe
> - use v4l2_async_unregister_subdev instead of v4l2_device_unregister_subdev
> - use dynamic debug
> - split start_stream to two steps: imx274_mode_regs() and
> imx274_start_stream()
> frame rate & exposure can be updated
> between imx274_mode_regs() & imx274_start_stream()
>
> v2:
> - Fix Kconfig to not remove existing options
> ---
> drivers/media/i2c/Kconfig | 7 +
> drivers/media/i2c/Makefile | 1 +
> drivers/media/i2c/imx274.c | 1810
> ++++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 1818 insertions(+)
> create mode 100644 drivers/media/i2c/imx274.c
>
> diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig
> index 47113774a297..9659849e33a0 100644
> --- a/drivers/media/i2c/Kconfig
> +++ b/drivers/media/i2c/Kconfig
> @@ -555,6 +555,13 @@ config VIDEO_APTINA_PLL
> config VIDEO_SMIAPP_PLL
> tristate
>
> +config VIDEO_IMX274
> + tristate "Sony IMX274 sensor support"
> + depends on I2C && VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API
> + ---help---
> + This is a V4L2 sensor-level driver for the Sony IMX274
> + CMOS image sensor.
> +
> config VIDEO_OV2640
> tristate "OmniVision OV2640 sensor support"
> depends on VIDEO_V4L2 && I2C
> diff --git a/drivers/media/i2c/Makefile b/drivers/media/i2c/Makefile
> index c843c181dfb9..f8d57e453936 100644
> --- a/drivers/media/i2c/Makefile
> +++ b/drivers/media/i2c/Makefile
> @@ -92,5 +92,6 @@ obj-$(CONFIG_VIDEO_IR_I2C) += ir-kbd-i2c.o
> obj-$(CONFIG_VIDEO_ML86V7667) += ml86v7667.o
> obj-$(CONFIG_VIDEO_OV2659) += ov2659.o
> obj-$(CONFIG_VIDEO_TC358743) += tc358743.o
> +obj-$(CONFIG_VIDEO_IMX274) += imx274.o
>
> obj-$(CONFIG_SDR_MAX2175) += max2175.o
> diff --git a/drivers/media/i2c/imx274.c b/drivers/media/i2c/imx274.c
> new file mode 100644
> index 000000000000..737dbf59a0d2
> --- /dev/null
> +++ b/drivers/media/i2c/imx274.c
> @@ -0,0 +1,1810 @@
> +/*
> + * imx274.c - IMX274 CMOS Image Sensor driver
> + *
> + * Copyright (C) 2017, Leopard Imaging, Inc.
> + *
> + * Leon Luo <leonl@leopardimaging.com>
> + * Edwin Zou <edwinz@leopardimaging.com>
> + *
> + * This program is free software; you can redistribute it and/or modify it
> + * under the terms and conditions of the GNU General Public License,
> + * version 2, as published by the Free Software Foundation.
> + *
> + * This program is distributed in the hope it will be useful, but WITHOUT
> + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
> or
> + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
> for
> + * more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program. If not, see <http://www.gnu.org/licenses/>.
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/delay.h>
> +#include <linux/gpio.h>
> +#include <linux/gpio/consumer.h>
> +#include <linux/i2c.h>
> +#include <linux/init.h>
> +#include <linux/module.h>
> +#include <linux/of_gpio.h>
> +#include <linux/regmap.h>
> +#include <linux/slab.h>
> +#include <linux/v4l2-mediabus.h>
> +#include <linux/videodev2.h>
> +
> +#include <media/v4l2-ctrls.h>
> +#include <media/v4l2-device.h>
> +#include <media/v4l2-subdev.h>
> +
> +/*
> + * See "SHR, SVR Setting" in datasheet
> + */
> +#define IMX274_DEFAULT_FRAME_LENGTH (4550)
> +#define IMX274_MAX_FRAME_LENGTH (0x000fffff)
> +
> +/*
> + * See "Frame Rate Adjustment" in datasheet
> + */
> +#define IMX274_PIXCLK_CONST1 (72000000)
> +#define IMX274_PIXCLK_CONST2 (1000000)
> +
> +/*
> + * The input gain is shifted by IMX274_GAIN_SHIFT to get
> + * decimal number. The real gain is
> + * (float)input_gain_value / (1 << IMX274_GAIN_SHIFT)
> + */
> +#define IMX274_GAIN_SHIFT (8)
> +#define IMX274_GAIN_SHIFT_MASK ((1 <<
> IMX274_GAIN_SHIFT) - 1)
> +
> +/*
> + * See "Analog Gain" and "Digital Gain" in datasheet
> + * min gain is 1X
> + * max gain is calculated based on IMX274_GAIN_REG_MAX
> + */
> +#define IMX274_GAIN_REG_MAX (1957)
> +#define IMX274_MIN_GAIN (0x01 <<
> IMX274_GAIN_SHIFT)
> +#define IMX274_MAX_ANALOG_GAIN ((2048 <<
> IMX274_GAIN_SHIFT)\
> + / (2048 - IMX274_GAIN_REG_MAX))
> +#define IMX274_MAX_DIGITAL_GAIN (8)
> +#define IMX274_DEF_GAIN (20 <<
> IMX274_GAIN_SHIFT)
> +#define IMX274_GAIN_CONST (2048) /* for gain formula */
> +
> +/*
> + * 1 line time in us = (HMAX / 72), minimal is 4 lines
> + */
> +#define IMX274_MIN_EXPOSURE_TIME (4 * 260 / 72)
> +
> +#define IMX274_DEFAULT_MODE
> IMX274_MODE_3840X2160
> +#define IMX274_MAX_WIDTH (3840)
> +#define IMX274_MAX_HEIGHT (2160)
> +#define IMX274_MAX_FRAME_RATE (120)
> +#define IMX274_MIN_FRAME_RATE (5)
> +#define IMX274_DEF_FRAME_RATE (60)
> +
> +/*
> + * register SHR is limited to (SVR value + 1) x VMAX value - 4
> + */
> +#define IMX274_SHR_LIMIT_CONST (4)
> +
> +/*
> + * Constants for sensor reset delay
> + */
> +#define IMX274_RESET_DELAY1 (2000)
> +#define IMX274_RESET_DELAY2 (2200)
> +
> +/*
> + * shift and mask constants
> + */
> +#define IMX274_SHIFT_8_BITS (8)
> +#define IMX274_SHIFT_16_BITS (16)
> +#define IMX274_MASK_LSB_2_BITS (0x03)
> +#define IMX274_MASK_LSB_3_BITS (0x07)
> +#define IMX274_MASK_LSB_4_BITS (0x0f)
> +#define IMX274_MASK_LSB_8_BITS (0x00ff)
> +
> +#define DRIVER_NAME "IMX274"
> +
> +/*
> + * IMX274 register definitions
> + */
> +#define IMX274_FRAME_LENGTH_ADDR_1 0x30FA /* VMAX, MSB
> */
> +#define IMX274_FRAME_LENGTH_ADDR_2 0x30F9 /* VMAX */
> +#define IMX274_FRAME_LENGTH_ADDR_3 0x30F8 /* VMAX, LSB
> */
> +#define IMX274_SVR_REG_MSB 0x300F /* SVR */
> +#define IMX274_SVR_REG_LSB 0x300E /* SVR */
> +#define IMX274_HMAX_REG_MSB 0x30F7 /* HMAX */
> +#define IMX274_HMAX_REG_LSB 0x30F6 /* HMAX */
> +#define IMX274_COARSE_TIME_ADDR_MSB 0x300D /* SHR */
> +#define IMX274_COARSE_TIME_ADDR_LSB 0x300C /* SHR */
> +#define IMX274_ANALOG_GAIN_ADDR_LSB 0x300A /* ANALOG
> GAIN LSB */
> +#define IMX274_ANALOG_GAIN_ADDR_MSB 0x300B /* ANALOG
> GAIN MSB */
> +#define IMX274_DIGITAL_GAIN_REG 0x3012 /* Digital Gain
> */
> +#define IMX274_VFLIP_REG 0x301A /* VERTICAL FLIP */
> +#define IMX274_TEST_PATTERN_REG 0x303D /* TEST
> PATTERN */
> +#define IMX274_STANDBY_REG 0x3000 /* STANDBY */
> +
> +#define IMX274_TABLE_WAIT_MS 0
> +#define IMX274_TABLE_END 1
> +
> +/*
> + * imx274 I2C operation related structure
> + */
> +struct reg_8 {
> + u16 addr;
> + u8 val;
> +};
> +
> +static const struct regmap_config imx274_regmap_config = {
> + .reg_bits = 16,
> + .val_bits = 8,
> + .cache_type = REGCACHE_RBTREE,
> +};
> +
> +enum imx274_mode {
> + IMX274_MODE_3840X2160,
> + IMX274_MODE_1920X1080,
> + IMX274_MODE_1280X720,
> +
> + IMX274_MODE_START_STREAM_1,
> + IMX274_MODE_START_STREAM_2,
> + IMX274_MODE_START_STREAM_3,
> + IMX274_MODE_START_STREAM_4,
> + IMX274_MODE_STOP_STREAM
> +};
> +
> +/*
> + * imx274 format related structure
> + */
> +struct imx274_frmfmt {
> + u32 mbus_code;
> + enum v4l2_colorspace colorspace;
> + struct v4l2_frmsize_discrete size;
> + enum imx274_mode mode;
> +};
> +
> +/*
> + * imx274 test pattern related structure
> + */
> +enum {
> + TEST_PATTERN_DISABLED = 0,
> + TEST_PATTERN_ALL_000H,
> + TEST_PATTERN_ALL_FFFH,
> + TEST_PATTERN_ALL_555H,
> + TEST_PATTERN_ALL_AAAH,
> + TEST_PATTERN_VSP_5AH, /* VERTICAL STRIPE PATTERN 555H/AAAH */
> + TEST_PATTERN_VSP_A5H, /* VERTICAL STRIPE PATTERN AAAH/555H */
> + TEST_PATTERN_VSP_05H, /* VERTICAL STRIPE PATTERN 000H/555H */
> + TEST_PATTERN_VSP_50H, /* VERTICAL STRIPE PATTERN 555H/000H */
> + TEST_PATTERN_VSP_0FH, /* VERTICAL STRIPE PATTERN 000H/FFFH */
> + TEST_PATTERN_VSP_F0H, /* VERTICAL STRIPE PATTERN FFFH/000H */
> + TEST_PATTERN_H_COLOR_BARS,
> + TEST_PATTERN_V_COLOR_BARS,
> +};
> +
> +static const char * const tp_qmenu[] = {
> + "Disabled",
> + "All 000h Pattern",
> + "All FFFh Pattern",
> + "All 555h Pattern",
> + "All AAAh Pattern",
> + "Vertical Stripe (555h / AAAh)",
> + "Vertical Stripe (AAAh / 555h)",
> + "Vertical Stripe (000h / 555h)",
> + "Vertical Stripe (555h / 000h)",
> + "Vertical Stripe (000h / FFFh)",
> + "Vertical Stripe (FFFh / 000h)",
> + "Horizontal Color Bars",
> + "Vertical Color Bars",
> +};
> +
> +/*
> + * All-pixel scan mode (10-bit)
> + * imx274 mode1(refer to datasheet) register configuration with
> + * 3840x2160 resolution, raw10 data and mipi four lane output
> + */
> +static const struct reg_8 imx274_mode1_3840x2160_raw10[] = {
> + {0x3004, 0x01},
> + {0x3005, 0x01},
> + {0x3006, 0x00},
> + {0x3007, 0x02},
> +
> + {0x3018, 0xA2}, /* output XVS, HVS */
> +
> + {0x306B, 0x05},
> + {0x30E2, 0x01},
> + {0x30F6, 0x07}, /* HMAX, 263 */
> + {0x30F7, 0x01}, /* HMAX */
> +
> + {0x30dd, 0x01}, /* crop to 2160 */
> + {0x30de, 0x06},
> + {0x30df, 0x00},
> + {0x30e0, 0x12},
> + {0x30e1, 0x00},
> + {0x3037, 0x01}, /* to crop to 3840 */
> + {0x3038, 0x0c},
> + {0x3039, 0x00},
> + {0x303a, 0x0c},
> + {0x303b, 0x0f},
> +
> + {0x30EE, 0x01},
> + {0x3130, 0x86},
> + {0x3131, 0x08},
> + {0x3132, 0x7E},
> + {0x3133, 0x08},
> + {0x3342, 0x0A},
> + {0x3343, 0x00},
> + {0x3344, 0x16},
> + {0x3345, 0x00},
> + {0x33A6, 0x01},
> + {0x3528, 0x0E},
> + {0x3554, 0x1F},
> + {0x3555, 0x01},
> + {0x3556, 0x01},
> + {0x3557, 0x01},
> + {0x3558, 0x01},
> + {0x3559, 0x00},
> + {0x355A, 0x00},
> + {0x35BA, 0x0E},
> + {0x366A, 0x1B},
> + {0x366B, 0x1A},
> + {0x366C, 0x19},
> + {0x366D, 0x17},
> + {0x3A41, 0x08},
> +
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +/*
> + * Horizontal/vertical 2/2-line binning
> + * (Horizontal and vertical weightedbinning, 10-bit)
> + * imx274 mode3(refer to datasheet) register configuration with
> + * 1920x1080 resolution, raw10 data and mipi four lane output
> + */
> +static const struct reg_8 imx274_mode3_1920x1080_raw10[] = {
> + {0x3004, 0x02},
> + {0x3005, 0x21},
> + {0x3006, 0x00},
> + {0x3007, 0x11},
> +
> + {0x3018, 0xA2}, /* output XVS, HVS */
> +
> + {0x306B, 0x05},
> + {0x30E2, 0x02},
> +
> + {0x30F6, 0x04}, /* HMAX, 260 */
> + {0x30F7, 0x01}, /* HMAX */
> +
> + {0x30dd, 0x01}, /* to crop to 1920x1080 */
> + {0x30de, 0x05},
> + {0x30df, 0x00},
> + {0x30e0, 0x04},
> + {0x30e1, 0x00},
> + {0x3037, 0x01},
> + {0x3038, 0x0c},
> + {0x3039, 0x00},
> + {0x303a, 0x0c},
> + {0x303b, 0x0f},
> +
> + {0x30EE, 0x01},
> + {0x3130, 0x4E},
> + {0x3131, 0x04},
> + {0x3132, 0x46},
> + {0x3133, 0x04},
> + {0x3342, 0x0A},
> + {0x3343, 0x00},
> + {0x3344, 0x1A},
> + {0x3345, 0x00},
> + {0x33A6, 0x01},
> + {0x3528, 0x0E},
> + {0x3554, 0x00},
> + {0x3555, 0x01},
> + {0x3556, 0x01},
> + {0x3557, 0x01},
> + {0x3558, 0x01},
> + {0x3559, 0x00},
> + {0x355A, 0x00},
> + {0x35BA, 0x0E},
> + {0x366A, 0x1B},
> + {0x366B, 0x1A},
> + {0x366C, 0x19},
> + {0x366D, 0x17},
> + {0x3A41, 0x08},
> +
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +/*
> + * Vertical 2/3 subsampling binning horizontal 3 binning
> + * imx274 mode5(refer to datasheet) register configuration with
> + * 1280x720 resolution, raw10 data and mipi four lane output
> + */
> +static const struct reg_8 imx274_mode5_1280x720_raw10[] = {
> + {0x3004, 0x03},
> + {0x3005, 0x31},
> + {0x3006, 0x00},
> + {0x3007, 0x09},
> +
> + {0x3018, 0xA2}, /* output XVS, HVS */
> +
> + {0x306B, 0x05},
> + {0x30E2, 0x03},
> +
> + {0x30F6, 0x04}, /* HMAX, 260 */
> + {0x30F7, 0x01}, /* HMAX */
> +
> + {0x30DD, 0x01},
> + {0x30DE, 0x07},
> + {0x30DF, 0x00},
> + {0x40E0, 0x04},
> + {0x30E1, 0x00},
> + {0x3030, 0xD4},
> + {0x3031, 0x02},
> + {0x3032, 0xD0},
> + {0x3033, 0x02},
> +
> + {0x30EE, 0x01},
> + {0x3130, 0xE2},
> + {0x3131, 0x02},
> + {0x3132, 0xDE},
> + {0x3133, 0x02},
> + {0x3342, 0x0A},
> + {0x3343, 0x00},
> + {0x3344, 0x1B},
> + {0x3345, 0x00},
> + {0x33A6, 0x01},
> + {0x3528, 0x0E},
> + {0x3554, 0x00},
> + {0x3555, 0x01},
> + {0x3556, 0x01},
> + {0x3557, 0x01},
> + {0x3558, 0x01},
> + {0x3559, 0x00},
> + {0x355A, 0x00},
> + {0x35BA, 0x0E},
> + {0x366A, 0x1B},
> + {0x366B, 0x19},
> + {0x366C, 0x17},
> + {0x366D, 0x17},
> + {0x3A41, 0x04},
> +
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +/*
> + * imx274 first step register configuration for
> + * starting stream
> + */
> +static const struct reg_8 imx274_start_1[] = {
> + {IMX274_STANDBY_REG, 0x12},
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +/*
> + * imx274 second step register configuration for
> + * starting stream
> + */
> +static const struct reg_8 imx274_start_2[] = {
> + {0x3120, 0xF0}, /* clock settings */
> + {0x3121, 0x00}, /* clock settings */
> + {0x3122, 0x02}, /* clock settings */
> + {0x3129, 0x9C}, /* clock settings */
> + {0x312A, 0x02}, /* clock settings */
> + {0x312D, 0x02}, /* clock settings */
> +
> + {0x310B, 0x00},
> +
> + /* PLSTMG */
> + {0x304C, 0x00}, /* PLSTMG01 */
> + {0x304D, 0x03},
> + {0x331C, 0x1A},
> + {0x331D, 0x00},
> + {0x3502, 0x02},
> + {0x3529, 0x0E},
> + {0x352A, 0x0E},
> + {0x352B, 0x0E},
> + {0x3538, 0x0E},
> + {0x3539, 0x0E},
> + {0x3553, 0x00},
> + {0x357D, 0x05},
> + {0x357F, 0x05},
> + {0x3581, 0x04},
> + {0x3583, 0x76},
> + {0x3587, 0x01},
> + {0x35BB, 0x0E},
> + {0x35BC, 0x0E},
> + {0x35BD, 0x0E},
> + {0x35BE, 0x0E},
> + {0x35BF, 0x0E},
> + {0x366E, 0x00},
> + {0x366F, 0x00},
> + {0x3670, 0x00},
> + {0x3671, 0x00},
> +
> + /* PSMIPI */
> + {0x3304, 0x32}, /* PSMIPI1 */
> + {0x3305, 0x00},
> + {0x3306, 0x32},
> + {0x3307, 0x00},
> + {0x3590, 0x32},
> + {0x3591, 0x00},
> + {0x3686, 0x32},
> + {0x3687, 0x00},
> +
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +/*
> + * imx274 third step register configuration for
> + * starting stream
> + */
> +static const struct reg_8 imx274_start_3[] = {
> + {IMX274_STANDBY_REG, 0x00},
> + {0x303E, 0x02}, /* SYS_MODE = 2 */
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +/*
> + * imx274 forth step register configuration for
> + * starting stream
> + */
> +static const struct reg_8 imx274_start_4[] = {
> + {0x30F4, 0x00},
> + {0x3018, 0xA2}, /* XHS VHS OUTUPT */
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +/*
> + * imx274 register configuration for stoping stream
> + */
> +static const struct reg_8 imx274_stop[] = {
> + {IMX274_STANDBY_REG, 0x01},
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +/*
> + * imx274 disable test pattern register configuration
> + */
> +static const struct reg_8 imx274_tp_disabled[] = {
> + {0x303C, 0x00},
> + {0x377F, 0x00},
> + {0x3781, 0x00},
> + {0x370B, 0x00},
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +/*
> + * imx274 test pattern register configuration
> + * reg 0x303D defines the test pattern modes
> + */
> +static const struct reg_8 imx274_tp_regs[] = {
> + {0x303C, 0x11},
> + {0x370E, 0x01},
> + {0x377F, 0x01},
> + {0x3781, 0x01},
> + {0x370B, 0x11},
> + {IMX274_TABLE_END, 0x00}
> +};
> +
> +static const struct reg_8 *mode_table[] = {
> + [IMX274_MODE_3840X2160] =
> imx274_mode1_3840x2160_raw10,
> + [IMX274_MODE_1920X1080] =
> imx274_mode3_1920x1080_raw10,
> + [IMX274_MODE_1280X720] =
> imx274_mode5_1280x720_raw10,
> +
> + [IMX274_MODE_START_STREAM_1] = imx274_start_1,
> + [IMX274_MODE_START_STREAM_2] = imx274_start_2,
> + [IMX274_MODE_START_STREAM_3] = imx274_start_3,
> + [IMX274_MODE_START_STREAM_4] = imx274_start_4,
> + [IMX274_MODE_STOP_STREAM] = imx274_stop,
> +};
> +
> +/*
> + * imx274 format related structure
> + */
> +static const struct imx274_frmfmt imx274_formats[] = {
> + {MEDIA_BUS_FMT_SRGGB10_1X10, V4L2_COLORSPACE_SRGB, {3840,
> 2160},
> + IMX274_MODE_3840X2160},
> + {MEDIA_BUS_FMT_SRGGB10_1X10, V4L2_COLORSPACE_SRGB, {1920,
> 1080},
> + IMX274_MODE_1920X1080},
> + {MEDIA_BUS_FMT_SRGGB10_1X10, V4L2_COLORSPACE_SRGB, {1280,
> 720},
> + IMX274_MODE_1280X720},
> +};
> +
> +/*
> + * minimal frame length for each mode
> + * refer to datasheet section "Frame Rate Adjustment (CSI-2)"
> + */
> +static const int min_frame_len[] = {
> + 4550, /* mode 1, 4K */
> + 2310, /* mode 3, 1080p */
> + 2310 /* mode 5, 720p */
> +};
> +
> +/*
> + * minimal numbers of SHR register
> + * refer to datasheet table "Shutter Setting (CSI-2)"
> + */
> +static const int min_SHR[] = {
> + 12, /* mode 1, 4K */
> + 8, /* mode 3, 1080p */
> + 8 /* mode 5, 720p */
> +};
> +
> +static const int max_frame_rate[] = {
> + 60, /* mode 1 , 4K */
> + 120, /* mode 3, 1080p */
> + 120 /* mode 5, 720p */
> +};
> +
> +/*
> + * Number of clocks per internal offset period
> + * a constant based on mode
> + * refer to section "Integration Time in Each Readout Drive Mode (CSI-2)"
> + * in the datasheet
> + * for the implemented 3 modes, it happens to be the same number
> + */
> +static const int nocpiop[] = {
> + 112, /* mode 1 , 4K */
> + 112, /* mode 3, 1080p */
> + 112 /* mode 5, 720p */
> +};
> +
> +/*
> + * struct imx274_ctrls - imx274 ctrl structure
> + * @handler: V4L2 ctrl handler structure
> + * @exposure: Pointer to expsure ctrl structure
> + * @gain: Pointer to gain ctrl structure
> + * @vflip: Pointer to vflip ctrl structure
> + * @test_pattern: Pointer to test pattern ctrl structure
> + */
> +struct imx274_ctrls {
> + struct v4l2_ctrl_handler handler;
> + struct v4l2_ctrl *exposure;
> + struct v4l2_ctrl *gain;
> + struct v4l2_ctrl *vflip;
> + struct v4l2_ctrl *test_pattern;
> +};
> +
> +/*
> + * struct stim274 - imx274 device structure
> + * @sd: V4L2 subdevice structure
> + * @pd: Media pad structure
> + * @client: Pointer to I2C client
> + * @ctrls: imx274 control structure
> + * @format: V4L2 media bus frame format structure
> + * @frame_rate: V4L2 frame rate structure
> + * @regmap: Pointer to regmap structure
> + * @reset_gpio: Pointer to reset gpio
> + * @lock: Mutex structure
> + * @mode_index: Resolution mode index
> + */
> +struct stimx274 {
> + struct v4l2_subdev sd;
> + struct media_pad pad;
> + struct i2c_client *client;
> + struct imx274_ctrls ctrls;
> + struct v4l2_mbus_framefmt format;
> + struct v4l2_fract frame_interval;
> + struct regmap *regmap;
> + struct gpio_desc *reset_gpio;
> + struct mutex lock; /* mutex lock for operations */
> + u32 mode_index;
> +};
> +
> +/*
> + * Function declaration
> + */
> +static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl);
> +static int imx274_set_exposure(struct stimx274 *priv, int val);
> +static int imx274_set_vflip(struct stimx274 *priv, int val);
> +static int imx274_set_test_pattern(struct stimx274 *priv, int val);
> +static int imx274_set_frame_interval(struct stimx274 *priv,
> + struct v4l2_fract frame_interval);
> +
> +static inline void msleep_range(unsigned int delay_base)
> +{
> + usleep_range(delay_base * 1000, delay_base * 1000 + 500);
> +}
> +
> +/*
> + * v4l2_ctrl and v4l2_subdev related operations
> + */
> +static inline struct v4l2_subdev *ctrl_to_sd(struct v4l2_ctrl *ctrl)
> +{
> + return &container_of(ctrl->handler,
> + struct stimx274, ctrls.handler)->sd;
> +}
> +
> +static inline struct stimx274 *to_imx274(struct v4l2_subdev *sd)
> +{
> + return container_of(sd, struct stimx274, sd);
> +}
> +
> +/*
> + * imx274_regmap_util_write_table_8 - Function for writing register table
> + * @regmap: Pointer to device reg map structure
> + * @table: Table containing register values
> + * @wait_ms_addr: Flag for performing delay
> + * @end_addr: Flag for incating end of table
> + *
> + * This is used to write register table into sensor's reg map.
> + *
> + * Return: 0 on success, errors otherwise
> + */
> +static int imx274_regmap_util_write_table_8(struct regmap *regmap,
> + const struct reg_8 table[],
> + u16 wait_ms_addr, u16 end_addr)
> +{
> + int err;
> + const struct reg_8 *next;
> + u8 val;
> +
> + int range_start = -1;
> + int range_count = 0;
> + u8 range_vals[16];
> + int max_range_vals = ARRAY_SIZE(range_vals);
> +
> + for (next = table;; next++) {
> + if ((next->addr != range_start + range_count) ||
> + (next->addr == end_addr) ||
> + (next->addr == wait_ms_addr) ||
> + (range_count == max_range_vals)) {
> + if (range_count == 1)
> + err = regmap_write(regmap,
> + range_start, range_vals[0]);
> + else if (range_count > 1)
> + err = regmap_bulk_write(regmap, range_start,
> + &range_vals[0],
> + range_count);
> +
> + if (err)
> + return err;
> +
> + range_start = -1;
> + range_count = 0;
> +
> + /* Handle special address values */
> + if (next->addr == end_addr)
> + break;
> +
> + if (next->addr == wait_ms_addr) {
> + msleep_range(next->val);
> + continue;
> + }
> + }
> +
> + val = next->val;
> +
> + if (range_start == -1)
> + range_start = next->addr;
> +
> + range_vals[range_count++] = val;
> + }
> + return 0;
> +}
> +
> +static inline int imx274_read_reg(struct stimx274 *priv, u16 addr, u8 *val)
> +{
> + int err;
> +
> + err = regmap_read(priv->regmap, addr, (unsigned int *)val);
> + if (err)
> + dev_err(&priv->client->dev,
> + "%s : i2c read failed, addr = %x\n", __func__, addr);
> + else
> + dev_dbg(&priv->client->dev,
> + "%s : addr 0x%x, val=0x%x\n", __func__,
> + addr, *val);
> + return err;
> +}
> +
> +static inline int imx274_write_reg(struct stimx274 *priv, u16 addr, u8 val)
> +{
> + int err;
> +
> + err = regmap_write(priv->regmap, addr, val);
> + if (err)
> + dev_err(&priv->client->dev,
> + "%s : i2c write failed, %x = %x\n", __func__,
> + addr, val);
> + else
> + dev_dbg(&priv->client->dev,
> + "%s : addr 0x%x, val=0x%x\n", __func__,
> + addr, val);
> + return err;
> +}
> +
> +static int imx274_write_table(struct stimx274 *priv, const struct reg_8 table[])
> +{
> + return imx274_regmap_util_write_table_8(priv->regmap,
> + table, IMX274_TABLE_WAIT_MS, IMX274_TABLE_END);
> +}
> +
> +/*
> + * imx274_mode_regs - Function for set mode registers per mode index
> + * @priv: Pointer to device structure
> + * @mode: Mode index value
> + *
> + * This is used to start steam per mode index.
> + * mode = 0, start stream for sensor Mode 1: 4K/raw10
> + * mode = 1, start stream for sensor Mode 3: 1080p/raw10
> + * mode = 2, start stream for sensor Mode 5: 720p/raw10
> + *
> + * Return: 0 on success, errors otherwise
> + */
> +static int imx274_mode_regs(struct stimx274 *priv, int mode)
> +{
> + int err = 0;
> +
> + err = imx274_write_table(priv,
> mode_table[IMX274_MODE_START_STREAM_1]);
> + if (err)
> + return err;
> +
> + err = imx274_write_table(priv,
> mode_table[IMX274_MODE_START_STREAM_2]);
> + if (err)
> + return err;
> +
> + err = imx274_write_table(priv, mode_table[mode]);
> +
> + return err;
> +}
> +
> +/*
> + * imx274_start_stream - Function for starting stream per mode index
> + * @priv: Pointer to device structure
> + *
> + * Return: 0 on success, errors otherwise
> + */
> +static int imx274_start_stream(struct stimx274 *priv)
> +{
> + int err = 0;
> +
> + /*
> + * Refer to "Standby Cancel Sequence when using CSI-2" in
> + * imx274 datasheet, it should wait 10ms or more here.
> + * give it 1 extra ms for margin
> + */
> + msleep_range(11);
> + err = imx274_write_table(priv,
> mode_table[IMX274_MODE_START_STREAM_3]);
> + if (err)
> + return err;
> +
> + /*
> + * Refer to "Standby Cancel Sequence when using CSI-2" in
> + * imx274 datasheet, it should wait 7ms or more here.
> + * give it 1 extra ms for margin
> + */
> + msleep_range(8);
> + err = imx274_write_table(priv,
> mode_table[IMX274_MODE_START_STREAM_4]);
> + if (err)
> + return err;
> +
> + return 0;
> +}
> +
> +/*
> + * imx274_reset - Function called to reset the sensor
> + * @priv: Pointer to device structure
> + * @rst: Input value for determining the sensor's end state after reset
> + *
> + * Set the senor in reset and then
> + * if rst = 0, keep it in reset;
> + * if rst = 1, bring it out of reset.
> + *
> + */
> +static void imx274_reset(struct stimx274 *priv, int rst)
> +{
> + gpiod_set_value_cansleep(priv->reset_gpio, 0);
> + usleep_range(IMX274_RESET_DELAY1, IMX274_RESET_DELAY2);
> + gpiod_set_value_cansleep(priv->reset_gpio, !!rst);
> + usleep_range(IMX274_RESET_DELAY1, IMX274_RESET_DELAY2);
> +}
> +
> +/**
> + * imx274_s_ctrl - This is used to set the imx274 V4L2 controls
> + * @ctrl: V4L2 control to be set
> + *
> + * This function is used to set the V4L2 controls for the imx274 sensor.
> + *
> + * Return: 0 on success, errors otherwise
> + */
> +static int imx274_s_ctrl(struct v4l2_ctrl *ctrl)
> +{
> + struct v4l2_subdev *sd = ctrl_to_sd(ctrl);
> + struct stimx274 *imx274 = to_imx274(sd);
> + int ret = -EINVAL;
> +
> + dev_dbg(&imx274->client->dev,
> + "%s : s_ctrl: %s, value: %d\n", __func__,
> + ctrl->name, ctrl->val);
> +
> + switch (ctrl->id) {
> + case V4L2_CID_EXPOSURE:
> + dev_dbg(&imx274->client->dev,
> + "%s : set V4L2_CID_EXPOSURE\n", __func__);
> + ret = imx274_set_exposure(imx274, ctrl->val);
> + break;
> +
> + case V4L2_CID_GAIN:
> + dev_dbg(&imx274->client->dev,
> + "%s : set V4L2_CID_GAIN\n", __func__);
> + ret = imx274_set_gain(imx274, ctrl);
> + break;
> +
> + case V4L2_CID_VFLIP:
> + dev_dbg(&imx274->client->dev,
> + "%s : set V4L2_CID_VFLIP\n", __func__);
> + ret = imx274_set_vflip(imx274, ctrl->val);
> + break;
> +
> + case V4L2_CID_TEST_PATTERN:
> + dev_dbg(&imx274->client->dev,
> + "%s : set V4L2_CID_TEST_PATTERN\n", __func__);
> + ret = imx274_set_test_pattern(imx274, ctrl->val);
> + break;
> + }
> +
> + return ret;
> +}
> +
> +/**
> + * imx274_get_fmt - Get the pad format
> + * @sd: Pointer to V4L2 Sub device structure
> + * @cfg: Pointer to sub device pad information structure
> + * @fmt: Pointer to pad level media bus format
> + *
> + * This function is used to get the pad format information.
> + *
> + * Return: 0 on success
> + */
> +static int imx274_get_fmt(struct v4l2_subdev *sd,
> + struct v4l2_subdev_pad_config *cfg,
> + struct v4l2_subdev_format *fmt)
> +{
> + struct stimx274 *imx274 = to_imx274(sd);
> +
> + mutex_lock(&imx274->lock);
> + fmt->format = imx274->format;
> + mutex_unlock(&imx274->lock);
> + return 0;
> +}
> +
> +/**
> + * imx274_set_fmt - This is used to set the pad format
> + * @sd: Pointer to V4L2 Sub device structure
> + * @cfg: Pointer to sub device pad information structure
> + * @format: Pointer to pad level media bus format
> + *
> + * This function is used to set the pad format.
> + *
> + * Return: 0 on success
> + */
> +static int imx274_set_fmt(struct v4l2_subdev *sd,
> + struct v4l2_subdev_pad_config *cfg,
> + struct v4l2_subdev_format *format)
> +{
> + struct v4l2_mbus_framefmt *fmt = &format->format;
> + struct stimx274 *imx274 = to_imx274(sd);
> + struct i2c_client *client = imx274->client;
> + int index;
> +
> + dev_dbg(&client->dev,
> + "%s: width = %d height = %d code = %d mbus_code = %d\n",
> + __func__, fmt->width, fmt->height, fmt->code,
> + imx274_formats[imx274->mode_index].mbus_code);
> +
> + mutex_lock(&imx274->lock);
> +
> + for (index = 0; index < ARRAY_SIZE(imx274_formats); index++) {
> + if (imx274_formats[index].size.width == fmt->width &&
> + imx274_formats[index].size.height == fmt->height)
> + break;
> + }
> +
> + if (index >= ARRAY_SIZE(imx274_formats)) {
> + /* default to first format */
> + index = 0;
> + }
> +
> + imx274->mode_index = index;
> +
> + if (fmt->width > IMX274_MAX_WIDTH)
> + fmt->width = IMX274_MAX_WIDTH;
> + if (fmt->height > IMX274_MAX_HEIGHT)
> + fmt->height = IMX274_MAX_HEIGHT;
> + fmt->width = fmt->width & (~IMX274_MASK_LSB_2_BITS);
> + fmt->height = fmt->height & (~IMX274_MASK_LSB_2_BITS);
> + fmt->field = V4L2_FIELD_NONE;
> +
> + if (format->which == V4L2_SUBDEV_FORMAT_TRY)
> + cfg->try_fmt = *fmt;
> + else
> + imx274->format = *fmt;
> +
> + mutex_unlock(&imx274->lock);
> + return 0;
> +}
> +
> +/**
> + * imx274_g_frame_interval - Get the frame interval
> + * @sd: Pointer to V4L2 Sub device structure
> + * @fi: Pointer to V4l2 Sub device frame interval structure
> + *
> + * This function is used to get the frame interval.
> + *
> + * Return: 0 on success
> + */
> +static int imx274_g_frame_interval(struct v4l2_subdev *sd,
> + struct v4l2_subdev_frame_interval *fi)
> +{
> + struct stimx274 *imx274 = to_imx274(sd);
> +
> + fi->interval = imx274->frame_interval;
> + dev_dbg(&imx274->client->dev, "%s frame rate = %d / %d\n",
> + __func__, imx274->frame_interval.numerator,
> + imx274->frame_interval.denominator);
> +
> + return 0;
> +}
> +
> +/**
> + * imx274_s_frame_interval - Set the frame interval
> + * @sd: Pointer to V4L2 Sub device structure
> + * @fi: Pointer to V4l2 Sub device frame interval structure
> + *
> + * This function is used to set the frame intervavl.
> + *
> + * Return: 0 on success
> + */
> +static int imx274_s_frame_interval(struct v4l2_subdev *sd,
> + struct v4l2_subdev_frame_interval *fi)
> +{
> + struct stimx274 *imx274 = to_imx274(sd);
> + struct v4l2_ctrl *ctrl = imx274->ctrls.exposure;
> + int min, max, def;
> + int ret;
> +
> + mutex_lock(&imx274->lock);
> + ret = imx274_set_frame_interval(imx274, fi->interval);
> +
> + if (!ret) {
> + /*
> + * exposure time range is decided by frame interval
> + * need to update it after frame interal changes
> + */
> + min = IMX274_MIN_EXPOSURE_TIME;
> + max = fi->interval.numerator * 1000000
> + / fi->interval.denominator;
> + def = max;
> + if (__v4l2_ctrl_modify_range(ctrl, min, max, 1, def)) {
> + dev_err(&imx274->client->dev,
> + "Exposure ctrl range update failed\n");
> + goto unlock;
> + }
> +
> + /* update exposure time accordingly */
> + imx274_set_exposure(imx274, imx274->ctrls.exposure->val);
> +
> + dev_dbg(&imx274->client->dev, "set frame interval to %uus\n",
> + fi->interval.numerator * 1000000
> + / fi->interval.denominator);
> + }
> +
> +unlock:
> + mutex_unlock(&imx274->lock);
> +
> + return ret;
> +}
> +
> +/**
> + * imx274_load_default - load default control values
> + * @priv: Pointer to device structure
> + *
> + * Return: 0 on success, errors otherwise
> + */
> +static int imx274_load_default(struct stimx274 *priv)
> +{
> + int ret;
> +
> + /* load default control values */
> + priv->frame_interval.numerator = 1;
> + priv->frame_interval.denominator = IMX274_DEF_FRAME_RATE;
> + priv->ctrls.exposure->val = 1000000 / IMX274_DEF_FRAME_RATE;
> + priv->ctrls.gain->val = IMX274_DEF_GAIN;
> + priv->ctrls.vflip->val = 0;
> + priv->ctrls.test_pattern->val = TEST_PATTERN_DISABLED;
> +
> + /* update frame rate */
> + ret = imx274_set_frame_interval(priv,
> + priv->frame_interval);
> + if (ret)
> + return ret;
> +
> + /* update exposure time */
> + ret = v4l2_ctrl_s_ctrl(priv->ctrls.exposure, priv->ctrls.exposure->val);
> + if (ret)
> + return ret;
> +
> + /* update gain */
> + ret = v4l2_ctrl_s_ctrl(priv->ctrls.gain, priv->ctrls.gain->val);
> + if (ret)
> + return ret;
> +
> + /* update vflip */
> + ret = v4l2_ctrl_s_ctrl(priv->ctrls.vflip, priv->ctrls.vflip->val);
> + if (ret)
> + return ret;
> +
> + return 0;
> +}
> +
> +/**
> + * imx274_s_stream - It is used to start/stop the streaming.
> + * @sd: V4L2 Sub device
> + * @on: Flag (True / False)
> + *
> + * This function controls the start or stop of streaming for the
> + * imx274 sensor.
> + *
> + * Return: 0 on success, errors otherwise
> + */
> +static int imx274_s_stream(struct v4l2_subdev *sd, int on)
> +{
> + struct stimx274 *imx274 = to_imx274(sd);
> + int ret = 0;
> +
> + dev_dbg(&imx274->client->dev, "%s : %s, mode index = %d\n",
> __func__,
> + on ? "Stream Start" : "Stream Stop", imx274->mode_index);
> +
> + mutex_lock(&imx274->lock);
> +
> + if (on) {
> + /* load mode registers */
> + imx274_mode_regs(imx274, imx274->mode_index);
> + if (ret)
> + goto fail;
> +
> + /*
> + * update frame rate & expsoure. if the last mode is different,
> + * HMAX could be changed. As the result, frame rate & exposure
> + * are changed.
> + * gain is not affected.
> + */
> + ret = imx274_set_frame_interval(imx274,
> + imx274->frame_interval);
> + if (ret)
> + goto fail;
> +
> + /* update exposure time */
> + ret = __v4l2_ctrl_s_ctrl(imx274->ctrls.exposure,
> + imx274->ctrls.exposure->val);
> + if (ret)
> + goto fail;
> +
> + /* start stream */
> + ret = imx274_start_stream(imx274);
> + if (ret)
> + goto fail;
> + } else {
> + /* stop stream */
> + ret = imx274_write_table(imx274,
> +
> mode_table[IMX274_MODE_STOP_STREAM]);
> + if (ret)
> + goto fail;
> + }
> +
> + mutex_unlock(&imx274->lock);
> + dev_dbg(&imx274->client->dev,
> + "%s : Done: mode = %d\n", __func__, imx274->mode_index);
> + return 0;
> +
> +fail:
> + mutex_unlock(&imx274->lock);
> + dev_err(&imx274->client->dev, "s_stream failed\n");
> + return ret;
> +}
> +
> +/*
> + * imx274_get_frame_length - Function for obtaining current frame length
> + * @priv: Pointer to device structure
> + * @val: Pointer to obainted value
> + *
> + * frame_length = vmax x (svr + 1), in unit of hmax.
> + *
> + * Return: 0 on success
> + */
> +static int imx274_get_frame_length(struct stimx274 *priv, u32 *val)
> +{
> + int err;
> + u16 svr;
> + u32 vmax;
> + u8 reg_val[3];
> +
> + /* svr */
> + err = imx274_read_reg(priv, IMX274_SVR_REG_LSB, ®_val[0]);
> + if (err)
> + goto fail;
> +
> + err = imx274_read_reg(priv, IMX274_SVR_REG_MSB, ®_val[1]);
> + if (err)
> + goto fail;
> +
> + svr = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
> +
> + /* vmax */
> + err = imx274_read_reg(priv, IMX274_FRAME_LENGTH_ADDR_3,
> ®_val[0]);
> + if (err)
> + goto fail;
> +
> + err = imx274_read_reg(priv, IMX274_FRAME_LENGTH_ADDR_2,
> ®_val[1]);
> + if (err)
> + goto fail;
> +
> + err = imx274_read_reg(priv, IMX274_FRAME_LENGTH_ADDR_1,
> ®_val[2]);
> + if (err)
> + goto fail;
> +
> + vmax = ((reg_val[2] & IMX274_MASK_LSB_3_BITS) <<
> IMX274_SHIFT_16_BITS)
> + + (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
> +
> + *val = vmax * (svr + 1);
> +
> + return 0;
> +
> +fail:
> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
> + return err;
> +}
> +
> +static int imx274_clamp_coarse_time(struct stimx274 *priv, u32 *val,
> + u32 *frame_length)
> +{
> + int err;
> +
> + err = imx274_get_frame_length(priv, frame_length);
> + if (err)
> + return err;
> +
> + if (*frame_length < min_frame_len[priv->mode_index])
> + *frame_length = min_frame_len[priv->mode_index];
> +
> + *val = *frame_length - *val; /* convert to raw shr */
> + if (*val > *frame_length - IMX274_SHR_LIMIT_CONST)
> + *val = *frame_length - IMX274_SHR_LIMIT_CONST;
> + else if (*val < min_SHR[priv->mode_index])
> + *val = min_SHR[priv->mode_index];
> +
> + return 0;
> +}
> +
> +/*
> + * imx274_set_digital gain - Function called when setting digital gain
> + * @priv: Pointer to device structure
> + * @dgain: Value of digital gain.
> + *
> + * Digital gain has only 4 steps: 1x, 2x, 4x, and 8x
> + *
> + * Return: 0 on success
> + */
> +static int imx274_set_digital_gain(struct stimx274 *priv, u32 dgain)
> +{
> + u8 reg_val;
> +
> + reg_val = ffs(dgain);
> +
> + if (reg_val)
> + reg_val--;
> +
> + reg_val = clamp(reg_val, (u8)0, (u8)3);
> +
> + return imx274_write_reg(priv, IMX274_DIGITAL_GAIN_REG,
> + reg_val & IMX274_MASK_LSB_4_BITS);
> +}
> +
> +static inline void imx274_calculate_gain_regs(struct reg_8 regs[2], u16 gain)
> +{
> + regs->addr = IMX274_ANALOG_GAIN_ADDR_MSB;
> + regs->val = (gain >> IMX274_SHIFT_8_BITS) &
> IMX274_MASK_LSB_3_BITS;
> +
> + (regs + 1)->addr = IMX274_ANALOG_GAIN_ADDR_LSB;
> + (regs + 1)->val = (gain) & IMX274_MASK_LSB_8_BITS;
> +}
> +
> +/*
> + * imx274_set_gain - Function called when setting gain
> + * @priv: Pointer to device structure
> + * @val: Value of gain. the real value = val << IMX274_GAIN_SHIFT;
> + * @ctrl: v4l2 control pointer
> + *
> + * Set the gain based on input value.
> + * The caller should hold the mutex lock imx274->lock if necessary
> + *
> + * Return: 0 on success
> + */
> +static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl)
> +{
> + struct reg_8 reg_list[2];
> + int err;
> + u32 gain, analog_gain, digital_gain, gain_reg;
> + int i;
> +
> + gain = (u32)(ctrl->val);
> +
> + dev_dbg(&priv->client->dev,
> + "%s : input gain = %d.%d\n", __func__,
> + gain >> IMX274_GAIN_SHIFT,
> + ((gain & IMX274_GAIN_SHIFT_MASK) * 100) >>
> IMX274_GAIN_SHIFT);
> +
> + if (gain > IMX274_MAX_DIGITAL_GAIN * IMX274_MAX_ANALOG_GAIN)
> + gain = IMX274_MAX_DIGITAL_GAIN *
> IMX274_MAX_ANALOG_GAIN;
> + else if (gain < IMX274_MIN_GAIN)
> + gain = IMX274_MIN_GAIN;
> +
> + if (gain <= IMX274_MAX_ANALOG_GAIN)
> + digital_gain = 1;
> + else if (gain <= IMX274_MAX_ANALOG_GAIN * 2)
> + digital_gain = 2;
> + else if (gain <= IMX274_MAX_ANALOG_GAIN * 4)
> + digital_gain = 4;
> + else
> + digital_gain = IMX274_MAX_DIGITAL_GAIN;
> +
> + analog_gain = gain / digital_gain;
> +
> + dev_dbg(&priv->client->dev,
> + "%s : digital gain = %d, analog gain = %d.%d\n",
> + __func__, digital_gain, analog_gain >> IMX274_GAIN_SHIFT,
> + ((analog_gain & IMX274_GAIN_SHIFT_MASK) * 100)
> + >> IMX274_GAIN_SHIFT);
> +
> + err = imx274_set_digital_gain(priv, digital_gain);
> + if (err)
> + goto fail;
> +
> + /* convert to register value, refer to imx274 datasheet */
> + gain_reg = (u32)IMX274_GAIN_CONST -
> + (IMX274_GAIN_CONST << IMX274_GAIN_SHIFT) / analog_gain;
> + if (gain_reg > IMX274_GAIN_REG_MAX)
> + gain_reg = IMX274_GAIN_REG_MAX;
> +
> + imx274_calculate_gain_regs(reg_list, (u16)gain_reg);
> +
> + for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
> + err = imx274_write_reg(priv, reg_list[i].addr,
> + reg_list[i].val);
> + if (err)
> + goto fail;
> + }
> +
> + if (IMX274_GAIN_CONST - gain_reg == 0) {
> + err = -EINVAL;
> + goto fail;
> + }
> +
> + /* convert register value back to gain value */
> + ctrl->val = (IMX274_GAIN_CONST << IMX274_GAIN_SHIFT)
> + / (IMX274_GAIN_CONST - gain_reg) * digital_gain;
> +
> + dev_dbg(&priv->client->dev,
> + "%s : GAIN control success, gain_reg = %d, new gain = %d\n",
> + __func__, gain_reg, ctrl->val);
> +
> + return 0;
> +
> +fail:
> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
> + return err;
> +}
> +
> +static inline void imx274_calculate_coarse_time_regs(struct reg_8 regs[2],
> + u32 coarse_time)
> +{
> + regs->addr = IMX274_COARSE_TIME_ADDR_MSB;
> + regs->val = (coarse_time >> IMX274_SHIFT_8_BITS)
> + & IMX274_MASK_LSB_8_BITS;
> + (regs + 1)->addr = IMX274_COARSE_TIME_ADDR_LSB;
> + (regs + 1)->val = (coarse_time) & IMX274_MASK_LSB_8_BITS;
> +}
> +
> +/*
> + * imx274_set_coarse_time - Function called when setting SHR value
> + * @priv: Pointer to device structure
> + * @val: Value for exposure time in number of line_length, or [HMAX]
> + *
> + * Set SHR value based on input value.
> + *
> + * Return: 0 on success
> + */
> +static int imx274_set_coarse_time(struct stimx274 *priv, u32 *val)
> +{
> + struct reg_8 reg_list[2];
> + int err;
> + u32 coarse_time, frame_length;
> + int i;
> +
> + coarse_time = *val;
> +
> + /* convert exposure_time to appropriate SHR value */
> + err = imx274_clamp_coarse_time(priv, &coarse_time, &frame_length);
> + if (err)
> + goto fail;
> +
> + /* prepare SHR registers */
> + imx274_calculate_coarse_time_regs(reg_list, coarse_time);
> +
> + /* write to SHR registers */
> + for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
> + err = imx274_write_reg(priv, reg_list[i].addr,
> + reg_list[i].val);
> + if (err)
> + goto fail;
> + }
> +
> + *val = frame_length - coarse_time;
> + return 0;
> +
> +fail:
> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
> + return err;
> +}
> +
> +/*
> + * imx274_set_exposure - Function called when setting exposure time
> + * @priv: Pointer to device structure
> + * @val: Variable for exposure time, in the unit of micro-second
> + *
> + * Set exposure time based on input value.
> + * The caller should hold the mutex lock imx274->lock if necessary
> + *
> + * Return: 0 on success
> + */
> +static int imx274_set_exposure(struct stimx274 *priv, int val)
> +{
> + int err;
> + u16 hmax;
> + u8 reg_val[2];
> + u32 coarse_time; /* exposure time in unit of line (HMAX)*/
> +
> + dev_dbg(&priv->client->dev,
> + "%s : EXPOSURE control input = %d\n", __func__, val);
> +
> + /* step 1: convert input exposure_time (val) into number of 1[HMAX] */
> +
> + /* obtain HMAX value */
> + err = imx274_read_reg(priv, IMX274_HMAX_REG_LSB, ®_val[0]);
> + if (err)
> + goto fail;
> + err = imx274_read_reg(priv, IMX274_HMAX_REG_MSB, ®_val[1]);
> + if (err)
> + goto fail;
> + hmax = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
> + if (hmax == 0) {
> + err = -EINVAL;
> + goto fail;
> + }
> +
> + coarse_time = (IMX274_PIXCLK_CONST1 / IMX274_PIXCLK_CONST2 *
> val
> + - nocpiop[priv->mode_index]) / hmax;
> +
> + /* step 2: convert exposure_time into SHR value */
> +
> + /* set SHR */
> + err = imx274_set_coarse_time(priv, &coarse_time);
> + if (err)
> + goto fail;
> +
> + priv->ctrls.exposure->val =
> + (coarse_time * hmax + nocpiop[priv->mode_index])
> + / (IMX274_PIXCLK_CONST1 /
> IMX274_PIXCLK_CONST2);
> +
> + dev_dbg(&priv->client->dev,
> + "%s : EXPOSURE control success\n", __func__);
> + return 0;
> +
> +fail:
> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
> +
> + return err;
> +}
> +
> +/*
> + * imx274_set_vflip - Function called when setting vertical flip
> + * @priv: Pointer to device structure
> + * @val: Value for vflip setting
> + *
> + * Set vertical flip based on input value.
> + * val = 0: normal, no vertical flip
> + * val = 1: vertical flip enabled
> + * The caller should hold the mutex lock imx274->lock if necessary
> + *
> + * Return: 0 on success
> + */
> +static int imx274_set_vflip(struct stimx274 *priv, int val)
> +{
> + int err;
> +
> + err = imx274_write_reg(priv, IMX274_VFLIP_REG, val);
> + if (err) {
> + dev_err(&priv->client->dev, "VFILP control error\n");
> + return err;
> + }
> +
> + dev_dbg(&priv->client->dev,
> + "%s : VFLIP control success\n", __func__);
> +
> + return 0;
> +}
> +
> +/*
> + * imx274_set_test_pattern - Function called when setting test pattern
> + * @priv: Pointer to device structure
> + * @val: Variable for test pattern
> + *
> + * Set to different test patterns based on input value.
> + *
> + * Return: 0 on success
> + */
> +static int imx274_set_test_pattern(struct stimx274 *priv, int val)
> +{
> + int err = 0;
> +
> + if (val == TEST_PATTERN_DISABLED) {
> + err = imx274_write_table(priv, imx274_tp_disabled);
> + } else if (val <= TEST_PATTERN_V_COLOR_BARS) {
> + err = imx274_write_reg(priv, IMX274_TEST_PATTERN_REG, val -
> 1);
> + if (!err)
> + err = imx274_write_table(priv, imx274_tp_regs);
> + } else {
> + err = -EINVAL;
> + }
> +
> + if (!err)
> + dev_dbg(&priv->client->dev,
> + "%s : TEST PATTERN control success\n", __func__);
> + else
> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
> +
> + return err;
> +}
> +
> +static inline void imx274_calculate_frame_length_regs(struct reg_8 regs[3],
> + u32 frame_length)
> +{
> + regs->addr = IMX274_FRAME_LENGTH_ADDR_1;
> + regs->val = (frame_length >> IMX274_SHIFT_16_BITS)
> + & IMX274_MASK_LSB_4_BITS;
> + (regs + 1)->addr = IMX274_FRAME_LENGTH_ADDR_2;
> + (regs + 1)->val = (frame_length >> IMX274_SHIFT_8_BITS)
> + & IMX274_MASK_LSB_8_BITS;
> + (regs + 2)->addr = IMX274_FRAME_LENGTH_ADDR_3;
> + (regs + 2)->val = (frame_length) & IMX274_MASK_LSB_8_BITS;
> +}
> +
> +/*
> + * imx274_set_frame_length - Function called when setting frame length
> + * @priv: Pointer to device structure
> + * @val: Variable for frame length (= VMAX, i.e. vertical drive period length)
> + *
> + * Set frame length based on input value.
> + *
> + * Return: 0 on success
> + */
> +static int imx274_set_frame_length(struct stimx274 *priv, u32 val)
> +{
> + struct reg_8 reg_list[3];
> + int err;
> + u32 frame_length;
> + int i;
> +
> + dev_dbg(&priv->client->dev, "%s : input length = %d\n",
> + __func__, val);
> +
> + frame_length = (u32)val;
> +
> + imx274_calculate_frame_length_regs(reg_list, frame_length);
> + for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
> + err = imx274_write_reg(priv, reg_list[i].addr,
> + reg_list[i].val);
> + if (err)
> + goto fail;
> + }
> +
> + return 0;
> +
> +fail:
> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
> + return err;
> +}
> +
> +/*
> + * imx274_set_frame_interval - Function called when setting frame interval
> + * @priv: Pointer to device structure
> + * @frame_interval: Variable for frame interval
> + *
> + * Change frame interval by updating VMAX value
> + * The caller should hold the mutex lock imx274->lock if necessary
> + *
> + * Return: 0 on success
> + */
> +static int imx274_set_frame_interval(struct stimx274 *priv,
> + struct v4l2_fract frame_interval)
> +{
> + int err;
> + u32 frame_length, req_frame_rate;
> + u16 svr;
> + u16 hmax;
> + u8 reg_val[2];
> +
> + dev_dbg(&priv->client->dev, "%s: input frame interval = %d / %d",
> + __func__, frame_interval.numerator,
> + frame_interval.denominator);
> +
> + if (frame_interval.numerator == 0) {
> + err = -EINVAL;
> + goto fail;
> + }
> +
> + req_frame_rate = (u32)(frame_interval.denominator
> + / frame_interval.numerator);
> +
> + /* boundary check */
> + if (req_frame_rate > max_frame_rate[priv->mode_index]) {
> + frame_interval.numerator = 1;
> + frame_interval.denominator =
> + max_frame_rate[priv->mode_index];
> + } else if (req_frame_rate < IMX274_MIN_FRAME_RATE) {
> + frame_interval.numerator = 1;
> + frame_interval.denominator = IMX274_MIN_FRAME_RATE;
> + }
> +
> + /*
> + * VMAX = 1/frame_rate x 72M / (SVR+1) / HMAX
> + * frame_length (i.e. VMAX) = (frame_interval) x 72M /(SVR+1) / HMAX
> + */
> +
> + /* SVR */
> + err = imx274_read_reg(priv, IMX274_SVR_REG_LSB, ®_val[0]);
> + if (err)
> + goto fail;
> + err = imx274_read_reg(priv, IMX274_SVR_REG_MSB, ®_val[1]);
> + if (err)
> + goto fail;
> + svr = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
> + dev_dbg(&priv->client->dev,
> + "%s : register SVR = %d\n", __func__, svr);
> +
> + /* HMAX */
> + err = imx274_read_reg(priv, IMX274_HMAX_REG_LSB, ®_val[0]);
> + if (err)
> + goto fail;
> + err = imx274_read_reg(priv, IMX274_HMAX_REG_MSB, ®_val[1]);
> + if (err)
> + goto fail;
> + hmax = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
> + dev_dbg(&priv->client->dev,
> + "%s : register HMAX = %d\n", __func__, hmax);
> +
> + if (hmax == 0 || frame_interval.denominator == 0) {
> + err = -EINVAL;
> + goto fail;
> + }
> +
> + frame_length = IMX274_PIXCLK_CONST1 / (svr + 1) / hmax
> + * frame_interval.numerator
> + / frame_interval.denominator;
> +
> + err = imx274_set_frame_length(priv, frame_length);
> + if (err)
> + goto fail;
> +
> + priv->frame_interval = frame_interval;
> + return 0;
> +
> +fail:
> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
> + return err;
> +}
> +
> +static const struct v4l2_subdev_pad_ops imx274_pad_ops = {
> + .get_fmt = imx274_get_fmt,
> + .set_fmt = imx274_set_fmt,
> +};
> +
> +static const struct v4l2_subdev_video_ops imx274_video_ops = {
> + .g_frame_interval = imx274_g_frame_interval,
> + .s_frame_interval = imx274_s_frame_interval,
> + .s_stream = imx274_s_stream,
> +};
> +
> +static const struct v4l2_subdev_ops imx274_subdev_ops = {
> + .pad = &imx274_pad_ops,
> + .video = &imx274_video_ops,
> +};
> +
> +static const struct v4l2_ctrl_ops imx274_ctrl_ops = {
> + .s_ctrl = imx274_s_ctrl,
> +};
> +
> +static const struct of_device_id imx274_of_id_table[] = {
> + { .compatible = "sony,imx274" },
> + { }
> +};
> +MODULE_DEVICE_TABLE(of, imx274_of_id_table);
> +
> +static const struct i2c_device_id imx274_id[] = {
> + { "IMX274", 0 },
> + { }
> +};
> +MODULE_DEVICE_TABLE(i2c, imx274_id);
> +
> +static int imx274_probe(struct i2c_client *client,
> + const struct i2c_device_id *id)
> +{
> + struct v4l2_subdev *sd;
> + struct stimx274 *imx274;
> + int ret;
> +
> + /* initialize imx274 */
> + imx274 = devm_kzalloc(&client->dev, sizeof(*imx274), GFP_KERNEL);
> + if (!imx274)
> + return -ENOMEM;
> +
> + mutex_init(&imx274->lock);
> +
> + /* initialize regmap */
> + imx274->regmap = devm_regmap_init_i2c(client,
> &imx274_regmap_config);
> + if (IS_ERR(imx274->regmap)) {
> + dev_err(&client->dev,
> + "regmap init failed: %ld\n", PTR_ERR(imx274-
> >regmap));
> + ret = -ENODEV;
> + goto err_regmap;
> + }
> +
> + /* initialize subdevice */
> + imx274->client = client;
> + sd = &imx274->sd;
> + v4l2_i2c_subdev_init(sd, client, &imx274_subdev_ops);
> + strlcpy(sd->name, DRIVER_NAME, sizeof(sd->name));
> + sd->flags |= V4L2_SUBDEV_FL_HAS_DEVNODE |
> V4L2_SUBDEV_FL_HAS_EVENTS;
> +
> + /* initialize subdev media pad */
> + imx274->pad.flags = MEDIA_PAD_FL_SOURCE;
> + sd->entity.function = MEDIA_ENT_F_CAM_SENSOR;
> + ret = media_entity_pads_init(&sd->entity, 1, &imx274->pad);
> + if (ret < 0) {
> + dev_err(&client->dev,
> + "%s : media entity init Failed %d\n", __func__, ret);
> + goto err_regmap;
> + }
> +
> + /* initialize sensor reset gpio */
> + imx274->reset_gpio = devm_gpiod_get_optional(&client->dev, "reset",
> + GPIOD_OUT_HIGH);
> + if (IS_ERR(imx274->reset_gpio)) {
> + if (PTR_ERR(imx274->reset_gpio) != -EPROBE_DEFER)
> + dev_err(&client->dev, "Reset GPIO not setup in DT");
> + ret = PTR_ERR(imx274->reset_gpio);
> + goto err_me;
> + }
> +
> + /* pull sensor out of reset */
> + imx274_reset(imx274, 1);
> +
> + /* initialize controls */
> + ret = v4l2_ctrl_handler_init(&imx274->ctrls.handler, 2);
> + if (ret < 0) {
> + dev_err(&client->dev,
> + "%s : ctrl handler init Failed\n", __func__);
> + goto err_me;
> + }
> +
> + imx274->ctrls.handler.lock = &imx274->lock;
> +
> + /* add new controls */
> + imx274->ctrls.test_pattern = v4l2_ctrl_new_std_menu_items(
> + &imx274->ctrls.handler, &imx274_ctrl_ops,
> + V4L2_CID_TEST_PATTERN,
> + ARRAY_SIZE(tp_qmenu) - 1, 0, 0, tp_qmenu);
> +
> + imx274->ctrls.gain = v4l2_ctrl_new_std(
> + &imx274->ctrls.handler,
> + &imx274_ctrl_ops,
> + V4L2_CID_GAIN, IMX274_MIN_GAIN,
> + IMX274_MAX_DIGITAL_GAIN * IMX274_MAX_ANALOG_GAIN,
> 1,
> + IMX274_DEF_GAIN);
> +
> + imx274->ctrls.exposure = v4l2_ctrl_new_std(
> + &imx274->ctrls.handler,
> + &imx274_ctrl_ops,
> + V4L2_CID_EXPOSURE, IMX274_MIN_EXPOSURE_TIME,
> + 1000000 / IMX274_DEF_FRAME_RATE, 1,
> + IMX274_MIN_EXPOSURE_TIME);
> +
> + imx274->ctrls.vflip = v4l2_ctrl_new_std(
> + &imx274->ctrls.handler,
> + &imx274_ctrl_ops,
> + V4L2_CID_VFLIP, 0, 1, 1, 0);
> +
> + imx274->sd.ctrl_handler = &imx274->ctrls.handler;
> + if (imx274->ctrls.handler.error) {
> + ret = imx274->ctrls.handler.error;
> + goto err_ctrls;
> + }
> +
> + /* setup default controls */
> + ret = v4l2_ctrl_handler_setup(&imx274->ctrls.handler);
> + if (ret) {
> + dev_err(&client->dev,
> + "Error %d setup default controls\n", ret);
> + goto err_ctrls;
> + }
> +
> + /* initialize format */
> + imx274->mode_index = IMX274_MODE_3840X2160;
> + imx274->format.width = imx274_formats[0].size.width;
> + imx274->format.height = imx274_formats[0].size.height;
> + imx274->format.field = V4L2_FIELD_NONE;
> + imx274->format.code = MEDIA_BUS_FMT_SRGGB10_1X10;
> + imx274->format.colorspace = V4L2_COLORSPACE_SRGB;
> + imx274->frame_interval.numerator = 1;
> + imx274->frame_interval.denominator = IMX274_DEF_FRAME_RATE;
> +
> + /* load default control values */
> + ret = imx274_load_default(imx274);
> + if (ret) {
> + dev_err(&client->dev,
> + "%s : imx274_load_default failed %d\n",
> + __func__, ret);
> + goto err_ctrls;
> + }
> +
> + /* register subdevice */
> + ret = v4l2_async_register_subdev(sd);
> + if (ret < 0) {
> + dev_err(&client->dev,
> + "%s : v4l2_async_register_subdev failed %d\n",
> + __func__, ret);
> + goto err_ctrls;
> + }
> +
> + dev_info(&client->dev, "imx274 : imx274 probe success !\n");
> + return 0;
> +
> +err_ctrls:
> + v4l2_ctrl_handler_free(&imx274->ctrls.handler);
> +err_me:
> + media_entity_cleanup(&sd->entity);
> +err_regmap:
> + mutex_destroy(&imx274->lock);
> + return ret;
> +}
> +
> +static int imx274_remove(struct i2c_client *client)
> +{
> + struct v4l2_subdev *sd = i2c_get_clientdata(client);
> + struct stimx274 *imx274 = to_imx274(sd);
> +
> + /* stop stream */
> + imx274_write_table(imx274,
> mode_table[IMX274_MODE_STOP_STREAM]);
> +
> + v4l2_async_unregister_subdev(sd);
> + v4l2_ctrl_handler_free(&imx274->ctrls.handler);
> + media_entity_cleanup(&sd->entity);
> + mutex_destroy(&imx274->lock);
> + return 0;
> +}
> +
> +static struct i2c_driver imx274_i2c_driver = {
> + .driver = {
> + .name = DRIVER_NAME,
> + .of_match_table = imx274_of_id_table,
> + },
> + .probe = imx274_probe,
> + .remove = imx274_remove,
> + .id_table = imx274_id,
> +};
> +
> +module_i2c_driver(imx274_i2c_driver);
> +
> +MODULE_AUTHOR("Leon Luo <leonl@leopardimaging.com>");
> +MODULE_DESCRIPTION("IMX274 CMOS Image Sensor driver");
> +MODULE_LICENSE("GPL v2");
> --
> 2.14.0.rc1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v9 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor
[not found] ` <DM2PR0201MB0685547EBAD0AF883C4BADD9A75D0-mGsVAZgi3o+bjIxu8aGtFxrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
@ 2017-11-03 6:08 ` Leon Luo
0 siblings, 0 replies; 4+ messages in thread
From: Leon Luo @ 2017-11-03 6:08 UTC (permalink / raw)
To: Vishal Sagar
Cc: mchehab-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
mark.rutland-5wv7dgnIgG8@public.gmane.org,
hans.verkuil-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org,
sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org,
linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Soren Brinkmann
Hi Vishal,
In probe(), it calls imx274_load_default(imx274), which has I2C
register read/write to the IMX274. If it fails, it will exit probe().
So it works as a sensor detection function as you suggested.
Regards,
Leon Luo
1130 Cadillac CT
Milpitas, CA 95035
Phone: (510)371-1169
Fax: (408) 217-1960
Email: leonl-+iDsysdzPxZTt5vwKEO50dBPR1lH4CV8@public.gmane.org
www.leopardimaging.com
On Thu, Nov 2, 2017 at 9:36 PM, Vishal Sagar <vsagar-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> wrote:
> Hi Leon,
>
> I understand this fixes correctly freeing the v4l control handlers in probe().
>
> But if there is a scenario where the sensor is mounted on a removable daughter card,
> shouldn't the probe fail if the daughter card is not connected?
> A sample read/write to an IMX274 register should be sufficient to confirm this in the probe() and fail.
>
> Does it make sense to add this in the probe()?
>
> Regards
> Vishal Sagar
>
>> -----Original Message-----
>> From: linux-media-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org [mailto:linux-media-
>> owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org] On Behalf Of Leon Luo
>> Sent: Thursday, October 26, 2017 12:21 PM
>> To: mchehab-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; mark.rutland-5wv7dgnIgG8@public.gmane.org;
>> hans.verkuil-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org; sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org
>> Cc: linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-
>> kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; leonl-+iDsysdzPxZTt5vwKEO50dBPR1lH4CV8@public.gmane.org; Soren Brinkmann
>> <sorenb-nlV6foIrjw1B+CpRQNdd2w@public.gmane.org>
>> Subject: [PATCH v9 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor
>>
>> The imx274 is a Sony CMOS image sensor that has 1/2.5 image size.
>> It supports up to 3840x2160 (4K) 60fps, 1080p 120fps. The interface
>> is 4-lane MIPI CSI-2 running at 1.44Gbps each.
>>
>> This driver has been tested on Xilinx ZCU102 platform with a Leopard
>> LI-IMX274MIPI-FMC camera board.
>>
>> Support for the following features:
>> -Resolutions: 3840x2160, 1920x1080, 1280x720
>> -Frame rate: 3840x2160 : 5 – 60fps
>> 1920x1080 : 5 – 120fps
>> 1280x720 : 5 – 120fps
>> -Exposure time: 16 – (frame interval) micro-seconds
>> -Gain: 1x - 180x
>> -VFLIP: enable/disabledrivers/media/i2c/imx274.c
>> -Test pattern: 12 test patterns
>>
>> Signed-off-by: Leon Luo <leonl-+iDsysdzPxZTt5vwKEO50dBPR1lH4CV8@public.gmane.org>
>> Tested-by: Sören Brinkmann <soren.brinkmann-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
>> Acked-by: Sakari Ailus <sakari.ailus-X3B1VOXEql0@public.gmane.org>
>> Acked-by: Chris Kohn <ckohn-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
>> ---
>> v9:
>> - remove v4l2_async_unregister_subdev from probe error
>> v8:
>> - check division by zero error
>> v7:
>> - use __v4l2_ctrl_s_ctrl instead of v4l2_ctrl_s_ctrl to have
>> clean mutex_lock/mutex_unlock in imx274_s_stream()
>> - define imx274_tp_regs[] as static, move the test pattern reg
>> out of imx274_tp_regs[], and define it as a macro
>> IMX274_TEST_PATTERN_REG
>> v6:
>> - remove media/v4l2-image-sizes.h from include header
>> - make the header file alphabetical order
>> - remove fmt->pad check in imx274_get_fmt,
>> the V4L2 subdev framework does it already
>> - change 'struct reg_8 *regs' to 'struct reg_8 regs[n]',
>> where n is the exact numbers needed for this function
>> - move MODULE_DEVICE_TABLE(of, imx274_of_id_table); closer
>> to imx274_of_id_table definition
>> - remove return check of imx274_write_table in imx274_remove,
>> because it should remove all resources even i2c fails here
>> - move imx274_load_default before v4l2_async_register_subdev
>> v5:
>> - no changes
>> v4:
>> - use 32-bit data type to avoid __divdi3 compile error for i386
>> - clean up OR together error codesdrivers/media/i2c/imx274.c
>> v3:
>> - clean up header files
>> - use struct directly instead of alias #define
>> - use v4l2_ctrl_s_ctrl instead of v4l2_s_ctrl
>> - revise debug output
>> - move static helpers closer to their call site
>> - don't OR toegether error codes
>> - use closest valid gain value instead of erroring out
>> - assigne lock to the control handler and omit explicit locking
>> - acquire mutex lock for imx274_get_fmt
>> - remove format->pad check in imx274_set_fmt since the pad is always 0
>> - pass struct v4l2_ctrl pointer in gain/exposure/vlip/test pattern controls
>> - remove priv->ctrls.vflip->val = val in imx274_set_vflip()
>> - remove priv->ctrls.test_pattern->val = val in imx274_set_test_pattern()
>> - remove empty open/close callbacks
>> - remove empty core ops
>> - add more error labels in probe
>> - use v4l2_async_unregister_subdev instead of v4l2_device_unregister_subdev
>> - use dynamic debug
>> - split start_stream to two steps: imx274_mode_regs() and
>> imx274_start_stream()
>> frame rate & exposure can be updated
>> between imx274_mode_regs() & imx274_start_stream()
>>
>> v2:
>> - Fix Kconfig to not remove existing options
>> ---
>> drivers/media/i2c/Kconfig | 7 +
>> drivers/media/i2c/Makefile | 1 +
>> drivers/media/i2c/imx274.c | 1810
>> ++++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 1818 insertions(+)
>> create mode 100644 drivers/media/i2c/imx274.c
>>
>> diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig
>> index 47113774a297..9659849e33a0 100644
>> --- a/drivers/media/i2c/Kconfig
>> +++ b/drivers/media/i2c/Kconfig
>> @@ -555,6 +555,13 @@ config VIDEO_APTINA_PLL
>> config VIDEO_SMIAPP_PLL
>> tristate
>>
>> +config VIDEO_IMX274
>> + tristate "Sony IMX274 sensor support"
>> + depends on I2C && VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API
>> + ---help---
>> + This is a V4L2 sensor-level driver for the Sony IMX274
>> + CMOS image sensor.
>> +
>> config VIDEO_OV2640
>> tristate "OmniVision OV2640 sensor support"
>> depends on VIDEO_V4L2 && I2C
>> diff --git a/drivers/media/i2c/Makefile b/drivers/media/i2c/Makefile
>> index c843c181dfb9..f8d57e453936 100644
>> --- a/drivers/media/i2c/Makefile
>> +++ b/drivers/media/i2c/Makefile
>> @@ -92,5 +92,6 @@ obj-$(CONFIG_VIDEO_IR_I2C) += ir-kbd-i2c.o
>> obj-$(CONFIG_VIDEO_ML86V7667) += ml86v7667.o
>> obj-$(CONFIG_VIDEO_OV2659) += ov2659.o
>> obj-$(CONFIG_VIDEO_TC358743) += tc358743.o
>> +obj-$(CONFIG_VIDEO_IMX274) += imx274.o
>>
>> obj-$(CONFIG_SDR_MAX2175) += max2175.o
>> diff --git a/drivers/media/i2c/imx274.c b/drivers/media/i2c/imx274.c
>> new file mode 100644
>> index 000000000000..737dbf59a0d2
>> --- /dev/null
>> +++ b/drivers/media/i2c/imx274.c
>> @@ -0,0 +1,1810 @@
>> +/*
>> + * imx274.c - IMX274 CMOS Image Sensor driver
>> + *
>> + * Copyright (C) 2017, Leopard Imaging, Inc.
>> + *
>> + * Leon Luo <leonl-+iDsysdzPxZTt5vwKEO50dBPR1lH4CV8@public.gmane.org>
>> + * Edwin Zou <edwinz-+iDsysdzPxZTt5vwKEO50dBPR1lH4CV8@public.gmane.org>
>> + *
>> + * This program is free software; you can redistribute it and/or modify it
>> + * under the terms and conditions of the GNU General Public License,
>> + * version 2, as published by the Free Software Foundation.
>> + *
>> + * This program is distributed in the hope it will be useful, but WITHOUT
>> + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
>> or
>> + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
>> for
>> + * more details.
>> + *
>> + * You should have received a copy of the GNU General Public License
>> + * along with this program. If not, see <http://www.gnu.org/licenses/>.
>> + */
>> +
>> +#include <linux/clk.h>
>> +#include <linux/delay.h>
>> +#include <linux/gpio.h>
>> +#include <linux/gpio/consumer.h>
>> +#include <linux/i2c.h>
>> +#include <linux/init.h>
>> +#include <linux/module.h>
>> +#include <linux/of_gpio.h>
>> +#include <linux/regmap.h>
>> +#include <linux/slab.h>
>> +#include <linux/v4l2-mediabus.h>
>> +#include <linux/videodev2.h>
>> +
>> +#include <media/v4l2-ctrls.h>
>> +#include <media/v4l2-device.h>
>> +#include <media/v4l2-subdev.h>
>> +
>> +/*
>> + * See "SHR, SVR Setting" in datasheet
>> + */
>> +#define IMX274_DEFAULT_FRAME_LENGTH (4550)
>> +#define IMX274_MAX_FRAME_LENGTH (0x000fffff)
>> +
>> +/*
>> + * See "Frame Rate Adjustment" in datasheet
>> + */
>> +#define IMX274_PIXCLK_CONST1 (72000000)
>> +#define IMX274_PIXCLK_CONST2 (1000000)
>> +
>> +/*
>> + * The input gain is shifted by IMX274_GAIN_SHIFT to get
>> + * decimal number. The real gain is
>> + * (float)input_gain_value / (1 << IMX274_GAIN_SHIFT)
>> + */
>> +#define IMX274_GAIN_SHIFT (8)
>> +#define IMX274_GAIN_SHIFT_MASK ((1 <<
>> IMX274_GAIN_SHIFT) - 1)
>> +
>> +/*
>> + * See "Analog Gain" and "Digital Gain" in datasheet
>> + * min gain is 1X
>> + * max gain is calculated based on IMX274_GAIN_REG_MAX
>> + */
>> +#define IMX274_GAIN_REG_MAX (1957)
>> +#define IMX274_MIN_GAIN (0x01 <<
>> IMX274_GAIN_SHIFT)
>> +#define IMX274_MAX_ANALOG_GAIN ((2048 <<
>> IMX274_GAIN_SHIFT)\
>> + / (2048 - IMX274_GAIN_REG_MAX))
>> +#define IMX274_MAX_DIGITAL_GAIN (8)
>> +#define IMX274_DEF_GAIN (20 <<
>> IMX274_GAIN_SHIFT)
>> +#define IMX274_GAIN_CONST (2048) /* for gain formula */
>> +
>> +/*
>> + * 1 line time in us = (HMAX / 72), minimal is 4 lines
>> + */
>> +#define IMX274_MIN_EXPOSURE_TIME (4 * 260 / 72)
>> +
>> +#define IMX274_DEFAULT_MODE
>> IMX274_MODE_3840X2160
>> +#define IMX274_MAX_WIDTH (3840)
>> +#define IMX274_MAX_HEIGHT (2160)
>> +#define IMX274_MAX_FRAME_RATE (120)
>> +#define IMX274_MIN_FRAME_RATE (5)
>> +#define IMX274_DEF_FRAME_RATE (60)
>> +
>> +/*
>> + * register SHR is limited to (SVR value + 1) x VMAX value - 4
>> + */
>> +#define IMX274_SHR_LIMIT_CONST (4)
>> +
>> +/*
>> + * Constants for sensor reset delay
>> + */
>> +#define IMX274_RESET_DELAY1 (2000)
>> +#define IMX274_RESET_DELAY2 (2200)
>> +
>> +/*
>> + * shift and mask constants
>> + */
>> +#define IMX274_SHIFT_8_BITS (8)
>> +#define IMX274_SHIFT_16_BITS (16)
>> +#define IMX274_MASK_LSB_2_BITS (0x03)
>> +#define IMX274_MASK_LSB_3_BITS (0x07)
>> +#define IMX274_MASK_LSB_4_BITS (0x0f)
>> +#define IMX274_MASK_LSB_8_BITS (0x00ff)
>> +
>> +#define DRIVER_NAME "IMX274"
>> +
>> +/*
>> + * IMX274 register definitions
>> + */
>> +#define IMX274_FRAME_LENGTH_ADDR_1 0x30FA /* VMAX, MSB
>> */
>> +#define IMX274_FRAME_LENGTH_ADDR_2 0x30F9 /* VMAX */
>> +#define IMX274_FRAME_LENGTH_ADDR_3 0x30F8 /* VMAX, LSB
>> */
>> +#define IMX274_SVR_REG_MSB 0x300F /* SVR */
>> +#define IMX274_SVR_REG_LSB 0x300E /* SVR */
>> +#define IMX274_HMAX_REG_MSB 0x30F7 /* HMAX */
>> +#define IMX274_HMAX_REG_LSB 0x30F6 /* HMAX */
>> +#define IMX274_COARSE_TIME_ADDR_MSB 0x300D /* SHR */
>> +#define IMX274_COARSE_TIME_ADDR_LSB 0x300C /* SHR */
>> +#define IMX274_ANALOG_GAIN_ADDR_LSB 0x300A /* ANALOG
>> GAIN LSB */
>> +#define IMX274_ANALOG_GAIN_ADDR_MSB 0x300B /* ANALOG
>> GAIN MSB */
>> +#define IMX274_DIGITAL_GAIN_REG 0x3012 /* Digital Gain
>> */
>> +#define IMX274_VFLIP_REG 0x301A /* VERTICAL FLIP */
>> +#define IMX274_TEST_PATTERN_REG 0x303D /* TEST
>> PATTERN */
>> +#define IMX274_STANDBY_REG 0x3000 /* STANDBY */
>> +
>> +#define IMX274_TABLE_WAIT_MS 0
>> +#define IMX274_TABLE_END 1
>> +
>> +/*
>> + * imx274 I2C operation related structure
>> + */
>> +struct reg_8 {
>> + u16 addr;
>> + u8 val;
>> +};
>> +
>> +static const struct regmap_config imx274_regmap_config = {
>> + .reg_bits = 16,
>> + .val_bits = 8,
>> + .cache_type = REGCACHE_RBTREE,
>> +};
>> +
>> +enum imx274_mode {
>> + IMX274_MODE_3840X2160,
>> + IMX274_MODE_1920X1080,
>> + IMX274_MODE_1280X720,
>> +
>> + IMX274_MODE_START_STREAM_1,
>> + IMX274_MODE_START_STREAM_2,
>> + IMX274_MODE_START_STREAM_3,
>> + IMX274_MODE_START_STREAM_4,
>> + IMX274_MODE_STOP_STREAM
>> +};
>> +
>> +/*
>> + * imx274 format related structure
>> + */
>> +struct imx274_frmfmt {
>> + u32 mbus_code;
>> + enum v4l2_colorspace colorspace;
>> + struct v4l2_frmsize_discrete size;
>> + enum imx274_mode mode;
>> +};
>> +
>> +/*
>> + * imx274 test pattern related structure
>> + */
>> +enum {
>> + TEST_PATTERN_DISABLED = 0,
>> + TEST_PATTERN_ALL_000H,
>> + TEST_PATTERN_ALL_FFFH,
>> + TEST_PATTERN_ALL_555H,
>> + TEST_PATTERN_ALL_AAAH,
>> + TEST_PATTERN_VSP_5AH, /* VERTICAL STRIPE PATTERN 555H/AAAH */
>> + TEST_PATTERN_VSP_A5H, /* VERTICAL STRIPE PATTERN AAAH/555H */
>> + TEST_PATTERN_VSP_05H, /* VERTICAL STRIPE PATTERN 000H/555H */
>> + TEST_PATTERN_VSP_50H, /* VERTICAL STRIPE PATTERN 555H/000H */
>> + TEST_PATTERN_VSP_0FH, /* VERTICAL STRIPE PATTERN 000H/FFFH */
>> + TEST_PATTERN_VSP_F0H, /* VERTICAL STRIPE PATTERN FFFH/000H */
>> + TEST_PATTERN_H_COLOR_BARS,
>> + TEST_PATTERN_V_COLOR_BARS,
>> +};
>> +
>> +static const char * const tp_qmenu[] = {
>> + "Disabled",
>> + "All 000h Pattern",
>> + "All FFFh Pattern",
>> + "All 555h Pattern",
>> + "All AAAh Pattern",
>> + "Vertical Stripe (555h / AAAh)",
>> + "Vertical Stripe (AAAh / 555h)",
>> + "Vertical Stripe (000h / 555h)",
>> + "Vertical Stripe (555h / 000h)",
>> + "Vertical Stripe (000h / FFFh)",
>> + "Vertical Stripe (FFFh / 000h)",
>> + "Horizontal Color Bars",
>> + "Vertical Color Bars",
>> +};
>> +
>> +/*
>> + * All-pixel scan mode (10-bit)
>> + * imx274 mode1(refer to datasheet) register configuration with
>> + * 3840x2160 resolution, raw10 data and mipi four lane output
>> + */
>> +static const struct reg_8 imx274_mode1_3840x2160_raw10[] = {
>> + {0x3004, 0x01},
>> + {0x3005, 0x01},
>> + {0x3006, 0x00},
>> + {0x3007, 0x02},
>> +
>> + {0x3018, 0xA2}, /* output XVS, HVS */
>> +
>> + {0x306B, 0x05},
>> + {0x30E2, 0x01},
>> + {0x30F6, 0x07}, /* HMAX, 263 */
>> + {0x30F7, 0x01}, /* HMAX */
>> +
>> + {0x30dd, 0x01}, /* crop to 2160 */
>> + {0x30de, 0x06},
>> + {0x30df, 0x00},
>> + {0x30e0, 0x12},
>> + {0x30e1, 0x00},
>> + {0x3037, 0x01}, /* to crop to 3840 */
>> + {0x3038, 0x0c},
>> + {0x3039, 0x00},
>> + {0x303a, 0x0c},
>> + {0x303b, 0x0f},
>> +
>> + {0x30EE, 0x01},
>> + {0x3130, 0x86},
>> + {0x3131, 0x08},
>> + {0x3132, 0x7E},
>> + {0x3133, 0x08},
>> + {0x3342, 0x0A},
>> + {0x3343, 0x00},
>> + {0x3344, 0x16},
>> + {0x3345, 0x00},
>> + {0x33A6, 0x01},
>> + {0x3528, 0x0E},
>> + {0x3554, 0x1F},
>> + {0x3555, 0x01},
>> + {0x3556, 0x01},
>> + {0x3557, 0x01},
>> + {0x3558, 0x01},
>> + {0x3559, 0x00},
>> + {0x355A, 0x00},
>> + {0x35BA, 0x0E},
>> + {0x366A, 0x1B},
>> + {0x366B, 0x1A},
>> + {0x366C, 0x19},
>> + {0x366D, 0x17},
>> + {0x3A41, 0x08},
>> +
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +/*
>> + * Horizontal/vertical 2/2-line binning
>> + * (Horizontal and vertical weightedbinning, 10-bit)
>> + * imx274 mode3(refer to datasheet) register configuration with
>> + * 1920x1080 resolution, raw10 data and mipi four lane output
>> + */
>> +static const struct reg_8 imx274_mode3_1920x1080_raw10[] = {
>> + {0x3004, 0x02},
>> + {0x3005, 0x21},
>> + {0x3006, 0x00},
>> + {0x3007, 0x11},
>> +
>> + {0x3018, 0xA2}, /* output XVS, HVS */
>> +
>> + {0x306B, 0x05},
>> + {0x30E2, 0x02},
>> +
>> + {0x30F6, 0x04}, /* HMAX, 260 */
>> + {0x30F7, 0x01}, /* HMAX */
>> +
>> + {0x30dd, 0x01}, /* to crop to 1920x1080 */
>> + {0x30de, 0x05},
>> + {0x30df, 0x00},
>> + {0x30e0, 0x04},
>> + {0x30e1, 0x00},
>> + {0x3037, 0x01},
>> + {0x3038, 0x0c},
>> + {0x3039, 0x00},
>> + {0x303a, 0x0c},
>> + {0x303b, 0x0f},
>> +
>> + {0x30EE, 0x01},
>> + {0x3130, 0x4E},
>> + {0x3131, 0x04},
>> + {0x3132, 0x46},
>> + {0x3133, 0x04},
>> + {0x3342, 0x0A},
>> + {0x3343, 0x00},
>> + {0x3344, 0x1A},
>> + {0x3345, 0x00},
>> + {0x33A6, 0x01},
>> + {0x3528, 0x0E},
>> + {0x3554, 0x00},
>> + {0x3555, 0x01},
>> + {0x3556, 0x01},
>> + {0x3557, 0x01},
>> + {0x3558, 0x01},
>> + {0x3559, 0x00},
>> + {0x355A, 0x00},
>> + {0x35BA, 0x0E},
>> + {0x366A, 0x1B},
>> + {0x366B, 0x1A},
>> + {0x366C, 0x19},
>> + {0x366D, 0x17},
>> + {0x3A41, 0x08},
>> +
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +/*
>> + * Vertical 2/3 subsampling binning horizontal 3 binning
>> + * imx274 mode5(refer to datasheet) register configuration with
>> + * 1280x720 resolution, raw10 data and mipi four lane output
>> + */
>> +static const struct reg_8 imx274_mode5_1280x720_raw10[] = {
>> + {0x3004, 0x03},
>> + {0x3005, 0x31},
>> + {0x3006, 0x00},
>> + {0x3007, 0x09},
>> +
>> + {0x3018, 0xA2}, /* output XVS, HVS */
>> +
>> + {0x306B, 0x05},
>> + {0x30E2, 0x03},
>> +
>> + {0x30F6, 0x04}, /* HMAX, 260 */
>> + {0x30F7, 0x01}, /* HMAX */
>> +
>> + {0x30DD, 0x01},
>> + {0x30DE, 0x07},
>> + {0x30DF, 0x00},
>> + {0x40E0, 0x04},
>> + {0x30E1, 0x00},
>> + {0x3030, 0xD4},
>> + {0x3031, 0x02},
>> + {0x3032, 0xD0},
>> + {0x3033, 0x02},
>> +
>> + {0x30EE, 0x01},
>> + {0x3130, 0xE2},
>> + {0x3131, 0x02},
>> + {0x3132, 0xDE},
>> + {0x3133, 0x02},
>> + {0x3342, 0x0A},
>> + {0x3343, 0x00},
>> + {0x3344, 0x1B},
>> + {0x3345, 0x00},
>> + {0x33A6, 0x01},
>> + {0x3528, 0x0E},
>> + {0x3554, 0x00},
>> + {0x3555, 0x01},
>> + {0x3556, 0x01},
>> + {0x3557, 0x01},
>> + {0x3558, 0x01},
>> + {0x3559, 0x00},
>> + {0x355A, 0x00},
>> + {0x35BA, 0x0E},
>> + {0x366A, 0x1B},
>> + {0x366B, 0x19},
>> + {0x366C, 0x17},
>> + {0x366D, 0x17},
>> + {0x3A41, 0x04},
>> +
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +/*
>> + * imx274 first step register configuration for
>> + * starting stream
>> + */
>> +static const struct reg_8 imx274_start_1[] = {
>> + {IMX274_STANDBY_REG, 0x12},
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +/*
>> + * imx274 second step register configuration for
>> + * starting stream
>> + */
>> +static const struct reg_8 imx274_start_2[] = {
>> + {0x3120, 0xF0}, /* clock settings */
>> + {0x3121, 0x00}, /* clock settings */
>> + {0x3122, 0x02}, /* clock settings */
>> + {0x3129, 0x9C}, /* clock settings */
>> + {0x312A, 0x02}, /* clock settings */
>> + {0x312D, 0x02}, /* clock settings */
>> +
>> + {0x310B, 0x00},
>> +
>> + /* PLSTMG */
>> + {0x304C, 0x00}, /* PLSTMG01 */
>> + {0x304D, 0x03},
>> + {0x331C, 0x1A},
>> + {0x331D, 0x00},
>> + {0x3502, 0x02},
>> + {0x3529, 0x0E},
>> + {0x352A, 0x0E},
>> + {0x352B, 0x0E},
>> + {0x3538, 0x0E},
>> + {0x3539, 0x0E},
>> + {0x3553, 0x00},
>> + {0x357D, 0x05},
>> + {0x357F, 0x05},
>> + {0x3581, 0x04},
>> + {0x3583, 0x76},
>> + {0x3587, 0x01},
>> + {0x35BB, 0x0E},
>> + {0x35BC, 0x0E},
>> + {0x35BD, 0x0E},
>> + {0x35BE, 0x0E},
>> + {0x35BF, 0x0E},
>> + {0x366E, 0x00},
>> + {0x366F, 0x00},
>> + {0x3670, 0x00},
>> + {0x3671, 0x00},
>> +
>> + /* PSMIPI */
>> + {0x3304, 0x32}, /* PSMIPI1 */
>> + {0x3305, 0x00},
>> + {0x3306, 0x32},
>> + {0x3307, 0x00},
>> + {0x3590, 0x32},
>> + {0x3591, 0x00},
>> + {0x3686, 0x32},
>> + {0x3687, 0x00},
>> +
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +/*
>> + * imx274 third step register configuration for
>> + * starting stream
>> + */
>> +static const struct reg_8 imx274_start_3[] = {
>> + {IMX274_STANDBY_REG, 0x00},
>> + {0x303E, 0x02}, /* SYS_MODE = 2 */
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +/*
>> + * imx274 forth step register configuration for
>> + * starting stream
>> + */
>> +static const struct reg_8 imx274_start_4[] = {
>> + {0x30F4, 0x00},
>> + {0x3018, 0xA2}, /* XHS VHS OUTUPT */
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +/*
>> + * imx274 register configuration for stoping stream
>> + */
>> +static const struct reg_8 imx274_stop[] = {
>> + {IMX274_STANDBY_REG, 0x01},
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +/*
>> + * imx274 disable test pattern register configuration
>> + */
>> +static const struct reg_8 imx274_tp_disabled[] = {
>> + {0x303C, 0x00},
>> + {0x377F, 0x00},
>> + {0x3781, 0x00},
>> + {0x370B, 0x00},
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +/*
>> + * imx274 test pattern register configuration
>> + * reg 0x303D defines the test pattern modes
>> + */
>> +static const struct reg_8 imx274_tp_regs[] = {
>> + {0x303C, 0x11},
>> + {0x370E, 0x01},
>> + {0x377F, 0x01},
>> + {0x3781, 0x01},
>> + {0x370B, 0x11},
>> + {IMX274_TABLE_END, 0x00}
>> +};
>> +
>> +static const struct reg_8 *mode_table[] = {
>> + [IMX274_MODE_3840X2160] =
>> imx274_mode1_3840x2160_raw10,
>> + [IMX274_MODE_1920X1080] =
>> imx274_mode3_1920x1080_raw10,
>> + [IMX274_MODE_1280X720] =
>> imx274_mode5_1280x720_raw10,
>> +
>> + [IMX274_MODE_START_STREAM_1] = imx274_start_1,
>> + [IMX274_MODE_START_STREAM_2] = imx274_start_2,
>> + [IMX274_MODE_START_STREAM_3] = imx274_start_3,
>> + [IMX274_MODE_START_STREAM_4] = imx274_start_4,
>> + [IMX274_MODE_STOP_STREAM] = imx274_stop,
>> +};
>> +
>> +/*
>> + * imx274 format related structure
>> + */
>> +static const struct imx274_frmfmt imx274_formats[] = {
>> + {MEDIA_BUS_FMT_SRGGB10_1X10, V4L2_COLORSPACE_SRGB, {3840,
>> 2160},
>> + IMX274_MODE_3840X2160},
>> + {MEDIA_BUS_FMT_SRGGB10_1X10, V4L2_COLORSPACE_SRGB, {1920,
>> 1080},
>> + IMX274_MODE_1920X1080},
>> + {MEDIA_BUS_FMT_SRGGB10_1X10, V4L2_COLORSPACE_SRGB, {1280,
>> 720},
>> + IMX274_MODE_1280X720},
>> +};
>> +
>> +/*
>> + * minimal frame length for each mode
>> + * refer to datasheet section "Frame Rate Adjustment (CSI-2)"
>> + */
>> +static const int min_frame_len[] = {
>> + 4550, /* mode 1, 4K */
>> + 2310, /* mode 3, 1080p */
>> + 2310 /* mode 5, 720p */
>> +};
>> +
>> +/*
>> + * minimal numbers of SHR register
>> + * refer to datasheet table "Shutter Setting (CSI-2)"
>> + */
>> +static const int min_SHR[] = {
>> + 12, /* mode 1, 4K */
>> + 8, /* mode 3, 1080p */
>> + 8 /* mode 5, 720p */
>> +};
>> +
>> +static const int max_frame_rate[] = {
>> + 60, /* mode 1 , 4K */
>> + 120, /* mode 3, 1080p */
>> + 120 /* mode 5, 720p */
>> +};
>> +
>> +/*
>> + * Number of clocks per internal offset period
>> + * a constant based on mode
>> + * refer to section "Integration Time in Each Readout Drive Mode (CSI-2)"
>> + * in the datasheet
>> + * for the implemented 3 modes, it happens to be the same number
>> + */
>> +static const int nocpiop[] = {
>> + 112, /* mode 1 , 4K */
>> + 112, /* mode 3, 1080p */
>> + 112 /* mode 5, 720p */
>> +};
>> +
>> +/*
>> + * struct imx274_ctrls - imx274 ctrl structure
>> + * @handler: V4L2 ctrl handler structure
>> + * @exposure: Pointer to expsure ctrl structure
>> + * @gain: Pointer to gain ctrl structure
>> + * @vflip: Pointer to vflip ctrl structure
>> + * @test_pattern: Pointer to test pattern ctrl structure
>> + */
>> +struct imx274_ctrls {
>> + struct v4l2_ctrl_handler handler;
>> + struct v4l2_ctrl *exposure;
>> + struct v4l2_ctrl *gain;
>> + struct v4l2_ctrl *vflip;
>> + struct v4l2_ctrl *test_pattern;
>> +};
>> +
>> +/*
>> + * struct stim274 - imx274 device structure
>> + * @sd: V4L2 subdevice structure
>> + * @pd: Media pad structure
>> + * @client: Pointer to I2C client
>> + * @ctrls: imx274 control structure
>> + * @format: V4L2 media bus frame format structure
>> + * @frame_rate: V4L2 frame rate structure
>> + * @regmap: Pointer to regmap structure
>> + * @reset_gpio: Pointer to reset gpio
>> + * @lock: Mutex structure
>> + * @mode_index: Resolution mode index
>> + */
>> +struct stimx274 {
>> + struct v4l2_subdev sd;
>> + struct media_pad pad;
>> + struct i2c_client *client;
>> + struct imx274_ctrls ctrls;
>> + struct v4l2_mbus_framefmt format;
>> + struct v4l2_fract frame_interval;
>> + struct regmap *regmap;
>> + struct gpio_desc *reset_gpio;
>> + struct mutex lock; /* mutex lock for operations */
>> + u32 mode_index;
>> +};
>> +
>> +/*
>> + * Function declaration
>> + */
>> +static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl);
>> +static int imx274_set_exposure(struct stimx274 *priv, int val);
>> +static int imx274_set_vflip(struct stimx274 *priv, int val);
>> +static int imx274_set_test_pattern(struct stimx274 *priv, int val);
>> +static int imx274_set_frame_interval(struct stimx274 *priv,
>> + struct v4l2_fract frame_interval);
>> +
>> +static inline void msleep_range(unsigned int delay_base)
>> +{
>> + usleep_range(delay_base * 1000, delay_base * 1000 + 500);
>> +}
>> +
>> +/*
>> + * v4l2_ctrl and v4l2_subdev related operations
>> + */
>> +static inline struct v4l2_subdev *ctrl_to_sd(struct v4l2_ctrl *ctrl)
>> +{
>> + return &container_of(ctrl->handler,
>> + struct stimx274, ctrls.handler)->sd;
>> +}
>> +
>> +static inline struct stimx274 *to_imx274(struct v4l2_subdev *sd)
>> +{
>> + return container_of(sd, struct stimx274, sd);
>> +}
>> +
>> +/*
>> + * imx274_regmap_util_write_table_8 - Function for writing register table
>> + * @regmap: Pointer to device reg map structure
>> + * @table: Table containing register values
>> + * @wait_ms_addr: Flag for performing delay
>> + * @end_addr: Flag for incating end of table
>> + *
>> + * This is used to write register table into sensor's reg map.
>> + *
>> + * Return: 0 on success, errors otherwise
>> + */
>> +static int imx274_regmap_util_write_table_8(struct regmap *regmap,
>> + const struct reg_8 table[],
>> + u16 wait_ms_addr, u16 end_addr)
>> +{
>> + int err;
>> + const struct reg_8 *next;
>> + u8 val;
>> +
>> + int range_start = -1;
>> + int range_count = 0;
>> + u8 range_vals[16];
>> + int max_range_vals = ARRAY_SIZE(range_vals);
>> +
>> + for (next = table;; next++) {
>> + if ((next->addr != range_start + range_count) ||
>> + (next->addr == end_addr) ||
>> + (next->addr == wait_ms_addr) ||
>> + (range_count == max_range_vals)) {
>> + if (range_count == 1)
>> + err = regmap_write(regmap,
>> + range_start, range_vals[0]);
>> + else if (range_count > 1)
>> + err = regmap_bulk_write(regmap, range_start,
>> + &range_vals[0],
>> + range_count);
>> +
>> + if (err)
>> + return err;
>> +
>> + range_start = -1;
>> + range_count = 0;
>> +
>> + /* Handle special address values */
>> + if (next->addr == end_addr)
>> + break;
>> +
>> + if (next->addr == wait_ms_addr) {
>> + msleep_range(next->val);
>> + continue;
>> + }
>> + }
>> +
>> + val = next->val;
>> +
>> + if (range_start == -1)
>> + range_start = next->addr;
>> +
>> + range_vals[range_count++] = val;
>> + }
>> + return 0;
>> +}
>> +
>> +static inline int imx274_read_reg(struct stimx274 *priv, u16 addr, u8 *val)
>> +{
>> + int err;
>> +
>> + err = regmap_read(priv->regmap, addr, (unsigned int *)val);
>> + if (err)
>> + dev_err(&priv->client->dev,
>> + "%s : i2c read failed, addr = %x\n", __func__, addr);
>> + else
>> + dev_dbg(&priv->client->dev,
>> + "%s : addr 0x%x, val=0x%x\n", __func__,
>> + addr, *val);
>> + return err;
>> +}
>> +
>> +static inline int imx274_write_reg(struct stimx274 *priv, u16 addr, u8 val)
>> +{
>> + int err;
>> +
>> + err = regmap_write(priv->regmap, addr, val);
>> + if (err)
>> + dev_err(&priv->client->dev,
>> + "%s : i2c write failed, %x = %x\n", __func__,
>> + addr, val);
>> + else
>> + dev_dbg(&priv->client->dev,
>> + "%s : addr 0x%x, val=0x%x\n", __func__,
>> + addr, val);
>> + return err;
>> +}
>> +
>> +static int imx274_write_table(struct stimx274 *priv, const struct reg_8 table[])
>> +{
>> + return imx274_regmap_util_write_table_8(priv->regmap,
>> + table, IMX274_TABLE_WAIT_MS, IMX274_TABLE_END);
>> +}
>> +
>> +/*
>> + * imx274_mode_regs - Function for set mode registers per mode index
>> + * @priv: Pointer to device structure
>> + * @mode: Mode index value
>> + *
>> + * This is used to start steam per mode index.
>> + * mode = 0, start stream for sensor Mode 1: 4K/raw10
>> + * mode = 1, start stream for sensor Mode 3: 1080p/raw10
>> + * mode = 2, start stream for sensor Mode 5: 720p/raw10
>> + *
>> + * Return: 0 on success, errors otherwise
>> + */
>> +static int imx274_mode_regs(struct stimx274 *priv, int mode)
>> +{
>> + int err = 0;
>> +
>> + err = imx274_write_table(priv,
>> mode_table[IMX274_MODE_START_STREAM_1]);
>> + if (err)
>> + return err;
>> +
>> + err = imx274_write_table(priv,
>> mode_table[IMX274_MODE_START_STREAM_2]);
>> + if (err)
>> + return err;
>> +
>> + err = imx274_write_table(priv, mode_table[mode]);
>> +
>> + return err;
>> +}
>> +
>> +/*
>> + * imx274_start_stream - Function for starting stream per mode index
>> + * @priv: Pointer to device structure
>> + *
>> + * Return: 0 on success, errors otherwise
>> + */
>> +static int imx274_start_stream(struct stimx274 *priv)
>> +{
>> + int err = 0;
>> +
>> + /*
>> + * Refer to "Standby Cancel Sequence when using CSI-2" in
>> + * imx274 datasheet, it should wait 10ms or more here.
>> + * give it 1 extra ms for margin
>> + */
>> + msleep_range(11);
>> + err = imx274_write_table(priv,
>> mode_table[IMX274_MODE_START_STREAM_3]);
>> + if (err)
>> + return err;
>> +
>> + /*
>> + * Refer to "Standby Cancel Sequence when using CSI-2" in
>> + * imx274 datasheet, it should wait 7ms or more here.
>> + * give it 1 extra ms for margin
>> + */
>> + msleep_range(8);
>> + err = imx274_write_table(priv,
>> mode_table[IMX274_MODE_START_STREAM_4]);
>> + if (err)
>> + return err;
>> +
>> + return 0;
>> +}
>> +
>> +/*
>> + * imx274_reset - Function called to reset the sensor
>> + * @priv: Pointer to device structure
>> + * @rst: Input value for determining the sensor's end state after reset
>> + *
>> + * Set the senor in reset and then
>> + * if rst = 0, keep it in reset;
>> + * if rst = 1, bring it out of reset.
>> + *
>> + */
>> +static void imx274_reset(struct stimx274 *priv, int rst)
>> +{
>> + gpiod_set_value_cansleep(priv->reset_gpio, 0);
>> + usleep_range(IMX274_RESET_DELAY1, IMX274_RESET_DELAY2);
>> + gpiod_set_value_cansleep(priv->reset_gpio, !!rst);
>> + usleep_range(IMX274_RESET_DELAY1, IMX274_RESET_DELAY2);
>> +}
>> +
>> +/**
>> + * imx274_s_ctrl - This is used to set the imx274 V4L2 controls
>> + * @ctrl: V4L2 control to be set
>> + *
>> + * This function is used to set the V4L2 controls for the imx274 sensor.
>> + *
>> + * Return: 0 on success, errors otherwise
>> + */
>> +static int imx274_s_ctrl(struct v4l2_ctrl *ctrl)
>> +{
>> + struct v4l2_subdev *sd = ctrl_to_sd(ctrl);
>> + struct stimx274 *imx274 = to_imx274(sd);
>> + int ret = -EINVAL;
>> +
>> + dev_dbg(&imx274->client->dev,
>> + "%s : s_ctrl: %s, value: %d\n", __func__,
>> + ctrl->name, ctrl->val);
>> +
>> + switch (ctrl->id) {
>> + case V4L2_CID_EXPOSURE:
>> + dev_dbg(&imx274->client->dev,
>> + "%s : set V4L2_CID_EXPOSURE\n", __func__);
>> + ret = imx274_set_exposure(imx274, ctrl->val);
>> + break;
>> +
>> + case V4L2_CID_GAIN:
>> + dev_dbg(&imx274->client->dev,
>> + "%s : set V4L2_CID_GAIN\n", __func__);
>> + ret = imx274_set_gain(imx274, ctrl);
>> + break;
>> +
>> + case V4L2_CID_VFLIP:
>> + dev_dbg(&imx274->client->dev,
>> + "%s : set V4L2_CID_VFLIP\n", __func__);
>> + ret = imx274_set_vflip(imx274, ctrl->val);
>> + break;
>> +
>> + case V4L2_CID_TEST_PATTERN:
>> + dev_dbg(&imx274->client->dev,
>> + "%s : set V4L2_CID_TEST_PATTERN\n", __func__);
>> + ret = imx274_set_test_pattern(imx274, ctrl->val);
>> + break;
>> + }
>> +
>> + return ret;
>> +}
>> +
>> +/**
>> + * imx274_get_fmt - Get the pad format
>> + * @sd: Pointer to V4L2 Sub device structure
>> + * @cfg: Pointer to sub device pad information structure
>> + * @fmt: Pointer to pad level media bus format
>> + *
>> + * This function is used to get the pad format information.
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_get_fmt(struct v4l2_subdev *sd,
>> + struct v4l2_subdev_pad_config *cfg,
>> + struct v4l2_subdev_format *fmt)
>> +{
>> + struct stimx274 *imx274 = to_imx274(sd);
>> +
>> + mutex_lock(&imx274->lock);
>> + fmt->format = imx274->format;
>> + mutex_unlock(&imx274->lock);
>> + return 0;
>> +}
>> +
>> +/**
>> + * imx274_set_fmt - This is used to set the pad format
>> + * @sd: Pointer to V4L2 Sub device structure
>> + * @cfg: Pointer to sub device pad information structure
>> + * @format: Pointer to pad level media bus format
>> + *
>> + * This function is used to set the pad format.
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_set_fmt(struct v4l2_subdev *sd,
>> + struct v4l2_subdev_pad_config *cfg,
>> + struct v4l2_subdev_format *format)
>> +{
>> + struct v4l2_mbus_framefmt *fmt = &format->format;
>> + struct stimx274 *imx274 = to_imx274(sd);
>> + struct i2c_client *client = imx274->client;
>> + int index;
>> +
>> + dev_dbg(&client->dev,
>> + "%s: width = %d height = %d code = %d mbus_code = %d\n",
>> + __func__, fmt->width, fmt->height, fmt->code,
>> + imx274_formats[imx274->mode_index].mbus_code);
>> +
>> + mutex_lock(&imx274->lock);
>> +
>> + for (index = 0; index < ARRAY_SIZE(imx274_formats); index++) {
>> + if (imx274_formats[index].size.width == fmt->width &&
>> + imx274_formats[index].size.height == fmt->height)
>> + break;
>> + }
>> +
>> + if (index >= ARRAY_SIZE(imx274_formats)) {
>> + /* default to first format */
>> + index = 0;
>> + }
>> +
>> + imx274->mode_index = index;
>> +
>> + if (fmt->width > IMX274_MAX_WIDTH)
>> + fmt->width = IMX274_MAX_WIDTH;
>> + if (fmt->height > IMX274_MAX_HEIGHT)
>> + fmt->height = IMX274_MAX_HEIGHT;
>> + fmt->width = fmt->width & (~IMX274_MASK_LSB_2_BITS);
>> + fmt->height = fmt->height & (~IMX274_MASK_LSB_2_BITS);
>> + fmt->field = V4L2_FIELD_NONE;
>> +
>> + if (format->which == V4L2_SUBDEV_FORMAT_TRY)
>> + cfg->try_fmt = *fmt;
>> + else
>> + imx274->format = *fmt;
>> +
>> + mutex_unlock(&imx274->lock);
>> + return 0;
>> +}
>> +
>> +/**
>> + * imx274_g_frame_interval - Get the frame interval
>> + * @sd: Pointer to V4L2 Sub device structure
>> + * @fi: Pointer to V4l2 Sub device frame interval structure
>> + *
>> + * This function is used to get the frame interval.
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_g_frame_interval(struct v4l2_subdev *sd,
>> + struct v4l2_subdev_frame_interval *fi)
>> +{
>> + struct stimx274 *imx274 = to_imx274(sd);
>> +
>> + fi->interval = imx274->frame_interval;
>> + dev_dbg(&imx274->client->dev, "%s frame rate = %d / %d\n",
>> + __func__, imx274->frame_interval.numerator,
>> + imx274->frame_interval.denominator);
>> +
>> + return 0;
>> +}
>> +
>> +/**
>> + * imx274_s_frame_interval - Set the frame interval
>> + * @sd: Pointer to V4L2 Sub device structure
>> + * @fi: Pointer to V4l2 Sub device frame interval structure
>> + *
>> + * This function is used to set the frame intervavl.
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_s_frame_interval(struct v4l2_subdev *sd,
>> + struct v4l2_subdev_frame_interval *fi)
>> +{
>> + struct stimx274 *imx274 = to_imx274(sd);
>> + struct v4l2_ctrl *ctrl = imx274->ctrls.exposure;
>> + int min, max, def;
>> + int ret;
>> +
>> + mutex_lock(&imx274->lock);
>> + ret = imx274_set_frame_interval(imx274, fi->interval);
>> +
>> + if (!ret) {
>> + /*
>> + * exposure time range is decided by frame interval
>> + * need to update it after frame interal changes
>> + */
>> + min = IMX274_MIN_EXPOSURE_TIME;
>> + max = fi->interval.numerator * 1000000
>> + / fi->interval.denominator;
>> + def = max;
>> + if (__v4l2_ctrl_modify_range(ctrl, min, max, 1, def)) {
>> + dev_err(&imx274->client->dev,
>> + "Exposure ctrl range update failed\n");
>> + goto unlock;
>> + }
>> +
>> + /* update exposure time accordingly */
>> + imx274_set_exposure(imx274, imx274->ctrls.exposure->val);
>> +
>> + dev_dbg(&imx274->client->dev, "set frame interval to %uus\n",
>> + fi->interval.numerator * 1000000
>> + / fi->interval.denominator);
>> + }
>> +
>> +unlock:
>> + mutex_unlock(&imx274->lock);
>> +
>> + return ret;
>> +}
>> +
>> +/**
>> + * imx274_load_default - load default control values
>> + * @priv: Pointer to device structure
>> + *
>> + * Return: 0 on success, errors otherwise
>> + */
>> +static int imx274_load_default(struct stimx274 *priv)
>> +{
>> + int ret;
>> +
>> + /* load default control values */
>> + priv->frame_interval.numerator = 1;
>> + priv->frame_interval.denominator = IMX274_DEF_FRAME_RATE;
>> + priv->ctrls.exposure->val = 1000000 / IMX274_DEF_FRAME_RATE;
>> + priv->ctrls.gain->val = IMX274_DEF_GAIN;
>> + priv->ctrls.vflip->val = 0;
>> + priv->ctrls.test_pattern->val = TEST_PATTERN_DISABLED;
>> +
>> + /* update frame rate */
>> + ret = imx274_set_frame_interval(priv,
>> + priv->frame_interval);
>> + if (ret)
>> + return ret;
>> +
>> + /* update exposure time */
>> + ret = v4l2_ctrl_s_ctrl(priv->ctrls.exposure, priv->ctrls.exposure->val);
>> + if (ret)
>> + return ret;
>> +
>> + /* update gain */
>> + ret = v4l2_ctrl_s_ctrl(priv->ctrls.gain, priv->ctrls.gain->val);
>> + if (ret)
>> + return ret;
>> +
>> + /* update vflip */
>> + ret = v4l2_ctrl_s_ctrl(priv->ctrls.vflip, priv->ctrls.vflip->val);
>> + if (ret)
>> + return ret;
>> +
>> + return 0;
>> +}
>> +
>> +/**
>> + * imx274_s_stream - It is used to start/stop the streaming.
>> + * @sd: V4L2 Sub device
>> + * @on: Flag (True / False)
>> + *
>> + * This function controls the start or stop of streaming for the
>> + * imx274 sensor.
>> + *
>> + * Return: 0 on success, errors otherwise
>> + */
>> +static int imx274_s_stream(struct v4l2_subdev *sd, int on)
>> +{
>> + struct stimx274 *imx274 = to_imx274(sd);
>> + int ret = 0;
>> +
>> + dev_dbg(&imx274->client->dev, "%s : %s, mode index = %d\n",
>> __func__,
>> + on ? "Stream Start" : "Stream Stop", imx274->mode_index);
>> +
>> + mutex_lock(&imx274->lock);
>> +
>> + if (on) {
>> + /* load mode registers */
>> + imx274_mode_regs(imx274, imx274->mode_index);
>> + if (ret)
>> + goto fail;
>> +
>> + /*
>> + * update frame rate & expsoure. if the last mode is different,
>> + * HMAX could be changed. As the result, frame rate & exposure
>> + * are changed.
>> + * gain is not affected.
>> + */
>> + ret = imx274_set_frame_interval(imx274,
>> + imx274->frame_interval);
>> + if (ret)
>> + goto fail;
>> +
>> + /* update exposure time */
>> + ret = __v4l2_ctrl_s_ctrl(imx274->ctrls.exposure,
>> + imx274->ctrls.exposure->val);
>> + if (ret)
>> + goto fail;
>> +
>> + /* start stream */
>> + ret = imx274_start_stream(imx274);
>> + if (ret)
>> + goto fail;
>> + } else {
>> + /* stop stream */
>> + ret = imx274_write_table(imx274,
>> +
>> mode_table[IMX274_MODE_STOP_STREAM]);
>> + if (ret)
>> + goto fail;
>> + }
>> +
>> + mutex_unlock(&imx274->lock);
>> + dev_dbg(&imx274->client->dev,
>> + "%s : Done: mode = %d\n", __func__, imx274->mode_index);
>> + return 0;
>> +
>> +fail:
>> + mutex_unlock(&imx274->lock);
>> + dev_err(&imx274->client->dev, "s_stream failed\n");
>> + return ret;
>> +}
>> +
>> +/*
>> + * imx274_get_frame_length - Function for obtaining current frame length
>> + * @priv: Pointer to device structure
>> + * @val: Pointer to obainted value
>> + *
>> + * frame_length = vmax x (svr + 1), in unit of hmax.
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_get_frame_length(struct stimx274 *priv, u32 *val)
>> +{
>> + int err;
>> + u16 svr;
>> + u32 vmax;
>> + u8 reg_val[3];
>> +
>> + /* svr */
>> + err = imx274_read_reg(priv, IMX274_SVR_REG_LSB, ®_val[0]);
>> + if (err)
>> + goto fail;
>> +
>> + err = imx274_read_reg(priv, IMX274_SVR_REG_MSB, ®_val[1]);
>> + if (err)
>> + goto fail;
>> +
>> + svr = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
>> +
>> + /* vmax */
>> + err = imx274_read_reg(priv, IMX274_FRAME_LENGTH_ADDR_3,
>> ®_val[0]);
>> + if (err)
>> + goto fail;
>> +
>> + err = imx274_read_reg(priv, IMX274_FRAME_LENGTH_ADDR_2,
>> ®_val[1]);
>> + if (err)
>> + goto fail;
>> +
>> + err = imx274_read_reg(priv, IMX274_FRAME_LENGTH_ADDR_1,
>> ®_val[2]);
>> + if (err)
>> + goto fail;
>> +
>> + vmax = ((reg_val[2] & IMX274_MASK_LSB_3_BITS) <<
>> IMX274_SHIFT_16_BITS)
>> + + (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
>> +
>> + *val = vmax * (svr + 1);
>> +
>> + return 0;
>> +
>> +fail:
>> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
>> + return err;
>> +}
>> +
>> +static int imx274_clamp_coarse_time(struct stimx274 *priv, u32 *val,
>> + u32 *frame_length)
>> +{
>> + int err;
>> +
>> + err = imx274_get_frame_length(priv, frame_length);
>> + if (err)
>> + return err;
>> +
>> + if (*frame_length < min_frame_len[priv->mode_index])
>> + *frame_length = min_frame_len[priv->mode_index];
>> +
>> + *val = *frame_length - *val; /* convert to raw shr */
>> + if (*val > *frame_length - IMX274_SHR_LIMIT_CONST)
>> + *val = *frame_length - IMX274_SHR_LIMIT_CONST;
>> + else if (*val < min_SHR[priv->mode_index])
>> + *val = min_SHR[priv->mode_index];
>> +
>> + return 0;
>> +}
>> +
>> +/*
>> + * imx274_set_digital gain - Function called when setting digital gain
>> + * @priv: Pointer to device structure
>> + * @dgain: Value of digital gain.
>> + *
>> + * Digital gain has only 4 steps: 1x, 2x, 4x, and 8x
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_set_digital_gain(struct stimx274 *priv, u32 dgain)
>> +{
>> + u8 reg_val;
>> +
>> + reg_val = ffs(dgain);
>> +
>> + if (reg_val)
>> + reg_val--;
>> +
>> + reg_val = clamp(reg_val, (u8)0, (u8)3);
>> +
>> + return imx274_write_reg(priv, IMX274_DIGITAL_GAIN_REG,
>> + reg_val & IMX274_MASK_LSB_4_BITS);
>> +}
>> +
>> +static inline void imx274_calculate_gain_regs(struct reg_8 regs[2], u16 gain)
>> +{
>> + regs->addr = IMX274_ANALOG_GAIN_ADDR_MSB;
>> + regs->val = (gain >> IMX274_SHIFT_8_BITS) &
>> IMX274_MASK_LSB_3_BITS;
>> +
>> + (regs + 1)->addr = IMX274_ANALOG_GAIN_ADDR_LSB;
>> + (regs + 1)->val = (gain) & IMX274_MASK_LSB_8_BITS;
>> +}
>> +
>> +/*
>> + * imx274_set_gain - Function called when setting gain
>> + * @priv: Pointer to device structure
>> + * @val: Value of gain. the real value = val << IMX274_GAIN_SHIFT;
>> + * @ctrl: v4l2 control pointer
>> + *
>> + * Set the gain based on input value.
>> + * The caller should hold the mutex lock imx274->lock if necessary
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl)
>> +{
>> + struct reg_8 reg_list[2];
>> + int err;
>> + u32 gain, analog_gain, digital_gain, gain_reg;
>> + int i;
>> +
>> + gain = (u32)(ctrl->val);
>> +
>> + dev_dbg(&priv->client->dev,
>> + "%s : input gain = %d.%d\n", __func__,
>> + gain >> IMX274_GAIN_SHIFT,
>> + ((gain & IMX274_GAIN_SHIFT_MASK) * 100) >>
>> IMX274_GAIN_SHIFT);
>> +
>> + if (gain > IMX274_MAX_DIGITAL_GAIN * IMX274_MAX_ANALOG_GAIN)
>> + gain = IMX274_MAX_DIGITAL_GAIN *
>> IMX274_MAX_ANALOG_GAIN;
>> + else if (gain < IMX274_MIN_GAIN)
>> + gain = IMX274_MIN_GAIN;
>> +
>> + if (gain <= IMX274_MAX_ANALOG_GAIN)
>> + digital_gain = 1;
>> + else if (gain <= IMX274_MAX_ANALOG_GAIN * 2)
>> + digital_gain = 2;
>> + else if (gain <= IMX274_MAX_ANALOG_GAIN * 4)
>> + digital_gain = 4;
>> + else
>> + digital_gain = IMX274_MAX_DIGITAL_GAIN;
>> +
>> + analog_gain = gain / digital_gain;
>> +
>> + dev_dbg(&priv->client->dev,
>> + "%s : digital gain = %d, analog gain = %d.%d\n",
>> + __func__, digital_gain, analog_gain >> IMX274_GAIN_SHIFT,
>> + ((analog_gain & IMX274_GAIN_SHIFT_MASK) * 100)
>> + >> IMX274_GAIN_SHIFT);
>> +
>> + err = imx274_set_digital_gain(priv, digital_gain);
>> + if (err)
>> + goto fail;
>> +
>> + /* convert to register value, refer to imx274 datasheet */
>> + gain_reg = (u32)IMX274_GAIN_CONST -
>> + (IMX274_GAIN_CONST << IMX274_GAIN_SHIFT) / analog_gain;
>> + if (gain_reg > IMX274_GAIN_REG_MAX)
>> + gain_reg = IMX274_GAIN_REG_MAX;
>> +
>> + imx274_calculate_gain_regs(reg_list, (u16)gain_reg);
>> +
>> + for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
>> + err = imx274_write_reg(priv, reg_list[i].addr,
>> + reg_list[i].val);
>> + if (err)
>> + goto fail;
>> + }
>> +
>> + if (IMX274_GAIN_CONST - gain_reg == 0) {
>> + err = -EINVAL;
>> + goto fail;
>> + }
>> +
>> + /* convert register value back to gain value */
>> + ctrl->val = (IMX274_GAIN_CONST << IMX274_GAIN_SHIFT)
>> + / (IMX274_GAIN_CONST - gain_reg) * digital_gain;
>> +
>> + dev_dbg(&priv->client->dev,
>> + "%s : GAIN control success, gain_reg = %d, new gain = %d\n",
>> + __func__, gain_reg, ctrl->val);
>> +
>> + return 0;
>> +
>> +fail:
>> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
>> + return err;
>> +}
>> +
>> +static inline void imx274_calculate_coarse_time_regs(struct reg_8 regs[2],
>> + u32 coarse_time)
>> +{
>> + regs->addr = IMX274_COARSE_TIME_ADDR_MSB;
>> + regs->val = (coarse_time >> IMX274_SHIFT_8_BITS)
>> + & IMX274_MASK_LSB_8_BITS;
>> + (regs + 1)->addr = IMX274_COARSE_TIME_ADDR_LSB;
>> + (regs + 1)->val = (coarse_time) & IMX274_MASK_LSB_8_BITS;
>> +}
>> +
>> +/*
>> + * imx274_set_coarse_time - Function called when setting SHR value
>> + * @priv: Pointer to device structure
>> + * @val: Value for exposure time in number of line_length, or [HMAX]
>> + *
>> + * Set SHR value based on input value.
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_set_coarse_time(struct stimx274 *priv, u32 *val)
>> +{
>> + struct reg_8 reg_list[2];
>> + int err;
>> + u32 coarse_time, frame_length;
>> + int i;
>> +
>> + coarse_time = *val;
>> +
>> + /* convert exposure_time to appropriate SHR value */
>> + err = imx274_clamp_coarse_time(priv, &coarse_time, &frame_length);
>> + if (err)
>> + goto fail;
>> +
>> + /* prepare SHR registers */
>> + imx274_calculate_coarse_time_regs(reg_list, coarse_time);
>> +
>> + /* write to SHR registers */
>> + for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
>> + err = imx274_write_reg(priv, reg_list[i].addr,
>> + reg_list[i].val);
>> + if (err)
>> + goto fail;
>> + }
>> +
>> + *val = frame_length - coarse_time;
>> + return 0;
>> +
>> +fail:
>> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
>> + return err;
>> +}
>> +
>> +/*
>> + * imx274_set_exposure - Function called when setting exposure time
>> + * @priv: Pointer to device structure
>> + * @val: Variable for exposure time, in the unit of micro-second
>> + *
>> + * Set exposure time based on input value.
>> + * The caller should hold the mutex lock imx274->lock if necessary
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_set_exposure(struct stimx274 *priv, int val)
>> +{
>> + int err;
>> + u16 hmax;
>> + u8 reg_val[2];
>> + u32 coarse_time; /* exposure time in unit of line (HMAX)*/
>> +
>> + dev_dbg(&priv->client->dev,
>> + "%s : EXPOSURE control input = %d\n", __func__, val);
>> +
>> + /* step 1: convert input exposure_time (val) into number of 1[HMAX] */
>> +
>> + /* obtain HMAX value */
>> + err = imx274_read_reg(priv, IMX274_HMAX_REG_LSB, ®_val[0]);
>> + if (err)
>> + goto fail;
>> + err = imx274_read_reg(priv, IMX274_HMAX_REG_MSB, ®_val[1]);
>> + if (err)
>> + goto fail;
>> + hmax = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
>> + if (hmax == 0) {
>> + err = -EINVAL;
>> + goto fail;
>> + }
>> +
>> + coarse_time = (IMX274_PIXCLK_CONST1 / IMX274_PIXCLK_CONST2 *
>> val
>> + - nocpiop[priv->mode_index]) / hmax;
>> +
>> + /* step 2: convert exposure_time into SHR value */
>> +
>> + /* set SHR */
>> + err = imx274_set_coarse_time(priv, &coarse_time);
>> + if (err)
>> + goto fail;
>> +
>> + priv->ctrls.exposure->val =
>> + (coarse_time * hmax + nocpiop[priv->mode_index])
>> + / (IMX274_PIXCLK_CONST1 /
>> IMX274_PIXCLK_CONST2);
>> +
>> + dev_dbg(&priv->client->dev,
>> + "%s : EXPOSURE control success\n", __func__);
>> + return 0;
>> +
>> +fail:
>> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
>> +
>> + return err;
>> +}
>> +
>> +/*
>> + * imx274_set_vflip - Function called when setting vertical flip
>> + * @priv: Pointer to device structure
>> + * @val: Value for vflip setting
>> + *
>> + * Set vertical flip based on input value.
>> + * val = 0: normal, no vertical flip
>> + * val = 1: vertical flip enabled
>> + * The caller should hold the mutex lock imx274->lock if necessary
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_set_vflip(struct stimx274 *priv, int val)
>> +{
>> + int err;
>> +
>> + err = imx274_write_reg(priv, IMX274_VFLIP_REG, val);
>> + if (err) {
>> + dev_err(&priv->client->dev, "VFILP control error\n");
>> + return err;
>> + }
>> +
>> + dev_dbg(&priv->client->dev,
>> + "%s : VFLIP control success\n", __func__);
>> +
>> + return 0;
>> +}
>> +
>> +/*
>> + * imx274_set_test_pattern - Function called when setting test pattern
>> + * @priv: Pointer to device structure
>> + * @val: Variable for test pattern
>> + *
>> + * Set to different test patterns based on input value.
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_set_test_pattern(struct stimx274 *priv, int val)
>> +{
>> + int err = 0;
>> +
>> + if (val == TEST_PATTERN_DISABLED) {
>> + err = imx274_write_table(priv, imx274_tp_disabled);
>> + } else if (val <= TEST_PATTERN_V_COLOR_BARS) {
>> + err = imx274_write_reg(priv, IMX274_TEST_PATTERN_REG, val -
>> 1);
>> + if (!err)
>> + err = imx274_write_table(priv, imx274_tp_regs);
>> + } else {
>> + err = -EINVAL;
>> + }
>> +
>> + if (!err)
>> + dev_dbg(&priv->client->dev,
>> + "%s : TEST PATTERN control success\n", __func__);
>> + else
>> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
>> +
>> + return err;
>> +}
>> +
>> +static inline void imx274_calculate_frame_length_regs(struct reg_8 regs[3],
>> + u32 frame_length)
>> +{
>> + regs->addr = IMX274_FRAME_LENGTH_ADDR_1;
>> + regs->val = (frame_length >> IMX274_SHIFT_16_BITS)
>> + & IMX274_MASK_LSB_4_BITS;
>> + (regs + 1)->addr = IMX274_FRAME_LENGTH_ADDR_2;
>> + (regs + 1)->val = (frame_length >> IMX274_SHIFT_8_BITS)
>> + & IMX274_MASK_LSB_8_BITS;
>> + (regs + 2)->addr = IMX274_FRAME_LENGTH_ADDR_3;
>> + (regs + 2)->val = (frame_length) & IMX274_MASK_LSB_8_BITS;
>> +}
>> +
>> +/*
>> + * imx274_set_frame_length - Function called when setting frame length
>> + * @priv: Pointer to device structure
>> + * @val: Variable for frame length (= VMAX, i.e. vertical drive period length)
>> + *
>> + * Set frame length based on input value.
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_set_frame_length(struct stimx274 *priv, u32 val)
>> +{
>> + struct reg_8 reg_list[3];
>> + int err;
>> + u32 frame_length;
>> + int i;
>> +
>> + dev_dbg(&priv->client->dev, "%s : input length = %d\n",
>> + __func__, val);
>> +
>> + frame_length = (u32)val;
>> +
>> + imx274_calculate_frame_length_regs(reg_list, frame_length);
>> + for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
>> + err = imx274_write_reg(priv, reg_list[i].addr,
>> + reg_list[i].val);
>> + if (err)
>> + goto fail;
>> + }
>> +
>> + return 0;
>> +
>> +fail:
>> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
>> + return err;
>> +}
>> +
>> +/*
>> + * imx274_set_frame_interval - Function called when setting frame interval
>> + * @priv: Pointer to device structure
>> + * @frame_interval: Variable for frame interval
>> + *
>> + * Change frame interval by updating VMAX value
>> + * The caller should hold the mutex lock imx274->lock if necessary
>> + *
>> + * Return: 0 on success
>> + */
>> +static int imx274_set_frame_interval(struct stimx274 *priv,
>> + struct v4l2_fract frame_interval)
>> +{
>> + int err;
>> + u32 frame_length, req_frame_rate;
>> + u16 svr;
>> + u16 hmax;
>> + u8 reg_val[2];
>> +
>> + dev_dbg(&priv->client->dev, "%s: input frame interval = %d / %d",
>> + __func__, frame_interval.numerator,
>> + frame_interval.denominator);
>> +
>> + if (frame_interval.numerator == 0) {
>> + err = -EINVAL;
>> + goto fail;
>> + }
>> +
>> + req_frame_rate = (u32)(frame_interval.denominator
>> + / frame_interval.numerator);
>> +
>> + /* boundary check */
>> + if (req_frame_rate > max_frame_rate[priv->mode_index]) {
>> + frame_interval.numerator = 1;
>> + frame_interval.denominator =
>> + max_frame_rate[priv->mode_index];
>> + } else if (req_frame_rate < IMX274_MIN_FRAME_RATE) {
>> + frame_interval.numerator = 1;
>> + frame_interval.denominator = IMX274_MIN_FRAME_RATE;
>> + }
>> +
>> + /*
>> + * VMAX = 1/frame_rate x 72M / (SVR+1) / HMAX
>> + * frame_length (i.e. VMAX) = (frame_interval) x 72M /(SVR+1) / HMAX
>> + */
>> +
>> + /* SVR */
>> + err = imx274_read_reg(priv, IMX274_SVR_REG_LSB, ®_val[0]);
>> + if (err)
>> + goto fail;
>> + err = imx274_read_reg(priv, IMX274_SVR_REG_MSB, ®_val[1]);
>> + if (err)
>> + goto fail;
>> + svr = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
>> + dev_dbg(&priv->client->dev,
>> + "%s : register SVR = %d\n", __func__, svr);
>> +
>> + /* HMAX */
>> + err = imx274_read_reg(priv, IMX274_HMAX_REG_LSB, ®_val[0]);
>> + if (err)
>> + goto fail;
>> + err = imx274_read_reg(priv, IMX274_HMAX_REG_MSB, ®_val[1]);
>> + if (err)
>> + goto fail;
>> + hmax = (reg_val[1] << IMX274_SHIFT_8_BITS) + reg_val[0];
>> + dev_dbg(&priv->client->dev,
>> + "%s : register HMAX = %d\n", __func__, hmax);
>> +
>> + if (hmax == 0 || frame_interval.denominator == 0) {
>> + err = -EINVAL;
>> + goto fail;
>> + }
>> +
>> + frame_length = IMX274_PIXCLK_CONST1 / (svr + 1) / hmax
>> + * frame_interval.numerator
>> + / frame_interval.denominator;
>> +
>> + err = imx274_set_frame_length(priv, frame_length);
>> + if (err)
>> + goto fail;
>> +
>> + priv->frame_interval = frame_interval;
>> + return 0;
>> +
>> +fail:
>> + dev_err(&priv->client->dev, "%s error = %d\n", __func__, err);
>> + return err;
>> +}
>> +
>> +static const struct v4l2_subdev_pad_ops imx274_pad_ops = {
>> + .get_fmt = imx274_get_fmt,
>> + .set_fmt = imx274_set_fmt,
>> +};
>> +
>> +static const struct v4l2_subdev_video_ops imx274_video_ops = {
>> + .g_frame_interval = imx274_g_frame_interval,
>> + .s_frame_interval = imx274_s_frame_interval,
>> + .s_stream = imx274_s_stream,
>> +};
>> +
>> +static const struct v4l2_subdev_ops imx274_subdev_ops = {
>> + .pad = &imx274_pad_ops,
>> + .video = &imx274_video_ops,
>> +};
>> +
>> +static const struct v4l2_ctrl_ops imx274_ctrl_ops = {
>> + .s_ctrl = imx274_s_ctrl,
>> +};
>> +
>> +static const struct of_device_id imx274_of_id_table[] = {
>> + { .compatible = "sony,imx274" },
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(of, imx274_of_id_table);
>> +
>> +static const struct i2c_device_id imx274_id[] = {
>> + { "IMX274", 0 },
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(i2c, imx274_id);
>> +
>> +static int imx274_probe(struct i2c_client *client,
>> + const struct i2c_device_id *id)
>> +{
>> + struct v4l2_subdev *sd;
>> + struct stimx274 *imx274;
>> + int ret;
>> +
>> + /* initialize imx274 */
>> + imx274 = devm_kzalloc(&client->dev, sizeof(*imx274), GFP_KERNEL);
>> + if (!imx274)
>> + return -ENOMEM;
>> +
>> + mutex_init(&imx274->lock);
>> +
>> + /* initialize regmap */
>> + imx274->regmap = devm_regmap_init_i2c(client,
>> &imx274_regmap_config);
>> + if (IS_ERR(imx274->regmap)) {
>> + dev_err(&client->dev,
>> + "regmap init failed: %ld\n", PTR_ERR(imx274-
>> >regmap));
>> + ret = -ENODEV;
>> + goto err_regmap;
>> + }
>> +
>> + /* initialize subdevice */
>> + imx274->client = client;
>> + sd = &imx274->sd;
>> + v4l2_i2c_subdev_init(sd, client, &imx274_subdev_ops);
>> + strlcpy(sd->name, DRIVER_NAME, sizeof(sd->name));
>> + sd->flags |= V4L2_SUBDEV_FL_HAS_DEVNODE |
>> V4L2_SUBDEV_FL_HAS_EVENTS;
>> +
>> + /* initialize subdev media pad */
>> + imx274->pad.flags = MEDIA_PAD_FL_SOURCE;
>> + sd->entity.function = MEDIA_ENT_F_CAM_SENSOR;
>> + ret = media_entity_pads_init(&sd->entity, 1, &imx274->pad);
>> + if (ret < 0) {
>> + dev_err(&client->dev,
>> + "%s : media entity init Failed %d\n", __func__, ret);
>> + goto err_regmap;
>> + }
>> +
>> + /* initialize sensor reset gpio */
>> + imx274->reset_gpio = devm_gpiod_get_optional(&client->dev, "reset",
>> + GPIOD_OUT_HIGH);
>> + if (IS_ERR(imx274->reset_gpio)) {
>> + if (PTR_ERR(imx274->reset_gpio) != -EPROBE_DEFER)
>> + dev_err(&client->dev, "Reset GPIO not setup in DT");
>> + ret = PTR_ERR(imx274->reset_gpio);
>> + goto err_me;
>> + }
>> +
>> + /* pull sensor out of reset */
>> + imx274_reset(imx274, 1);
>> +
>> + /* initialize controls */
>> + ret = v4l2_ctrl_handler_init(&imx274->ctrls.handler, 2);
>> + if (ret < 0) {
>> + dev_err(&client->dev,
>> + "%s : ctrl handler init Failed\n", __func__);
>> + goto err_me;
>> + }
>> +
>> + imx274->ctrls.handler.lock = &imx274->lock;
>> +
>> + /* add new controls */
>> + imx274->ctrls.test_pattern = v4l2_ctrl_new_std_menu_items(
>> + &imx274->ctrls.handler, &imx274_ctrl_ops,
>> + V4L2_CID_TEST_PATTERN,
>> + ARRAY_SIZE(tp_qmenu) - 1, 0, 0, tp_qmenu);
>> +
>> + imx274->ctrls.gain = v4l2_ctrl_new_std(
>> + &imx274->ctrls.handler,
>> + &imx274_ctrl_ops,
>> + V4L2_CID_GAIN, IMX274_MIN_GAIN,
>> + IMX274_MAX_DIGITAL_GAIN * IMX274_MAX_ANALOG_GAIN,
>> 1,
>> + IMX274_DEF_GAIN);
>> +
>> + imx274->ctrls.exposure = v4l2_ctrl_new_std(
>> + &imx274->ctrls.handler,
>> + &imx274_ctrl_ops,
>> + V4L2_CID_EXPOSURE, IMX274_MIN_EXPOSURE_TIME,
>> + 1000000 / IMX274_DEF_FRAME_RATE, 1,
>> + IMX274_MIN_EXPOSURE_TIME);
>> +
>> + imx274->ctrls.vflip = v4l2_ctrl_new_std(
>> + &imx274->ctrls.handler,
>> + &imx274_ctrl_ops,
>> + V4L2_CID_VFLIP, 0, 1, 1, 0);
>> +
>> + imx274->sd.ctrl_handler = &imx274->ctrls.handler;
>> + if (imx274->ctrls.handler.error) {
>> + ret = imx274->ctrls.handler.error;
>> + goto err_ctrls;
>> + }
>> +
>> + /* setup default controls */
>> + ret = v4l2_ctrl_handler_setup(&imx274->ctrls.handler);
>> + if (ret) {
>> + dev_err(&client->dev,
>> + "Error %d setup default controls\n", ret);
>> + goto err_ctrls;
>> + }
>> +
>> + /* initialize format */
>> + imx274->mode_index = IMX274_MODE_3840X2160;
>> + imx274->format.width = imx274_formats[0].size.width;
>> + imx274->format.height = imx274_formats[0].size.height;
>> + imx274->format.field = V4L2_FIELD_NONE;
>> + imx274->format.code = MEDIA_BUS_FMT_SRGGB10_1X10;
>> + imx274->format.colorspace = V4L2_COLORSPACE_SRGB;
>> + imx274->frame_interval.numerator = 1;
>> + imx274->frame_interval.denominator = IMX274_DEF_FRAME_RATE;
>> +
>> + /* load default control values */
>> + ret = imx274_load_default(imx274);
>> + if (ret) {
>> + dev_err(&client->dev,
>> + "%s : imx274_load_default failed %d\n",
>> + __func__, ret);
>> + goto err_ctrls;
>> + }
>> +
>> + /* register subdevice */
>> + ret = v4l2_async_register_subdev(sd);
>> + if (ret < 0) {
>> + dev_err(&client->dev,
>> + "%s : v4l2_async_register_subdev failed %d\n",
>> + __func__, ret);
>> + goto err_ctrls;
>> + }
>> +
>> + dev_info(&client->dev, "imx274 : imx274 probe success !\n");
>> + return 0;
>> +
>> +err_ctrls:
>> + v4l2_ctrl_handler_free(&imx274->ctrls.handler);
>> +err_me:
>> + media_entity_cleanup(&sd->entity);
>> +err_regmap:
>> + mutex_destroy(&imx274->lock);
>> + return ret;
>> +}
>> +
>> +static int imx274_remove(struct i2c_client *client)
>> +{
>> + struct v4l2_subdev *sd = i2c_get_clientdata(client);
>> + struct stimx274 *imx274 = to_imx274(sd);
>> +
>> + /* stop stream */
>> + imx274_write_table(imx274,
>> mode_table[IMX274_MODE_STOP_STREAM]);
>> +
>> + v4l2_async_unregister_subdev(sd);
>> + v4l2_ctrl_handler_free(&imx274->ctrls.handler);
>> + media_entity_cleanup(&sd->entity);
>> + mutex_destroy(&imx274->lock);
>> + return 0;
>> +}
>> +
>> +static struct i2c_driver imx274_i2c_driver = {
>> + .driver = {
>> + .name = DRIVER_NAME,
>> + .of_match_table = imx274_of_id_table,
>> + },
>> + .probe = imx274_probe,
>> + .remove = imx274_remove,
>> + .id_table = imx274_id,
>> +};
>> +
>> +module_i2c_driver(imx274_i2c_driver);
>> +
>> +MODULE_AUTHOR("Leon Luo <leonl-+iDsysdzPxZTt5vwKEO50dBPR1lH4CV8@public.gmane.org>");
>> +MODULE_DESCRIPTION("IMX274 CMOS Image Sensor driver");
>> +MODULE_LICENSE("GPL v2");
>> --
>> 2.14.0.rc1
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-11-03 6:08 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-10-26 6:51 [PATCH v9 1/2] media:imx274 device tree binding file Leon Luo
2017-10-26 6:51 ` [PATCH v9 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor Leon Luo
2017-11-03 4:36 ` Vishal Sagar
[not found] ` <DM2PR0201MB0685547EBAD0AF883C4BADD9A75D0-mGsVAZgi3o+bjIxu8aGtFxrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
2017-11-03 6:08 ` Leon Luo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).