[PATCH 12/14] xfree86: IOADDRESS is PCIism and shouldn't be used by common header files

Vignatti Tiago (Nokia-D/Helsinki) tiago.vignatti at nokia.com
Tue Jun 1 11:37:27 PDT 2010


On Tue, Jun 01, 2010 at 05:50:17PM +0200, ext Mark Kettenis wrote:
> > From: Tiago Vignatti <tiago.vignatti at nokia.com>
> > Date: Tue,  1 Jun 2010 17:59:08 +0300
> 
> This change does not make it easier to understand the code.  Imagine
> someone looking at it after you apply this diff.  What is the meaning
> of 'vertsyncreg'?  
> 
> Really, if the type is a PCIism, so are the variables that are
> declared using that type.  That may be undesirable, but at least the
> use of IOADDRESS as a type makes that clear.

If I move the IOADDRESS definition to xf86str.h will help then?

             Tiago


More information about the xorg-devel mailing list