From: Hans de Goede <hdegoede@redhat.com>
To: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Cc: linuxarm@huawei.com, mauro.chehab@huawei.com,
Bhaskar Chowdhury <unixbhaskar@gmail.com>,
Deepak R Varma <drv@mailo.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Randy Dunlap <rdunlap@infradead.org>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
linux-staging@lists.linux.dev
Subject: Re: [PATCH 1/2] media: atomisp-ov2680: uncomment other resolutions
Date: Thu, 11 Nov 2021 11:03:58 +0100 [thread overview]
Message-ID: <f38e8b72-bc55-312f-ebd0-95c27b2daa8d@redhat.com> (raw)
In-Reply-To: <440a1ecd3e1ef3a58bff40cde2ae57a32e5b0f60.1636624633.git.mchehab+huawei@kernel.org>
Hi,
On 11/11/21 10:57, Mauro Carvalho Chehab wrote:
> There are several resolutions defined for this sensor, but just
> two are currently uncommented. Change the driver to allow other
> resolutions to also be used.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Thanks, series looks good to me:
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
for the series.
Regards,
Hans
> ---
> drivers/staging/media/atomisp/i2c/ov2680.h | 159 ++++++++++++++++++++-
> 1 file changed, 153 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/media/atomisp/i2c/ov2680.h b/drivers/staging/media/atomisp/i2c/ov2680.h
> index e53be612a1e4..ed68510bb0b9 100644
> --- a/drivers/staging/media/atomisp/i2c/ov2680.h
> +++ b/drivers/staging/media/atomisp/i2c/ov2680.h
> @@ -282,7 +282,6 @@ static struct ov2680_reg const ov2680_global_setting[] = {
> {}
> };
>
> -#if 0 /* None of the definitions below are used currently */
> /*
> * 176x144 30fps VBlanking 1lane 10Bit (binning)
> */
> @@ -442,7 +441,7 @@ static struct ov2680_reg const ov2680_656x496_30fps[] = {
> };
>
> /*
> - * 800x600 30fps VBlanking 1lane 10Bit (binning)
> + * 720x592 30fps VBlanking 1lane 10Bit (binning)
> */
> static struct ov2680_reg const ov2680_720x592_30fps[] = {
> {0x3086, 0x01},
> @@ -637,7 +636,6 @@ static struct ov2680_reg const ov2680_1456x1096_30fps[] = {
> // {0x5090, 0x0c},
> {}
> };
> -#endif
>
> /*
> *1616x916 30fps VBlanking 1lane 10bit
> @@ -680,9 +678,8 @@ static struct ov2680_reg const ov2680_1616x916_30fps[] = {
> };
>
> /*
> - * 1612x1212 30fps VBlanking 1lane 10Bit
> + * 1616x1082 30fps VBlanking 1lane 10Bit
> */
> -#if 0
> static struct ov2680_reg const ov2680_1616x1082_30fps[] = {
> {0x3086, 0x00},
> {0x370a, 0x21},
> @@ -718,7 +715,7 @@ static struct ov2680_reg const ov2680_1616x1082_30fps[] = {
> {0x5081, 0x41},
> {}
> };
> -#endif
> +
> /*
> * 1616x1216 30fps VBlanking 1lane 10Bit
> */
> @@ -774,6 +771,21 @@ static struct ov2680_resolution ov2680_res_preview[] = {
> .skip_frames = 3,
> .regs = ov2680_1616x1216_30fps,
> },
> + {
> + .desc = "ov2680_1616x1082_30fps",
> + .width = 1616,
> + .height = 1082,
> + .pix_clk_freq = 66,
> + .fps = 30,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_1616x1082_30fps,
> + },
> {
> .desc = "ov2680_1616x916_30fps",
> .width = 1616,
> @@ -789,6 +801,141 @@ static struct ov2680_resolution ov2680_res_preview[] = {
> .skip_frames = 3,
> .regs = ov2680_1616x916_30fps,
> },
> + {
> + .desc = "ov2680_1456x1096_30fps",
> + .width = 1456,
> + .height = 1096,
> + .fps = 30,
> + .pix_clk_freq = 66,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_1456x1096_30fps,
> + },
> + {
> + .desc = "ov2680_1296x976_30fps",
> + .width = 1296,
> + .height = 976,
> + .fps = 30,
> + .pix_clk_freq = 66,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_1296x976_30fps,
> + },
> + {
> + .desc = "ov2680_720p_30fps",
> + .width = 1280,
> + .height = 720,
> + .fps = 30,
> + .pix_clk_freq = 66,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_720p_30fps,
> + },
> + {
> + .desc = "ov2680_800x600_30fps",
> + .width = 800,
> + .height = 600,
> + .fps = 30,
> + .pix_clk_freq = 66,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_800x600_30fps,
> + },
> + {
> + .desc = "ov2680_720x592_30fps",
> + .width = 720,
> + .height = 592,
> + .fps = 30,
> + .pix_clk_freq = 66,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_720x592_30fps,
> + },
> + {
> + .desc = "ov2680_656x496_30fps",
> + .width = 656,
> + .height = 496,
> + .fps = 30,
> + .pix_clk_freq = 66,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_656x496_30fps,
> + },
> + {
> + .desc = "ov2680_QVGA_30fps",
> + .width = 336,
> + .height = 256,
> + .fps = 30,
> + .pix_clk_freq = 66,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_QVGA_30fps,
> + },
> + {
> + .desc = "ov2680_CIF_30fps",
> + .width = 352,
> + .height = 288,
> + .fps = 30,
> + .pix_clk_freq = 66,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_CIF_30fps,
> + },
> + {
> + .desc = "ov2680_QCIF_30fps",
> + .width = 176,
> + .height = 144,
> + .fps = 30,
> + .pix_clk_freq = 66,
> + .used = 0,
> + .pixels_per_line = 1698,//1704,
> + .lines_per_frame = 1294,
> + .bin_factor_x = 0,
> + .bin_factor_y = 0,
> + .bin_mode = 0,
> + .skip_frames = 3,
> + .regs = ov2680_QCIF_30fps,
> + },
> };
>
> #define N_RES_PREVIEW (ARRAY_SIZE(ov2680_res_preview))
>
prev parent reply other threads:[~2021-11-11 10:04 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-11 9:57 Mauro Carvalho Chehab
2021-11-11 9:57 ` [PATCH 2/2] media: atomisp-ov2680: remove some unused fields Mauro Carvalho Chehab
2021-11-11 10:03 ` Hans de Goede [this message]
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=f38e8b72-bc55-312f-ebd0-95c27b2daa8d@redhat.com \
--to=hdegoede@redhat.com \
--cc=drv@mailo.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=linuxarm@huawei.com \
--cc=mauro.chehab@huawei.com \
--cc=mchehab+huawei@kernel.org \
--cc=mchehab@kernel.org \
--cc=rdunlap@infradead.org \
--cc=sakari.ailus@linux.intel.com \
--cc=unixbhaskar@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
all inboxes | Powered by JetHome®