On Fri, 2016-08-05 at 18:49 +0100, Chris Wilson wrote: > On Fri, Aug 05, 2016 at 08:41:34PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä <ville.syrj...@linux.intel.com> > > > > intel_enable_pipe() looks rather confusing when one side doesn't have > > the curly braces, and the other one does. And what's even worse, > > there's another if-else inside the braceless side. Let's put braces > > around it to make it clear which branch goes where. > > > > Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com> > > Easiest review of the day. > Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> > -Chris > I was just about getting started to do my first review!
_______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
