[PATCH v2 0/5] Support autoloading of non-PCI drivers

Aaron Plattner aplattner at nvidia.com
Wed Feb 19 13:27:35 PST 2014


On 02/18/2014 02:19 PM, Thierry Reding wrote:
> From: Thierry Reding <treding at nvidia.com>
>
> This series introduces a new configuration section, OutputClass, which
> can be used to match an output device against a set of attributes and
> pass in additional configuration. The idea is that each DDX module for
> output drivers can ship configuration snippets that give a hint as to
> what devices they support.
>
> Patch 1 modifies xf86MatchDriverFromFiles() to behave more like the
> other driver enumeration functions. This makes the code in subsequent
> patches simpler.
>
> Patch 2 is preparatory work to store the name of a device's kernel
> driver for later use.
>
> Patch 3 adds the new OutputClass section that for now only supports
> matching by kernel driver name and providing the name of the DDX module
> to load for a matching device.
>
> Patch 4 in this series makes use of these configuration snippets to
> auto-load drivers for a particular device.
>
> Patch 5 makes an error message more readable and consistent with code
> introduced in an earlier patch.
>
> Changes in v2:
> - address review comments from Aaron Plattner:
>    - drivers matched by OutputClass take precedence over other drivers
>    - fix typos in manpage, start sentences on new lines
>    - make error messages more readable
>
> Thierry
>
> Thierry Reding (5):
>    xfree86: Make driver matching consistent
>    xfree86: Store kernel driver name in platform device attribute
>    xfree86: Introduce OutputClass configuration
>    xfree86: Support driver loading via OutputClass
>    xfree86: Make error message more readable
>
>   hw/xfree86/common/xf86AutoConfig.c         |   2 +-
>   hw/xfree86/common/xf86pciBus.c             |  24 ++---
>   hw/xfree86/common/xf86pciBus.h             |   5 +-
>   hw/xfree86/common/xf86platformBus.c        |  85 ++++++++++++++-
>   hw/xfree86/man/xorg.conf.man               |  77 +++++++++++++
>   hw/xfree86/os-support/linux/lnx_platform.c |  17 ++-
>   hw/xfree86/parser/Makefile.am              |   1 +
>   hw/xfree86/parser/OutputClass.c            | 167 +++++++++++++++++++++++++++++
>   hw/xfree86/parser/configProcs.h            |   5 +
>   hw/xfree86/parser/read.c                   |   6 ++
>   hw/xfree86/parser/write.c                  |   2 +
>   hw/xfree86/parser/xf86Parser.h             |   9 ++
>   include/hotplug.h                          |   2 +
>   13 files changed, 378 insertions(+), 24 deletions(-)
>   create mode 100644 hw/xfree86/parser/OutputClass.c
>

For the series,

Reviewed-By: Aaron Plattner <aplattner at nvidia.com>
Tested-By: Aaron Plattner <aplattner at nvidia.com>

-- 
Aaron


More information about the xorg-devel mailing list