From: Andy Shevchenko <andy@kernel.org>
To: Hans de Goede <hdegoede@redhat.com>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Tsuchiya Yuto <kitakar@gmail.com>,
Yury Luneff <yury.lunev@gmail.com>,
Nable <nable.maininbox@googlemail.com>,
andrey.i.trufanov@gmail.com, Fabio Aiuto <fabioaiuto83@gmail.com>,
linux-media@vger.kernel.org, linux-staging@lists.linux.dev
Subject: Re: [PATCH 08/57] media: atomisp: Remove custom ATOMISP_IOC_ISP_MAKERNOTE ioctl
Date: Mon, 23 Jan 2023 16:30:08 +0200 [thread overview]
Message-ID: <Y86Z8GGiJM/GCzAO@smile.fi.intel.com> (raw)
In-Reply-To: <20230123125205.622152-9-hdegoede@redhat.com>
On Mon, Jan 23, 2023 at 01:51:16PM +0100, Hans de Goede wrote:
> This ioctl simply returns a couple of fixed sensor parameters.
>
> With libcamera these fixed parameters are instead stored in a table
> with sensor-name to parameters mappings (camera_sensor_properties.cpp),
> so these custom ioctl is not necessary; and it currently has no users.
s/ioctl is/IOCTLs are/
or
s/these/this/
> Remove the ioctl and also remove the custom v4l2-ctrls underpinning
> the ioctl.
>
> This is part of a patch-series which tries to remove atomisp specific /
> custom code from the sensor drivers, with as end goal to make the atomisp
> drivers regular camera sensor drivers.
Reviewed-by: Andy Shevchenko <andy@kernel.org>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
> .../media/atomisp/i2c/atomisp-gc0310.c | 63 ------------------
> .../media/atomisp/i2c/atomisp-gc2235.c | 63 ------------------
> .../media/atomisp/i2c/atomisp-mt9m114.c | 64 -------------------
> .../media/atomisp/i2c/atomisp-ov2680.c | 64 -------------------
> .../media/atomisp/i2c/atomisp-ov2722.c | 63 ------------------
> drivers/staging/media/atomisp/i2c/gc0310.h | 3 -
> drivers/staging/media/atomisp/i2c/gc2235.h | 3 -
> drivers/staging/media/atomisp/i2c/mt9m114.h | 3 -
> drivers/staging/media/atomisp/i2c/ov2680.h | 3 -
> drivers/staging/media/atomisp/i2c/ov2722.h | 3 -
> .../media/atomisp/i2c/ov5693/atomisp-ov5693.c | 63 ------------------
> .../media/atomisp/include/linux/atomisp.h | 20 ------
> .../staging/media/atomisp/pci/atomisp_cmd.c | 36 -----------
> .../staging/media/atomisp/pci/atomisp_cmd.h | 3 -
> .../staging/media/atomisp/pci/atomisp_ioctl.c | 7 --
> 15 files changed, 461 deletions(-)
>
> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-gc0310.c b/drivers/staging/media/atomisp/i2c/atomisp-gc0310.c
> index 87a634bf9ff5..a9c4724a9358 100644
> --- a/drivers/staging/media/atomisp/i2c/atomisp-gc0310.c
> +++ b/drivers/staging/media/atomisp/i2c/atomisp-gc0310.c
> @@ -241,27 +241,6 @@ static int gc0310_write_reg_array(struct i2c_client *client,
> return __gc0310_flush_reg_array(client, &ctrl);
> }
>
> -static int gc0310_g_focal(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (GC0310_FOCAL_LENGTH_NUM << 16) | GC0310_FOCAL_LENGTH_DEM;
> - return 0;
> -}
> -
> -static int gc0310_g_fnumber(struct v4l2_subdev *sd, s32 *val)
> -{
> - /*const f number for imx*/
> - *val = (GC0310_F_NUMBER_DEFAULT_NUM << 16) | GC0310_F_NUMBER_DEM;
> - return 0;
> -}
> -
> -static int gc0310_g_fnumber_range(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (GC0310_F_NUMBER_DEFAULT_NUM << 24) |
> - (GC0310_F_NUMBER_DEM << 16) |
> - (GC0310_F_NUMBER_DEFAULT_NUM << 8) | GC0310_F_NUMBER_DEM;
> - return 0;
> -}
> -
> static int gc0310_g_bin_factor_x(struct v4l2_subdev *sd, s32 *val)
> {
> struct gc0310_device *dev = to_gc0310_sensor(sd);
> @@ -596,15 +575,6 @@ static int gc0310_g_volatile_ctrl(struct v4l2_ctrl *ctrl)
> case V4L2_CID_EXPOSURE_ABSOLUTE:
> ret = gc0310_q_exposure(&dev->sd, &ctrl->val);
> break;
> - case V4L2_CID_FOCAL_ABSOLUTE:
> - ret = gc0310_g_focal(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_ABSOLUTE:
> - ret = gc0310_g_fnumber(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_RANGE:
> - ret = gc0310_g_fnumber_range(&dev->sd, &ctrl->val);
> - break;
> case V4L2_CID_BIN_FACTOR_HORZ:
> ret = gc0310_g_bin_factor_x(&dev->sd, &ctrl->val);
> break;
> @@ -655,39 +625,6 @@ static const struct v4l2_ctrl_config gc0310_controls[] = {
> .step = 1,
> .def = 0,
> },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FOCAL_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "focal length",
> - .min = GC0310_FOCAL_LENGTH_DEFAULT,
> - .max = GC0310_FOCAL_LENGTH_DEFAULT,
> - .step = 0x01,
> - .def = GC0310_FOCAL_LENGTH_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number",
> - .min = GC0310_F_NUMBER_DEFAULT,
> - .max = GC0310_F_NUMBER_DEFAULT,
> - .step = 0x01,
> - .def = GC0310_F_NUMBER_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_RANGE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number range",
> - .min = GC0310_F_NUMBER_RANGE,
> - .max = GC0310_F_NUMBER_RANGE,
> - .step = 0x01,
> - .def = GC0310_F_NUMBER_RANGE,
> - .flags = 0,
> - },
> {
> .ops = &ctrl_ops,
> .id = V4L2_CID_BIN_FACTOR_HORZ,
> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-gc2235.c b/drivers/staging/media/atomisp/i2c/atomisp-gc2235.c
> index 4d5a7e335f85..e6df10bcab8c 100644
> --- a/drivers/staging/media/atomisp/i2c/atomisp-gc2235.c
> +++ b/drivers/staging/media/atomisp/i2c/atomisp-gc2235.c
> @@ -220,27 +220,6 @@ static int gc2235_write_reg_array(struct i2c_client *client,
> return __gc2235_flush_reg_array(client, &ctrl);
> }
>
> -static int gc2235_g_focal(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (GC2235_FOCAL_LENGTH_NUM << 16) | GC2235_FOCAL_LENGTH_DEM;
> - return 0;
> -}
> -
> -static int gc2235_g_fnumber(struct v4l2_subdev *sd, s32 *val)
> -{
> - /* const f number for imx */
> - *val = (GC2235_F_NUMBER_DEFAULT_NUM << 16) | GC2235_F_NUMBER_DEM;
> - return 0;
> -}
> -
> -static int gc2235_g_fnumber_range(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (GC2235_F_NUMBER_DEFAULT_NUM << 24) |
> - (GC2235_F_NUMBER_DEM << 16) |
> - (GC2235_F_NUMBER_DEFAULT_NUM << 8) | GC2235_F_NUMBER_DEM;
> - return 0;
> -}
> -
> static int gc2235_get_intg_factor(struct i2c_client *client,
> struct camera_mipi_info *info,
> const struct gc2235_resolution *res)
> @@ -467,15 +446,6 @@ static int gc2235_g_volatile_ctrl(struct v4l2_ctrl *ctrl)
> case V4L2_CID_EXPOSURE_ABSOLUTE:
> ret = gc2235_q_exposure(&dev->sd, &ctrl->val);
> break;
> - case V4L2_CID_FOCAL_ABSOLUTE:
> - ret = gc2235_g_focal(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_ABSOLUTE:
> - ret = gc2235_g_fnumber(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_RANGE:
> - ret = gc2235_g_fnumber_range(&dev->sd, &ctrl->val);
> - break;
> default:
> ret = -EINVAL;
> }
> @@ -499,39 +469,6 @@ static struct v4l2_ctrl_config gc2235_controls[] = {
> .def = 0x00,
> .flags = 0,
> },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FOCAL_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "focal length",
> - .min = GC2235_FOCAL_LENGTH_DEFAULT,
> - .max = GC2235_FOCAL_LENGTH_DEFAULT,
> - .step = 0x01,
> - .def = GC2235_FOCAL_LENGTH_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number",
> - .min = GC2235_F_NUMBER_DEFAULT,
> - .max = GC2235_F_NUMBER_DEFAULT,
> - .step = 0x01,
> - .def = GC2235_F_NUMBER_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_RANGE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number range",
> - .min = GC2235_F_NUMBER_RANGE,
> - .max = GC2235_F_NUMBER_RANGE,
> - .step = 0x01,
> - .def = GC2235_F_NUMBER_RANGE,
> - .flags = 0,
> - },
> };
>
> static int __gc2235_init(struct v4l2_subdev *sd)
> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c b/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
> index a0e8e94b2412..eb34b5cadb33 100644
> --- a/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
> +++ b/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
> @@ -841,28 +841,6 @@ static int mt9m114_set_fmt(struct v4l2_subdev *sd,
> return 0;
> }
>
> -/* TODO: Update to SOC functions, remove exposure and gain */
> -static int mt9m114_g_focal(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (MT9M114_FOCAL_LENGTH_NUM << 16) | MT9M114_FOCAL_LENGTH_DEM;
> - return 0;
> -}
> -
> -static int mt9m114_g_fnumber(struct v4l2_subdev *sd, s32 *val)
> -{
> - /* const f number for mt9m114 */
> - *val = (MT9M114_F_NUMBER_DEFAULT_NUM << 16) | MT9M114_F_NUMBER_DEM;
> - return 0;
> -}
> -
> -static int mt9m114_g_fnumber_range(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (MT9M114_F_NUMBER_DEFAULT_NUM << 24) |
> - (MT9M114_F_NUMBER_DEM << 16) |
> - (MT9M114_F_NUMBER_DEFAULT_NUM << 8) | MT9M114_F_NUMBER_DEM;
> - return 0;
> -}
> -
> /* Horizontal flip the image. */
> static int mt9m114_g_hflip(struct v4l2_subdev *sd, s32 *val)
> {
> @@ -1271,15 +1249,6 @@ static int mt9m114_g_volatile_ctrl(struct v4l2_ctrl *ctrl)
> case V4L2_CID_HFLIP:
> ret = mt9m114_g_hflip(&dev->sd, &ctrl->val);
> break;
> - case V4L2_CID_FOCAL_ABSOLUTE:
> - ret = mt9m114_g_focal(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_ABSOLUTE:
> - ret = mt9m114_g_fnumber(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_RANGE:
> - ret = mt9m114_g_fnumber_range(&dev->sd, &ctrl->val);
> - break;
> case V4L2_CID_EXPOSURE_ABSOLUTE:
> ret = mt9m114_g_exposure(&dev->sd, &ctrl->val);
> break;
> @@ -1331,39 +1300,6 @@ static struct v4l2_ctrl_config mt9m114_controls[] = {
> .step = 1,
> .def = 0,
> },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FOCAL_ABSOLUTE,
> - .name = "focal length",
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .min = MT9M114_FOCAL_LENGTH_DEFAULT,
> - .max = MT9M114_FOCAL_LENGTH_DEFAULT,
> - .step = 1,
> - .def = MT9M114_FOCAL_LENGTH_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_ABSOLUTE,
> - .name = "f-number",
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .min = MT9M114_F_NUMBER_DEFAULT,
> - .max = MT9M114_F_NUMBER_DEFAULT,
> - .step = 1,
> - .def = MT9M114_F_NUMBER_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_RANGE,
> - .name = "f-number range",
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .min = MT9M114_F_NUMBER_RANGE,
> - .max = MT9M114_F_NUMBER_RANGE,
> - .step = 1,
> - .def = MT9M114_F_NUMBER_RANGE,
> - .flags = 0,
> - },
> {
> .ops = &ctrl_ops,
> .id = V4L2_CID_EXPOSURE_ABSOLUTE,
> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-ov2680.c b/drivers/staging/media/atomisp/i2c/atomisp-ov2680.c
> index fa1de45b7a2d..39f86c7fd12e 100644
> --- a/drivers/staging/media/atomisp/i2c/atomisp-ov2680.c
> +++ b/drivers/staging/media/atomisp/i2c/atomisp-ov2680.c
> @@ -119,28 +119,6 @@ static int ov2680_write_reg_array(struct i2c_client *client,
> return 0;
> }
>
> -static int ov2680_g_focal(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (OV2680_FOCAL_LENGTH_NUM << 16) | OV2680_FOCAL_LENGTH_DEM;
> - return 0;
> -}
> -
> -static int ov2680_g_fnumber(struct v4l2_subdev *sd, s32 *val)
> -{
> - /* const f number for ov2680 */
> -
> - *val = (OV2680_F_NUMBER_DEFAULT_NUM << 16) | OV2680_F_NUMBER_DEM;
> - return 0;
> -}
> -
> -static int ov2680_g_fnumber_range(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (OV2680_F_NUMBER_DEFAULT_NUM << 24) |
> - (OV2680_F_NUMBER_DEM << 16) |
> - (OV2680_F_NUMBER_DEFAULT_NUM << 8) | OV2680_F_NUMBER_DEM;
> - return 0;
> -}
> -
> static int ov2680_g_bin_factor_x(struct v4l2_subdev *sd, s32 *val)
> {
> struct ov2680_device *dev = to_ov2680_sensor(sd);
> @@ -517,15 +495,6 @@ static int ov2680_g_volatile_ctrl(struct v4l2_ctrl *ctrl)
> case V4L2_CID_EXPOSURE_ABSOLUTE:
> ret = ov2680_q_exposure(&dev->sd, &ctrl->val);
> break;
> - case V4L2_CID_FOCAL_ABSOLUTE:
> - ret = ov2680_g_focal(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_ABSOLUTE:
> - ret = ov2680_g_fnumber(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_RANGE:
> - ret = ov2680_g_fnumber_range(&dev->sd, &ctrl->val);
> - break;
> case V4L2_CID_BIN_FACTOR_HORZ:
> ret = ov2680_g_bin_factor_x(&dev->sd, &ctrl->val);
> break;
> @@ -556,39 +525,6 @@ static const struct v4l2_ctrl_config ov2680_controls[] = {
> .def = 0x00,
> .flags = 0,
> },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FOCAL_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "focal length",
> - .min = OV2680_FOCAL_LENGTH_DEFAULT,
> - .max = OV2680_FOCAL_LENGTH_DEFAULT,
> - .step = 0x01,
> - .def = OV2680_FOCAL_LENGTH_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number",
> - .min = OV2680_F_NUMBER_DEFAULT,
> - .max = OV2680_F_NUMBER_DEFAULT,
> - .step = 0x01,
> - .def = OV2680_F_NUMBER_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_RANGE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number range",
> - .min = OV2680_F_NUMBER_RANGE,
> - .max = OV2680_F_NUMBER_RANGE,
> - .step = 0x01,
> - .def = OV2680_F_NUMBER_RANGE,
> - .flags = 0,
> - },
> {
> .ops = &ctrl_ops,
> .id = V4L2_CID_BIN_FACTOR_HORZ,
> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c b/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c
> index 887b6f99f6ca..47eefaccbe0b 100644
> --- a/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c
> +++ b/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c
> @@ -261,27 +261,6 @@ static int ov2722_write_reg_array(struct i2c_client *client,
> return __ov2722_flush_reg_array(client, &ctrl);
> }
>
> -static int ov2722_g_focal(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (OV2722_FOCAL_LENGTH_NUM << 16) | OV2722_FOCAL_LENGTH_DEM;
> - return 0;
> -}
> -
> -static int ov2722_g_fnumber(struct v4l2_subdev *sd, s32 *val)
> -{
> - /*const f number for imx*/
> - *val = (OV2722_F_NUMBER_DEFAULT_NUM << 16) | OV2722_F_NUMBER_DEM;
> - return 0;
> -}
> -
> -static int ov2722_g_fnumber_range(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (OV2722_F_NUMBER_DEFAULT_NUM << 24) |
> - (OV2722_F_NUMBER_DEM << 16) |
> - (OV2722_F_NUMBER_DEFAULT_NUM << 8) | OV2722_F_NUMBER_DEM;
> - return 0;
> -}
> -
> static int ov2722_get_intg_factor(struct i2c_client *client,
> struct camera_mipi_info *info,
> const struct ov2722_resolution *res)
> @@ -547,15 +526,6 @@ static int ov2722_g_volatile_ctrl(struct v4l2_ctrl *ctrl)
> case V4L2_CID_EXPOSURE_ABSOLUTE:
> ret = ov2722_q_exposure(&dev->sd, &ctrl->val);
> break;
> - case V4L2_CID_FOCAL_ABSOLUTE:
> - ret = ov2722_g_focal(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_ABSOLUTE:
> - ret = ov2722_g_fnumber(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_RANGE:
> - ret = ov2722_g_fnumber_range(&dev->sd, &ctrl->val);
> - break;
> case V4L2_CID_LINK_FREQ:
> val = dev->res->mipi_freq;
> if (val == 0)
> @@ -586,39 +556,6 @@ static const struct v4l2_ctrl_config ov2722_controls[] = {
> .def = 0x00,
> .flags = 0,
> },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FOCAL_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "focal length",
> - .min = OV2722_FOCAL_LENGTH_DEFAULT,
> - .max = OV2722_FOCAL_LENGTH_DEFAULT,
> - .step = 0x01,
> - .def = OV2722_FOCAL_LENGTH_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number",
> - .min = OV2722_F_NUMBER_DEFAULT,
> - .max = OV2722_F_NUMBER_DEFAULT,
> - .step = 0x01,
> - .def = OV2722_F_NUMBER_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_RANGE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number range",
> - .min = OV2722_F_NUMBER_RANGE,
> - .max = OV2722_F_NUMBER_RANGE,
> - .step = 0x01,
> - .def = OV2722_F_NUMBER_RANGE,
> - .flags = 0,
> - },
> {
> .ops = &ctrl_ops,
> .id = V4L2_CID_LINK_FREQ,
> diff --git a/drivers/staging/media/atomisp/i2c/gc0310.h b/drivers/staging/media/atomisp/i2c/gc0310.h
> index 4b9ce681bd93..52b4c07e5cf0 100644
> --- a/drivers/staging/media/atomisp/i2c/gc0310.h
> +++ b/drivers/staging/media/atomisp/i2c/gc0310.h
> @@ -38,9 +38,6 @@
> #define I2C_RETRY_COUNT 5
>
> #define GC0310_FOCAL_LENGTH_NUM 278 /*2.78mm*/
> -#define GC0310_FOCAL_LENGTH_DEM 100
> -#define GC0310_F_NUMBER_DEFAULT_NUM 26
> -#define GC0310_F_NUMBER_DEM 10
>
> #define MAX_FMTS 1
>
> diff --git a/drivers/staging/media/atomisp/i2c/gc2235.h b/drivers/staging/media/atomisp/i2c/gc2235.h
> index 806be5dff7a5..dd2d44b40e22 100644
> --- a/drivers/staging/media/atomisp/i2c/gc2235.h
> +++ b/drivers/staging/media/atomisp/i2c/gc2235.h
> @@ -44,9 +44,6 @@
> #define I2C_RETRY_COUNT 5
>
> #define GC2235_FOCAL_LENGTH_NUM 278 /*2.78mm*/
> -#define GC2235_FOCAL_LENGTH_DEM 100
> -#define GC2235_F_NUMBER_DEFAULT_NUM 26
> -#define GC2235_F_NUMBER_DEM 10
>
> #define MAX_FMTS 1
>
> diff --git a/drivers/staging/media/atomisp/i2c/mt9m114.h b/drivers/staging/media/atomisp/i2c/mt9m114.h
> index bcce18b65fa6..831875071cbb 100644
> --- a/drivers/staging/media/atomisp/i2c/mt9m114.h
> +++ b/drivers/staging/media/atomisp/i2c/mt9m114.h
> @@ -136,9 +136,6 @@
> #define MT9M114_BPAT_BGBGGRGR BIT(3)
>
> #define MT9M114_FOCAL_LENGTH_NUM 208 /*2.08mm*/
> -#define MT9M114_FOCAL_LENGTH_DEM 100
> -#define MT9M114_F_NUMBER_DEFAULT_NUM 24
> -#define MT9M114_F_NUMBER_DEM 10
> #define MT9M114_WAIT_STAT_TIMEOUT 100
> #define MT9M114_FLICKER_MODE_50HZ 1
> #define MT9M114_FLICKER_MODE_60HZ 2
> diff --git a/drivers/staging/media/atomisp/i2c/ov2680.h b/drivers/staging/media/atomisp/i2c/ov2680.h
> index 7ab337b859ad..2bc350c67711 100644
> --- a/drivers/staging/media/atomisp/i2c/ov2680.h
> +++ b/drivers/staging/media/atomisp/i2c/ov2680.h
> @@ -37,9 +37,6 @@
> #define I2C_RETRY_COUNT 5
>
> #define OV2680_FOCAL_LENGTH_NUM 334 /*3.34mm*/
> -#define OV2680_FOCAL_LENGTH_DEM 100
> -#define OV2680_F_NUMBER_DEFAULT_NUM 24
> -#define OV2680_F_NUMBER_DEM 10
>
> #define OV2680_BIN_FACTOR_MAX 4
>
> diff --git a/drivers/staging/media/atomisp/i2c/ov2722.h b/drivers/staging/media/atomisp/i2c/ov2722.h
> index d6e2510bc01c..d4cd6f27ee8d 100644
> --- a/drivers/staging/media/atomisp/i2c/ov2722.h
> +++ b/drivers/staging/media/atomisp/i2c/ov2722.h
> @@ -39,9 +39,6 @@
> #define I2C_RETRY_COUNT 5
>
> #define OV2722_FOCAL_LENGTH_NUM 278 /*2.78mm*/
> -#define OV2722_FOCAL_LENGTH_DEM 100
> -#define OV2722_F_NUMBER_DEFAULT_NUM 26
> -#define OV2722_F_NUMBER_DEM 10
>
> #define MAX_FMTS 1
>
> diff --git a/drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c b/drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c
> index c1cd631455e6..9adaf2fc940a 100644
> --- a/drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c
> +++ b/drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c
> @@ -415,27 +415,6 @@ static int ov5693_write_reg_array(struct i2c_client *client,
> return __ov5693_flush_reg_array(client, &ctrl);
> }
>
> -static int ov5693_g_focal(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (OV5693_FOCAL_LENGTH_NUM << 16) | OV5693_FOCAL_LENGTH_DEM;
> - return 0;
> -}
> -
> -static int ov5693_g_fnumber(struct v4l2_subdev *sd, s32 *val)
> -{
> - /*const f number for imx*/
> - *val = (OV5693_F_NUMBER_DEFAULT_NUM << 16) | OV5693_F_NUMBER_DEM;
> - return 0;
> -}
> -
> -static int ov5693_g_fnumber_range(struct v4l2_subdev *sd, s32 *val)
> -{
> - *val = (OV5693_F_NUMBER_DEFAULT_NUM << 24) |
> - (OV5693_F_NUMBER_DEM << 16) |
> - (OV5693_F_NUMBER_DEFAULT_NUM << 8) | OV5693_F_NUMBER_DEM;
> - return 0;
> -}
> -
> static int ov5693_g_bin_factor_x(struct v4l2_subdev *sd, s32 *val)
> {
> struct ov5693_device *dev = to_ov5693_sensor(sd);
> @@ -1107,15 +1086,6 @@ static int ov5693_g_volatile_ctrl(struct v4l2_ctrl *ctrl)
> case V4L2_CID_EXPOSURE_ABSOLUTE:
> ret = ov5693_q_exposure(&dev->sd, &ctrl->val);
> break;
> - case V4L2_CID_FOCAL_ABSOLUTE:
> - ret = ov5693_g_focal(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_ABSOLUTE:
> - ret = ov5693_g_fnumber(&dev->sd, &ctrl->val);
> - break;
> - case V4L2_CID_FNUMBER_RANGE:
> - ret = ov5693_g_fnumber_range(&dev->sd, &ctrl->val);
> - break;
> case V4L2_CID_FOCUS_ABSOLUTE:
> ret = ov5693_q_focus_abs(&dev->sd, &ctrl->val);
> break;
> @@ -1152,39 +1122,6 @@ static const struct v4l2_ctrl_config ov5693_controls[] = {
> .def = 0x00,
> .flags = 0,
> },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FOCAL_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "focal length",
> - .min = OV5693_FOCAL_LENGTH_DEFAULT,
> - .max = OV5693_FOCAL_LENGTH_DEFAULT,
> - .step = 0x01,
> - .def = OV5693_FOCAL_LENGTH_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_ABSOLUTE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number",
> - .min = OV5693_F_NUMBER_DEFAULT,
> - .max = OV5693_F_NUMBER_DEFAULT,
> - .step = 0x01,
> - .def = OV5693_F_NUMBER_DEFAULT,
> - .flags = 0,
> - },
> - {
> - .ops = &ctrl_ops,
> - .id = V4L2_CID_FNUMBER_RANGE,
> - .type = V4L2_CTRL_TYPE_INTEGER,
> - .name = "f-number range",
> - .min = OV5693_F_NUMBER_RANGE,
> - .max = OV5693_F_NUMBER_RANGE,
> - .step = 0x01,
> - .def = OV5693_F_NUMBER_RANGE,
> - .flags = 0,
> - },
> {
> .ops = &ctrl_ops,
> .id = V4L2_CID_FOCUS_ABSOLUTE,
> diff --git a/drivers/staging/media/atomisp/include/linux/atomisp.h b/drivers/staging/media/atomisp/include/linux/atomisp.h
> index 3f602b5aaff9..e70e57695300 100644
> --- a/drivers/staging/media/atomisp/include/linux/atomisp.h
> +++ b/drivers/staging/media/atomisp/include/linux/atomisp.h
> @@ -586,20 +586,6 @@ struct atomisp_shading_table {
> __u16 *data[ATOMISP_NUM_SC_COLORS];
> };
>
> -struct atomisp_makernote_info {
> - /* bits 31-16: numerator, bits 15-0: denominator */
> - unsigned int focal_length;
> - /* bits 31-16: numerator, bits 15-0: denominator*/
> - unsigned int f_number_curr;
> - /*
> - * bits 31-24: max f-number numerator
> - * bits 23-16: max f-number denominator
> - * bits 15-8: min f-number numerator
> - * bits 7-0: min f-number denominator
> - */
> - unsigned int f_number_range;
> -};
> -
> /* parameter for MACC */
> #define ATOMISP_NUM_MACC_AXES 16
> struct atomisp_macc_table {
> @@ -914,8 +900,6 @@ struct atomisp_sensor_ae_bracketing_lut {
> _IOR('v', BASE_VIDIOC_PRIVATE + 10, struct atomisp_morph_table)
> #define ATOMISP_IOC_S_ISP_GDC_TAB \
> _IOW('v', BASE_VIDIOC_PRIVATE + 10, struct atomisp_morph_table)
> -#define ATOMISP_IOC_ISP_MAKERNOTE \
> - _IOWR('v', BASE_VIDIOC_PRIVATE + 11, struct atomisp_makernote_info)
>
> /* macc parameter control*/
> #define ATOMISP_IOC_G_ISP_MACC \
> @@ -1093,10 +1077,6 @@ struct atomisp_sensor_ae_bracketing_lut {
> * Exposure, Flash and privacy (indicator) light controls, to be upstreamed */
> #define V4L2_CID_CAMERA_LASTP1 (V4L2_CID_CAMERA_CLASS_BASE + 1024)
>
> -#define V4L2_CID_FOCAL_ABSOLUTE (V4L2_CID_CAMERA_LASTP1 + 0)
> -#define V4L2_CID_FNUMBER_ABSOLUTE (V4L2_CID_CAMERA_LASTP1 + 1)
> -#define V4L2_CID_FNUMBER_RANGE (V4L2_CID_CAMERA_LASTP1 + 2)
> -
> /* Flash related CIDs, see also:
> * http://linuxtv.org/downloads/v4l-dvb-apis/extended-controls.html\
> * #flash-controls */
> diff --git a/drivers/staging/media/atomisp/pci/atomisp_cmd.c b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
> index 5cea1df48b7d..b167ee32a952 100644
> --- a/drivers/staging/media/atomisp/pci/atomisp_cmd.c
> +++ b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
> @@ -5492,42 +5492,6 @@ int atomisp_set_shading_table(struct atomisp_sub_device *asd,
> return ret;
> }
>
> -int atomisp_exif_makernote(struct atomisp_sub_device *asd,
> - struct atomisp_makernote_info *config)
> -{
> - struct v4l2_control ctrl;
> - struct atomisp_device *isp = asd->isp;
> -
> - ctrl.id = V4L2_CID_FOCAL_ABSOLUTE;
> - if (v4l2_g_ctrl
> - (isp->inputs[asd->input_curr].camera->ctrl_handler, &ctrl)) {
> - dev_warn(isp->dev, "failed to g_ctrl for focal length\n");
> - return -EINVAL;
> - } else {
> - config->focal_length = ctrl.value;
> - }
> -
> - ctrl.id = V4L2_CID_FNUMBER_ABSOLUTE;
> - if (v4l2_g_ctrl
> - (isp->inputs[asd->input_curr].camera->ctrl_handler, &ctrl)) {
> - dev_warn(isp->dev, "failed to g_ctrl for f-number\n");
> - return -EINVAL;
> - } else {
> - config->f_number_curr = ctrl.value;
> - }
> -
> - ctrl.id = V4L2_CID_FNUMBER_RANGE;
> - if (v4l2_g_ctrl
> - (isp->inputs[asd->input_curr].camera->ctrl_handler, &ctrl)) {
> - dev_warn(isp->dev, "failed to g_ctrl for f number range\n");
> - return -EINVAL;
> - } else {
> - config->f_number_range = ctrl.value;
> - }
> -
> - return 0;
> -}
> -
> int atomisp_offline_capture_configure(struct atomisp_sub_device *asd,
> struct atomisp_cont_capture_conf *cvf_config)
> {
> diff --git a/drivers/staging/media/atomisp/pci/atomisp_cmd.h b/drivers/staging/media/atomisp/pci/atomisp_cmd.h
> index b8911491581a..99bbab402c9c 100644
> --- a/drivers/staging/media/atomisp/pci/atomisp_cmd.h
> +++ b/drivers/staging/media/atomisp/pci/atomisp_cmd.h
> @@ -273,9 +273,6 @@ int atomisp_set_shading_table(struct atomisp_sub_device *asd,
> int atomisp_offline_capture_configure(struct atomisp_sub_device *asd,
> struct atomisp_cont_capture_conf *cvf_config);
>
> -int atomisp_exif_makernote(struct atomisp_sub_device *asd,
> - struct atomisp_makernote_info *config);
> -
> void atomisp_free_internal_buffers(struct atomisp_sub_device *asd);
>
> int atomisp_s_ae_window(struct atomisp_sub_device *asd,
> diff --git a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
> index 4f35e8f8250a..faf65387df56 100644
> --- a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
> +++ b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
> @@ -1631,7 +1631,6 @@ static int atomisp_g_ctrl(struct file *file, void *fh,
> switch (control->id) {
> case V4L2_CID_IRIS_ABSOLUTE:
> case V4L2_CID_EXPOSURE_ABSOLUTE:
> - case V4L2_CID_FNUMBER_ABSOLUTE:
> case V4L2_CID_2A_STATUS:
> case V4L2_CID_AUTO_N_PRESET_WHITE_BALANCE:
> case V4L2_CID_EXPOSURE:
> @@ -1828,7 +1827,6 @@ static int atomisp_camera_g_ext_ctrls(struct file *file, void *fh,
> case V4L2_CID_EXPOSURE_ABSOLUTE:
> case V4L2_CID_EXPOSURE_AUTO:
> case V4L2_CID_IRIS_ABSOLUTE:
> - case V4L2_CID_FNUMBER_ABSOLUTE:
> case V4L2_CID_BIN_FACTOR_HORZ:
> case V4L2_CID_BIN_FACTOR_VERT:
> case V4L2_CID_3A_LOCK:
> @@ -1940,7 +1938,6 @@ static int atomisp_camera_s_ext_ctrls(struct file *file, void *fh,
> case V4L2_CID_EXPOSURE_AUTO:
> case V4L2_CID_EXPOSURE_METERING:
> case V4L2_CID_IRIS_ABSOLUTE:
> - case V4L2_CID_FNUMBER_ABSOLUTE:
> case V4L2_CID_VCM_TIMING:
> case V4L2_CID_VCM_SLEW:
> case V4L2_CID_3A_LOCK:
> @@ -2276,10 +2273,6 @@ static long atomisp_vidioc_default(struct file *file, void *fh,
> err = atomisp_fixed_pattern_table(asd, arg);
> break;
>
> - case ATOMISP_IOC_ISP_MAKERNOTE:
> - err = atomisp_exif_makernote(asd, arg);
> - break;
> -
> case ATOMISP_IOC_G_SENSOR_MODE_DATA:
> err = atomisp_get_sensor_mode_data(asd, arg);
> break;
> --
> 2.39.0
>
--
With Best Regards,
Andy Shevchenko
next prev parent reply other threads:[~2023-01-23 14:30 UTC|newest]
Thread overview: 168+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-23 12:51 [PATCH 00/57] media: atomisp: Big power-management changes + lots of fixes Hans de Goede
2023-01-23 12:51 ` [PATCH 01/57] media: atomisp: fix videobuf2 Kconfig depenendency Hans de Goede
2023-01-23 14:12 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 02/57] media: atomisp: use vb2_start_streaming_called() Hans de Goede
2023-01-23 12:51 ` [PATCH 03/57] media: atomisp: Remove atomisp_sw_contex struct Hans de Goede
2023-01-23 14:14 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 04/57] media: atomisp: Move power-management over to a custom pm-domain Hans de Goede
2023-01-23 12:51 ` [PATCH 05/57] media: atomisp: Silence "isys dma store at addr, val" debug messages Hans de Goede
2023-01-23 14:18 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 06/57] media: atomisp: Remove non working doorbell check from punit_ddr_dvfs_enable() Hans de Goede
2023-01-23 12:51 ` [PATCH 07/57] media: atomisp: Remove useless msleep(10) before power-on on BYT Hans de Goede
2023-01-23 14:40 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 08/57] media: atomisp: Remove custom ATOMISP_IOC_ISP_MAKERNOTE ioctl Hans de Goede
2023-01-23 14:30 ` Andy Shevchenko [this message]
2023-01-23 12:51 ` [PATCH 09/57] media: atomisp: Remove custom ATOMISP_IOC_G_SENSOR_MODE_DATA ioctl Hans de Goede
2023-01-23 14:31 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 10/57] media: atomisp: Remove V4L2_CID_BIN_FACTOR_HORZ/_VERT Hans de Goede
2023-01-23 14:33 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 11/57] media: atomisp: Remove no longer used binning info from sensor resolution info Hans de Goede
2023-01-23 14:33 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 12/57] media: atomisp: Propagate set_fmt() errors in queue_setup() Hans de Goede
2023-01-23 14:35 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 13/57] media: atomisp: Remove deferred firmware loading support Hans de Goede
2023-01-23 14:37 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 14/57] media: atomisp: Check buffer index is in range inside atomisp_qbuf_wrapper() Hans de Goede
2023-01-23 14:38 ` Andy Shevchenko
2023-01-24 11:09 ` Hans de Goede
2023-01-23 12:51 ` [PATCH 15/57] media: atomisp: Drop atomisp_init_pipe() Hans de Goede
2023-01-23 15:30 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 16/57] media: atomisp: Remove unnecessary memset(foo, 0, sizeof(foo)) calls Hans de Goede
2023-01-23 17:41 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 17/57] media: atomisp: Only set default_run_mode on first open of a stream/asd Hans de Goede
2023-01-23 17:42 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 18/57] media: atomisp: Do not turn off sensor when the atomisp-sub-dev does not own it Hans de Goede
2023-01-23 12:51 ` [PATCH 19/57] media: atomisp: Allow sensor drivers without a s_power callback Hans de Goede
2023-01-23 17:49 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 20/57] media: atomisp: Fix regulator registers on BYT devices with CRC PMIC Hans de Goede
2023-01-23 17:51 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 21/57] media: atomisp: Remove atomisp_gmin_find_subdev() Hans de Goede
2023-01-23 17:53 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 22/57] media: atomisp: Add atomisp_register_sensor_no_gmin() helper Hans de Goede
2023-01-23 17:55 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 23/57] media: atomisp: Fix WARN() when the vb2 start_streaming callback fails Hans de Goede
2023-01-23 17:56 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 24/57] media: atomisp: Drop ffmt local var from atomisp_set_fmt() Hans de Goede
2023-01-23 12:51 ` [PATCH 25/57] media: atomisp: Stop overriding padding w/h to 12 on BYT Hans de Goede
2023-01-23 17:59 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 26/57] media: atomisp: Put sensor ACPI devices in D3 before disable ACPI power-resources Hans de Goede
2023-01-23 12:51 ` [PATCH 27/57] media: atomisp: Remove isp_subdev_link_setup() Hans de Goede
2023-01-23 18:02 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 28/57] media: Add ovxxxx_16bit_addr_reg_helpers.h Hans de Goede
2023-01-23 18:09 ` Andy Shevchenko
2023-01-24 11:21 ` Hans de Goede
2023-01-24 12:47 ` Andy Shevchenko
2023-01-23 18:15 ` Andy Shevchenko
2023-01-23 18:23 ` Andy Shevchenko
2023-01-24 11:25 ` Hans de Goede
2023-02-08 9:52 ` Laurent Pinchart
2023-02-08 11:27 ` Andy Shevchenko
2023-02-08 15:41 ` Laurent Pinchart
2023-02-08 15:50 ` Andy Shevchenko
2023-02-08 16:03 ` Laurent Pinchart
2023-02-08 17:31 ` Andy Shevchenko
2023-02-09 10:31 ` Laurent Pinchart
2023-02-09 15:03 ` Hans de Goede
2023-02-09 16:11 ` Laurent Pinchart
2023-02-10 10:21 ` Sakari Ailus
2023-02-10 10:29 ` Laurent Pinchart
2023-02-10 10:47 ` Sakari Ailus
2023-02-10 10:53 ` Andy Shevchenko
2023-02-10 11:05 ` Laurent Pinchart
2023-02-10 15:35 ` Andy Shevchenko
2023-02-10 16:01 ` Hans de Goede
2023-02-10 11:19 ` Hans de Goede
2023-02-10 11:35 ` Laurent Pinchart
2023-02-10 12:01 ` Hans de Goede
2023-02-10 11:04 ` Laurent Pinchart
2023-02-10 11:18 ` Sakari Ailus
2023-02-10 11:34 ` Laurent Pinchart
2023-02-10 11:20 ` Hans de Goede
2023-02-10 11:45 ` Laurent Pinchart
2023-02-10 11:56 ` Hans de Goede
2023-02-10 12:09 ` Laurent Pinchart
2023-02-10 12:17 ` Sakari Ailus
2023-02-10 12:59 ` Hans de Goede
2023-02-10 13:31 ` Sakari Ailus
2023-02-10 12:47 ` Hans de Goede
2023-02-10 13:18 ` Sakari Ailus
2023-02-10 14:43 ` Hans de Goede
2023-02-10 16:43 ` Laurent Pinchart
2023-02-10 20:16 ` Sakari Ailus
2023-02-10 12:26 ` Sakari Ailus
2023-02-10 15:42 ` Andy Shevchenko
2023-02-10 16:39 ` Laurent Pinchart
2023-02-10 20:18 ` Sakari Ailus
2023-02-10 16:40 ` Laurent Pinchart
2023-02-08 11:31 ` Sakari Ailus
2023-02-08 14:33 ` Mauro Carvalho Chehab
2023-02-08 15:39 ` Laurent Pinchart
2023-01-23 12:51 ` [PATCH 29/57] media: atomisp: ov2680: Use the new ovxxxx_16bit_addr_reg_helpers.h Hans de Goede
2023-01-23 18:13 ` Andy Shevchenko
2023-01-24 11:22 ` Hans de Goede
2023-01-23 12:51 ` [PATCH 30/57] media: atomisp: ov2680: Rework flip ctrls Hans de Goede
2023-01-23 18:33 ` Andy Shevchenko
2023-01-29 0:36 ` kernel test robot
2023-01-23 12:51 ` [PATCH 31/57] media: atomisp: ov2680: Drop custom ATOMISP_IOC_S_EXPOSURE support Hans de Goede
2023-01-23 18:35 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 32/57] media: atomisp: ov2680: Add exposure and gain controls Hans de Goede
2023-01-23 18:43 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 33/57] media: atomisp: ov2680: Add test pattern control Hans de Goede
2023-01-23 18:46 ` Andy Shevchenko
2023-01-24 11:27 ` Hans de Goede
2023-01-24 12:50 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 34/57] media: atomisp: ov2680: Fix window settings and enable window for all resolutions Hans de Goede
2023-01-23 18:48 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 35/57] media: atomisp: ov2680: Make setting the modes algorithm based Hans de Goede
2023-01-24 10:37 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 36/57] media: atomisp: ov2680: Use defines for fps, lines-per-frame and skip-frames Hans de Goede
2023-01-24 10:40 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 37/57] media: atomisp: ov2680: Drop unused res member from struct ov2680_device Hans de Goede
2023-01-24 10:39 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 38/57] media: atomisp: ov2680: Fix ov2680_enum_frame_interval() Hans de Goede
2023-01-24 10:42 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 39/57] media: atomisp: ov2680: Drop v4l2_find_nearest_size() call from set_fmt() Hans de Goede
2023-01-24 10:43 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 40/57] media: atomisp: ov2680: Drop struct ov2680_resolution / ov2680_res_preview Hans de Goede
2023-01-24 10:44 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 41/57] media: atomisp: ov2680: Fix frame_size list Hans de Goede
2023-01-24 10:46 ` Andy Shevchenko
2023-01-24 11:29 ` Hans de Goede
2023-01-23 12:51 ` [PATCH 42/57] media: atomisp: ov2680: Remove unused data-types and defines from ov2680.h Hans de Goede
2023-01-24 10:46 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 43/57] media: atomisp: ov2680: Drop MAX_FMTS define Hans de Goede
2023-01-24 10:48 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 44/57] media: atomisp: ov2680: Consistently indent define values Hans de Goede
2023-01-24 10:49 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 45/57] media: atomisp: ov2680: Cleanup includes Hans de Goede
2023-01-24 10:50 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 46/57] media: atomisp: ov2680: Delay power-on till streaming is started Hans de Goede
2023-01-24 10:51 ` Andy Shevchenko
2023-01-24 11:31 ` Hans de Goede
2023-01-24 12:52 ` Andy Shevchenko
2023-01-24 13:35 ` Hans de Goede
2023-01-23 12:51 ` [PATCH 47/57] media: atomisp: ov2680: Add runtime-pm support Hans de Goede
2023-01-24 10:53 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 48/57] media: atomisp: ov2680: s/dev/sensor/ Hans de Goede
2023-01-24 10:54 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 49/57] media: atomisp: ov2680: Use devm_kzalloc() for sensor data struct Hans de Goede
2023-01-24 10:55 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 50/57] media: atomisp: ov2680: Switch over to ACPI powermanagement Hans de Goede
2023-01-24 10:59 ` Andy Shevchenko
2023-01-23 12:51 ` [PATCH 51/57] media: atomisp: ov2722: Call atomisp_gmin_remove_subdev() on probe failure Hans de Goede
2023-01-23 18:36 ` Andy Shevchenko
2023-01-23 12:52 ` [PATCH 52/57] media: atomisp: ov2722: Fix GPIO1 polarity Hans de Goede
2023-01-23 18:39 ` Andy Shevchenko
2023-01-23 18:40 ` Andy Shevchenko
2023-01-23 12:52 ` [PATCH 53/57] media: atomisp: ov2722: Don't take the input_lock for try_fmt calls Hans de Goede
2023-01-23 18:39 ` Andy Shevchenko
2023-01-23 12:52 ` [PATCH 54/57] media: atomisp: ov2722: Power on sensor from set_fmt() callback Hans de Goede
2023-01-23 18:42 ` Andy Shevchenko
2023-01-24 11:32 ` Hans de Goede
2023-01-23 12:52 ` [PATCH 55/57] media: atomisp: pci: Replace bytes macros with functions Hans de Goede
2023-01-23 12:52 ` [PATCH 56/57] media: atomisp: pci: hive_isp_css_common: host: vmem: Replace SUBWORD " Hans de Goede
2023-01-23 18:27 ` Andy Shevchenko
2023-01-23 18:29 ` Andy Shevchenko
2023-01-23 12:52 ` [PATCH 57/57] media: atomisp: pci: sh_css: Inline single invocation of macro STATS_ENABLED() Hans de Goede
2023-01-24 11:01 ` [PATCH 00/57] media: atomisp: Big power-management changes + lots of fixes Andy Shevchenko
2023-01-24 11:05 ` Hans de Goede
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Y86Z8GGiJM/GCzAO@smile.fi.intel.com \
--to=andy@kernel.org \
--cc=andrey.i.trufanov@gmail.com \
--cc=fabioaiuto83@gmail.com \
--cc=hdegoede@redhat.com \
--cc=kitakar@gmail.com \
--cc=linux-media@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=mchehab@kernel.org \
--cc=nable.maininbox@googlemail.com \
--cc=sakari.ailus@linux.intel.com \
--cc=yury.lunev@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox