Commit 69e9ba6f authored by Hans Verkuil's avatar Hans Verkuil Committed by Mauro Carvalho Chehab

[media] adv7842: support YCrCb analog input, receive CEA formats as RGB on VGA input

Added support for YCrCb analog input.

If input is ADV7842_MODE_RGB and RGB quantization range is set to
V4L2_DV_RGB_RANGE_AUTO, then video with CEA timings will be received
as RGB. For ADV7842_MODE_COMP, automatic CSC mode will be selected.

See table 48 on page 281 in "ADV7842 Hardware Manual, Rev. 0, January 2011"
for details.

Make sure that when switching inputs the RGB quantization range is
updated as well.

Also updated the platform_data in ezkit to ensure that what was the old
default value is now explicitly specified, so the behavior for that board
is unchanged.
Signed-off-by: default avatarMats Randgaard <matrandg@cisco.com>
Signed-off-by: default avatarMartin Bugge <marbugge@cisco.com>
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Cc: Scott Jiang <scott.jiang.linux@gmail.com>
Signed-off-by: default avatarMauro Carvalho Chehab <m.chehab@samsung.com>
parent 32dbc8d4
...@@ -1025,7 +1025,6 @@ static struct adv7842_platform_data adv7842_data = { ...@@ -1025,7 +1025,6 @@ static struct adv7842_platform_data adv7842_data = {
.ain_sel = ADV7842_AIN10_11_12_NC_SYNC_4_1, .ain_sel = ADV7842_AIN10_11_12_NC_SYNC_4_1,
.prim_mode = ADV7842_PRIM_MODE_SDP, .prim_mode = ADV7842_PRIM_MODE_SDP,
.vid_std_select = ADV7842_SDP_VID_STD_CVBS_SD_4x1, .vid_std_select = ADV7842_SDP_VID_STD_CVBS_SD_4x1,
.inp_color_space = ADV7842_INP_COLOR_SPACE_AUTO,
.i2c_sdp_io = 0x40, .i2c_sdp_io = 0x40,
.i2c_sdp = 0x41, .i2c_sdp = 0x41,
.i2c_cp = 0x42, .i2c_cp = 0x42,
......
...@@ -1034,34 +1034,60 @@ static void set_rgb_quantization_range(struct v4l2_subdev *sd) ...@@ -1034,34 +1034,60 @@ static void set_rgb_quantization_range(struct v4l2_subdev *sd)
{ {
struct adv7842_state *state = to_state(sd); struct adv7842_state *state = to_state(sd);
v4l2_dbg(2, debug, sd, "%s: rgb_quantization_range = %d\n",
__func__, state->rgb_quantization_range);
switch (state->rgb_quantization_range) { switch (state->rgb_quantization_range) {
case V4L2_DV_RGB_RANGE_AUTO: case V4L2_DV_RGB_RANGE_AUTO:
/* automatic */ if (state->mode == ADV7842_MODE_RGB) {
if (is_digital_input(sd) && !(hdmi_read(sd, 0x05) & 0x80)) { /* Receiving analog RGB signal
/* receiving DVI-D signal */ * Set RGB full range (0-255) */
io_write_and_or(sd, 0x02, 0x0f, 0x10);
/* ADV7842 selects RGB limited range regardless of break;
input format (CE/IT) in automatic mode */ }
if (state->timings.bt.standards & V4L2_DV_BT_STD_CEA861) {
/* RGB limited range (16-235) */ if (state->mode == ADV7842_MODE_COMP) {
io_write_and_or(sd, 0x02, 0x0f, 0x00); /* Receiving analog YPbPr signal
* Set automode */
} else { io_write_and_or(sd, 0x02, 0x0f, 0xf0);
/* RGB full range (0-255) */ break;
io_write_and_or(sd, 0x02, 0x0f, 0x10); }
}
} else { if (hdmi_read(sd, 0x05) & 0x80) {
/* receiving HDMI or analog signal, set automode */ /* Receiving HDMI signal
* Set automode */
io_write_and_or(sd, 0x02, 0x0f, 0xf0); io_write_and_or(sd, 0x02, 0x0f, 0xf0);
break;
}
/* Receiving DVI-D signal
* ADV7842 selects RGB limited range regardless of
* input format (CE/IT) in automatic mode */
if (state->timings.bt.standards & V4L2_DV_BT_STD_CEA861) {
/* RGB limited range (16-235) */
io_write_and_or(sd, 0x02, 0x0f, 0x00);
} else {
/* RGB full range (0-255) */
io_write_and_or(sd, 0x02, 0x0f, 0x10);
} }
break; break;
case V4L2_DV_RGB_RANGE_LIMITED: case V4L2_DV_RGB_RANGE_LIMITED:
/* RGB limited range (16-235) */ if (state->mode == ADV7842_MODE_COMP) {
io_write_and_or(sd, 0x02, 0x0f, 0x00); /* YCrCb limited range (16-235) */
io_write_and_or(sd, 0x02, 0x0f, 0x20);
} else {
/* RGB limited range (16-235) */
io_write_and_or(sd, 0x02, 0x0f, 0x00);
}
break; break;
case V4L2_DV_RGB_RANGE_FULL: case V4L2_DV_RGB_RANGE_FULL:
/* RGB full range (0-255) */ if (state->mode == ADV7842_MODE_COMP) {
io_write_and_or(sd, 0x02, 0x0f, 0x10); /* YCrCb full range (0-255) */
io_write_and_or(sd, 0x02, 0x0f, 0x60);
} else {
/* RGB full range (0-255) */
io_write_and_or(sd, 0x02, 0x0f, 0x10);
}
break; break;
} }
} }
...@@ -1299,7 +1325,7 @@ static int adv7842_dv_timings_cap(struct v4l2_subdev *sd, ...@@ -1299,7 +1325,7 @@ static int adv7842_dv_timings_cap(struct v4l2_subdev *sd,
} }
/* Fill the optional fields .standards and .flags in struct v4l2_dv_timings /* Fill the optional fields .standards and .flags in struct v4l2_dv_timings
if the format is listed in adv7604_timings[] */ if the format is listed in adv7842_timings[] */
static void adv7842_fill_optional_dv_timings_fields(struct v4l2_subdev *sd, static void adv7842_fill_optional_dv_timings_fields(struct v4l2_subdev *sd,
struct v4l2_dv_timings *timings) struct v4l2_dv_timings *timings)
{ {
...@@ -1442,6 +1468,8 @@ static int adv7842_g_dv_timings(struct v4l2_subdev *sd, ...@@ -1442,6 +1468,8 @@ static int adv7842_g_dv_timings(struct v4l2_subdev *sd,
static void enable_input(struct v4l2_subdev *sd) static void enable_input(struct v4l2_subdev *sd)
{ {
struct adv7842_state *state = to_state(sd); struct adv7842_state *state = to_state(sd);
set_rgb_quantization_range(sd);
switch (state->mode) { switch (state->mode) {
case ADV7842_MODE_SDP: case ADV7842_MODE_SDP:
case ADV7842_MODE_COMP: case ADV7842_MODE_COMP:
...@@ -1586,6 +1614,13 @@ static void select_input(struct v4l2_subdev *sd, ...@@ -1586,6 +1614,13 @@ static void select_input(struct v4l2_subdev *sd,
afe_write(sd, 0x00, 0x00); /* power up ADC */ afe_write(sd, 0x00, 0x00); /* power up ADC */
afe_write(sd, 0xc8, 0x00); /* phase control */ afe_write(sd, 0xc8, 0x00); /* phase control */
if (state->mode == ADV7842_MODE_COMP) {
/* force to YCrCb */
io_write_and_or(sd, 0x02, 0x0f, 0x60);
} else {
/* force to RGB */
io_write_and_or(sd, 0x02, 0x0f, 0x10);
}
/* set ADI recommended settings for digitizer */ /* set ADI recommended settings for digitizer */
/* "ADV7842 Register Settings Recommendations /* "ADV7842 Register Settings Recommendations
...@@ -1681,19 +1716,19 @@ static int adv7842_s_routing(struct v4l2_subdev *sd, ...@@ -1681,19 +1716,19 @@ static int adv7842_s_routing(struct v4l2_subdev *sd,
switch (input) { switch (input) {
case ADV7842_SELECT_HDMI_PORT_A: case ADV7842_SELECT_HDMI_PORT_A:
/* TODO select HDMI_COMP or HDMI_GR */
state->mode = ADV7842_MODE_HDMI; state->mode = ADV7842_MODE_HDMI;
state->vid_std_select = ADV7842_HDMI_COMP_VID_STD_HD_1250P; state->vid_std_select = ADV7842_HDMI_COMP_VID_STD_HD_1250P;
state->hdmi_port_a = true; state->hdmi_port_a = true;
break; break;
case ADV7842_SELECT_HDMI_PORT_B: case ADV7842_SELECT_HDMI_PORT_B:
/* TODO select HDMI_COMP or HDMI_GR */
state->mode = ADV7842_MODE_HDMI; state->mode = ADV7842_MODE_HDMI;
state->vid_std_select = ADV7842_HDMI_COMP_VID_STD_HD_1250P; state->vid_std_select = ADV7842_HDMI_COMP_VID_STD_HD_1250P;
state->hdmi_port_a = false; state->hdmi_port_a = false;
break; break;
case ADV7842_SELECT_VGA_COMP: case ADV7842_SELECT_VGA_COMP:
v4l2_info(sd, "%s: VGA component: todo\n", __func__); state->mode = ADV7842_MODE_COMP;
state->vid_std_select = ADV7842_RGB_VID_STD_AUTO_GRAPH_MODE;
break;
case ADV7842_SELECT_VGA_RGB: case ADV7842_SELECT_VGA_RGB:
state->mode = ADV7842_MODE_RGB; state->mode = ADV7842_MODE_RGB;
state->vid_std_select = ADV7842_RGB_VID_STD_AUTO_GRAPH_MODE; state->vid_std_select = ADV7842_RGB_VID_STD_AUTO_GRAPH_MODE;
...@@ -2112,7 +2147,7 @@ static int adv7842_cp_log_status(struct v4l2_subdev *sd) ...@@ -2112,7 +2147,7 @@ static int adv7842_cp_log_status(struct v4l2_subdev *sd)
static const char * const input_color_space_txt[16] = { static const char * const input_color_space_txt[16] = {
"RGB limited range (16-235)", "RGB full range (0-255)", "RGB limited range (16-235)", "RGB full range (0-255)",
"YCbCr Bt.601 (16-235)", "YCbCr Bt.709 (16-235)", "YCbCr Bt.601 (16-235)", "YCbCr Bt.709 (16-235)",
"XvYCC Bt.601", "XvYCC Bt.709", "xvYCC Bt.601", "xvYCC Bt.709",
"YCbCr Bt.601 (0-255)", "YCbCr Bt.709 (0-255)", "YCbCr Bt.601 (0-255)", "YCbCr Bt.709 (0-255)",
"invalid", "invalid", "invalid", "invalid", "invalid", "invalid", "invalid", "invalid", "invalid", "invalid",
"invalid", "invalid", "automatic" "invalid", "invalid", "automatic"
...@@ -2341,9 +2376,10 @@ static int adv7842_g_std(struct v4l2_subdev *sd, v4l2_std_id *norm) ...@@ -2341,9 +2376,10 @@ static int adv7842_g_std(struct v4l2_subdev *sd, v4l2_std_id *norm)
/* ----------------------------------------------------------------------- */ /* ----------------------------------------------------------------------- */
static int adv7842_core_init(struct v4l2_subdev *sd, static int adv7842_core_init(struct v4l2_subdev *sd)
const struct adv7842_platform_data *pdata)
{ {
struct adv7842_state *state = to_state(sd);
struct adv7842_platform_data *pdata = &state->pdata;
hdmi_write(sd, 0x48, hdmi_write(sd, 0x48,
(pdata->disable_pwrdnb ? 0x80 : 0) | (pdata->disable_pwrdnb ? 0x80 : 0) |
(pdata->disable_cable_det_rst ? 0x40 : 0)); (pdata->disable_cable_det_rst ? 0x40 : 0));
...@@ -2356,7 +2392,7 @@ static int adv7842_core_init(struct v4l2_subdev *sd, ...@@ -2356,7 +2392,7 @@ static int adv7842_core_init(struct v4l2_subdev *sd,
/* video format */ /* video format */
io_write(sd, 0x02, io_write(sd, 0x02,
pdata->inp_color_space << 4 | 0xf0 |
pdata->alt_gamma << 3 | pdata->alt_gamma << 3 |
pdata->op_656_range << 2 | pdata->op_656_range << 2 |
pdata->rgb_out << 1 | pdata->rgb_out << 1 |
...@@ -2570,7 +2606,7 @@ static int adv7842_command_ram_test(struct v4l2_subdev *sd) ...@@ -2570,7 +2606,7 @@ static int adv7842_command_ram_test(struct v4l2_subdev *sd)
adv7842_rewrite_i2c_addresses(sd, pdata); adv7842_rewrite_i2c_addresses(sd, pdata);
/* and re-init chip and state */ /* and re-init chip and state */
adv7842_core_init(sd, pdata); adv7842_core_init(sd);
disable_input(sd); disable_input(sd);
......
...@@ -163,9 +163,6 @@ struct adv7842_platform_data { ...@@ -163,9 +163,6 @@ struct adv7842_platform_data {
/* Video standard */ /* Video standard */
enum adv7842_vid_std_select vid_std_select; enum adv7842_vid_std_select vid_std_select;
/* Input Color Space */
enum adv7842_inp_color_space inp_color_space;
/* Select output format */ /* Select output format */
enum adv7842_op_format_sel op_format_sel; enum adv7842_op_format_sel op_format_sel;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment