Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756672AbbEVM0M (ORCPT ); Fri, 22 May 2015 08:26:12 -0400 Received: from mga01.intel.com ([192.55.52.88]:57896 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756267AbbEVM0H (ORCPT ); Fri, 22 May 2015 08:26:07 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,475,1427785200"; d="scan'208";a="575406804" Date: Fri, 22 May 2015 17:56:52 +0530 From: Vinod Koul To: Maninder Singh Cc: "dan.j.williams@intel.com" , "dmaengine@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [EDT][PATCH 1/1] dma/pl330.c : Initialize pl330 pointer after NULL check of pch pointer Message-ID: <20150522122652.GV3140@localhost> References: <1507989167.25151432264740747.JavaMail.weblogic@epmlwas06d> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1507989167.25151432264740747.JavaMail.weblogic@epmlwas06d> 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 Content-Length: 3003 Lines: 93 On Fri, May 22, 2015 at 03:19:00AM +0000, Maninder Singh wrote: > EP-AA9D1F29B02341529D96C06444D8471D > Hi Vinod > > >On Tue, May 19, 2015 at 05:00:27AM +0000, Maninder Singh wrote: > >> EP-AA9D1F29B02341529D96C06444D8471D > >Whats this, please sned patch ina proper format, git format-patch is your > >friend, use it > > > > This is auto generated no. from editor, so please ignore it, and i am resubmitting the patch, thanks for your feedback. > > >> > >> Hi, > >> Subject: [PATCH 1/1] dma/pl330.c : Initialize pl330 pointer after NULL check of pch pointer > >patch title doesn't match expectations, use the right subsystem name, and > >also please explain why yoy are doing this > > Ok according to comments resubmitting . Please check new patch if it is appropriate. because this patch is also generated by git, > But i think editor does some malfunctioning. Hint here is NOT to use editor to send patches, git send-email is your friend. Also do consider wrapping your lines appropriately Below looks much better, but the mailer has screwed it up and am unable to apply, so please resend with git send-mail. Thanks -- ~Vinod > > >> > >> Signed-off-by: Maninder Singh > >> Reviewed-By: Vaneet Narang > >> --- > >> drivers/dma/pl330.c | 4 +++- > ---cut--- > >> Thanks > >> Maninder Singh > >Nice signature! > > > >-- > >~Vinod > > From 76a6ef435065c10a0476038e8a08016878488dcb Mon Sep 17 00:00:00 2001 > From: Maninder Singh > Date: Mon, 18 May 2015 17:53:11 +0530 > Subject: [PATCH 1/1] dmaengine: pl330: Initialize pl330 for pl330_prep_dma_memcpy after NULL check of pch > > Currently pch pointer is already dereferenced before NULL check and thus we are getting below warning: > warn: variable dereferenced before check 'pch' > > So initialize struct pl330_dmac *pl330 after NULL check of dma_pl330_chan *pch. > > Signed-off-by: Maninder Singh > Reviewed-by: Vaneet Narang > --- > drivers/dma/pl330.c | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c > index a7d9d30..7e27144 100644 > --- a/drivers/dma/pl330.c > +++ b/drivers/dma/pl330.c > @@ -2581,12 +2581,14 @@ pl330_prep_dma_memcpy(struct dma_chan *chan, dma_addr_t dst, > { > struct dma_pl330_desc *desc; > struct dma_pl330_chan *pch = to_pchan(chan); > - struct pl330_dmac *pl330 = pch->dmac; > + struct pl330_dmac *pl330; > int burst; > > if (unlikely(!pch || !len)) > return NULL; > > + pl330 = pch->dmac; > + > desc = __pl330_prep_dma_memcpy(pch, dst, src, len); > if (!desc) > return NULL; > -- > 1.7.1 > > Thanks > Maninder > ............ -- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/