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

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to