From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1031013AbbEEI6s (ORCPT ); Tue, 5 May 2015 04:58:48 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:27581 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756684AbbEEI6g (ORCPT ); Tue, 5 May 2015 04:58:36 -0400 Date: Tue, 5 May 2015 11:58:23 +0300 From: Dan Carpenter To: Leonardo Carreras Cc: Dan Williams , Vinod Koul , dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: Re: [PATCH] drivers: dma: amba-pl08x: Supress spaces in indentation Message-ID: <20150505085823.GR14154@mwanda> References: <20150505013227.GA11053@leo-Inspiron-N5110> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150505013227.GA11053@leo-Inspiron-N5110> User-Agent: Mutt/1.5.21 (2010-09-15) X-Source-IP: aserv0022.oracle.com [141.146.126.234] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, May 04, 2015 at 09:32:27PM -0400, Leonardo Carreras wrote: > Removed checkpatch reported spaces in indentation. > > Signed-off-by: Leonardo Carreras > --- > drivers/dma/amba-pl08x.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c > index 49d396e..a76df61 100644 > --- a/drivers/dma/amba-pl08x.c > +++ b/drivers/dma/amba-pl08x.c > @@ -474,7 +474,7 @@ static void pl08x_terminate_phy_chan(struct pl08x_driver_data *pl08x, > u32 val = readl(ch->reg_config); > > val &= ~(PL080_CONFIG_ENABLE | PL080_CONFIG_ERR_IRQ_MASK | > - PL080_CONFIG_TC_IRQ_MASK); > + PL080_CONFIG_TC_IRQ_MASK); Yeah, but now it doesn't line up correctly. Do it like this: val &= ~(PL080_CONFIG_ENABLE | PL080_CONFIG_ERR_IRQ_MASK | PL080_CONFIG_TC_IRQ_MASK); In other words: [tab][space]PL080_CONFIG_TC_IRQ_MASK); regards, dan carpenter