On Tue, Apr 23, 2019 at 12:00:38PM +1000, Dave Airlie wrote: > From: Dave Airlie <airlied@xxxxxxxxxx> > > This could probably be done with Kconfig somehow, but I failed in my > first 2 minute attempt. > > Signed-off-by: Dave Airlie <airlied@xxxxxxxxxx> config DRM_ATI_PCIGART bool default y depens on PCI && DRM_LEGACY Should give you a nice hidden Kconfig symbol that dtrt and can be used in the Makefile. With that Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> Other option (no idea it's preferred): config DRM_ATI_PCIGART bool and then in config DRM_LEGACY select DRM_ATI_PCIGART if PCI Either should work I think, the latter more closely follows what we do already. -Daniel > --- > drivers/gpu/drm/ati_pcigart.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/ati_pcigart.c b/drivers/gpu/drm/ati_pcigart.c > index 2362f07fe1fc..0420a28f770a 100644 > --- a/drivers/gpu/drm/ati_pcigart.c > +++ b/drivers/gpu/drm/ati_pcigart.c > @@ -5,6 +5,7 @@ > * \author Gareth Hughes <gareth@xxxxxxxxxxx> > */ > > +#if IS_ENABLED(CONFIG_DRM_LEGACY) > /* > * Created: Wed Dec 13 21:52:19 2000 by gareth@xxxxxxxxxxx > * > @@ -203,3 +204,4 @@ int drm_ati_pcigart_init(struct drm_device *dev, struct drm_ati_pcigart_info *ga > return ret; > } > EXPORT_SYMBOL(drm_ati_pcigart_init); > +#endif > -- > 2.20.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
