On Wed, 23 Nov 2016, Chris Wilson <chris at chris-wilson.co.uk> wrote: > +#define drm_fb_helper_for_each_connector(fbh, i__) \ > + for (({lockdep_assert_held(&(fbh)->dev->mode_config.mutex); 1;}), \ > + i__ = 0; i__ < (fbh)->connector_count; i__++)
No comments on the substance, but just curious, why is that "1;" required there? Or is it? BR, Jani. -- Jani Nikula, Intel Open Source Technology Center