On Tue, Feb 20, 2018 at 10:49:52AM +0100, Thierry Reding wrote: > On Mon, Feb 19, 2018 at 11:53:56PM +0100, Daniel Vetter wrote: > > Note: This is untested because the new stuff hasn't landed in upstream > > yet. But it should work. Ok Jani put the patch into topic/core-for-CI temporarily, so I could test. It works. > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx> > > --- > > include/drm/drm_vblank.h | 16 ++++++++++++++++ > > 1 file changed, 16 insertions(+) > > I couldn't find a reference to the new stuff, so maybe this was already > discussed, but wouldn't it be possible for the parser to figure out the > substruct by itself? That way the event. prefix could be avoided. https://lkml.org/lkml/2018/2/16/339 Jon said he pulled it all in already, I guess this way was simplest, the actual fix is a one-liner: https://lkml.org/lkml/2018/2/16/344 The reason for this is likely that you'll end up with name clashes everywhere if you don't insist on the fully qualified name. I merged the other patches meanwhile, thanks a lot for your review. -Daniel -- 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
