Skip to content

Commit

Permalink
drivers: media: arducam_64mp: Add V4L2_CID_LINK_FREQ control
Browse files Browse the repository at this point in the history
Add V4L2_CID_LINK_FREQ as a read-only control with a value of 456 Mhz.
This will be used by the CFE driver to corretly setup the DPHY timing
parameters in the CSI-2 block.

Signed-off-by: Lee Jackson <[email protected]>
  • Loading branch information
Lee Jackson authored and popcornmix committed Jan 11, 2024
1 parent 9d6cdeb commit 2256ccc
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions drivers/media/i2c/arducam_64mp.c
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,10 @@ struct arducam_64mp_mode {
struct arducam_64mp_reg_list reg_list;
};

static const s64 arducam_64mp_link_freq_menu[] = {
ARDUCAM_64MP_DEFAULT_LINK_FREQ,
};

static const struct arducam_64mp_reg mode_common_regs[] = {
{0x0100, 0x00},
{0x0136, 0x18},
Expand Down Expand Up @@ -2272,9 +2276,11 @@ static int arducam_64mp_init_controls(struct arducam_64mp *arducam_64mp)
struct v4l2_ctrl_handler *ctrl_hdlr;
struct i2c_client *client = v4l2_get_subdevdata(&arducam_64mp->sd);
struct v4l2_fwnode_device_properties props;
struct v4l2_ctrl *link_freq;
unsigned int i;
int ret;
u8 test_pattern_max;
u8 link_freq_max;

ctrl_hdlr = &arducam_64mp->ctrl_handler;
ret = v4l2_ctrl_handler_init(ctrl_hdlr, 16);
Expand All @@ -2292,6 +2298,16 @@ static int arducam_64mp_init_controls(struct arducam_64mp *arducam_64mp)
ARDUCAM_64MP_PIXEL_RATE, 1,
ARDUCAM_64MP_PIXEL_RATE);

/* LINK_FREQ is also read only */
link_freq_max = ARRAY_SIZE(arducam_64mp_link_freq_menu) - 1;
link_freq =
v4l2_ctrl_new_int_menu(ctrl_hdlr, &arducam_64mp_ctrl_ops,
V4L2_CID_LINK_FREQ,
link_freq_max, 0,
arducam_64mp_link_freq_menu);
if (link_freq)
link_freq->flags |= V4L2_CTRL_FLAG_READ_ONLY;

/*
* Create the controls here, but mode specific limits are setup
* in the arducam_64mp_set_framing_limits() call below.
Expand Down

0 comments on commit 2256ccc

Please sign in to comment.