Re: [PATCH 4/6] drm/i915: Dynamically allocate s0ix struct for VLV

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux