[PATCH 1/2] xfree86: add pci id matching to OutputClass

[Dave Airlie]

On 18 February 2016 at 11:28, Aaron Plattner <aplattner at nvidia.com> wrote:
> On 02/08/2016 07:24 PM, Dave Airlie wrote:
>> This will be used by drivers to denote PCI ids in the future
>> instead of hardcoding them in the server.
>
> What benefit does this provide over MatchDriver?  Is this to just load X
> drivers for PCI devices that don't have corresponding DRM drivers?
>
> I ask because your suggested nouveau outputclass match from the cover
> letter will match an NVIDIA GPU that's being driven by the nvidia driver
> rather than the nouveau driver, if that config snippet is loaded first,
> because it doesn't require a match against the nouveau DRM driver too.
>
> The other problem, of course, is that it won't match Tegra devices being
> driven by nouveau since they don't have a pdev.  So in both cases, it
> seems like
>
>   MatchDriver "nouveau"
>
> would be more appropriate than
>
>   MatchPCIID "10de:"

Yes I guess for nouveau that match makes more sense.

The main idea is to remove all the matching from the X server
and have driver ship the files. Which means for drivers with no
kernel component we need to just match on the pci id list they ship.

Then I'd like to let distros not bind intel/ati/nouveau in favour of modeset
unless the user overrides things. So I'd like to only have the legacy DDX
match older hardware where glamor isn't possible.

So I'd probably end up wanting nouveau just matching pre-nv40 anyways,
in which case I'd specify a bunch of PCI IDs ripped from the driver.

or maybe install two match files per driver, one for older GPUs, one for
glamor possible ones, etc.

Dave.