From mboxrd@z Thu Jan 1 00:00:00 1970 From: Damien Lespiau Subject: Re: [v2] drm/i915: Add support for Generic MIPI panel driver Date: Tue, 27 May 2014 13:26:04 +0100 Message-ID: <20140527122604.GD18389@strange.amr.corp.intel.com> References: <1397454507-10273-5-git-send-email-shobhit.kumar@intel.com> <1400861127-11475-1-git-send-email-shobhit.kumar@intel.com> <20140527110243.GA18389@strange.amr.corp.intel.com> <53847553.2040703@intel.com> <20140527113941.GG14357@phenom.ffwll.local> <53847A2D.3030903@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id D0CBD6E61A for ; Tue, 27 May 2014 05:26:10 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: Jani Nikula , intel-gfx , Daniel Vetter List-Id: intel-gfx@lists.freedesktop.org On Tue, May 27, 2014 at 01:51:22PM +0200, Daniel Vetter wrote: > > Hmm, I will check my vim config and be more careful next time. Thanks for > > pointing out. Have you taken care of these or I will push another patch to > > fix the alignment issues ? > > Yeah, a quick fixup patch would be pretty. For the vim configuration I > highly recommend to use > > set cinoptions=:0,(0 " no indent for case and align function arguments to ( > > Then you can realign function parameter lists or multi-line if > condiditions with the = command, e.g. =a( FWIW, this is what I use: set cinoptions=:0,t0,(0,u0,w1,m1 -- Damien