<div dir="ltr"><div><div><div><div><div>Ok I will try to write a better commit message.<br></div>But for frame buffer understanding, the buffer is often allowed for a screen with the double of height as showed. Then is possible to set the buffer with two screens, one above the other. After we switch the display from the first middle buffer to the next one with "yoffset" and the FIBIOPAN_DISPLAY. Normally it's used to have double buffering and improve the number of frames per second (double buffering).<br>
</div>But the problem is that yoffset is not always configured to 0 at the startup and we have to set it.<br><br></div>I plan to add double buffering but in few days.<br></div>Marc.<br><br></div>PS: I have to support SurfaceFlinger (Android) as wayland client. It's the reason of my work. But I can stay a long time on my weston patches...<br>
<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/10 Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Fri, 7 Jun 2013 17:20:23 +0200<br>
Marc Chalain <<a href="mailto:marc.chalain@gmail.com">marc.chalain@gmail.com</a>> wrote:<br>
<br>
> From 6553ed3ad9845113f3b2280e6fc94e2b326a7eca Mon Sep 17 00:00:00 2001<br>
> From: mchalain <<a href="mailto:marc.chalain@gmail.com">marc.chalain@gmail.com</a>><br>
> Date: Fri, 7 Jun 2013 17:18:23 +0200<br>
> Subject: [PATCH 2/2] [fbdev_backend] initialization of the first frame<br>
> buffer<br>
><br>
> ---<br>
>  src/compositor-fbdev.c |    8 +++++++-<br>
>  1 file changed, 7 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/compositor-fbdev.c b/src/compositor-fbdev.c<br>
> index d6d911a..e6c761f 100644<br>
> --- a/src/compositor-fbdev.c<br>
> +++ b/src/compositor-fbdev.c<br>
> @@ -277,7 +277,7 @@ calculate_pixman_format(struct fb_var_screeninfo *vinfo,<br>
>      if (finfo->visual == FB_VISUAL_DIRECTCOLOR)<br>
>          build_cmap(vinfo, cmap);<br>
>      else<br>
> -     /* We only handle true-colour frame buffers at the moment. */<br>
> +    /* We only handle true-colour frame buffers at the moment. */<br>
>      if (finfo->visual != FB_VISUAL_TRUECOLOR || vinfo->grayscale != 0)<br>
>          return 0;<br>
><br>
> @@ -373,6 +373,11 @@ fbdev_query_screen_info(struct fbdev_output *output,<br>
> int fd,<br>
>          return -1;<br>
>      }<br>
><br>
> +    if (varinfo.yoffset != 0) {<br>
> +        varinfo.yoffset    = 0;<br>
> +        if (ioctl(fd, FBIOPAN_DISPLAY, &varinfo) < 0)<br>
> +            return -1;<br>
> +    }<br>
>      return 1;<br>
>  }<br>
><br>
> @@ -409,6 +414,7 @@ fbdev_set_screen_info(struct fbdev_output *output, int<br>
> fd,<br>
>      varinfo.blue.length = 8;<br>
>      varinfo.blue.msb_right = 0;<br>
><br>
> +    varinfo.yoffset = 0;<br>
>      /* Set the device's screen information. */<br>
>      if (ioctl(fd, FBIOPUT_VSCREENINFO, &varinfo) < 0) {<br>
>          return -1;<br>
<br>
</div></div>Hi Marc,<br>
<br>
I admit I am not familiar with the fbdev interface, but this patch is<br>
missing a commit message. I have no clue what is going on here, and I<br>
could not understand the note for this patch in the previous email.<br>
<br>
If you don't know what to write in a commit message, the following two<br>
links give a good idea:<br>
<a href="https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages" target="_blank">https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages</a><br>
<a href="http://who-t.blogspot.de/2009/12/on-commit-messages.html" target="_blank">http://who-t.blogspot.de/2009/12/on-commit-messages.html</a><br>
<br>
The same comment about a missing commit message applies to your<br>
latest "DIRECTCOLOR" patch, too.<br>
<br>
<br>
Thanks,<br>
pq<br>
</blockquote></div><br></div>