On Thu, 2017-02-09 at 08:08 +0000, Chris Wilson wrote: > On Wed, Feb 08, 2017 at 10:38:07PM -0800, Dhinakaran Pandiyan wrote: > > +#define for_each_private_obj(__state, obj_funcs, obj, obj_state, __i, > > __funcs) \ > > + for ((__i) = 0; \ > > + (__i) < (__state)->num_private_objs && \ > > + ((obj) = (__state)->private_objs[__i].obj, \ > > + (__funcs) = (__state)->private_objs[__i].funcs, \ > > + (obj_state) = (__state)->private_objs[__i].obj_state, 1); \ > > Align to ( and put the trailing 1 on its own line so it stands out.
Sure, will do that. Looks like I have to change other macros in that file too. -DK > > (__i) < (__state)->num_private_objs && \ > ((obj) = (__state)->private_objs[__i].obj, \ > (__funcs) = (__state)->private_objs[__i].funcs, \ > (obj_state) = (__state)->private_objs[__i].obj_state, \ > 1); \ > (__i)++) \ > > + for_each_if (__funcs == obj_funcs) > > + > > +/** > > * drm_atomic_crtc_needs_modeset - compute combined modeset need > > * @state: &drm_crtc_state for the CRTC > > * > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
