On Thu, Dec 27, 2018 at 02:56:40PM +0200, Jani Nikula wrote: > Make it easier to drop drmP.h includes. Switch from "" to <> includes > while at it. > > Cc: Sam Ravnborg <s...@ravnborg.org> > Cc: Daniel Vetter <dan...@ffwll.ch> > Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > Signed-off-by: Jani Nikula <jani.nik...@intel.com> > --- > include/drm/drm_syncobj.h | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/include/drm/drm_syncobj.h b/include/drm/drm_syncobj.h > index 7c6ed845c70d..93884da3f9fe 100644 > --- a/include/drm/drm_syncobj.h > +++ b/include/drm/drm_syncobj.h > @@ -26,7 +26,9 @@ > #ifndef __DRM_SYNCOBJ_H__ > #define __DRM_SYNCOBJ_H__ > > -#include "linux/dma-fence.h" > +#include <linux/dma-fence.h> > + > +#include <drm/drm_file.h>
I think you need only a struct drm_file; pre-decl instead of the full include. With that: Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> > > /** > * struct drm_syncobj - sync object. > -- > 2.11.0 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
