[PATCH] xserver: remove all of RAC and access code

Eric Anholt eric at anholt.net
Wed Jul 29 14:51:12 PDT 2009


On Tue, 2009-07-28 at 09:17 +1000, Dave Airlie wrote:
> On Thu, 2009-07-23 at 12:32 +1000, Dave Airlie wrote:
> > this is a major API + ABI break, we'll be adding back vga arb support after
> > this hopefully.
> 
> So after a bit of messing around I think removing everything makes it
> hard to build drivers for 1.6 and 1.7, so I'm going to add some api
> compat for now.
> 
> Mainly I'm thinking of keeping the third member the pci 
> 
> > @@ -285,27 +273,29 @@ extern _X_EXPORT pointer xf86FindXvOptions(int scrnIndex, int adapt_index, char
> >  extern _X_EXPORT void xf86GetOS(const char **name, int *major, int *minor, int *teeny);
> >  extern _X_EXPORT ScrnInfoPtr xf86ConfigPciEntity(ScrnInfoPtr pScrn, int scrnFlag,
> >  				int entityIndex,PciChipsets *p_chip,
> > -				resList res, EntityProc init,
> > +				EntityProc init,
> >  				EntityProc enter, EntityProc leave,
> >  				pointer private);
> >  extern _X_EXPORT ScrnInfoPtr xf86ConfigFbEntity(ScrnInfoPtr pScrn, int scrnFlag,
> >  			       int entityIndex, EntityProc init,
> >  			       EntityProc enter, EntityProc leave,
> >  			       pointer private);
> > +
> >  /* Obsolete! don't use */
> >  extern _X_EXPORT Bool xf86ConfigActivePciEntity(ScrnInfoPtr pScrn,
> >  				int entityIndex,PciChipsets *p_chip,
> > -				resList res, EntityProc init,
> > +				EntityProc init,
> >  				EntityProc enter, EntityProc leave,
> >  				pointer private);
> 
> Add a spare member instead of resList, so the function signatures don't
> change.
> > diff --git a/hw/xfree86/common/xf86Resources.h b/hw/xfree86/common/xf86Resources.h
> > deleted file mode 100644
> > index 012fa69..0000000
> > --- a/hw/xfree86/common/xf86Resources.h
> > +++ /dev/null
> > @@ -1,137 +0,0 @@
> > -
> > -/* Old style names */
> > -#define RES_EXCLUSIVE_VGA   resVgaExclusive
> > -#define RES_SHARED_VGA      resVgaShared
> > -#define RES_EXCLUSIVE_8514  res8514Exclusive
> > -#define RES_SHARED_8514     res8514Shared
> 
> add some dummy defines for these.
> 
> > -
> >  typedef struct _PciChipsets {
> >      /**
> >       * Key used to match this device with its name in an array of
> > @@ -720,10 +581,6 @@ typedef struct _PciChipsets {
> >       */
> >      int PCIid;
> >  
> > -    /**
> > -     * Resources associated with this type of device.
> > -     */
> > -    resRange *resList;
> >  } PciChipsets;
> 
> add a dummy for resRange here so we don't have to rewrite the pci ids
> structs in lots of drivers.
> 
> good idea or bad idea?

Love it!

-- 
Eric Anholt
eric at anholt.net                         eric.anholt at intel.com


-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : http://lists.x.org/archives/xorg-devel/attachments/20090729/8a354e0e/attachment.pgp 


More information about the xorg-devel mailing list