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: <20190119154522.GD9170@smile.fi.intel.com> Date: Sat, 19 Jan 2019 17:45:22 +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: T24gVHVlLCBKYW4gMTUsIDIwMTkgYXQgMTI6NDg6MzRQTSArMDAwMCwgR3VzdGF2byBQaW1lbnRl bCB3cm90ZToKPiBPbiAxNS8wMS8yMDE5IDA1OjQzLCBBbmR5IFNoZXZjaGVua28gd3JvdGU6Cj4g PiBPbiBNb24sIEphbiAxNCwgMjAxOSBhdCAxMTozODowMkFNICswMDAwLCBHdXN0YXZvIFBpbWVu dGVsIHdyb3RlOgo+ID4+IE9uIDExLzAxLzIwMTkgMTk6NDcsIEFuZHkgU2hldmNoZW5rbyB3cm90 ZToKPiA+Pj4gT24gRnJpLCBKYW4gMTEsIDIwMTkgYXQgMDc6MzM6NDFQTSArMDEwMCwgR3VzdGF2 byBQaW1lbnRlbCB3cm90ZToKCj4gPj4+PiArc3RhdGljIGJvb2wgZGlzYWJsZV9tc2l4Owo+ID4+ Pj4gK21vZHVsZV9wYXJhbShkaXNhYmxlX21zaXgsIGJvb2wsIDA2NDQpOwo+ID4+Pj4gK01PRFVM RV9QQVJNX0RFU0MoZGlzYWJsZV9tc2l4LCAiRGlzYWJsZSBNU0ktWCBpbnRlcnJ1cHRzIik7Cj4g Pj4+Cj4gPj4+IFdoeT8hCj4gPj4+IFdlIGFyZSBubyBhbGxvdyBuZXcgbW9kdWxlIHBhcmFtZXRl cnMgd2l0aG91dCB2ZXJ5IHN0cm9uZyBhcmd1bWVudHMuCj4gPj4KPiA+PiBTaW5jZSB0aGlzIGlz IGEgcmVmZXJlbmNlIGRyaXZlciBhbmQgbWlnaHQgYmUgdXNlZCB0byB0ZXN0IGN1c3RvbWl6ZWQg SFcKPiA+PiBzb2x1dGlvbnMsIEkgYWRkZWQgdGhpcyBwYXJhbWV0ZXIgdG8gYWxsb3cgdGhlIHBv c3NpYmlsaXR5IHRvIHRlc3QgdGhlIHNvbHV0aW9uCj4gPj4gZm9yY2luZyB0aGUgTVNJIGZlYXR1 cmUgYmluZGluZy4gVGhpcyBpcyByZXF1aXJlZCBzcGVjaWFsbHkgaWYgd2hvIHdpbGwgdGVzdAo+ ID4+IHRoaXMgc29sdXRpb24gaGFzIGEgUm9vdCBDb21wbGV4IHdpdGggYm90aCBmZWF0dXJlcyBh dmFpbGFibGUgKE1TSSBhbmQgTVNJLVgpLAo+ID4+IGJlY2F1c2UgdGhlIEtlcm5lbCB3aWxsIGdp dmUgYWx3YXlzIHByZWZlcmVuY2UgdG8gTVNJLVggYmluZGluZyAoYXNzdW1pbmcgdGhhdAo+ID4+ IHRoZSBFUCBoYXMgYWxzbyBib3RoIGZlYXR1cmVzIGF2YWlsYWJsZSkuCj4gPiAKPiA+IFllcywg eW91IG1heSBkbyBpdCBmb3IgdGVzdGluZyBwdXJwb3NlcywgYnV0IGl0IGRvZXNuJ3QgZml0IHRo ZSBrZXJuZWwgc3RhbmRhcmRzLgo+IAo+IE9rLCBidXQgaG93IHNob3VsZCBJIHByb2NlZWQ/IE1h eSBJIGxlYXZlIGl0IG9yIHN1YnN0aXR1dGUgYnkgYW5vdGhlciB3YXkgdG8gZG8KPiBpdD8gSWYg c28sIGhvdz8KPiBBcyBJIHNhaWQsIHRoZSBpbnRlbmRlZCBpcyB0byBiZSBvbmx5IHVzZWQgZm9y IHRoaXMgdGVzdCBjYXNlLCBvbiBub3JtYWwKPiBvcGVyYXRpb24gdGhlIHBhcmFtZXRlciBpdCBz aG91bGQgYmUgYWx3YXlzIGZhbHNlLgoKS2VlcCBvdXQtb2YtdHJlZSBwYXRjaCBmb3IgeW91ciBu ZWVkcy4KCj4gPj4+PiArCWlmICghcGRhdGEpIHsKPiA+Pj4+ICsJCWRldl9lcnIoZGV2LCAiJXMg bWlzc2luZyBkYXRhIHN0cnVjdHVyZVxuIiwgcGNpX25hbWUocGRldikpOwo+ID4+Pj4gKwkJcmV0 dXJuIC1FRkFVTFQ7Cj4gPj4+PiArCX0KPiA+Pj4KPiA+Pj4gVXNlbGVzcyBjaGVjay4KPiA+Pgo+ ID4+IFdoeT8gSXQncyBqdXN0IGEgcHJlY2F1dGlvbiwgaXNuJ3QgaXQgYSBnb29kIHByYWN0aWNl IGFsd2F5cyB0byB0aGluayBvZiB0aGUKPiA+PiB3b3JzdCBjYXNlPwo+ID4gCj4gPiBZb3UganVz dCBjYW4gcHV0IGFuIGltcGxpY2l0IHJlcXVpcmVtZW50IG9mIHBkYXRhIHJhdGhlciB0aGFuIGRv aW5nIHRoaXMKPiAKPiBPaywgaG93IGNhbiBJIGRvIGl0PyBXaGF0IEkgc2hvdWxkIGFkZCB0byB0 aGUgY29kZSB0byBmb3JjZSB0aGF0PwoKTm90IGFkZGluZywgcmVtb3ZpbmcuIFRoYXQncyB3aGF0 IEkgcHV0IGJlZm9yZS4KCj4gCj4gPiB1c2VsZXNzIGNoZWNrLiBJIGRvbid0IGJlbGlldmUgaXQg d291bGQgbWFrZSBzZW5zZSB0byBoYXZlIE5VTEwgcGRhdGEgZm9yIHRoZQo+ID4gZHJpdmVyIHNp bmNlIGl0IHdvdWxkbid0IGJlIGZ1bmN0aW9uYWwgYW55aG93Lgo+IAo+IFllcywgeW91J3JlIHJp Z2h0IHdpdGhvdXQgcGRhdGEgdGhlIGRyaXZlciBjYW4ndCBkbyBhbnl0aGluZy4K 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 CB7F8C61CE4 for ; Sat, 19 Jan 2019 15:45:28 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id A19E52054F for ; Sat, 19 Jan 2019 15:45:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728294AbfASPp2 (ORCPT ); Sat, 19 Jan 2019 10:45:28 -0500 Received: from mga06.intel.com ([134.134.136.31]:54463 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728260AbfASPp1 (ORCPT ); Sat, 19 Jan 2019 10:45:27 -0500 X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Jan 2019 07:45:27 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,496,1539673200"; d="scan'208";a="115928429" Received: from smile.fi.intel.com (HELO smile) ([10.237.72.86]) by fmsmga007.fm.intel.com with ESMTP; 19 Jan 2019 07:45:23 -0800 Received: from andy by smile with local (Exim 4.92-RC4) (envelope-from ) id 1gksoE-00022K-B5; Sat, 19 Jan 2019 17:45:22 +0200 Date: Sat, 19 Jan 2019 17:45:22 +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: <20190119154522.GD9170@smile.fi.intel.com> References: <20190111194705.GU9170@smile.fi.intel.com> <916f319b-9e45-6d78-4ecc-850feda84bb3@synopsys.com> <20190115054358.GB9170@smile.fi.intel.com> <9b887e39-9de3-2285-f3b4-174a72e6c4d3@synopsys.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <9b887e39-9de3-2285-f3b4-174a72e6c4d3@synopsys.com> 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 Tue, Jan 15, 2019 at 12:48:34PM +0000, Gustavo Pimentel wrote: > On 15/01/2019 05:43, Andy Shevchenko wrote: > > On Mon, Jan 14, 2019 at 11:38:02AM +0000, Gustavo Pimentel wrote: > >> On 11/01/2019 19:47, Andy Shevchenko wrote: > >>> On Fri, Jan 11, 2019 at 07:33:41PM +0100, Gustavo Pimentel wrote: > >>>> +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. > >> > >> Since this is a reference driver and might be used to test customized HW > >> solutions, I added this parameter to allow the possibility to test the solution > >> forcing the MSI feature binding. This is required specially if who will test > >> this solution has a Root Complex with both features available (MSI and MSI-X), > >> because the Kernel will give always preference to MSI-X binding (assuming that > >> the EP has also both features available). > > > > Yes, you may do it for testing purposes, but it doesn't fit the kernel standards. > > Ok, but how should I proceed? May I leave it or substitute by another way to do > it? If so, how? > As I said, the intended is to be only used for this test case, on normal > operation the parameter it should be always false. Keep out-of-tree patch for your needs. > >>>> + if (!pdata) { > >>>> + dev_err(dev, "%s missing data structure\n", pci_name(pdev)); > >>>> + return -EFAULT; > >>>> + } > >>> > >>> Useless check. > >> > >> Why? It's just a precaution, isn't it a good practice always to think of the > >> worst case? > > > > You just can put an implicit requirement of pdata rather than doing this > > Ok, how can I do it? What I should add to the code to force that? Not adding, removing. That's what I put before. > > > useless check. I don't believe it would make sense to have NULL pdata for the > > driver since it wouldn't be functional anyhow. > > Yes, you're right without pdata the driver can't do anything. -- With Best Regards, Andy Shevchenko