[PATCH xf86-video-geode] Remove UploadToScreen and DownloadFromScreen

Matthieu Herrb matthieu at herrb.eu
Tue Aug 25 23:02:42 PDT 2015


On Tue, Aug 25, 2015 at 11:32:18PM -0400, Connor Behan wrote:
> The only reason these might be a tiny bit faster than other (recently
> removed) unaccelerated hooks is that they use assembly rather than
> memcpy. However, it still seems like a good idea to remove things that
> make the code less portable.

Hi,

I'm not sure if the Geode GX exists as a discrete GPU or if it's only
bundled into Geode CPU, but I think it's the latter.

So portability doesn't really matter, as you will not find a Geode GX GPU
on ARM or so...

Also there is a dedicated list  <xorg-driver-geode at lists.x.org> for
this driver, maybe it's better to move the discussion there. I'm
puting it on carbon copy.

> 
> Signed-off-by: Connor Behan <connor.behan at gmail.com>
> ---
>  src/gx_accel.c | 33 ---------------------------------
>  1 file changed, 33 deletions(-)
> 
> diff --git a/src/gx_accel.c b/src/gx_accel.c
> index c3b3bc4..48587f2 100644
> --- a/src/gx_accel.c
> +++ b/src/gx_accel.c
> @@ -1485,36 +1485,6 @@ amd_gx_exa_Done(PixmapPtr p)
>  {
>  }
>  
> -static Bool
> -amd_gx_exa_UploadToScreen(PixmapPtr pDst, int x, int y, int w, int h,
> -                          char *src, int src_pitch)
> -{
> -    char *dst = pDst->devPrivate.ptr;
> -    int dst_pitch = exaGetPixmapPitch(pDst);
> -    int bpp = pDst->drawable.bitsPerPixel;
> -
> -    dst += y * dst_pitch + x * (bpp >> 3);
> -    GU2_WAIT_BUSY;
> -    geode_memory_to_screen_blt((unsigned long) src, (unsigned long) dst,
> -                               src_pitch, dst_pitch, w, h, bpp);
> -    return TRUE;
> -}
> -
> -static Bool
> -amd_gx_exa_DownloadFromScreen(PixmapPtr pSrc, int x, int y, int w, int h,
> -                              char *dst, int dst_pitch)
> -{
> -    char *src = pSrc->devPrivate.ptr;
> -    int src_pitch = exaGetPixmapPitch(pSrc);
> -    int bpp = pSrc->drawable.bitsPerPixel;
> -
> -    src += (y * src_pitch) + (x * (bpp >> 3));
> -    GU2_WAIT_BUSY;
> -    geode_memory_to_screen_blt((unsigned long) src, (unsigned long) dst,
> -                               src_pitch, dst_pitch, w, h, bpp);
> -    return TRUE;
> -}
> -
>  /* Solid */
>  
>  static Bool
> @@ -1944,9 +1914,6 @@ GXAccelInit(ScreenPtr pScrn)
>  
>          /* Sync */
>          pExa->WaitMarker = amd_gx_exa_WaitMarker;
> -        /* UploadToScreen */
> -        pExa->UploadToScreen = amd_gx_exa_UploadToScreen;
> -        pExa->DownloadFromScreen = amd_gx_exa_DownloadFromScreen;
>  
>          /* Solid fill */
>          pExa->PrepareSolid = amd_gx_exa_PrepareSolid;
> -- 
> 2.5.0
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel

-- 
Matthieu Herrb


More information about the xorg-devel mailing list