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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 48AF6C04A68 for ; Thu, 28 Jul 2022 12:21:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238219AbiG1MVu (ORCPT ); Thu, 28 Jul 2022 08:21:50 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57846 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236444AbiG1MVf (ORCPT ); Thu, 28 Jul 2022 08:21:35 -0400 Received: from mail-lf1-x12a.google.com (mail-lf1-x12a.google.com [IPv6:2a00:1450:4864:20::12a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BFD7C49B7F; Thu, 28 Jul 2022 05:21:32 -0700 (PDT) Received: by mail-lf1-x12a.google.com with SMTP id t17so2653623lfk.0; Thu, 28 Jul 2022 05:21:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to; bh=n3olH6gq/B3RKr1Nv0cHF1AA/miXKUeZTOf6sJbeJCU=; b=ks3WtqHxXxLhHY0jUGfcMt1266HUTP8DsN3TWQ5ee39lU2Z3iBYkh/+f2kLMF7r12n q9AdKcoSsJraxVwqWqni2qZpd+Nqs+8Awfbrjg5osRexlF9KyzEjTqObMMVoZMvaemVo Tr+HvILpXlwNP3Pr3hXYvg07YRf342ySKLRq5B/stjlgD2lExzP8+EkiGyomBCSSrkG1 2YTZv4GXpwH1wDb5+KdWfvpybgUmiOPeCScTRoNVhOLvhPyzmOkEjoVkyUaDeBnH74vg aTFIX4530a1SjlnN3NZhsBPfDBg4V4YYpwK29hyajKPMeBYyZrZovP4Avr7d1WEZ/w12 /JQw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=n3olH6gq/B3RKr1Nv0cHF1AA/miXKUeZTOf6sJbeJCU=; b=so/m0bmDsW6HKJML7DAUIwhreBVfR0IQycjo3/fy3/xOGorscJtRSJvJiqLmzzXNUi /QIHIAXtmx9mAIuw5T23YHl2BI9iDCuxrkUM/GouSZqDGEb5DcCQ90X7V9z8GnmFLgPd wiUd6acs1ytjonj9o6ehlyO9Y/MPIhYBqLUTdrBZTWBJFIuqFkz7EuQi5lIHlSlnO+pN Vm7zl7AiPEUgby17qrEacgi2ruamI94Ww27jopRfnleKu2F5dMRqPZeWtd+IucOzm/Rt 02rG+GESYKEB6zbg7N+i/toPRoCyBHWuMS1/dh5J0IdUYX+4C/+nilJLkpFXQ6FzoovC xHmw== X-Gm-Message-State: AJIora8CEsvev5E43MUxY7x6ZixrVmIHE3S58bCmoMh4ozmL22dne+WN iY9gzgzIojLCRHktUPn+vsI= X-Google-Smtp-Source: AGRyM1vlYJp3Kg1WE9mOHGV2AbiuMTusSwWyOgGYz6c+f4J8+DSKBzG+Dwnrp2qkWWYbuKBj5UshFw== X-Received: by 2002:a05:6512:2625:b0:48a:a77a:5b08 with SMTP id bt37-20020a056512262500b0048aa77a5b08mr4669666lfb.116.1659010890831; Thu, 28 Jul 2022 05:21:30 -0700 (PDT) Received: from mobilestation ([95.79.140.178]) by smtp.gmail.com with ESMTPSA id i27-20020a2ea23b000000b0025a67779931sm122822ljm.57.2022.07.28.05.21.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 Jul 2022 05:21:30 -0700 (PDT) Date: Thu, 28 Jul 2022 15:21:27 +0300 From: Serge Semin To: Bjorn Helgaas Cc: Serge Semin , Rob Herring , Bjorn Helgaas , Lorenzo Pieralisi , Jingoo Han , Gustavo Pimentel , Alexey Malahov , Pavel Parkhomenko , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Frank Li , Manivannan Sadhasivam , Rob Herring , linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH RESEND v4 00/15] PCI: dwc: Add hw version and dma-ranges support Message-ID: <20220728122127.lsncoxbpoopd3w6r@mobilestation> References: <20220624143947.8991-1-Sergey.Semin@baikalelectronics.ru> <20220727224904.GA253904@bhelgaas> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220727224904.GA253904@bhelgaas> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Bjorn On Wed, Jul 27, 2022 at 05:49:04PM -0500, Bjorn Helgaas wrote: > On Fri, Jun 24, 2022 at 05:39:32PM +0300, Serge Semin wrote: > > This patchset is a second one in the series created in the framework of > > my Baikal-T1 PCIe/eDMA-related work: > > > > [1: In-progress v5] PCI: dwc: Various fixes and cleanups > > Link: https://lore.kernel.org/linux-pci/20220624143428.8334-1-Sergey.Semin@baikalelectronics.ru/ > > [2: In-progress v4] PCI: dwc: Add hw version and dma-ranges support > > Link: ---you are looking at it--- > > https://lore.kernel.org/r/20220624143947.8991-1-Sergey.Semin@baikalelectronics.ru > > > [3: In-progress v3] PCI: dwc: Add extended YAML-schema and Baikal-T1 support > > Link: https://lore.kernel.org/linux-pci/20220610085706.15741-1-Sergey.Semin@baikalelectronics.ru/ > > [4: In-progress v3] dmaengine: dw-edma: Add RP/EP local DMA support > > Link: https://lore.kernel.org/linux-pci/20220610091459.17612-1-Sergey.Semin@baikalelectronics.ru > > Hi Serge, is the above the latest and greatest list? The 3rd series > doesn't apply cleanly for me: Thanks for getting back to my patchsets. The 3rd series is still under review (see below for details). Here is the summary regarding the patchsets state: [1: Done v5] PCI: dwc: Various fixes and cleanups The latest lore Link: https://lore.kernel.org/linux-pci/20220624143428.8334-1-Sergey.Semin@baikalelectronics.ru/ You've already merged it in here: Link: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git/log/?h=pci/ctrl/dwc [2: Done v4] PCI: dwc: Add hw version and dma-ranges support The latest lore Link: https://lore.kernel.org/linux-pci/20220624143947.8991-1-Sergey.Semin@baikalelectronics.ru [3: In-progress v3] PCI: dwc: Add extended YAML-schema and Baikal-T1 support The latest lore Link: https://lore.kernel.org/linux-pci/20220610085706.15741-1-Sergey.Semin@baikalelectronics.ru/ This series, you are having difficulties to apply, is still under Rob' review. I've addressed all your comments locally in my repo, but Rob' didn't respond on some of my questions (there are several principal aspects of the dt-bindings patches he didn't like). First he used to be in vacation, then he just missed several of my email messages, but responded on another ones. Pings didn't work either. Then I thought the merge-window was opened so I has stopped bothering him for some time.) So I'll resend the series as is one more time today then. Hopefully he'll get back to it soon. [4: Done v3] dmaengine: dw-edma: Add RP/EP local DMA support The latest lore Link: https://lore.kernel.org/linux-pci/20220610091459.17612-1-Sergey.Semin@baikalelectronics.ru/ The series review is done. It has even got Vinod' Ab tag. But I need to resend it (will do this today) since it needs to be rebased on the latest Frank Li patchset. Frank series is in your repo already: Link: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git/log/?h=pci/ctrl/dwc-edma So if you aren't going to merge the DW PCIe/eDMA related series in on this merge window, please collect my and Frank work in a single branch of yours (pci/ctrl/dwc-edma), since my patchset #4 depends on his series and the patchsets #3, #2 and #1 described above. -Sergey > > 05:40:34 ~/linux (main)$ git checkout -b wip/serge v5.19-rc1 > Switched to a new branch 'wip/serge' > > # fetch 1: PCI: dwc: Various fixes and cleanups > 05:40:45 ~/linux (wip/serge)$ b4 am -om/ https://lore.kernel.org/linux-pci/20220624143428.8334-1-Sergey.Semin@baikalelectronics.ru/ > Analyzing 37 messages in the thread > Checking attestation on all messages, may take a moment... > --- > ✓ [PATCH v5 1/18] PCI: dwc: Stop link in the host init error and de-initialization > ✓ [PATCH v5 2/18] PCI: dwc: Add unroll iATU space support to the regions disable method > ✓ [PATCH v5 3/18] PCI: dwc: Disable outbound windows for controllers with iATU > ✓ [PATCH v5 4/18] PCI: dwc: Set INCREASE_REGION_SIZE flag based on limit address > ✓ [PATCH v5 5/18] PCI: dwc: Deallocate EPC memory on EP init error > + Reviewed-by: Manivannan Sadhasivam (✓ DKIM/linaro.org) > ✓ [PATCH v5 6/18] PCI: dwc: Enable CDM-check independently from the num_lanes value > + Reviewed-by: Manivannan Sadhasivam (✓ DKIM/linaro.org) > ✓ [PATCH v5 7/18] PCI: dwc: Add braces to the multi-line if-else statements > ✓ [PATCH v5 8/18] PCI: dwc: Add trailing new-line literals to the log messages > ✓ [PATCH v5 9/18] PCI: dwc: Discard IP-core version checking on unrolled iATU detection > ✓ [PATCH v5 10/18] PCI: dwc: Convert Link-up status method to using dw_pcie_readl_dbi() > ✓ [PATCH v5 11/18] PCI: dwc: Organize local variables usage > + Reviewed-by: Manivannan Sadhasivam (✓ DKIM/linaro.org) > ✓ [PATCH v5 12/18] PCI: dwc: Re-use local pointer to the resource data > + Reviewed-by: Manivannan Sadhasivam (✓ DKIM/linaro.org) > ✓ [PATCH v5 13/18] PCI: dwc: Add start_link/stop_link inliners > + Reviewed-by: Manivannan Sadhasivam (✓ DKIM/linaro.org) > ✓ [PATCH v5 14/18] PCI: dwc: Move io_cfg_atu_shared to the Root Port descriptor > + Reviewed-by: Manivannan Sadhasivam (✓ DKIM/linaro.org) > ✓ [PATCH v5 15/18] PCI: dwc: Add dw_ prefix to the pcie_port structure name > + Acked-by: Jesper Nilsson (✓ DKIM/axis.com) > + Reviewed-by: Manivannan Sadhasivam (✓ DKIM/linaro.org) > + Reviewed-by: Neil Armstrong (✓ DKIM/baylibre-com.20210112.gappssmtp.com) > ✓ [PATCH v5 16/18] PCI: dwc-plat: Simplify the probe method return value handling > ✓ [PATCH v5 17/18] PCI: dwc-plat: Discard unused regmap pointer > ✓ [PATCH v5 18/18] PCI: dwc-plat: Drop dw_plat_pcie_of_match forward declaration > --- > ✓ Signed: DKIM/baikalelectronics.ru > --- > Total patches: 18 > --- > NOTE: some trailers ignored due to from/email mismatches: > ! Trailer: Link: https://lore.kernel.org/linux-pci/20220616152048.gcqacgs2ed66vsl4@mobilestation/ > Msg From: Serge Semin > NOTE: Rerun with -S to apply them anyway > --- > Cover: m/v5_20220624_sergey_semin_pci_dwc_various_fixes_and_cleanups.cover > Link: https://lore.kernel.org/r/20220624143428.8334-1-Sergey.Semin@baikalelectronics.ru > Base: applies clean to current tree > git am m/v5_20220624_sergey_semin_pci_dwc_various_fixes_and_cleanups.mbx > > # apply 1: PCI: dwc: Various fixes and cleanups > 05:41:24 ~/linux (wip/serge)$ git am m/v5_20220624_sergey_semin_pci_dwc_various_fixes_and_cleanups.mbx > Applying: PCI: dwc: Stop link in the host init error and de-initialization > Applying: PCI: dwc: Add unroll iATU space support to the regions disable method > Applying: PCI: dwc: Disable outbound windows for controllers with iATU > Applying: PCI: dwc: Set INCREASE_REGION_SIZE flag based on limit address > Applying: PCI: dwc: Deallocate EPC memory on EP init error > Applying: PCI: dwc: Enable CDM-check independently from the num_lanes value > Applying: PCI: dwc: Add braces to the multi-line if-else statements > Applying: PCI: dwc: Add trailing new-line literals to the log messages > Applying: PCI: dwc: Discard IP-core version checking on unrolled iATU detection > Applying: PCI: dwc: Convert Link-up status method to using dw_pcie_readl_dbi() > Applying: PCI: dwc: Organize local variables usage > Applying: PCI: dwc: Re-use local pointer to the resource data > Applying: PCI: dwc: Add start_link/stop_link inliners > Applying: PCI: dwc: Move io_cfg_atu_shared to the Root Port descriptor > Applying: PCI: dwc: Add dw_ prefix to the pcie_port structure name > Applying: PCI: dwc-plat: Simplify the probe method return value handling > Applying: PCI: dwc-plat: Discard unused regmap pointer > Applying: PCI: dwc-plat: Drop dw_plat_pcie_of_match forward declaration > > # fetch 2: PCI: dwc: Add hw version and dma-ranges support > 05:41:46 ~/linux (wip/serge)$ b4 am -om/ https://lore.kernel.org/r/20220624143947.8991-1-Sergey.Semin@baikalelectronics.ru > Looking up https://lore.kernel.org/r/20220624143947.8991-1-Sergey.Semin%40baikalelectronics.ru > Analyzing 19 messages in the thread > Checking attestation on all messages, may take a moment... > --- > ✓ [PATCH v4 1/15] PCI: dwc: Add more verbose link-up message > ✓ [PATCH v4 2/15] PCI: dwc: Detect iATU settings after getting "addr_space" resource > ✓ [PATCH v4 3/15] PCI: dwc: Convert to using native IP-core versions representation > ✓ [PATCH v4 4/15] PCI: dwc: Add IP-core version detection procedure > ✓ [PATCH v4 5/15] PCI: dwc: Introduce Synopsys IP-core versions/types interface > ✓ [PATCH v4 6/15] PCI: intel-gw: Drop manual DW PCIe controller version setup > ✓ [PATCH v4 7/15] PCI: tegra194: Drop manual DW PCIe controller version setup > ✓ [PATCH v4 8/15] PCI: dwc: Add host de-initialization callback > ✓ [PATCH v4 9/15] PCI: dwc: Drop inbound iATU types enumeration - dw_pcie_as_type > ✓ [PATCH v4 10/15] PCI: dwc: Drop iATU regions enumeration - dw_pcie_region_type > ✓ [PATCH v4 11/15] PCI: dwc: Simplify in/outbound iATU setup methods > ✓ [PATCH v4 12/15] PCI: dwc: Add iATU regions size detection procedure > ✓ [PATCH v4 13/15] PCI: dwc: Verify in/out regions against iATU constraints > ✓ [PATCH v4 14/15] PCI: dwc: Check iATU in/outbound ranges setup methods status > ✓ [PATCH v4 15/15] PCI: dwc: Introduce dma-ranges property support for RC-host > --- > ✓ Signed: DKIM/baikalelectronics.ru > --- > Total patches: 15 > --- > NOTE: some trailers ignored due to from/email mismatches: > ! Trailer: Link: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git/log/?h=pci%2Fctrl%2Fdwc-fixes > Msg From: Serge Semin > ! Trailer: Link: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git/log/?h=pci/edma > Msg From: Serge Semin > NOTE: Rerun with -S to apply them anyway > --- > Cover: m/v4_20220624_sergey_semin_pci_dwc_add_hw_version_and_dma_ranges_support.cover > Link: https://lore.kernel.org/r/20220624143947.8991-1-Sergey.Semin@baikalelectronics.ru > Base: applies clean to current tree > git am m/v4_20220624_sergey_semin_pci_dwc_add_hw_version_and_dma_ranges_support.mbx > > # apply 2: PCI: dwc: Add hw version and dma-ranges support > 05:42:05 ~/linux (wip/serge)$ git am m/v4_20220624_sergey_semin_pci_dwc_add_hw_version_and_dma_ranges_support.mbx > Applying: PCI: dwc: Add more verbose link-up message > Applying: PCI: dwc: Detect iATU settings after getting "addr_space" resource > Applying: PCI: dwc: Convert to using native IP-core versions representation > Applying: PCI: dwc: Add IP-core version detection procedure > Applying: PCI: dwc: Introduce Synopsys IP-core versions/types interface > Applying: PCI: intel-gw: Drop manual DW PCIe controller version setup > Applying: PCI: tegra194: Drop manual DW PCIe controller version setup > Applying: PCI: dwc: Add host de-initialization callback > Applying: PCI: dwc: Drop inbound iATU types enumeration - dw_pcie_as_type > Applying: PCI: dwc: Drop iATU regions enumeration - dw_pcie_region_type > Applying: PCI: dwc: Simplify in/outbound iATU setup methods > Applying: PCI: dwc: Add iATU regions size detection procedure > Applying: PCI: dwc: Verify in/out regions against iATU constraints > Applying: PCI: dwc: Check iATU in/outbound ranges setup methods status > Applying: PCI: dwc: Introduce dma-ranges property support for RC-host > > # fetch 3: PCI: dwc: Add extended YAML-schema and Baikal-T1 support > 05:42:40 ~/linux (wip/serge)$ b4 am -om/ https://lore.kernel.org/linux-pci/20220610085706.15741-1-Sergey.Semin@baikalelectronics.ru/ > Analyzing 51 messages in the thread > Checking attestation on all messages, may take a moment... > --- > ✓ [PATCH v3 1/17] dt-bindings: PCI: dwc: Detach common RP/EP DT bindings > ✓ [PATCH v3 2/17] dt-bindings: PCI: dwc: Remove bus node from the examples > + Reviewed-by: Rob Herring > ✓ [PATCH v3 3/17] dt-bindings: PCI: dwc: Add phys/phy-names common properties > ✓ [PATCH v3 4/17] dt-bindings: PCI: dwc: Add max-link-speed common property > ✓ [PATCH v3 5/17] dt-bindings: PCI: dwc: Stop selecting generic bindings by default > ✓ [PATCH v3 6/17] dt-bindings: PCI: dwc: Add max-functions EP property > + Reviewed-by: Rob Herring > ✓ [PATCH v3 7/17] dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties > ✓ [PATCH v3 8/17] dt-bindings: PCI: dwc: Add reg/reg-names common properties > ✓ [PATCH v3 9/17] dt-bindings: PCI: dwc: Add clocks/resets common properties > ✓ [PATCH v3 10/17] dt-bindings: PCI: dwc: Add dma-coherent property > ✓ [PATCH v3 11/17] dt-bindings: PCI: dwc: Apply common schema to Rockchip DW PCIe nodes > ✓ [PATCH v3 12/17] dt-bindings: PCI: dwc: Add Baikal-T1 PCIe Root Port bindings > ✓ [PATCH v3 13/17] PCI: dwc: Introduce generic controller capabilities interface > + Reviewed-by: Rob Herring > ✓ [PATCH v3 14/17] PCI: dwc: Introduce generic resources getter > + Reviewed-by: Rob Herring > ✓ [PATCH v3 15/17] PCI: dwc: Combine iATU detection procedures > + Reviewed-by: Rob Herring > ✓ [PATCH v3 16/17] PCI: dwc: Introduce generic platform clocks and resets > ✓ [PATCH v3 17/17] PCI: dwc: Add Baikal-T1 PCIe controller support > --- > ✓ Signed: DKIM/baikalelectronics.ru > --- > Total patches: 17 > --- > NOTE: some trailers ignored due to from/email mismatches: > ! Trailer: Link: https://lore.kernel.org/linux-pci/20220610091459.17612-23-Sergey.Semin@baikalelectronics.ru/ > Msg From: Serge Semin > ! Trailer: Link: https://lore.kernel.org/linux-pci/20220610091459.17612-1-Sergey.Semin@baikalelectronics.ru/ > Msg From: Serge Semin > NOTE: Rerun with -S to apply them anyway > --- > Cover: m/v3_20220610_sergey_semin_pci_dwc_add_generic_resources_and_baikal_t1_support.cover > Link: https://lore.kernel.org/r/20220610085706.15741-1-Sergey.Semin@baikalelectronics.ru > Base: not specified > git am m/v3_20220610_sergey_semin_pci_dwc_add_generic_resources_and_baikal_t1_support.mbx > > # apply 3: PCI: dwc: Add extended YAML-schema and Baikal-T1 support > 05:42:58 ~/linux (wip/serge)$ git am m/v3_20220610_sergey_semin_pci_dwc_add_generic_resources_and_baikal_t1_support.mbx > Applying: dt-bindings: PCI: dwc: Detach common RP/EP DT bindings > error: patch failed: Documentation/devicetree/bindings/pci/snps,dw-pcie-ep.yaml:36 > error: Documentation/devicetree/bindings/pci/snps,dw-pcie-ep.yaml: patch does not apply > error: patch failed: Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml:37 > error: Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml: patch does not apply > Patch failed at 0001 dt-bindings: PCI: dwc: Detach common RP/EP DT bindings > hint: Use 'git am --show-current-patch' to see the failed patch > When you have resolved this problem, run "git am --continue". > If you prefer to skip this patch, run "git am --skip" instead. > To restore the original branch and stop patching, run "git am --abort". > 05:43:07 ~/linux (wip/serge|AM 1/17)$