From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1763326AbdLSOJY (ORCPT ); Tue, 19 Dec 2017 09:09:24 -0500 Received: from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]:37746 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1763309AbdLSOJT (ORCPT ); Tue, 19 Dec 2017 09:09:19 -0500 Date: Tue, 19 Dec 2017 14:10:04 +0000 From: Lorenzo Pieralisi To: Gustavo Pimentel , Joao Pinto , Jingoo Han , "kishon@ti.com" Cc: Niklas Cassel , Bjorn Helgaas , "linux-pci@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v5 01/18] PCI: dwc: Use the DMA-API to get the MSI address Message-ID: <20171219141004.GE19911@red-moon> References: <20171120133222.27771-2-niklas.cassel@axis.com> <20171130152843.GC12096@red-moon> <20171213135953.GA30833@axis.com> <20171214122229.GB27348@e107981-ln.cambridge.arm.com> <816956be-90ff-be0e-748f-39c64e4e7833@synopsys.com> <20171218155726.GB8157@red-moon> <6ef8e160-1bf6-7fb2-d9c8-c9d27c04541d@synopsys.com> <95866c7e-5177-69d6-ac64-1ba7f8b3e36d@synopsys.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <95866c7e-5177-69d6-ac64-1ba7f8b3e36d@synopsys.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 19, 2017 at 12:45:30PM +0000, Gustavo Pimentel wrote: > Tested-By: Gustavo Pimentel > Using an arc board with USB and SATA end points with a root complex PCIe > IP core version 5.00, I ran the the following tests: > - I checked that all end points were listed correctly; > - I checked that no PCI related errors was present in dmesg; > - I copied a file from arc to the USB flash drive and checked the > checksum; > - I copied a file from USB flash drive to the ARC and checked the > checksum; > - I copied a file from arc to the SATA disk and checked the checksum; > - I copied a file from SATA disk to the arc and checked the checksum; > - I copied a file from USB flash drive to the SATA disk and checked the > checksum; > - I copied a file from SATA disk to the USB flash drive and checked the > checksum; > - I checked that no PCI related errors was present in dmesg; > > Everything seemed normal to me. Thank you very much. I would still need ACKs from the respective maintainers (Joao/Jingoo/Kishon) in order to push this upstream please. Thanks, Lorenzo > On 18/12/2017 16:11, Gustavo Pimentel wrote: > > Hi Lorenzo, > > Yes, I have retrieve patches and performed some basic tests without having any > problems till now. > > However, I'm still trying to test the changes with our automated system. It's > being very difficult to find a time slot available for running this tests. > > I hope in this week I could give you go/not go flag. Sorry the delay... > > Regards, > > Gustavo > > > On 18/12/2017 15:57, Lorenzo Pieralisi wrote: > > Hi Gustavo, > > On Thu, Dec 14, 2017 at 12:38:04PM +0000, Gustavo Pimentel wrote: > > Ok. I will compile and test it now. > > Have you managed to retrieve the patches and test them ? > > Thank you, > Lorenzo > > > Thanks. > > > On 14/12/2017 12:22, Lorenzo Pieralisi wrote: > > On Thu, Dec 14, 2017 at 12:16:38PM +0000, Gustavo Pimentel wrote: > > Hi Niklas and Lorenzo, > > I'm going to work on PCI software development now as told by Joao and I will > test your code now. > > I was retrieving the patches through the patchwork > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.ozlabs.org_project_linux-2Dpci_list_-3Fsubmitter-3D65580&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=bkWxpLoW-f-E3EdiDCCa0_h0PicsViasSlvIpzZvPxs&m=paO6WYeoTil-HWIJVBhwaWcIZP7r95MZNRzKMN6kprU&s=yjtNhJKPJdsHD_A52xWSD2g_YNHuLhO1YADKsPaPsio&e= and I > notice that its missing the patch 13 and 17, is this right? > > No, you just have to refine the filter, I marked the DT bindings as > ready to be applied that's why they do not show up (and by the way > you can test the series even without those two patches - that are just > DT bindings). > > Thanks, > Lorenzo > > > If not, can you give me both patches location? > > Thanks. > > > On 13/12/2017 17:21, Joao Pinto wrote: > > Hi Niklas, > > ??s 1:59 PM de 12/13/2017, Niklas Cassel escreveu: > > On Thu, Nov 30, 2017 at 03:28:43PM +0000, Lorenzo Pieralisi wrote: > > Jingoo, Joao, > > I am expecting your testing on the series and ACKs on the dwc related > patches please, according to v4 review - I will mark them as needs > review/ACK waiting for you to chime in. > > v4 thread: > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.ozlabs.org_patch_833882_&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=s2fO0hii0OGNOv9qQy_HRXy-xAJUD1NNoEcc3io_kx0&m=CQAV8rCgm2jd7m3iSWL5vBbGTqSc7yN3N7zeQDz-ZUY&s=mqyDi45qm4qE0jcm0KviYAoLHmMXMMIWEW-gAR0RKOw&e= > > > Hello Lorenzo, Jingoo, Joao, > > Tomorrow another 2 weeks has passed. > V1 of this patch series was posted on 2017-10-13. > > Sorry, I have been tight up with a debug session and not able to check this out. > Adding Gustavo in CC that is now also working in PCI software development. > I am going to check the code ASAP and we will test it as soon as the debug is > finished. > > Thanks. > > > I'm a bit worried that this patch series will not make it to linux-next > in time for this patch series to be included in the 4.16 pull request. > > Here is the V5 patch series: > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.ozlabs.org_project_linux-2Dpci_list_-3Fseries-3D14364&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=s2fO0hii0OGNOv9qQy_HRXy-xAJUD1NNoEcc3io_kx0&m=CQAV8rCgm2jd7m3iSWL5vBbGTqSc7yN3N7zeQDz-ZUY&s=qXtUHFXK7_nCndQob0I0qDByBdJsL7rRjZ9cx7c847U&e= > > Perhaps any maintainer of a designware based PCIe driver could > help out and test this patch series? > > Regards, > Niklas