On Wed, 14 Dec 2016, Daniel Vetter <daniel.vetter@xxxxxxxx> wrote: > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index 20bc04d5e617..7e25a7fffb1c 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -481,6 +481,9 @@ struct i915_hotplug { > &(dev)->mode_config.connector_list, \ > base.head) > > +#define for_each_intel_connector_iter(intel_connector, iter) \ > + while ((intel_connector = to_intel_connector(drm_connector_list_iter_next(iter)))) I occasionally worry about us relying on the base struct always being the first member. We don't even document this requirement anywhere. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx