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: dmaengine: qcom_hidma: simplify DT resource parsing From: Vinod Koul Message-Id: <20180108170224.GI18649@localhost> Date: Mon, 8 Jan 2018 22:32:24 +0530 To: Rob Herring Cc: linux-kernel@vger.kernel.org, Andy Gross , David Brown , Dan Williams , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, dmaengine@vger.kernel.org List-ID: T24gVGh1LCBKYW4gMDQsIDIwMTggYXQgMDQ6NDU6NDBQTSAtMDYwMCwgUm9iIEhlcnJpbmcgd3Jv dGU6Cj4gVGhlIGhpZG1hIGRyaXZlciBvcGVuIGNvZGVzIHBvcHVsYXRpbmcgYWRkcmVzcyBhbmQg SVJRIHJlc291cmNlcyBmcm9tIERULgo+IFdlIGhhdmUgc3RhbmRhcmQgZnVuY3Rpb25zIG9mX2Fk ZHJlc3NfdG9fcmVzb3VyY2UgYW5kIG9mX2lycV90b19yZXNvdXJjZQo+IGZvciB0aGlzLCBzbyB1 c2UgdGhlbSBpbnN0ZWFkLgo+IAo+IFRoZSBEVCBiaW5kaW5nIHN0YXRlcyBlYWNoIGNoaWxkIHNo b3VsZCBoYXZlIDIgYWRkcmVzc2VzIGFuZCAxIElSUSwgc28gd2UKPiBjYW4gc2ltcGxpZnkgdGhl IGxvZ2ljIGFuZCBkbyBhIGZpeGVkIHNpemUgcmVzb3VyY2UgYWxsb2NhdGlvbi4gVXNpbmcgdGhl Cj4gc3RhbmRhcmQgb2ZfYWRkcmVzc190b19yZXNvdXJjZSB3aWxsIGFsc28gZG8gYW55IGFkZHJl c3MgdHJhbnNsYXRpb24gd2hpY2gKPiB3YXMgbWlzc2luZy4KCkFwcGxpZWQsIHRoYW5rcwo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Subject: Re: [PATCH] dmaengine: qcom_hidma: simplify DT resource parsing Date: Mon, 8 Jan 2018 22:32:24 +0530 Message-ID: <20180108170224.GI18649@localhost> References: <20180104224542.15333-2-robh@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mga05.intel.com ([192.55.52.43]:38741 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935329AbeAHQ6P (ORCPT ); Mon, 8 Jan 2018 11:58:15 -0500 Content-Disposition: inline In-Reply-To: <20180104224542.15333-2-robh@kernel.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Rob Herring Cc: linux-kernel@vger.kernel.org, Andy Gross , David Brown , Dan Williams , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, dmaengine@vger.kernel.org On Thu, Jan 04, 2018 at 04:45:40PM -0600, Rob Herring wrote: > The hidma driver open codes populating address and IRQ resources from DT. > We have standard functions of_address_to_resource and of_irq_to_resource > for this, so use them instead. > > The DT binding states each child should have 2 addresses and 1 IRQ, so we > can simplify the logic and do a fixed size resource allocation. Using the > standard of_address_to_resource will also do any address translation which > was missing. Applied, thanks -- ~Vinod