[PATCH 00/11] Clean up and reorganize BUS configuration code, take2

Alex Deucher alexdeucher at gmail.com
Fri May 21 09:03:25 PDT 2010


On Fri, May 21, 2010 at 7:43 AM, Tiago Vignatti
<tiago.vignatti at nokia.com> wrote:
> Hi,
>
> This set keeps with the cleaning up on the Xorg bus code. The most apparent
> benefit is that it makes xf86AutoConfig and xf86Helper independent from pci
> code, moving it to a more meaningful place (xf86pciBus.c).
>
> This patch set is a preparation of the rework of the bus entity code (RAC
> remnants) that I'll be planing to do some time soon. Will be a bit tought due
> it touches a lot on the driver side -- can we merge them in the xserver for
> 1.10, please? :)
>
>
> All patches should be very trivial to review, please do so:
>
>    git://anongit.freedesktop.org/~vignatti/xserver [bus-cleanup-take2]
>    http://cgit.freedesktop.org/~vignatti/xserver/log/?h=bus-cleanup-take2
>

Looks good to me.

Reviewed-by: Alex Deucher <alexdeucher at gmail.com>

for the series.

>
> Thanks,
>
>                 Tiago
>
> Tiago Vignatti (11):
>  xfree86: bus: remove unused pci macros
>  xfree86: bus: remove unused headers
>  xfree86: bus: move IOADDRESS definition to a common header file
>  xfree86: bus: remove superfluous and confused structures in BusRec
>  xfree86: bus: delete useless xf86FindPrimaryDevice
>  xfree86: bus: remove useless field from EntityRec
>  xfree86: remove all kind of bus and PCI dependency from the common helper file
>  xfree86: remove BUS_ISA type given we don't support anymore
>  xfree86: organize and group all pci related stuff inside xf86.h
>  xfree86: no need to check for the configuration case when matching devices
>  xfree86: move all pci code from auto configurator into a meaningful location
>
>  hw/xfree86/common/xf86.h            |   57 ++--
>  hw/xfree86/common/xf86AutoConfig.c  |  248 +-----------
>  hw/xfree86/common/xf86Bus.c         |   45 +--
>  hw/xfree86/common/xf86Bus.h         |    1 -
>  hw/xfree86/common/xf86Configure.c   |    2 -
>  hw/xfree86/common/xf86Helper.c      |  505 -----------------------
>  hw/xfree86/common/xf86Priv.h        |    4 -
>  hw/xfree86/common/xf86VGAarbiter.c  |    2 +-
>  hw/xfree86/common/xf86pciBus.c      |  762 ++++++++++++++++++++++++++++++++++-
>  hw/xfree86/common/xf86pciBus.h      |    1 +
>  hw/xfree86/common/xf86sbusBus.c     |    8 +-
>  hw/xfree86/common/xf86str.h         |   13 +-
>  hw/xfree86/os-support/bus/Pci.c     |    1 -
>  hw/xfree86/os-support/bus/Pci.h     |    8 -
>  hw/xfree86/os-support/bus/xf86Pci.h |    3 +-
>  hw/xfree86/vbe/vbe.c                |    1 -
>  16 files changed, 796 insertions(+), 865 deletions(-)
>
> _______________________________________________
> 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
>


More information about the xorg-devel mailing list