Re: çåï Re: [Xen-devel][PATCH]qemu-xen: let xenfb_guest_copy() handle depth=32 case

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Fri, 29 Oct 2010, Chun Yan Liu wrote:
> >Â Could you please resubmit a patch with both changes and a signed-off-by
> >Â line?
> 
> Sure. Following is the patch with both changes.
> 

Acked

> Signed-off by Chunyan Liu <cyliu@xxxxxxxxxx>
> 
> diff -r 1e5cb7d6a96c hw/xenfb.c
> --- a/hw/xenfb.cÂÂÂ Mon Oct 18 17:24:50 2010 +0100
> +++ b/hw/xenfb.cÂÂÂ Sat Oct 30 00:48:45 2010 +0800
> @@ -630,6 +630,18 @@
> ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ oops = 1;
> ÂÂÂÂÂÂÂÂÂÂÂÂ }
> ÂÂÂÂÂÂÂÂÂÂÂÂ break;
> +ÂÂÂ case 16:
> +ÂÂÂÂÂÂÂÂÂÂÂ if (bpp == 16) {
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ for (line = y; line < (y+h); line++) {
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ memcpy (data + (line * linesize) + (x * bpp / 8), 
> xenfb->pixels + xenfb->offset
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ + (line * xenfb->row_stride) + (x * 
> xenfb->depth / 8), w * xenfb->depth / 8);
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ }
> +ÂÂÂÂÂÂÂÂÂÂÂ } else if (bpp == 32) {
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ BLT(uint16_t, uint32_t,ÂÂ 5, 6, 5,ÂÂ 8, 8, 8);
> +ÂÂÂÂÂÂÂÂÂÂÂ } else {
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ oops = 1;
> +ÂÂÂÂÂÂÂÂÂÂÂ }
> +ÂÂÂÂÂÂÂÂÂÂÂ break;
> ÂÂÂÂÂÂÂÂ case 24:
> ÂÂÂÂÂÂÂÂÂÂÂÂ if (bpp == 16) {
>  BLT(uint32_t, uint16_t, 8, 8, 8, 5, 6, 5);
> @@ -639,6 +651,18 @@
> ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ oops = 1;
> ÂÂÂÂÂÂÂÂÂÂÂÂ }
> ÂÂÂÂÂÂÂÂÂÂÂÂ break;
> +ÂÂÂÂÂÂÂ case 32:
> +ÂÂÂÂÂÂÂÂÂÂÂ if (bpp == 16) {
> + BLT(uint32_t, uint16_t, 8, 8, 8, 5, 6, 5);
> +ÂÂÂÂÂÂÂÂÂÂÂ } else if (bpp == 32) {
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ for (line = y; line < (y+h); line++) {
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ memcpy (data + (line * linesize) + (x * bpp / 8), 
> xenfb->pixels + xenfb->offset
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ + (line * xenfb->row_stride) + (x * 
> xenfb->depth / 8), w * xenfb->depth / 8);
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ }
> +ÂÂÂÂÂÂÂÂÂÂÂ } else {
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ oops = 1;
> +ÂÂÂÂÂÂÂÂÂÂÂ }
> +ÂÂÂÂÂÂÂÂÂÂÂ break;
> ÂÂÂÂÂÂÂÂ default:
> ÂÂÂÂÂÂÂÂÂÂÂÂ oops = 1;
> ÂÂÂÂ }
> @@ -792,6 +816,7 @@
> Âstatic void xenfb_invalidate(void *opaque)
> Â{
> ÂÂÂÂ struct XenFB *xenfb = opaque;
> +ÂÂÂ xenfb->do_resize = 1;
> ÂÂÂÂ xenfb->up_fullscreen = 1;
> Â}
> Â
> 
> 
>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel