Re: [PATCH v1 1/1] drm: drop unused drm_display_mode.private

Hi Daniel.

...

I also checked private_flags - it is used in a few modules. And it looked legit.

Iirc i915 used this, before we went full overdrive with entire atomic state structure subclassing :-)

$ git grep -l private_flags gma500/psb_intel_drv.h i915/display/intel_display.c i915/display/intel_display_types.h i915/display/intel_tv.c i915/display/vlv_dsi.c i915/i915_irq.c msm/disp/dpu1/dpu_encoder.c msm/disp/dpu1/dpu_trace.h <= false hit

i915 still has a few uses of private_flags. Likewise msm and gma500

Looks doable to address this, but not on my TODO list.

Sam

Reviewed-by: Daniel Vetter daniel.vetter@ffwll.ch

Thanks - applied.

Sam