On 03/30/2015 01:10 PM, Philipp Zabel wrote:
Instead of only using the visible width and height, also add the
horizontal and vertical blanking to calculate the bit rate.
I am not a CSI expert and when I look at the spreadsheet from Toshiba I
understand that the calculation of the CSI parameters is quite complex.
I think you are right that there should be added some kind of blanking
to the active video, but adding all the blanking from the HDMI format
gives a too high value. It seems like the calculation will fail if "CSI
clock enable during LP" is set to "Disable", but that is not supported
by the driver at the moment.
I have tested all the resolutions between 640x480p60 and 1920x1080p60
that my video generator will give me, and it works perfect on our
product. Do you have to change this for your product?
Signed-off-by: Philipp Zabel<p.zabel@xxxxxxxxxxxxxx>
---
drivers/media/i2c/tc358743.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/media/i2c/tc358743.c b/drivers/media/i2c/tc358743.c
index dd2ea16..74e83c5 100644
--- a/drivers/media/i2c/tc358743.c
+++ b/drivers/media/i2c/tc358743.c
@@ -713,9 +713,11 @@ static unsigned tc358743_num_csi_lanes_needed(struct v4l2_subdev *sd)
{
struct tc358743_state *state = to_state(sd);
struct v4l2_bt_timings *bt = &state->timings.bt;
+ u32 htotal = bt->width + bt->hfrontporch + bt->hsync + bt->hbackporch;
+ u32 vtotal = bt->height + bt->vfrontporch + bt->vsync + bt->vbackporch;
By the way, V4L2_DV_BT_FRAME_WIDTH(bt) and V4L2_DV_BT_FRAME_HEIGHT(bt)
can be used to calculate total frame width and height.
Regards,
Mats Randgaard
u32 bits_pr_pixel =
(state->mbus_fmt_code == MEDIA_BUS_FMT_UYVY8_1X16) ? 16 : 24;
- u32 bps = bt->width * bt->height * fps(bt) * bits_pr_pixel;
+ u32 bps = htotal * vtotal * fps(bt) * bits_pr_pixel;
return DIV_ROUND_UP(bps, state->pdata.bps_pr_lane);
}
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
