On Wed, 14 Dec 2016, Daniel Vetter <daniel.vetter at ffwll.ch> 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

Reply via email to