From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [v5,1/6] dmaengine: fsldma: Replace DMA_IN/OUT by FSL_DMA_IN/OUT From: Vinod Koul Message-Id: <20180529102121.GK5666@vkoul-mobl> Date: Tue, 29 May 2018 15:51:21 +0530 To: Wen He Cc: "dmaengine@vger.kernel.org" , "robh+dt@kernel.org" , "devicetree@vger.kernel.org" , Leo Li , Jiafei Pan , Jiaheng Fan List-ID: T24gMjktMDUtMTgsIDEwOjE0LCBXZW4gSGUgd3JvdGU6Cj4gPiAKPiA+IFdlbGwgdGhhdCBwYXRj aCBzZWVtcyB0byBkbyBhIGJpdCBtb3JlIHRoYW4gcmVwbGFjZSBETUFfSU4vT1VUIGJ5Cj4gPiBG U0xfRE1BX0lOL09VVCBjYXJlIHRvIGV4cGxhaW4gdGhvc2UuIGJlc3Qgd291bGQgYmUgdG8gZG8g YSBwYXRjaCB3aGljaAo+ID4gZG9lcyByZXBsYWNlIGFuZCB0aGVuIGFkZCBuZXcgdGhpbmdzIGlu IHNlcGFyYXRlIHBhdGNoLCBlYXNpZXIgdG8gcmV2aWV3Cj4gPiAKPiAKPiBXaGF0IGRvZXMgbWVh bnM/Cj4gVGhpcyBhbHNvIGEgcGF0Y2guCgpZb3UgY25hIGRvIG9ubHkgcmVwbGFjZSBETUFfSU4v T1VUIGJ5IEZTTF9ETUFfSU4vT1VUIGluIHRoaXMgcGF0Y2guIFNlY29uZCBwYXRjaApjYW4gYWRk IGFkZGl0aW9uYWwgbWFjcm9zIHlvdSBoYXZlIGFkZGVkIGFuZCBleHBsYWluIHdoYXQgdGhleSBk byBhbmQgd2h5IHRoZXkKYXJlIG5lZWRlZAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 29 May 2018 15:51:21 +0530 From: Vinod Subject: Re: [v5 1/6] dmaengine: fsldma: Replace DMA_IN/OUT by FSL_DMA_IN/OUT Message-ID: <20180529102121.GK5666@vkoul-mobl> References: <20180525111920.24498-1-wen.he_1@nxp.com> <20180529044925.GD5666@vkoul-mobl> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: To: Wen He Cc: "dmaengine@vger.kernel.org" , "robh+dt@kernel.org" , "devicetree@vger.kernel.org" , Leo Li , Jiafei Pan , Jiaheng Fan List-ID: On 29-05-18, 10:14, Wen He wrote: > > > > Well that patch seems to do a bit more than replace DMA_IN/OUT by > > FSL_DMA_IN/OUT care to explain those. best would be to do a patch which > > does replace and then add new things in separate patch, easier to review > > > > What does means? > This also a patch. You cna do only replace DMA_IN/OUT by FSL_DMA_IN/OUT in this patch. Second patch can add additional macros you have added and explain what they do and why they are needed -- ~Vinod