[PATCH 5/6] xfree86: fbdevhw: remove secondary way to find fb PCI device

Vignatti Tiago (Nokia-D/Helsinki) tiago.vignatti at nokia.com
Mon May 31 08:21:12 PDT 2010


On Mon, May 31, 2010 at 05:03:56PM +0200, ext Michel Dänzer wrote:
> 
> sysfs didn't appear in its current form instantly but in evolutionary
> steps. AFAIR the files above (which with your change are required for
> the matching to work) were added relatively recently, and it might be
> possible for libpciaccess to otherwise work fine with kernels which
> didn't have them. Now I may well be wrong about this, but it would be
> nice if you could check rather than asserting the code is 'superfluous'.

Ah, okay. Now I understood what you're saying. 

X server's commit 46f55f5d, dating Jun 2006, introduced the code to use the
interface /sys/bus/pci/devices/*/graphics:fb*. So I'd presume sysfs was
working since that time and that we're safe enough to remove this secondary  
way to match PCI devices (the actual changes introduced by my patch). 

Therefore, if no one loudly scream now and if it's fine for you I'll keep my
patch as it is (in the worst case should be pretty easy to create a hook for
this on Pci.c though). Okay? 

             Tiago


More information about the xorg-devel mailing list