[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