Quoting Daniel Vetter (2019-05-02 08:27:27) > +static int drm_legacy_setup(struct drm_device * dev) > +{ > + int ret; > + > + if (dev->driver->firstopen && > + drm_core_check_feature(dev, DRIVER_LEGACY)) { > + ret = dev->driver->firstopen(dev); > + if (ret != 0) > + return ret; > + } Did airlied land his patch to move legacy to a separate compilation unit? If so, we could move this away and the compiler would have a field day simplifying drm_open(). -Chris _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel