From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755255AbbEEDbt (ORCPT ); Mon, 4 May 2015 23:31:49 -0400 Received: from mga11.intel.com ([192.55.52.93]:33775 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752428AbbEEDbi (ORCPT ); Mon, 4 May 2015 23:31:38 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,369,1427785200"; d="scan'208";a="705296154" Date: Tue, 5 May 2015 09:02:26 +0530 From: Vinod Koul To: Leonardo Carreras Cc: Dan Williams , 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: <20150505033226.GK3521@localhost> 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) 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. > You should look at git log to come up with patch tags for a subsystem, if you had done that you would have noticed that we use dmaengine:.... Also you should have run checkpatch and that would have told you a typo in changelog Lastly, you should mention which error/warn your are fixing in log. I have fixed first two and applied this -- ~Vinod > 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); > > writel(val, ch->reg_config); > > -- > 1.7.9.5 > --