[PATCH] Provide damage region for screen recording on Raspberry Pi

[Bryce Harrington]

On Thu, Oct 22, 2015 at 11:44:49AM +0900, Tomohito Esaki wrote:
> From: Shinya Saito <ssaito at igel.co.jp>
> 
> When trying to do sceen recording using the screenshooter, no screen data
> was ever processed because the rpi-renderer never set the previous frames
> damage area.
> 
> Update the rpi-renderer to copy the necessary data.
> 
> Signed-off-by: Shinya Saito <ssaito at igel.co.jp>
> Signed-off-by: Tomohito Esaki <etom at igel.co.jp>
> ---
> 
>  Tested on weston v1.8.0
>  Compile tested on v1.9.0. (I don't have access to a Raspberry Pi right now)

I see looking at both pixman_renderer_repaint_output() and
gl_renderer_repaint_output() that they both make this call, so this fix
looks appropriate, but can't test it either so dunno if it's sufficient.

Reviewed-by: Bryce Harrington <bryce at osg.samsung.com>

Anyone have a rpi device to check and give a Tested-by on this?
 
>  src/rpi-renderer.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/rpi-renderer.c b/src/rpi-renderer.c
> index d0a9797..76e1ddf 100644
> --- a/src/rpi-renderer.c
> +++ b/src/rpi-renderer.c
> @@ -1410,6 +1410,7 @@ rpi_renderer_repaint_output(struct weston_output *base,
>  	/* The frame_signal is emitted in rpi_renderer_finish_frame(),
>  	 * so that the firmware can capture the up-to-date contents.
>  	 */
> +	pixman_region32_copy(&base->previous_damage, output_damage);
>  }
>  
>  static void
> -- 
> 1.8.0
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel