On Wed, Mar 30, 2016 at 01:53:38PM +0100, Emil Velikov wrote: > On 30 March 2016 at 12:24, Daniel Vetter <[email protected]> wrote: > > It tries to do fancy things with excluding agp support if ttm is > > built-in, but agp isn't. Instead just express this depency like drm > > does and use CONFIG_AGP everywhere. > > > > Also use the neat Makefile magic to make the entire ttm_agp_backend > > file optional. > > > > v2: Use IS_ENABLED(CONFIG_AGP) as suggested by Ville > > > > v3: Review from Emil. > > > > v4: Actually get it right as spotted by 0-day. > > > Nice one. I didn't even spot the typo/thinko in v3 ;-) > > > Cc: Emil Velikov <[email protected]> > > Cc: Ville Syrjälä <[email protected]> > > Signed-off-by: Daniel Vetter <[email protected]> > Reviewed-by: Emil Velikov <[email protected]>
Applied to drm-misc, thanks for the review. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
