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: [4/7] dmaengine: sprd: Add device validation to support multiple controllers From: Baolin Wang Message-Id: Date: Tue, 30 Apr 2019 13:30:45 +0800 To: Vinod Koul Cc: Dan Williams , eric.long@unisoc.com, Orson Zhai , Chunyan Zhang , Mark Brown , dmaengine@vger.kernel.org, LKML List-ID: T24gTW9uLCAyOSBBcHIgMjAxOSBhdCAyMjowNSwgVmlub2QgS291bCA8dmtvdWxAa2VybmVsLm9y Zz4gd3JvdGU6Cj4KPiBPbiAyOS0wNC0xOSwgMjA6MjAsIEJhb2xpbiBXYW5nIHdyb3RlOgo+ID4g T24gTW9uLCAyOSBBcHIgMjAxOSBhdCAxOTo1NywgVmlub2QgS291bCA8dmtvdWxAa2VybmVsLm9y Zz4gd3JvdGU6Cj4gPiA+Cj4gPiA+IE9uIDE1LTA0LTE5LCAyMDoxNCwgQmFvbGluIFdhbmcgd3Jv dGU6Cj4gPiA+ID4gRnJvbTogRXJpYyBMb25nIDxlcmljLmxvbmdAdW5pc29jLmNvbT4KPiA+ID4g Pgo+ID4gPiA+IFNpbmNlIHdlIGNhbiBzdXBwb3J0IG11bHRpcGxlIERNQSBlbmdpbmUgY29udHJv bGxlcnMsIHdlIHNob3VsZCBhZGQKPiA+ID4gPiBkZXZpY2UgdmFsaWRhdGlvbiBpbiBmaWx0ZXIg ZnVuY3Rpb24gdG8gY2hlY2sgaWYgdGhlIGNvcnJlY3QgY29udHJvbGxlcgo+ID4gPiA+IHRvIGJl IHJlcXVlc3RlZC4KPiA+ID4gPgo+ID4gPiA+IFNpZ25lZC1vZmYtYnk6IEVyaWMgTG9uZyA8ZXJp Yy5sb25nQHVuaXNvYy5jb20+Cj4gPiA+ID4gU2lnbmVkLW9mZi1ieTogQmFvbGluIFdhbmcgPGJh b2xpbi53YW5nQGxpbmFyby5vcmc+Cj4gPiA+ID4gLS0tCj4gPiA+ID4gIGRyaXZlcnMvZG1hL3Nw cmQtZG1hLmMgfCAgICA1ICsrKysrCj4gPiA+ID4gIDEgZmlsZSBjaGFuZ2VkLCA1IGluc2VydGlv bnMoKykKPiA+ID4gPgo+ID4gPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2RtYS9zcHJkLWRtYS5j IGIvZHJpdmVycy9kbWEvc3ByZC1kbWEuYwo+ID4gPiA+IGluZGV4IDBmOTJlNjAuLjlmOTlkNGIg MTAwNjQ0Cj4gPiA+ID4gLS0tIGEvZHJpdmVycy9kbWEvc3ByZC1kbWEuYwo+ID4gPiA+ICsrKyBi L2RyaXZlcnMvZG1hL3NwcmQtZG1hLmMKPiA+ID4gPiBAQCAtMTAyMCw4ICsxMDIwLDEzIEBAIHN0 YXRpYyB2b2lkIHNwcmRfZG1hX2ZyZWVfZGVzYyhzdHJ1Y3QgdmlydF9kbWFfZGVzYyAqdmQpCj4g PiA+ID4gIHN0YXRpYyBib29sIHNwcmRfZG1hX2ZpbHRlcl9mbihzdHJ1Y3QgZG1hX2NoYW4gKmNo YW4sIHZvaWQgKnBhcmFtKQo+ID4gPiA+ICB7Cj4gPiA+ID4gICAgICAgc3RydWN0IHNwcmRfZG1h X2NobiAqc2NoYW4gPSB0b19zcHJkX2RtYV9jaGFuKGNoYW4pOwo+ID4gPiA+ICsgICAgIHN0cnVj dCBvZl9waGFuZGxlX2FyZ3MgKmRtYV9zcGVjID0KPiA+ID4gPiArICAgICAgICAgICAgIGNvbnRh aW5lcl9vZihwYXJhbSwgc3RydWN0IG9mX3BoYW5kbGVfYXJncywgYXJnc1swXSk7Cj4gPiA+ID4g ICAgICAgdTMyIHNsYXZlX2lkID0gKih1MzIgKilwYXJhbTsKPiA+ID4gPgo+ID4gPiA+ICsgICAg IGlmIChjaGFuLT5kZXZpY2UtPmRldi0+b2Zfbm9kZSAhPSBkbWFfc3BlYy0+bnApCj4gPiA+Cj4g PiA+IEFyZSB5b3Ugbm90IHVzaW5nIG9mX2RtYV9maW5kX2NvbnRyb2xsZXIoKSB0aGF0IGRvZXMg dGhpcywgc28gdGhpcyB3b3VsZAo+ID4gPiBiZSB1c2VsZXNzIQo+ID4KPiA+IFllcywgd2UgY2Fu IHVzZSBvZl9kbWFfZmluZF9jb250cm9sbGVyKCksIGJ1dCB0aGF0IHdpbGwgYmUgYSBsaXR0bGUK PiA+IGNvbXBsaWNhdGVkIHRoYW4gY3VycmVudCBzb2x1dGlvbi4gU2luY2Ugd2UgbmVlZCBpbnRy b2R1Y2Ugb25lCj4gPiBzdHJ1Y3R1cmUgdG8gc2F2ZSB0aGUgbm9kZSB0byB2YWxpZGF0ZSBpbiB0 aGUgZmlsdGVyIGZ1bmN0aW9uIGxpa2UKPiA+IGJlbG93LCB3aGljaCBzZWVtcyBtYWtlIHRoaW5n cyBjb21wbGljYXRlZC4gQnV0IGlmIHlvdSBzdGlsbCBsaWtlIHRvCj4gPiB1c2Ugb2ZfZG1hX2Zp bmRfY29udHJvbGxlcigpLCBJIGNhbiBjaGFuZ2UgdG8gdXNlIGl0IGluIG5leHQgdmVyc2lvbi4K Pgo+IFNvcnJ5IEkgc2hvdWxkIGhhdmUgY2xhcmlmaWVkIG1vcmUuLgo+Cj4gb2ZfZG1hX2ZpbmRf Y29udHJvbGxlcigpIGlzIGNhbGxlZCBieSB4bGF0ZSwgc28geW91IGFscmVhZHkgcnVuIHRoaXMK PiBjaGVjaywgc28gd2h5IHVzZSB0aGlzIDopCgpUaGUgb2ZfZG1hX2ZpbmRfY29udHJvbGxlcigp IGNhbiBzYXZlIHRoZSByZXF1ZXN0ZWQgZGV2aWNlIG5vZGUgaW50bwpkbWFfc3BlYywgYW5kIGlu IHRoZSBvZl9kbWFfc2ltcGxlX3hsYXRlKCkgZnVuY3Rpb24sIGl0IHdpbGwgY2FsbApkbWFfcmVx dWVzdF9jaGFubmVsKCkgdG8gcmVxdWVzdCBvbmUgY2hhbm5lbCwgYnV0IGl0IGRpZCBub3QgdmFs aWRhdGUKdGhlIGRldmljZSBub2RlIHRvIGZpbmQgdGhlIGNvcnJlc3BvbmRpbmcgZG1hIGRldmlj ZSBpbgpkbWFfcmVxdWVzdF9jaGFubmVsKCkuIFNvIHdlIHNob3VsZCBpbiBvdXIgZmlsdGVyIGZ1 bmN0aW9uIHRvIHZhbGlkYXRlCnRoZSBkZXZpY2Ugbm9kZSB3aXRoIHRoZSBkZXZpY2Ugbm9kZSBz cGVjaWZpZWQgYnkgdGhlIGRtYV9zcGVjLiBIb3BlIEkKbWFrZSB0aGluZ3MgY2xlYXIuCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.1 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E4003C43219 for ; Tue, 30 Apr 2019 05:31:02 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id B252721670 for ; Tue, 30 Apr 2019 05:31:02 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="dkPagJ0e" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726309AbfD3Fa5 (ORCPT ); Tue, 30 Apr 2019 01:30:57 -0400 Received: from mail-ot1-f67.google.com ([209.85.210.67]:45762 "EHLO mail-ot1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726256AbfD3Fa5 (ORCPT ); Tue, 30 Apr 2019 01:30:57 -0400 Received: by mail-ot1-f67.google.com with SMTP id a10so2009911otl.12 for ; Mon, 29 Apr 2019 22:30:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=aRCavcxLPADLPpht8Z6on0LEjWfPk/EUgU06FZDzQBs=; b=dkPagJ0eXYSA35T2kaZcmUZXB2OILKj0nMSYyBST4s2xfakoz85Omi7wWR1V27rKqH F8ZjntDv2PEYDd5G71kInVhcH+Sasi8brcqjOSI1puC5I+xu4nE5of/YwxdxOHbY1cWO C2u2VcCEessMML8+c3A2YJ/mndkHPDai8p6c/w2DDpqipF/Ck73ivkZwOhKwG59PxMl1 /IeZZycjferMWsS5sXZ5Eofl3ab6/TjgMvCTg1IzSYuOvzxbbppBp1kS5QZgkZsMH6FK wIo8m4/2Ha8dlyIBzDDqjcx3xhGQYwO2ftpBY72oKexg3Oc6G0nlfGEb32cy33w7pSkr GKKQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=aRCavcxLPADLPpht8Z6on0LEjWfPk/EUgU06FZDzQBs=; b=SIPtu1mdmw+11Pnk/RSaomeXjOaPBaTlXQVAv1HDylwoJi4m0Ntr+2D8fAMH5BOhSL zAABsqz4kVc4F6XkiX8lqh4PAA+Lm2/b79a5wuKrI4sx+bbLe0jT6p2LD2RHGnbe5bfy KZ+gKdr8DT2z7buomPntOFp+X56H9oYPnNmO8h2jMbikxs7OXpkawxCbhbFoYVnM+9gB 4J0KgsIg4Cew31tl4HISe6/ROqrG1k+mzP+3BS3ioFAwy0Gt+q8J9o/AyDKpdYQuMFjj gNCmmHca5YrbIn0im2XVmvsR4rNBmHEffPJAm0JEVLGaA6tIEaLth5kXflP8aonEqebx JHaA== X-Gm-Message-State: APjAAAXGMdaD+n2Z1lZbNbqmJ7g8Wgdb8CsS1xcV8etmlJaAAFWx/NPY rjxz3eUsqJsXnmnkaZ4svVPOJJ8oFERaCsetkX2J1g== X-Google-Smtp-Source: APXvYqyBqdmUVb6VbsoihMRU+eubsmilp+PDOhv3PWxHkWGUYXheXD440opXlENKb9i33kPaQEYmTctFj4rsQ3tn0NY= X-Received: by 2002:a9d:7ad9:: with SMTP id m25mr41338446otn.75.1556602256776; Mon, 29 Apr 2019 22:30:56 -0700 (PDT) MIME-Version: 1.0 References: <20190429115723.GK3845@vkoul-mobl.Dlink> <20190429140537.GN3845@vkoul-mobl.Dlink> In-Reply-To: <20190429140537.GN3845@vkoul-mobl.Dlink> From: Baolin Wang Date: Tue, 30 Apr 2019 13:30:45 +0800 Message-ID: Subject: Re: [PATCH 4/7] dmaengine: sprd: Add device validation to support multiple controllers To: Vinod Koul Cc: Dan Williams , eric.long@unisoc.com, Orson Zhai , Chunyan Zhang , Mark Brown , dmaengine@vger.kernel.org, LKML Content-Type: text/plain; charset="UTF-8" Sender: dmaengine-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: dmaengine@vger.kernel.org Message-ID: <20190430053045.YgfM9XthBe0irvboomHSoY3SmHz6CiG7jqVyo-yt9oI@z> On Mon, 29 Apr 2019 at 22:05, Vinod Koul wrote: > > On 29-04-19, 20:20, Baolin Wang wrote: > > On Mon, 29 Apr 2019 at 19:57, Vinod Koul wrote: > > > > > > On 15-04-19, 20:14, Baolin Wang wrote: > > > > From: Eric Long > > > > > > > > Since we can support multiple DMA engine controllers, we should add > > > > device validation in filter function to check if the correct controller > > > > to be requested. > > > > > > > > Signed-off-by: Eric Long > > > > Signed-off-by: Baolin Wang > > > > --- > > > > drivers/dma/sprd-dma.c | 5 +++++ > > > > 1 file changed, 5 insertions(+) > > > > > > > > diff --git a/drivers/dma/sprd-dma.c b/drivers/dma/sprd-dma.c > > > > index 0f92e60..9f99d4b 100644 > > > > --- a/drivers/dma/sprd-dma.c > > > > +++ b/drivers/dma/sprd-dma.c > > > > @@ -1020,8 +1020,13 @@ static void sprd_dma_free_desc(struct virt_dma_desc *vd) > > > > static bool sprd_dma_filter_fn(struct dma_chan *chan, void *param) > > > > { > > > > struct sprd_dma_chn *schan = to_sprd_dma_chan(chan); > > > > + struct of_phandle_args *dma_spec = > > > > + container_of(param, struct of_phandle_args, args[0]); > > > > u32 slave_id = *(u32 *)param; > > > > > > > > + if (chan->device->dev->of_node != dma_spec->np) > > > > > > Are you not using of_dma_find_controller() that does this, so this would > > > be useless! > > > > Yes, we can use of_dma_find_controller(), but that will be a little > > complicated than current solution. Since we need introduce one > > structure to save the node to validate in the filter function like > > below, which seems make things complicated. But if you still like to > > use of_dma_find_controller(), I can change to use it in next version. > > Sorry I should have clarified more.. > > of_dma_find_controller() is called by xlate, so you already run this > check, so why use this :) The of_dma_find_controller() can save the requested device node into dma_spec, and in the of_dma_simple_xlate() function, it will call dma_request_channel() to request one channel, but it did not validate the device node to find the corresponding dma device in dma_request_channel(). So we should in our filter function to validate the device node with the device node specified by the dma_spec. Hope I make things clear. -- Baolin Wang Best Regards