Adam Jackson <[email protected]> writes:

> On Wed, 2017-05-03 at 12:38 -0700, Eric Anholt wrote:
>> > Adam Jackson <[email protected]> writes:
>> > +#ifdef HAVE_LIBDRM
>> > +/* XXX kind of copypasta of drmCompareBusInfo */
>> > +static int
>> > +dri2_bus_info_equal(drmDevicePtr a, drmDevicePtr b)
>> > +{
>> > +    if (a == NULL || b == NULL)
>> > +  return 0;
>> > +
>> > +    if (a->bustype != b->bustype)
>> > +  return 0;
>> > +
>> > +    if (a->bustype == DRM_BUS_PCI)
>> > +  return !memcmp(a->businfo.pci, b->businfo.pci, sizeof(drmPciBusInfo));
>> > +
>> > +    return 0;
>> > +}
>> > +#endif
>> 
>> Yeah, this looks like a worse version of drmCompareBusInfo().  Would you
>> be willing to export it from libdrm, instead?  If not, please re-copy it
>> to get the usb/platform/host1x bits.
>
> Apologies, I copied this bit at a time when it only had the PCI case
> anyway. libdrm patch sent. I may need some guidance about which
> LIBDRM_REQUIRED in configure is the one to bump.

Thanks for doing that.  I think it would just be the top level
LIBDRM_REQUIRED.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to