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: [RFC,v3,5/7] dmaengine: Add Synopsys eDMA IP PCIe glue-logic From: Andy Shevchenko Message-Id: <20190111194705.GU9170@smile.fi.intel.com> Date: Fri, 11 Jan 2019 21:47:05 +0200 To: Gustavo Pimentel Cc: linux-pci@vger.kernel.org, dmaengine@vger.kernel.org, Vinod Koul , Dan Williams , Eugeniy Paltsev , Russell King , Niklas Cassel , Lorenzo Pieralisi , Joao Pinto , Jose Abreu , Luis Oliveira , Vitor Soares , Nelson Costa , Pedro Sousa List-ID: T24gRnJpLCBKYW4gMTEsIDIwMTkgYXQgMDc6MzM6NDFQTSArMDEwMCwgR3VzdGF2byBQaW1lbnRl bCB3cm90ZToKPiBTeW5vcHN5cyBlRE1BIElQIGlzIG5vcm1hbGx5IGRpc3RyaWJ1dGVkIGFsb25n IHdpdGggU3lub3BzeXMgUENJZQo+IEVuZFBvaW50IElQIChkZXBlbmRzIG9mIHRoZSB1c2UgYW5k IGxpY2Vuc2luZyBhZ3JlZW1lbnQpLgo+IAo+IFRoaXMgSVAgcmVxdWlyZXMgc29tZSBiYXNpYyBj b25maWd1cmF0aW9ucywgc3VjaCBhczoKPiAgLSBlRE1BIHJlZ2lzdGVycyBCQVIKPiAgLSBlRE1B IHJlZ2lzdGVycyBvZmZzZXQKPiAgLSBlRE1BIHJlZ2lzdGVycyBzaXplCj4gIC0gZURNQSBsaW5r ZWQgbGlzdCBtZW1vcnkgQkFSCj4gIC0gZURNQSBsaW5rZWQgbGlzdCBtZW1vcnkgb2Zmc2V0Cj4g IC0gZURNQSBsaW5rZWQgbGlzdCBtZW1vcnkgc3plCj4gIC0gZURNQSBkYXRhIG1lbW9yeSBCQVIK PiAgLSBlRE1BIGRhdGEgbWVtb3J5IG9mZnNldAo+ICAtIGVETUEgZGF0YSBtZW1vcnkgc2l6ZQo+ ICAtIGVETUEgdmVyc2lvbgo+ICAtIGVETUEgbW9kZQo+ICAtIElSUXMgYXZhaWxhYmxlIGZvciBl RE1BCj4gCj4gQXMgYSB3b3JraW5nIGV4YW1wbGUsIFBDSWUgZ2x1ZS1sb2dpYyB3aWxsIGF0dGFj aCB0byBhIFN5bm9wc3lzIFBDSWUKPiBFbmRQb2ludCBJUCBwcm90b3R5cGUga2l0IChWZW5kb3Ig SUQgPSAweDE2YzMsIERldmljZSBJRCA9IDB4ZWRkYSksCj4gd2hpY2ggaGFzIGJ1aWx0LWluIGFu IGVETUEgSVAgd2l0aCB0aGlzIGRlZmF1bHQgY29uZmlndXJhdGlvbjoKPiAgLSBlRE1BIHJlZ2lz dGVycyBCQVIgPSAwCj4gIC0gZURNQSByZWdpc3RlcnMgb2Zmc2V0ID0gMHgwMDAwMTAwMCAoNCBL Ynl0ZXMpCj4gIC0gZURNQSByZWdpc3RlcnMgc2l6ZSA9IDB4MDAwMDIwMDAgKDggS2J5dGVzKQo+ ICAtIGVETUEgbGlua2VkIGxpc3QgbWVtb3J5IEJBUiA9IDIKPiAgLSBlRE1BIGxpbmtlZCBsaXN0 IG1lbW9yeSBvZmZzZXQgPSAweDAwMDAwMDAwICgwIEtieXRlcykKPiAgLSBlRE1BIGxpbmtlZCBs aXN0IG1lbW9yeSBzaXplID0gMHgwMDgwMDAwMCAoOCBNYnl0ZXMpCj4gIC0gZURNQSBkYXRhIG1l bW9yeSBCQVIgPSAyCj4gIC0gZURNQSBkYXRhIG1lbW9yeSBvZmZzZXQgPSAweDAwODAwMDAwICg4 IE1ieXRlcykKPiAgLSBlRE1BIGRhdGEgbWVtb3J5IHNpemUgPSAweDAzODAwMDAwICg1NiBNYnl0 ZXMpCj4gIC0gZURNQSB2ZXJzaW9uID0gMAo+ICAtIGVETUEgbW9kZSA9IEVETUFfTU9ERV9VTlJP TEwKPiAgLSBJUlFzID0gMQo+IAo+IFRoaXMgZHJpdmVyIGNhbiBiZSBjb21waWxlIGFzIGJ1aWx0 LWluIG9yIGV4dGVybmFsIG1vZHVsZSBpbiBrZXJuZWwuCj4gCj4gVG8gZW5hYmxlIHRoaXMgZHJp dmVyIGp1c3Qgc2VsZWN0IERXX0VETUFfUENJRSBvcHRpb24gaW4ga2VybmVsCj4gY29uZmlndXJh dGlvbiwgaG93ZXZlciBpdCByZXF1aXJlcyBhbmQgc2VsZWN0cyBhdXRvbWF0aWNhbGx5IERXX0VE TUEKPiBvcHRpb24gdG9vLgo+IAoKPiBDaGFuZ2VzOgo+IFJGQyB2MS0+UkZDIHYyOgoKQ2hhbmdl cyBnbyBhZnRlciAnLS0tICcgbGluZS4KCj4gIC0gUmVwbGFjZSBjb21tZW50cyAvLyAoQzk5IHN0 eWxlKSBieSAvKiovCj4gIC0gTWVyZ2UgdHdvIHBjaW1faW9tYXBfcmVnaW9ucygpIGNhbGxzIGlu dG8ganVzdCBvbmUgY2FsbAo+ICAtIFJlbW92ZSBwY2lfdHJ5X3NldF9td2koKSBjYWxsCj4gIC0g UmVwbGFjZSBzb21lIGRldl9pbmZvKCkgYnkgZGV2X2RiZygpIHRvIHJlZHVjZSAqbm9pc2UqCj4g IC0gUmVtb3ZlIHBjaV9uYW1lKHBkZXYpIGNhbGwgYWZ0ZXIgYmVpbmcgY2FsbCBkd19lZG1hX3Jl bW92ZSgpCj4gIC0gUmVtb3ZlIGFsbCBwb3dlciBtYW5hZ2VtZW50IHN1cHBvcnQKPiAgLSBGaXgg dGhlIGhlYWRlcnMgb2YgdGhlIC5jIGFuZCAuaCBmaWxlcyBhY2NvcmRpbmcgdG8gdGhlIG1vc3Qg cmVjZW50Cj4gICAgY29udmVudGlvbgo+ICAtIEZpeCBlcnJvcnMgYW5kIGNoZWNrcyBwb2ludGVk IG91dCBieSBjaGVja3BhdGNoIHdpdGggLS1zdHJpY3Qgb3B0aW9uCj4gIC0gUmVwbGFjZSBwYXRj aCBzbWFsbCBkZXNjcmlwdGlvbiB0YWcgZnJvbSBkbWEgYnkgZG1hZW5naW5lCj4gUkZDIHYyLT5S RkMgdjM6Cj4gIC0gRml4IHByaW50ayB2YXJpYWJsZSBvZiBwaHlzX2FkZHJfdCB0eXBlCj4gIC0g Rml4IG1pc3NpbmcgdmFyaWFibGUgaW5pdGlhbGl6YXRpb24gKGNoYW4tPmNvbmZpZ3VyZWQpCj4g IC0gQ2hhbmdlIGxpbmtlZCBsaXN0IHNpemUgdG8gNTEyIEtieXRlcwo+ICAtIEFkZCBkYXRhIG1l bW9yeSBpbmZvcm1hdGlvbgo+ICAtIEFkZCByZWdpc3RlciBzaXplIGluZm9ybWF0aW9uCj4gIC0g QWRkIGNvbW1lbnRzIG9yIGltcHJvdmUgZXhpc3Rpbmcgb25lcwo+ICAtIEFkZCBwb3NzaWJpbGl0 eSB0byB3b3JrIHdpdGggbXVsdGlwbGUgSVJRcyBmZWF0dXJlCj4gIC0gUmVwbGFjZSBNU0kgYW5k IE1TSS1YIGVuYWJsZSBjb25kaXRpb24gYnkgcGNpX2Rldl9tc2lfZW5hYmxlZCgpCj4gIC0gUmVw bGFjZSBjb2RlIHRvIGFjcXVpcmUgTVNJKC1YKSBhZGRyZXNzIGFuZCBkYXRhIGJ5Cj4gICAgZ2V0 X2NhY2hlZF9tc2lfbXNnKCkKCj4gK2VudW0gZHdfZWRtYV9wY2llX2JhciB7Cj4gKwlCQVJfMCwK PiArCUJBUl8xLAo+ICsJQkFSXzIsCj4gKwlCQVJfMywKPiArCUJBUl80LAo+ICsJQkFSXzUKPiAr fTsKCnBjaS1lcGYuaCBoYXMgdGhpcy4KV2h5IGR1cGxpY2F0ZT8KCgpXaGF0IGVsc2UgaXMgYmVp bmcgZHVwbGljYXRlZCBmcm9tIFBDSSBjb3JlPwoKPiArc3RhdGljIGJvb2wgZGlzYWJsZV9tc2l4 Owo+ICttb2R1bGVfcGFyYW0oZGlzYWJsZV9tc2l4LCBib29sLCAwNjQ0KTsKPiArTU9EVUxFX1BB Uk1fREVTQyhkaXNhYmxlX21zaXgsICJEaXNhYmxlIE1TSS1YIGludGVycnVwdHMiKTsKCldoeT8h CldlIGFyZSBubyBhbGxvdyBuZXcgbW9kdWxlIHBhcmFtZXRlcnMgd2l0aG91dCB2ZXJ5IHN0cm9u ZyBhcmd1bWVudHMuCgo+ICsKPiArc3RhdGljIGludCBkd19lZG1hX3BjaWVfcHJvYmUoc3RydWN0 IHBjaV9kZXYgKnBkZXYsCj4gKwkJCSAgICAgIGNvbnN0IHN0cnVjdCBwY2lfZGV2aWNlX2lkICpw aWQpCj4gK3sKPiArCWNvbnN0IHN0cnVjdCBkd19lZG1hX3BjaWVfZGF0YSAqcGRhdGEgPSAodm9p ZCAqKXBpZC0+ZHJpdmVyX2RhdGE7Cj4gKwlzdHJ1Y3QgZGV2aWNlICpkZXYgPSAmcGRldi0+ZGV2 Owo+ICsJc3RydWN0IGR3X2VkbWFfY2hpcCAqY2hpcDsKPiArCXN0cnVjdCBkd19lZG1hICpkdzsK PiArCXVuc2lnbmVkIGludCBpcnFfZmxhZ3MgPSBQQ0lfSVJRX01TSTsKPiArCWludCBlcnIsIG5y X2lycXMsIGk7Cj4gKwoKPiArCWlmICghcGRhdGEpIHsKPiArCQlkZXZfZXJyKGRldiwgIiVzIG1p c3NpbmcgZGF0YSBzdHJ1Y3R1cmVcbiIsIHBjaV9uYW1lKHBkZXYpKTsKPiArCQlyZXR1cm4gLUVG QVVMVDsKPiArCX0KClVzZWxlc3MgY2hlY2suCgo+ICsKPiArCS8qIEVuYWJsZSBQQ0kgZGV2aWNl ICovCj4gKwllcnIgPSBwY2ltX2VuYWJsZV9kZXZpY2UocGRldik7Cj4gKwlpZiAoZXJyKSB7Cj4g KwkJZGV2X2VycihkZXYsICIlcyBlbmFibGluZyBkZXZpY2UgZmFpbGVkXG4iLCBwY2lfbmFtZShw ZGV2KSk7Cj4gKwkJcmV0dXJuIGVycjsKPiArCX0KPiArCj4gKwkvKiBNYXBwaW5nIFBDSSBCQVIg cmVnaW9ucyAqLwo+ICsJZXJyID0gcGNpbV9pb21hcF9yZWdpb25zKHBkZXYsIEJJVChwZGF0YS0+ cmdfYmFyKSB8Cj4gKwkJCQkgICAgICAgQklUKHBkYXRhLT5sbF9iYXIpIHwKPiArCQkJCSAgICAg ICBCSVQocGRhdGEtPmR0X2JhciksCj4gKwkJCQkgcGNpX25hbWUocGRldikpOwo+ICsJaWYgKGVy cikgewoKPiArCQlkZXZfZXJyKGRldiwgIiVzIGVETUEgQkFSIEkvTyByZW1hcHBpbmcgZmFpbGVk XG4iLAo+ICsJCQlwY2lfbmFtZShwZGV2KSk7CgpJc24ndCBpdCBwY2lfZXJyKCkgPwpTYW1lIGNv bW1lbnQgZm9yIHRoZSByZXN0IHNpbWlsYXIgY2FzZXMgYWJvdmUgYW5kIGJlbG93LgoKPiArCQly ZXR1cm4gZXJyOwo+ICsJfQo+ICsKPiArCXBjaV9zZXRfbWFzdGVyKHBkZXYpOwo+ICsKPiArCW5y X2lycXMgPSBwY2lfYWxsb2NfaXJxX3ZlY3RvcnMocGRldiwgMSwgcGRhdGEtPmlycXNfY250LCBp cnFfZmxhZ3MpOwo+ICsJaWYgKG5yX2lycXMgPCAxKSB7Cj4gKwkJZGV2X2VycihkZXYsICIlcyBm YWlsZWQgdG8gYWxsb2MgSVJRIHZlY3RvciAoTnVtYmVyIG9mIElSUXM9JXUpXG4iLAo+ICsJCQlw Y2lfbmFtZShwZGV2KSwgbnJfaXJxcyk7Cj4gKwkJcmV0dXJuIC1FUEVSTTsKPiArCX0KPiArCj4g KwkvKiBEYXRhIHN0cnVjdHVyZSBpbml0aWFsaXphdGlvbiAqLwo+ICsJY2hpcC0+ZHcgPSBkdzsK PiArCWNoaXAtPmRldiA9IGRldjsKPiArCWNoaXAtPmlkID0gcGRldi0+ZGV2Zm47Cj4gKwljaGlw LT5pcnEgPSBwZGV2LT5pcnE7Cj4gKwoKPiArCWlmICghcGNpbV9pb21hcF90YWJsZShwZGV2KSkK PiArCQlyZXR1cm4gLUVBQ0NFUzsKCk5ldmVyIGhhcHBlbiBjb25kaXRpb24uIFRodXMgdXNlbGVz cy4KCj4gKwlkZXZfaW5mbyhkZXYsICJEZXNpZ25XYXJlIGVETUEgUENJZSBkcml2ZXIgbG9hZGVk IGNvbXBsZXRlbHlcbiIpOwoKVXNlbGVzcy4KCj4gK30KPiArCj4gK3N0YXRpYyB2b2lkIGR3X2Vk bWFfcGNpZV9yZW1vdmUoc3RydWN0IHBjaV9kZXYgKnBkZXYpCj4gK3sKPiArCXN0cnVjdCBkd19l ZG1hX2NoaXAgKmNoaXAgPSBwY2lfZ2V0X2RydmRhdGEocGRldik7Cj4gKwlzdHJ1Y3QgZGV2aWNl ICpkZXYgPSAmcGRldi0+ZGV2Owo+ICsJaW50IGVycjsKPiArCj4gKwkvKiBTdG9wcGluZyBlRE1B IGRyaXZlciAqLwo+ICsJZXJyID0gZHdfZWRtYV9yZW1vdmUoY2hpcCk7Cj4gKwlpZiAoZXJyKQo+ ICsJCWRldl93YXJuKGRldiwgImNhbid0IHJlbW92ZSBkZXZpY2UgcHJvcGVybHk6ICVkXG4iLCBl cnIpOwo+ICsKPiArCS8qIEZyZWVpbmcgSVJRcyAqLwo+ICsJcGNpX2ZyZWVfaXJxX3ZlY3RvcnMo cGRldik7Cj4gKwo+ICsJZGV2X2luZm8oZGV2LCAiRGVzaWduV2FyZSBlRE1BIFBDSWUgZHJpdmVy IHVubG9hZGVkIGNvbXBsZXRlbHlcbiIpOwoKRGl0dG8uCgo+ICt9Cgo+ICtNT0RVTEVfREVWSUNF X1RBQkxFKHBjaSwgZHdfZWRtYV9wY2llX2lkX3RhYmxlKTsKPiArCj4gK3N0YXRpYyBzdHJ1Y3Qg cGNpX2RyaXZlciBkd19lZG1hX3BjaWVfZHJpdmVyID0gewo+ICsJLm5hbWUJCT0gImR3LWVkbWEt cGNpZSIsCj4gKwkuaWRfdGFibGUJPSBkd19lZG1hX3BjaWVfaWRfdGFibGUsCj4gKwkucHJvYmUJ CT0gZHdfZWRtYV9wY2llX3Byb2JlLAo+ICsJLnJlbW92ZQkJPSBkd19lZG1hX3BjaWVfcmVtb3Zl LAoKUG93ZXIgbWFuYWdlbWVudD8KCj4gK307Cg== 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=-2.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,USER_AGENT_MUTT autolearn=ham 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 1B781C43387 for ; Fri, 11 Jan 2019 19:53:14 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id E50142084C for ; Fri, 11 Jan 2019 19:53:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388024AbfAKTxN (ORCPT ); Fri, 11 Jan 2019 14:53:13 -0500 Received: from mga12.intel.com ([192.55.52.136]:34943 "EHLO mga12.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731874AbfAKTxM (ORCPT ); Fri, 11 Jan 2019 14:53:12 -0500 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Jan 2019 11:47:10 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,466,1539673200"; d="scan'208";a="125223936" Received: from smile.fi.intel.com (HELO smile) ([10.237.72.86]) by FMSMGA003.fm.intel.com with ESMTP; 11 Jan 2019 11:47:06 -0800 Received: from andy by smile with local (Exim 4.92-RC4) (envelope-from ) id 1gi2ll-0002Z9-9T; Fri, 11 Jan 2019 21:47:05 +0200 Date: Fri, 11 Jan 2019 21:47:05 +0200 From: Andy Shevchenko To: Gustavo Pimentel Cc: linux-pci@vger.kernel.org, dmaengine@vger.kernel.org, Vinod Koul , Dan Williams , Eugeniy Paltsev , Russell King , Niklas Cassel , Lorenzo Pieralisi , Joao Pinto , Jose Abreu , Luis Oliveira , Vitor Soares , Nelson Costa , Pedro Sousa Subject: Re: [RFC v3 5/7] dmaengine: Add Synopsys eDMA IP PCIe glue-logic Message-ID: <20190111194705.GU9170@smile.fi.intel.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org On Fri, Jan 11, 2019 at 07:33:41PM +0100, Gustavo Pimentel wrote: > Synopsys eDMA IP is normally distributed along with Synopsys PCIe > EndPoint IP (depends of the use and licensing agreement). > > This IP requires some basic configurations, such as: > - eDMA registers BAR > - eDMA registers offset > - eDMA registers size > - eDMA linked list memory BAR > - eDMA linked list memory offset > - eDMA linked list memory sze > - eDMA data memory BAR > - eDMA data memory offset > - eDMA data memory size > - eDMA version > - eDMA mode > - IRQs available for eDMA > > As a working example, PCIe glue-logic will attach to a Synopsys PCIe > EndPoint IP prototype kit (Vendor ID = 0x16c3, Device ID = 0xedda), > which has built-in an eDMA IP with this default configuration: > - eDMA registers BAR = 0 > - eDMA registers offset = 0x00001000 (4 Kbytes) > - eDMA registers size = 0x00002000 (8 Kbytes) > - eDMA linked list memory BAR = 2 > - eDMA linked list memory offset = 0x00000000 (0 Kbytes) > - eDMA linked list memory size = 0x00800000 (8 Mbytes) > - eDMA data memory BAR = 2 > - eDMA data memory offset = 0x00800000 (8 Mbytes) > - eDMA data memory size = 0x03800000 (56 Mbytes) > - eDMA version = 0 > - eDMA mode = EDMA_MODE_UNROLL > - IRQs = 1 > > This driver can be compile as built-in or external module in kernel. > > To enable this driver just select DW_EDMA_PCIE option in kernel > configuration, however it requires and selects automatically DW_EDMA > option too. > > Changes: > RFC v1->RFC v2: Changes go after '--- ' line. > - Replace comments // (C99 style) by /**/ > - Merge two pcim_iomap_regions() calls into just one call > - Remove pci_try_set_mwi() call > - Replace some dev_info() by dev_dbg() to reduce *noise* > - Remove pci_name(pdev) call after being call dw_edma_remove() > - Remove all power management support > - Fix the headers of the .c and .h files according to the most recent > convention > - Fix errors and checks pointed out by checkpatch with --strict option > - Replace patch small description tag from dma by dmaengine > RFC v2->RFC v3: > - Fix printk variable of phys_addr_t type > - Fix missing variable initialization (chan->configured) > - Change linked list size to 512 Kbytes > - Add data memory information > - Add register size information > - Add comments or improve existing ones > - Add possibility to work with multiple IRQs feature > - Replace MSI and MSI-X enable condition by pci_dev_msi_enabled() > - Replace code to acquire MSI(-X) address and data by > get_cached_msi_msg() > +enum dw_edma_pcie_bar { > + BAR_0, > + BAR_1, > + BAR_2, > + BAR_3, > + BAR_4, > + BAR_5 > +}; pci-epf.h has this. Why duplicate? What else is being duplicated from PCI core? > +static bool disable_msix; > +module_param(disable_msix, bool, 0644); > +MODULE_PARM_DESC(disable_msix, "Disable MSI-X interrupts"); Why?! We are no allow new module parameters without very strong arguments. > + > +static int dw_edma_pcie_probe(struct pci_dev *pdev, > + const struct pci_device_id *pid) > +{ > + const struct dw_edma_pcie_data *pdata = (void *)pid->driver_data; > + struct device *dev = &pdev->dev; > + struct dw_edma_chip *chip; > + struct dw_edma *dw; > + unsigned int irq_flags = PCI_IRQ_MSI; > + int err, nr_irqs, i; > + > + if (!pdata) { > + dev_err(dev, "%s missing data structure\n", pci_name(pdev)); > + return -EFAULT; > + } Useless check. > + > + /* Enable PCI device */ > + err = pcim_enable_device(pdev); > + if (err) { > + dev_err(dev, "%s enabling device failed\n", pci_name(pdev)); > + return err; > + } > + > + /* Mapping PCI BAR regions */ > + err = pcim_iomap_regions(pdev, BIT(pdata->rg_bar) | > + BIT(pdata->ll_bar) | > + BIT(pdata->dt_bar), > + pci_name(pdev)); > + if (err) { > + dev_err(dev, "%s eDMA BAR I/O remapping failed\n", > + pci_name(pdev)); Isn't it pci_err() ? Same comment for the rest similar cases above and below. > + return err; > + } > + > + pci_set_master(pdev); > + > + nr_irqs = pci_alloc_irq_vectors(pdev, 1, pdata->irqs_cnt, irq_flags); > + if (nr_irqs < 1) { > + dev_err(dev, "%s failed to alloc IRQ vector (Number of IRQs=%u)\n", > + pci_name(pdev), nr_irqs); > + return -EPERM; > + } > + > + /* Data structure initialization */ > + chip->dw = dw; > + chip->dev = dev; > + chip->id = pdev->devfn; > + chip->irq = pdev->irq; > + > + if (!pcim_iomap_table(pdev)) > + return -EACCES; Never happen condition. Thus useless. > + dev_info(dev, "DesignWare eDMA PCIe driver loaded completely\n"); Useless. > +} > + > +static void dw_edma_pcie_remove(struct pci_dev *pdev) > +{ > + struct dw_edma_chip *chip = pci_get_drvdata(pdev); > + struct device *dev = &pdev->dev; > + int err; > + > + /* Stopping eDMA driver */ > + err = dw_edma_remove(chip); > + if (err) > + dev_warn(dev, "can't remove device properly: %d\n", err); > + > + /* Freeing IRQs */ > + pci_free_irq_vectors(pdev); > + > + dev_info(dev, "DesignWare eDMA PCIe driver unloaded completely\n"); Ditto. > +} > +MODULE_DEVICE_TABLE(pci, dw_edma_pcie_id_table); > + > +static struct pci_driver dw_edma_pcie_driver = { > + .name = "dw-edma-pcie", > + .id_table = dw_edma_pcie_id_table, > + .probe = dw_edma_pcie_probe, > + .remove = dw_edma_pcie_remove, Power management? > +}; -- With Best Regards, Andy Shevchenko