[media] adv7604: pixel-clock depends on deep-color-mode
The frequency calculation has to take deep-color mode into account. While we're at it, also log the deep-color mode in log_status. Signed-off-by: Martin Bugge <marbugge@cisco.com> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
This commit is contained in:
parent
25a64ac950
commit
bb88f325a4
|
@ -992,6 +992,11 @@ static inline bool no_lock_tmds(struct v4l2_subdev *sd)
|
|||
return (io_read(sd, 0x6a) & 0xe0) != 0xe0;
|
||||
}
|
||||
|
||||
static inline bool is_hdmi(struct v4l2_subdev *sd)
|
||||
{
|
||||
return hdmi_read(sd, 0x05) & 0x80;
|
||||
}
|
||||
|
||||
static inline bool no_lock_sspd(struct v4l2_subdev *sd)
|
||||
{
|
||||
/* TODO channel 2 */
|
||||
|
@ -1244,12 +1249,21 @@ static int adv7604_query_dv_timings(struct v4l2_subdev *sd,
|
|||
V4L2_DV_INTERLACED : V4L2_DV_PROGRESSIVE;
|
||||
|
||||
if (DIGITAL_INPUT) {
|
||||
uint32_t freq;
|
||||
|
||||
timings->type = V4L2_DV_BT_656_1120;
|
||||
|
||||
bt->width = (hdmi_read(sd, 0x07) & 0x0f) * 256 + hdmi_read(sd, 0x08);
|
||||
bt->height = (hdmi_read(sd, 0x09) & 0x0f) * 256 + hdmi_read(sd, 0x0a);
|
||||
bt->pixelclock = (hdmi_read(sd, 0x06) * 1000000) +
|
||||
freq = (hdmi_read(sd, 0x06) * 1000000) +
|
||||
((hdmi_read(sd, 0x3b) & 0x30) >> 4) * 250000;
|
||||
if (is_hdmi(sd)) {
|
||||
/* adjust for deep color mode */
|
||||
unsigned bits_per_channel = ((hdmi_read(sd, 0x0b) & 0x60) >> 4) + 8;
|
||||
|
||||
freq = freq * 8 / bits_per_channel;
|
||||
}
|
||||
bt->pixelclock = freq;
|
||||
bt->hfrontporch = (hdmi_read(sd, 0x20) & 0x03) * 256 +
|
||||
hdmi_read(sd, 0x21);
|
||||
bt->hsync = (hdmi_read(sd, 0x22) & 0x03) * 256 +
|
||||
|
@ -1637,7 +1651,7 @@ static void print_avi_infoframe(struct v4l2_subdev *sd)
|
|||
u8 avi_len;
|
||||
u8 avi_ver;
|
||||
|
||||
if (!(hdmi_read(sd, 0x05) & 0x80)) {
|
||||
if (!is_hdmi(sd)) {
|
||||
v4l2_info(sd, "receive DVI-D signal (AVI infoframe not supported)\n");
|
||||
return;
|
||||
}
|
||||
|
@ -1698,6 +1712,12 @@ static int adv7604_log_status(struct v4l2_subdev *sd)
|
|||
"RGB limited range (16-235)",
|
||||
"RGB full range (0-255)",
|
||||
};
|
||||
char *deep_color_mode_txt[4] = {
|
||||
"8-bits per channel",
|
||||
"10-bits per channel",
|
||||
"12-bits per channel",
|
||||
"16-bits per channel (not supported)"
|
||||
};
|
||||
|
||||
v4l2_info(sd, "-----Chip status-----\n");
|
||||
v4l2_info(sd, "Chip power: %s\n", no_power(sd) ? "off" : "on");
|
||||
|
@ -1756,7 +1776,9 @@ static int adv7604_log_status(struct v4l2_subdev *sd)
|
|||
v4l2_info(sd, "-----HDMI status-----\n");
|
||||
v4l2_info(sd, "HDCP encrypted content: %s\n",
|
||||
hdmi_read(sd, 0x05) & 0x40 ? "true" : "false");
|
||||
|
||||
if (is_hdmi(sd))
|
||||
v4l2_info(sd, "deep color mode: %s\n",
|
||||
deep_color_mode_txt[(hdmi_read(sd, 0x0b) >> 5) & 0x3]);
|
||||
print_avi_infoframe(sd);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue