[Xquartz-changes] xserver: Changes to 'bus-cleanup-take4'

Jeremy Huddleston jeremyhu at freedesktop.org
Wed Sep 14 09:44:36 PDT 2011


New branch 'bus-cleanup-take4' available with the following commits:
commit d52a24e357526a75a4fa710a83f0ebc345e2aa51
Author: Tiago Vignatti <tiago.vignatti at nokia.com>
Date:   Tue Jun 1 17:43:04 2010 +0300

    xfree86: bus: remove unused macro checking
    
    DDX now can be used without PCI bus \o/
    
    Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
    Reviewed-by: Mikhail Gusarov <dottedmag at dottedmag.net>
    Reviewed-by: Jeremy Huddleston <jeremyhu at apple.com>
    Built-by: Jeremy Huddleston <jeremyhu at apple.com>

commit 1d7125f680c5ae0f30e3c445705cfcc5af5f47b4
Author: Tiago Vignatti <tiago.vignatti at nokia.com>
Date:   Tue Jun 1 13:05:47 2010 +0300

    configure: wrap PCI code with macro and set it at build time
    
    --disable-pciaccess, used together with --disable-module-int10, can be used to
    disable all pci code inside the server.
    
    Note that XSERVER_LIBPCIACCESS was previously used only in the driver side and
    now it defines also whether the library is used inside the server. Also,
    XORG_BUS_PCI automake variable is introduced to track PCI code needs.
    
    Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
    Reviewed-by: Mikhail Gusarov <dottedmag at dottedmag.net>
    Reviewed-by: Jeremy Huddleston <jeremyhu at apple.com>
    Built-by: Jeremy Huddleston <jeremyhu at apple.com>

commit fd769e10952f8082ec24bbe28f97e07881e16c82
Author: Tiago Vignatti <tiago.vignatti at nokia.com>
Date:   Mon May 31 19:27:07 2010 +0300

    configure: make PCI configuration more sane
    
    No semantical changes. Just moved code around, grouping PCI related stuff in a
    single chunk.
    
    Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
    Reviewed-by: Mikhail Gusarov <dottedmag at dottedmag.net>
    Reviewed-by: Jeremy Huddleston <jeremyhu at apple.com>
    Built-by: Jeremy Huddleston <jeremyhu at apple.com>

commit fe50bb4ee395646cd042f4cacb1429a59c3fb7af
Author: Tiago Vignatti <tiago.vignatti at nokia.com>
Date:   Mon May 31 19:13:42 2010 +0300

    configure: remove unused variable referencing PCI code
    
    commit f3490d3e missed to remove this chunk.
    
    Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
    Reviewed-by: Mikhail Gusarov <dottedmag at dottedmag.net>
    Reviewed-by: Jeremy Huddleston <jeremyhu at apple.com>
    Built-by: Jeremy Huddleston <jeremyhu at apple.com>

commit 1ecfc5a1aced01e232255ae82fe02e68edf6e2e8
Author: Tiago Vignatti <tiago.vignatti at nokia.com>
Date:   Mon May 31 18:50:50 2010 +0300

    configure: change PCI function checking by a meaningful version of the library
    
    Version 0.10.7 of libpciaccess contains all functions being checked by
    autoconf, so use it instead.
    
    People that don't want VGA arbiter active can go to the library and enable the
    stubs there.
    
    Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
    Reviewed-by: Mikhail Gusarov <dottedmag at dottedmag.net>
    Reviewed-by: Jeremy Huddleston <jeremyhu at apple.com>
    Built-by: Jeremy Huddleston <jeremyhu at apple.com>

commit a8b121391ebc3ebdaddfc5a0b3a252c49f54307f
Author: Tiago Vignatti <tiago.vignatti at nokia.com>
Date:   Tue Jun 1 16:49:08 2010 +0300

    xfree86: remove IOADDRESS definition from common header file
    
    IOADDRESS currently is used to store I/O addresses - actually the virtual
    address of I/O ports. In theory it should be used for any kind of bus that
    would access devices through this method. However, in practice we're abusing
    such definition in the implementation:
    
      1. we're narrowing it because it's defined inside os-support/bus/xf86Pci.h,
         i.e., for PCI buses only, and;
    
      2. we're extending its usage to common/ files, which conceptually shouldn't
      be aware of any hardware details.
    
    This patch try to contour 2., substituting all references of IOADDRESS inside
    common/ by unsigned long. Therefore IOADDRESS should be used only for PCI from
    now.
    
    One may want to improve this patch creating a similar type inside DDX common
    directory (common/) to deal with buses in general.
    
    Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>

commit 737b66f161e4f26a82c6807eb6d522b8cea20896
Author: Tiago Vignatti <tiago.vignatti at nokia.com>
Date:   Tue Jun 1 15:06:22 2010 +0300

    xfree86: fbdevhw: remove all pci symbols from the module
    
    xf86PciOpenFbDev was created and put inside its proper file.
    
    Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>

commit 0ab3a4a3eeeeed65a19f5db6839e5c981953c224
Author: Tiago Vignatti <tiago.vignatti at nokia.com>
Date:   Fri May 28 18:50:43 2010 +0300

    xfree86: fbdevhw: simplify fbdev_open_pci function
    
    delete unused namep.
    
    Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>

commit f50cd39f2d094bcfc1e14dba5774eb388253d12a
Author: Tiago Vignatti <tiago.vignatti at nokia.com>
Date:   Fri May 28 18:20:20 2010 +0300

    xfree86: fbdevhw: remove superfluous pci pointer from probe function
    
    The only drivers using this function are fbdev and glint. Though those driver
    initialize with NULL argument. Other drivers do some kind of implicit probe
    using fbdevHWInit instead.
    
    API break.
    
    Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>



More information about the Xquartz-changes mailing list