dix: Remove usage_hint from pixmaps, store it in ->drawable.class
The class field was unused for pixmaps, and we don't have enough classes to justify a whole uint32 anyway. Reviewed-by: Soren Sandmann <ssp@redhat.com> Signed-off-by: Adam Jackson <ajax@redhat.com>
This commit is contained in:
parent
d8caa78200
commit
1564c82417
|
@ -712,7 +712,7 @@ typedef struct _ExaDriver {
|
|||
* driver MUST fill out new_fb_pitch with valid pitch of pixmap
|
||||
*/
|
||||
void *(*CreatePixmap2)(ScreenPtr pScreen, int width, int height,
|
||||
int depth, int usage_hint, int bitsPerPixel,
|
||||
int depth, int class, int bitsPerPixel,
|
||||
int *new_fb_pitch);
|
||||
/** @} */
|
||||
} ExaDriverRec, *ExaDriverPtr;
|
||||
|
|
|
@ -54,7 +54,7 @@ ExaGetPixmapAddress(PixmapPtr p)
|
|||
*/
|
||||
PixmapPtr
|
||||
exaCreatePixmap_classic(ScreenPtr pScreen, int w, int h, int depth,
|
||||
unsigned usage_hint)
|
||||
unsigned class)
|
||||
{
|
||||
PixmapPtr pPixmap;
|
||||
ExaPixmapPrivPtr pExaPixmap;
|
||||
|
@ -66,7 +66,7 @@ exaCreatePixmap_classic(ScreenPtr pScreen, int w, int h, int depth,
|
|||
return NullPixmap;
|
||||
|
||||
swap(pExaScr, pScreen, CreatePixmap);
|
||||
pPixmap = pScreen->CreatePixmap (pScreen, w, h, depth, usage_hint);
|
||||
pPixmap = pScreen->CreatePixmap (pScreen, w, h, depth, class);
|
||||
swap(pExaScr, pScreen, CreatePixmap);
|
||||
|
||||
if (!pPixmap)
|
||||
|
|
|
@ -48,7 +48,7 @@ ExaGetPixmapAddress(PixmapPtr p)
|
|||
*/
|
||||
PixmapPtr
|
||||
exaCreatePixmap_driver(ScreenPtr pScreen, int w, int h, int depth,
|
||||
unsigned usage_hint)
|
||||
unsigned class)
|
||||
{
|
||||
PixmapPtr pPixmap;
|
||||
ExaPixmapPrivPtr pExaPixmap;
|
||||
|
@ -60,7 +60,7 @@ exaCreatePixmap_driver(ScreenPtr pScreen, int w, int h, int depth,
|
|||
return NullPixmap;
|
||||
|
||||
swap(pExaScr, pScreen, CreatePixmap);
|
||||
pPixmap = pScreen->CreatePixmap(pScreen, 0, 0, depth, usage_hint);
|
||||
pPixmap = pScreen->CreatePixmap(pScreen, 0, 0, depth, class);
|
||||
swap(pExaScr, pScreen, CreatePixmap);
|
||||
|
||||
if (!pPixmap)
|
||||
|
@ -78,7 +78,7 @@ exaCreatePixmap_driver(ScreenPtr pScreen, int w, int h, int depth,
|
|||
|
||||
if (pExaScr->info->CreatePixmap2) {
|
||||
int new_pitch = 0;
|
||||
pExaPixmap->driverPriv = pExaScr->info->CreatePixmap2(pScreen, w, h, depth, usage_hint, bpp, &new_pitch);
|
||||
pExaPixmap->driverPriv = pExaScr->info->CreatePixmap2(pScreen, w, h, depth, class, bpp, &new_pitch);
|
||||
paddedWidth = pExaPixmap->fb_pitch = new_pitch;
|
||||
}
|
||||
else {
|
||||
|
|
|
@ -39,7 +39,7 @@ exaCreateDriverPixmap_mixed(PixmapPtr pPixmap)
|
|||
ExaPixmapPriv(pPixmap);
|
||||
int w = pPixmap->drawable.width, h = pPixmap->drawable.height;
|
||||
int depth = pPixmap->drawable.depth, bpp = pPixmap->drawable.bitsPerPixel;
|
||||
int usage_hint = pPixmap->usage_hint;
|
||||
int class = pPixmap->drawable.class;
|
||||
int paddedWidth = pExaPixmap->sys_pitch;
|
||||
|
||||
/* Already done. */
|
||||
|
@ -55,7 +55,7 @@ exaCreateDriverPixmap_mixed(PixmapPtr pPixmap)
|
|||
|
||||
if (pExaScr->info->CreatePixmap2) {
|
||||
int new_pitch = 0;
|
||||
pExaPixmap->driverPriv = pExaScr->info->CreatePixmap2(pScreen, w, h, depth, usage_hint, bpp, &new_pitch);
|
||||
pExaPixmap->driverPriv = pExaScr->info->CreatePixmap2(pScreen, w, h, depth, class, bpp, &new_pitch);
|
||||
paddedWidth = pExaPixmap->fb_pitch = new_pitch;
|
||||
} else {
|
||||
if (paddedWidth < pExaPixmap->fb_pitch)
|
||||
|
|
|
@ -47,7 +47,7 @@ ExaGetPixmapAddress(PixmapPtr p)
|
|||
*/
|
||||
PixmapPtr
|
||||
exaCreatePixmap_mixed(ScreenPtr pScreen, int w, int h, int depth,
|
||||
unsigned usage_hint)
|
||||
unsigned class)
|
||||
{
|
||||
PixmapPtr pPixmap;
|
||||
ExaPixmapPrivPtr pExaPixmap;
|
||||
|
@ -59,7 +59,7 @@ exaCreatePixmap_mixed(ScreenPtr pScreen, int w, int h, int depth,
|
|||
return NullPixmap;
|
||||
|
||||
swap(pExaScr, pScreen, CreatePixmap);
|
||||
pPixmap = pScreen->CreatePixmap(pScreen, 0, 0, depth, usage_hint);
|
||||
pPixmap = pScreen->CreatePixmap(pScreen, 0, 0, depth, class);
|
||||
swap(pExaScr, pScreen, CreatePixmap);
|
||||
|
||||
if (!pPixmap)
|
||||
|
|
|
@ -588,7 +588,7 @@ extern const GCFuncs exaGCFuncs;
|
|||
/* exa_classic.c */
|
||||
PixmapPtr
|
||||
exaCreatePixmap_classic(ScreenPtr pScreen, int w, int h, int depth,
|
||||
unsigned usage_hint);
|
||||
unsigned class);
|
||||
|
||||
Bool
|
||||
exaModifyPixmapHeader_classic(PixmapPtr pPixmap, int width, int height, int depth,
|
||||
|
@ -603,7 +603,7 @@ exaPixmapHasGpuCopy_classic(PixmapPtr pPixmap);
|
|||
/* exa_driver.c */
|
||||
PixmapPtr
|
||||
exaCreatePixmap_driver(ScreenPtr pScreen, int w, int h, int depth,
|
||||
unsigned usage_hint);
|
||||
unsigned class);
|
||||
|
||||
Bool
|
||||
exaModifyPixmapHeader_driver(PixmapPtr pPixmap, int width, int height, int depth,
|
||||
|
@ -618,7 +618,7 @@ exaPixmapHasGpuCopy_driver(PixmapPtr pPixmap);
|
|||
/* exa_mixed.c */
|
||||
PixmapPtr
|
||||
exaCreatePixmap_mixed(ScreenPtr pScreen, int w, int h, int depth,
|
||||
unsigned usage_hint);
|
||||
unsigned class);
|
||||
|
||||
Bool
|
||||
exaModifyPixmapHeader_mixed(PixmapPtr pPixmap, int width, int height, int depth,
|
||||
|
|
4
fb/fb.h
4
fb/fb.h
|
@ -1625,11 +1625,11 @@ fbPictureInit (ScreenPtr pScreen,
|
|||
|
||||
extern _X_EXPORT PixmapPtr
|
||||
fbCreatePixmapBpp (ScreenPtr pScreen, int width, int height, int depth, int bpp,
|
||||
unsigned usage_hint);
|
||||
unsigned class);
|
||||
|
||||
extern _X_EXPORT PixmapPtr
|
||||
fbCreatePixmap (ScreenPtr pScreen, int width, int height, int depth,
|
||||
unsigned usage_hint);
|
||||
unsigned class);
|
||||
|
||||
extern _X_EXPORT Bool
|
||||
fbDestroyPixmap (PixmapPtr pPixmap);
|
||||
|
|
|
@ -546,7 +546,7 @@ fb24_32ReformatTile(PixmapPtr pOldTile, int bitsPerPixel)
|
|||
pNewTile = pScreen->CreatePixmap(pScreen, pOldTile->drawable.width,
|
||||
pOldTile->drawable.height,
|
||||
pOldTile->drawable.depth,
|
||||
pOldTile->usage_hint);
|
||||
pOldTile->drawable.class);
|
||||
if (!pNewTile)
|
||||
return 0;
|
||||
fbGetDrawable (&pOldTile->drawable,
|
||||
|
|
|
@ -30,7 +30,7 @@
|
|||
|
||||
PixmapPtr
|
||||
fbCreatePixmapBpp (ScreenPtr pScreen, int width, int height, int depth, int bpp,
|
||||
unsigned usage_hint)
|
||||
unsigned class)
|
||||
{
|
||||
PixmapPtr pPixmap;
|
||||
size_t datasize;
|
||||
|
@ -54,7 +54,7 @@ fbCreatePixmapBpp (ScreenPtr pScreen, int width, int height, int depth, int bpp,
|
|||
if (!pPixmap)
|
||||
return NullPixmap;
|
||||
pPixmap->drawable.type = DRAWABLE_PIXMAP;
|
||||
pPixmap->drawable.class = 0;
|
||||
pPixmap->drawable.class = class;
|
||||
pPixmap->drawable.pScreen = pScreen;
|
||||
pPixmap->drawable.depth = depth;
|
||||
pPixmap->drawable.bitsPerPixel = bpp;
|
||||
|
@ -78,14 +78,12 @@ fbCreatePixmapBpp (ScreenPtr pScreen, int width, int height, int depth, int bpp,
|
|||
pPixmap->screen_y = 0;
|
||||
#endif
|
||||
|
||||
pPixmap->usage_hint = usage_hint;
|
||||
|
||||
return pPixmap;
|
||||
}
|
||||
|
||||
PixmapPtr
|
||||
fbCreatePixmap (ScreenPtr pScreen, int width, int height, int depth,
|
||||
unsigned usage_hint)
|
||||
unsigned class)
|
||||
{
|
||||
int bpp;
|
||||
bpp = BitsPerPixel (depth);
|
||||
|
@ -93,7 +91,7 @@ fbCreatePixmap (ScreenPtr pScreen, int width, int height, int depth,
|
|||
if (bpp == 32 && depth <= 24)
|
||||
bpp = fbGetScreenPrivate(pScreen)->pix32bpp;
|
||||
#endif
|
||||
return fbCreatePixmapBpp (pScreen, width, height, depth, bpp, usage_hint);
|
||||
return fbCreatePixmapBpp (pScreen, width, height, depth, bpp, class);
|
||||
}
|
||||
|
||||
Bool
|
||||
|
|
|
@ -82,7 +82,7 @@ void dmxBECreatePixmap(PixmapPtr pPixmap)
|
|||
/** Create a pixmap for \a pScreen with the specified \a width, \a
|
||||
* height, and \a depth. */
|
||||
PixmapPtr dmxCreatePixmap(ScreenPtr pScreen, int width, int height, int depth,
|
||||
unsigned usage_hint)
|
||||
unsigned class)
|
||||
{
|
||||
DMXScreenInfo *dmxScreen = &dmxScreens[pScreen->myNum];
|
||||
PixmapPtr pPixmap;
|
||||
|
@ -104,7 +104,7 @@ PixmapPtr dmxCreatePixmap(ScreenPtr pScreen, int width, int height, int depth,
|
|||
return NullPixmap;
|
||||
|
||||
pPixmap->drawable.type = DRAWABLE_PIXMAP;
|
||||
pPixmap->drawable.class = 0;
|
||||
pPixmap->drawable.class = class;
|
||||
pPixmap->drawable.pScreen = pScreen;
|
||||
pPixmap->drawable.depth = depth;
|
||||
pPixmap->drawable.bitsPerPixel = bpp;
|
||||
|
@ -116,7 +116,6 @@ PixmapPtr dmxCreatePixmap(ScreenPtr pScreen, int width, int height, int depth,
|
|||
pPixmap->drawable.height = height;
|
||||
pPixmap->devKind = PixmapBytePad(width, bpp);
|
||||
pPixmap->refcnt = 1;
|
||||
pPixmap->usage_hint = usage_hint;
|
||||
|
||||
pPixPriv = DMX_GET_PIXMAP_PRIV(pPixmap);
|
||||
pPixPriv->pixmap = (Pixmap)0;
|
||||
|
|
|
@ -50,7 +50,7 @@ extern Bool dmxInitPixmap(ScreenPtr pScreen);
|
|||
|
||||
extern PixmapPtr dmxCreatePixmap(ScreenPtr pScreen,
|
||||
int width, int height, int depth,
|
||||
unsigned usage_hint);
|
||||
unsigned class);
|
||||
extern Bool dmxDestroyPixmap(PixmapPtr pPixmap);
|
||||
extern RegionPtr dmxBitmapToRegion(PixmapPtr pPixmap);
|
||||
|
||||
|
|
|
@ -369,13 +369,13 @@ VGAarbiterClearToBackground (
|
|||
}
|
||||
|
||||
static PixmapPtr
|
||||
VGAarbiterCreatePixmap(ScreenPtr pScreen, int w, int h, int depth, unsigned usage_hint)
|
||||
VGAarbiterCreatePixmap(ScreenPtr pScreen, int w, int h, int depth, unsigned class)
|
||||
{
|
||||
PixmapPtr pPix;
|
||||
|
||||
SCREEN_PROLOG ( CreatePixmap);
|
||||
VGAGet(pScreen);
|
||||
pPix = (*pScreen->CreatePixmap) (pScreen, w, h, depth, usage_hint);
|
||||
pPix = (*pScreen->CreatePixmap) (pScreen, w, h, depth, class);
|
||||
VGAPut();
|
||||
SCREEN_EPILOG (CreatePixmap, VGAarbiterCreatePixmap);
|
||||
|
||||
|
|
|
@ -155,7 +155,7 @@ static void VGAarbiterCopyWindow(WindowPtr pWin, DDXPointRec ptOldOrg,
|
|||
static void VGAarbiterClearToBackground (WindowPtr pWin, int x, int y, int w,
|
||||
int h, Bool generateExposures);
|
||||
static PixmapPtr VGAarbiterCreatePixmap(ScreenPtr pScreen, int w, int h,
|
||||
int depth, unsigned int usage_hint);
|
||||
int depth, unsigned int class);
|
||||
static Bool VGAarbiterCreateGC(GCPtr pGC);
|
||||
static Bool VGAarbiterSaveScreen(ScreenPtr pScreen, Bool unblank);
|
||||
static void VGAarbiterStoreColors (ColormapPtr pmap, int ndef, xColorItem
|
||||
|
|
|
@ -34,7 +34,7 @@ static void XAAGetImage(DrawablePtr pDrawable, int sx, int sy, int w, int h,
|
|||
static void XAAGetSpans(DrawablePtr pDrawable, int wMax, DDXPointPtr ppt,
|
||||
int *pwidth, int nspans, char *pdstStart);
|
||||
static PixmapPtr XAACreatePixmap(ScreenPtr pScreen, int w, int h, int depth,
|
||||
unsigned usage_hint);
|
||||
unsigned class);
|
||||
static Bool XAADestroyPixmap(PixmapPtr pPixmap);
|
||||
static Bool XAAEnterVT (int index, int flags);
|
||||
static void XAALeaveVT (int index, int flags);
|
||||
|
@ -331,7 +331,7 @@ XAAInitializeOffscreenDepths (ScreenPtr pScreen)
|
|||
}
|
||||
|
||||
static PixmapPtr
|
||||
XAACreatePixmap(ScreenPtr pScreen, int w, int h, int depth, unsigned usage_hint)
|
||||
XAACreatePixmap(ScreenPtr pScreen, int w, int h, int depth, unsigned class)
|
||||
{
|
||||
XAAInfoRecPtr infoRec = GET_XAAINFORECPTR_FROM_SCREEN(pScreen);
|
||||
ScrnInfoPtr pScrn = xf86Screens[pScreen->myNum];
|
||||
|
@ -346,7 +346,7 @@ XAACreatePixmap(ScreenPtr pScreen, int w, int h, int depth, unsigned usage_hint)
|
|||
XAAInitializeOffscreenDepths (pScreen);
|
||||
|
||||
if(pScrn->vtSema &&
|
||||
(usage_hint != CREATE_PIXMAP_USAGE_GLYPH_PICTURE) &&
|
||||
(class != CREATE_PIXMAP_USAGE_GLYPH_PICTURE) &&
|
||||
(infoRec->offscreenDepths & (1 << (depth - 1))) &&
|
||||
(size >= MIN_OFFPIX_SIZE) && !SwitchedOut &&
|
||||
(!infoRec->maxOffPixWidth || (w <= infoRec->maxOffPixWidth)) &&
|
||||
|
@ -379,7 +379,7 @@ XAACreatePixmap(ScreenPtr pScreen, int w, int h, int depth, unsigned usage_hint)
|
|||
}
|
||||
|
||||
XAA_SCREEN_PROLOGUE (pScreen, CreatePixmap);
|
||||
pPix = (*pScreen->CreatePixmap) (pScreen, 0, 0, depth, usage_hint);
|
||||
pPix = (*pScreen->CreatePixmap) (pScreen, 0, 0, depth, class);
|
||||
XAA_SCREEN_EPILOGUE (pScreen, CreatePixmap, XAACreatePixmap);
|
||||
|
||||
if (!pPix) {
|
||||
|
@ -411,7 +411,7 @@ XAACreatePixmap(ScreenPtr pScreen, int w, int h, int depth, unsigned usage_hint)
|
|||
}
|
||||
BAILOUT:
|
||||
XAA_SCREEN_PROLOGUE (pScreen, CreatePixmap);
|
||||
pPix = (*pScreen->CreatePixmap) (pScreen, w, h, depth, usage_hint);
|
||||
pPix = (*pScreen->CreatePixmap) (pScreen, w, h, depth, class);
|
||||
XAA_SCREEN_EPILOGUE (pScreen, CreatePixmap, XAACreatePixmap);
|
||||
|
||||
if(pPix) {
|
||||
|
|
|
@ -37,7 +37,7 @@ DevPrivateKeyRec xnestPixmapPrivateKeyRec;
|
|||
|
||||
PixmapPtr
|
||||
xnestCreatePixmap(ScreenPtr pScreen, int width, int height, int depth,
|
||||
unsigned usage_hint)
|
||||
unsigned class)
|
||||
{
|
||||
PixmapPtr pPixmap;
|
||||
|
||||
|
@ -45,7 +45,7 @@ xnestCreatePixmap(ScreenPtr pScreen, int width, int height, int depth,
|
|||
if (!pPixmap)
|
||||
return NullPixmap;
|
||||
pPixmap->drawable.type = DRAWABLE_PIXMAP;
|
||||
pPixmap->drawable.class = 0;
|
||||
pPixmap->drawable.class = class;
|
||||
pPixmap->drawable.depth = depth;
|
||||
pPixmap->drawable.bitsPerPixel = depth;
|
||||
pPixmap->drawable.id = 0;
|
||||
|
@ -57,7 +57,6 @@ xnestCreatePixmap(ScreenPtr pScreen, int width, int height, int depth,
|
|||
pPixmap->drawable.serialNumber = NEXT_SERIAL_NUMBER;
|
||||
pPixmap->refcnt = 1;
|
||||
pPixmap->devKind = PixmapBytePad(width, depth);
|
||||
pPixmap->usage_hint = usage_hint;
|
||||
if (width && height)
|
||||
xnestPixmapPriv(pPixmap)->pixmap =
|
||||
XCreatePixmap(xnestDisplay,
|
||||
|
|
|
@ -30,7 +30,7 @@ typedef struct {
|
|||
#define xnestSharePixmap(pPixmap) ((pPixmap)->refcnt++)
|
||||
|
||||
PixmapPtr xnestCreatePixmap(ScreenPtr pScreen, int width, int height,
|
||||
int depth, unsigned usage_hint);
|
||||
int depth, unsigned class);
|
||||
Bool xnestDestroyPixmap(PixmapPtr pPixmap);
|
||||
RegionPtr xnestPixmapToRegion(PixmapPtr pPixmap);
|
||||
|
||||
|
|
|
@ -1069,7 +1069,7 @@ winSetEngineFunctionsPrimaryDD (ScreenPtr pScreen);
|
|||
|
||||
PixmapPtr
|
||||
winCreatePixmapNativeGDI (ScreenPtr pScreen, int width, int height, int depth,
|
||||
unsigned usage_hint);
|
||||
unsigned class);
|
||||
|
||||
Bool
|
||||
winDestroyPixmapNativeGDI (PixmapPtr pPixmap);
|
||||
|
|
|
@ -57,7 +57,7 @@ winCopyRotatePixmapNativeGDI (PixmapPtr psrcPix, PixmapPtr *ppdstPix,
|
|||
PixmapPtr
|
||||
winCreatePixmapNativeGDI (ScreenPtr pScreen,
|
||||
int iWidth, int iHeight,
|
||||
int iDepth, unsigned usage_hint)
|
||||
int iDepth, unsigned class)
|
||||
{
|
||||
winPrivPixmapPtr pPixmapPriv = NULL;
|
||||
PixmapPtr pPixmap = NULL;
|
||||
|
@ -72,13 +72,13 @@ winCreatePixmapNativeGDI (ScreenPtr pScreen,
|
|||
|
||||
#if CYGDEBUG
|
||||
winDebug ("winCreatePixmap () - w %d h %d d %d uh %d bw %d\n",
|
||||
iWidth, iHeight, iDepth, usage_hint,
|
||||
iWidth, iHeight, iDepth, class,
|
||||
PixmapBytePad (iWidth, iDepth));
|
||||
#endif
|
||||
|
||||
/* Setup pixmap values */
|
||||
pPixmap->drawable.type = DRAWABLE_PIXMAP;
|
||||
pPixmap->drawable.class = 0;
|
||||
pPixmap->drawable.class = class;
|
||||
pPixmap->drawable.pScreen = pScreen;
|
||||
pPixmap->drawable.depth = iDepth;
|
||||
pPixmap->drawable.bitsPerPixel = BitsPerPixel (iDepth);
|
||||
|
@ -91,7 +91,6 @@ winCreatePixmapNativeGDI (ScreenPtr pScreen,
|
|||
pPixmap->devKind = 0;
|
||||
pPixmap->refcnt = 1;
|
||||
pPixmap->devPrivate.ptr = NULL;
|
||||
pPixmap->usage_hint = usage_hint;
|
||||
|
||||
/* Pixmap privates are allocated by AllocatePixmap */
|
||||
pPixmapPriv = winGetPixmapPriv (pPixmap);
|
||||
|
|
|
@ -80,7 +80,6 @@ typedef struct _Pixmap {
|
|||
short screen_x;
|
||||
short screen_y;
|
||||
#endif
|
||||
unsigned usage_hint; /* see CREATE_PIXMAP_USAGE_* */
|
||||
} PixmapRec;
|
||||
|
||||
#endif /* PIXMAPSTRUCT_H */
|
||||
|
|
|
@ -208,7 +208,7 @@ typedef PixmapPtr (* CreatePixmapProcPtr)(
|
|||
int /*width*/,
|
||||
int /*height*/,
|
||||
int /*depth*/,
|
||||
unsigned /*usage_hint*/);
|
||||
unsigned /*class*/);
|
||||
|
||||
typedef Bool (* DestroyPixmapProcPtr)(
|
||||
PixmapPtr /*pPixmap*/);
|
||||
|
|
Loading…
Reference in New Issue