Hi Kaneko-san, Hosoya-san,
On Sun, Mar 15, 2015 at 3:33 PM, Yoshihiro Kaneko <ykaneko0929@xxxxxxxxx> wrote:
> From: Yoshifumi Hosoya <yoshifumi.hosoya.wj@xxxxxxxxxxx>
>
> Change to VSPD hardware recommended value.
> Purpose is highest pixel clock without underruns.
> In the default R-Car Linux BSP config this value is
> wrong and therefore there are many underruns.
>
> Here are the original settings:
> HBTH = 1300 (VSPD stops when 1300 pixels are buffered)
> LBTH = 200 (VSPD resumes when buffer level has decreased
> below 200 pixels)
>
> The display underruns can be eliminated
> by applying the following settings:
> HBTH = 1504
> LBTH = 1248
> --- a/drivers/media/platform/vsp1/vsp1_lif.c
> +++ b/drivers/media/platform/vsp1/vsp1_lif.c
> @@ -44,9 +44,9 @@ static int lif_s_stream(struct v4l2_subdev *subdev, int enable)
> {
> const struct v4l2_mbus_framefmt *format;
> struct vsp1_lif *lif = to_lif(subdev);
> - unsigned int hbth = 1300;
> - unsigned int obth = 400;
> - unsigned int lbth = 200;
> + unsigned int hbth = 1536;
> + unsigned int obth = 128;
> + unsigned int lbth = 1520;
These values don't match the patch description?
BTW, what's the significance of changing obth?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
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
