Yeah, thought about that (checking name then -> try to parse it as PCI-ID) but didn't implement it because it won't work when there are multiple different PCI-ID bound to same 'name' (e.g. want to use a specific PCI-ID hsw). But wait a minite.... I think the opposite way (check if it's PCI-ID first) should cover that case....
I will upload v2 with this change shortly. On Wed, Mar 14, 2018 at 02:52:36PM -0700, Kenneth Graunke wrote: > On Monday, February 12, 2018 5:26:15 PM PDT Dongwon Kim wrote: > > Add a new option, '--pciid' to override a pci id of the target arch > > to support cross-architecture shader compilation. Not like "-p" option, > > it is for accepting any GFX devices supported by the driver. > > > > Setting both "-p" and "--pciid" is blocked to avoid conflict. > > > > Signed-off-by: Dongwon Kim <dongwon....@intel.com> > > --- > > run.c | 44 ++++++++++++++++++++++++++++++++++++++++++-- > > 1 file changed, 42 insertions(+), 2 deletions(-) > > Hi Dongwon, > > It looks like this does the exact same thing as -p, but it accepts > arbitrary numerical PCI IDs instead of a platform name. IMHO, I think > we should just make -p try parsing it as a number of it doesn't match > any of the names. Then either -p bdw or -p 0x1616 would work. > > It seems much less complicated that way. > > --Ken _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
