[PATCH v2] xfree86: configure: remove vendor and card name matching rules
keithp at keithp.com
Tue Jun 22 11:02:47 PDT 2010
On Sat, 19 Jun 2010 13:05:22 +0300, Tiago Vignatti <tiago.vignatti at nokia.com> wrote:
> Although vendor and board naming are used to create the configure file, the
> server doesn't actually use it when fetching such file and probing devices.
> Reported-by: Richard Barnette <jrbarnette at chromium.org>
> Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
> from v1:
> - remove two more superfluous assignments (kudos Keith).
> Richard and James put the review stamp before, but I preferred to remove given
> there was two bugs spotted by Keith. Anyway, your review again would be much
Ok, so you're no longer setting the dev_vendor and dev_board values in
this path. Why do dev_vendor and dev_board continue to exist in this
structure then? And how about the configDevice path in xf86Config.c? Why
is that still setting this values?
I commented out the vendor and board values in the GDevRec, and then the
dev_vendor and dev_board values in XF86ConfDeviceRec and discovered that
nothing in the server uses them, other than to write out to a config
It seems like a better patch would remove all of this stuff, lest we end
up with code depending on values which are not getting set in some
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: not available
More information about the xorg-devel