From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Subject: Re: [PATCH v5 11/16] mtd: nand: qcom: support for command descriptor formation Date: Sat, 19 Aug 2017 22:38:44 +0200 Message-ID: <20170819223844.6068e615@bbrezillon> References: <1502971674-13810-1-git-send-email-absahu@codeaurora.org> <1502971674-13810-11-git-send-email-absahu@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Return-path: Received: from mail.free-electrons.com ([62.4.15.54]:41288 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751586AbdHSUis (ORCPT ); Sat, 19 Aug 2017 16:38:48 -0400 In-Reply-To: Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Abhishek Sahu Cc: David Woodhouse , Brian Norris , Marek Vasut , Richard Weinberger , Cyrille Pitchen , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org, Andy Gross , Archit Taneja , Sricharan R , Vinod Koul , dmaengine@vger.kernel.org Le Sat, 19 Aug 2017 15:17:13 +0530, Abhishek Sahu a écrit : > On 2017-08-17 17:37, Abhishek Sahu wrote: > > 1. Add the function for command descriptor preparation which will > > be used only by BAM DMA and it will form the DMA descriptors > > containing command elements > > 2. DMA_PREP_CMD flag should be used for forming command DMA > > descriptors > > > > Reviewed-by: Archit Taneja > > Signed-off-by: Abhishek Sahu > > --- > > > > * Changes from v4: None > > > > * BUILD DEPENDENCY: > > > > This PATCH has build dependency over following BAM command descriptor > > patch posted in DMA engine mailing list > > > > http://www.spinics.net/lists/dmaengine/msg13665.html > > > > Hi Boris, > > These patch has build dependency over DMA engine patch > http://www.spinics.net/lists/dmaengine/msg13665.html > > but this patch has been applied in your github > https://github.com/bbrezillon/linux-0day/commits/nand/next > > The DMA change has not merged yet so could we drop this and > previous patch alone till the the DMA change is merged to prevent > build error. Oops. I saw this note when reviewing previous version of this and completely forgot when applying v5. I dropped it from my nand/next branch. ; Note that the linux-0day repo is just a personal repo I use to let Fenguang's robots test things before I push then to l2-mtd/nand/next. Anyway, thanks for the heads up. Boris