Quoting Jani Nikula (2019-08-16 10:35:08) > On Thu, 15 Aug 2019, Daniele Ceraolo Spurio <daniele.ceraolospurio@xxxxxxxxx> wrote: > > @@ -1622,7 +1560,7 @@ struct drm_i915_private { > > u32 suspend_count; > > bool power_domains_suspended; > > struct i915_suspend_saved_registers regfile; > > - struct vlv_s0ix_state vlv_s0ix_state; > > + void *s0ix_state; > > I'd keep the vlv_ prefix in the member name. And forward decl to avoid a naked void * -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx