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 2AA4BC433F5 for ; Thu, 7 Apr 2022 20:47:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229988AbiDGUtt (ORCPT ); Thu, 7 Apr 2022 16:49:49 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39834 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230282AbiDGUtn (ORCPT ); Thu, 7 Apr 2022 16:49:43 -0400 Received: from mail-pg1-x52b.google.com (mail-pg1-x52b.google.com [IPv6:2607:f8b0:4864:20::52b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 71A9431E1EC for ; Thu, 7 Apr 2022 13:41:58 -0700 (PDT) Received: by mail-pg1-x52b.google.com with SMTP id s137so3251248pgs.5 for ; Thu, 07 Apr 2022 13:41:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gateworks-com.20210112.gappssmtp.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=hsfikvGKS7EdmIl+2iOk9u6ngpZWIU8z/zrcARIqPHQ=; b=tOWDVf5evJdQHKhBdYX/UbCqVT4V8kWTzzwNIFGft9PveKaupbjTyquiq+3tylKDHy ZdqxtITaS7Hb31q313DCaJkJjL5Hk3PG7+W1/PXJkFX9VBoIOjA2Dldii8LNZ6iJ3Tf9 cjkJlKmBe3r4E7WHL2tqXyU8cyJ1TDtj7A0IA0Ka9I193d+UyUAT7Mnpz8C+FXx0B4Cn TXiYAH9JmD3kMUuf+ABMol4g0/m2sHuE7aWS6Bac0RwuLcnqnb9m8kuLJnRlf2uSe8fc apW8DPB7VP4WqlidUV4aYHwYdPdJeROzrwgkto6xvOkwcIKb5AClXNH+qZ9bA9nMoovF BKcg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=hsfikvGKS7EdmIl+2iOk9u6ngpZWIU8z/zrcARIqPHQ=; b=DJZD3Cra3WBZOPqe9wP3ZU8tD81EXW9zhh6P7Vl0dyVNnXIsIX3zYg6sbl493w22ao qk23TPIeb2/HH2roq3MjNfAlr4HeGGkSk6c2G9DIp8NWPeCgwPxA3D3Gw0vhpcANCT3U 40P7T7emgyT0+FvvhHPYKFOaP2mZ0qFi5+xTcOqoR2ycWTIQtFHMK1FjHnvq47m2UJXn tmbz2o08yQMP3dFOrOLw4ueUcZRgqA0IZqYVo+meMbIJjSXP2NAVyVijHEKPhPb+mHdS FrhSUbrxcXhON7YezcSCRlogf6YDDoCxFfr1aOhDmll2WbyrilfpvxGWbbCisR3at9UK l7Tg== X-Gm-Message-State: AOAM532lOmw77k0/aWHpq1ZWnqm7kJ9rxmBmphhnQ5Z90UPCUYE7VfqJ eLnULJlCuw4dSXNb8IH9BC8GJnj+TpB7tlogLtRTHQ== X-Google-Smtp-Source: ABdhPJyw/X9PAj5hQXkBHgIQHLkpVomaTRwQeIQ9NwMn/1m/zjbBSew3NGRBkecufGNq6ap7srYNrp4wut6BDAAtUN0= X-Received: by 2002:a05:6a00:14ca:b0:4fb:5d3e:5f77 with SMTP id w10-20020a056a0014ca00b004fb5d3e5f77mr16115636pfu.34.1649364117644; Thu, 07 Apr 2022 13:41:57 -0700 (PDT) MIME-Version: 1.0 References: <1646644054-24421-1-git-send-email-hongxing.zhu@nxp.com> In-Reply-To: <1646644054-24421-1-git-send-email-hongxing.zhu@nxp.com> From: Tim Harvey Date: Thu, 7 Apr 2022 13:41:46 -0700 Message-ID: Subject: Re: [PATCH v2 0/7] Add the iMX8MP PCIe support To: Richard Zhu , Lucas Stach Cc: Philipp Zabel , "bhelgaas@google.com" , Lorenzo Pieralisi , Rob Herring , Shawn Guo , Vinod Koul , Alexander Stein , linux-phy@lists.infradead.org, Device Tree Mailing List , linux-pci@vger.kernel.org, Linux ARM Mailing List , open list , Sascha Hauer , NXP Linux Team Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org On Mon, Mar 7, 2022 at 1:18 AM Richard Zhu wrote: > > Based on the i.MX8MP GPC and blk-ctrl patch-set[1] issued by Lucas and the > following commits. > - one codes refine patch-set[5]. > - two Fixes[2],[3]. > - one binding commit[4]. > - some dts changes in Shawn's git if you want to test PCIe on i.MX8MM EVK. > b4d36c10bf17 arm64: dts: imx8mm-evk: Add the pcie support on imx8mm evk board > aaeba6a8e226 arm64: dts: imx8mm: Add the pcie support > cfc5078432ca arm64: dts: imx8mm: Add the pcie phy support > > Sorry about that there may be some conflictions when do the codes merge. > I'm waiting for the ack now, and will re-base them in a proper sequence later. > > This series patches add the i.MX8MP PCIe support and tested on i.MX8MM EVK and > i.MX8MP EVk boards. The PCIe NVME works fine on both boards. > > - i.MX8MP PCIe PHY has two resets refer to the i.MX8MM PCIe PHY. > Add one more PHY reset for i.MX8MP PCIe PHY accordingly. > - Add the i.MX8MP PCIe PHY support in the i.MX8M PCIe PHY driver. > And share as much as possible codes with i.MX8MM PCIe PHY. > - Add the i.MX8MP PCIe support in binding document, DTS files, and PCIe > driver. > > Main changes v1-->v2: > - It's my fault forget including Vinod, re-send v2 after include Vinod > and linux-phy@lists.infradead.org. > - List the basements of this patch-set. The branch, codes changes and so on. > - Clean up some useless register and bit definitions in #3 patch. > > [1]https://patchwork.kernel.org/project/linux-arm-kernel/cover/20220228201731.3330192-1-l.stach@pengutronix.de/ > [2]https://patchwork.ozlabs.org/project/linux-pci/patch/1646289275-17813-1-git-send-email-hongxing.zhu@nxp.com/ > [3]https://patchwork.ozlabs.org/project/linux-pci/patch/1645672013-8949-1-git-send-email-hongxing.zhu@nxp.com/ > [4]https://patchwork.ozlabs.org/project/linux-pci/patch/1646293805-18248-1-git-send-email-hongxing.zhu@nxp.com/ > [5]https://patchwork.ozlabs.org/project/linux-pci/cover/1645760667-10510-1-git-send-email-hongxing.zhu@nxp.com/ > > NOTE: > Based git > Based branch > > Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml | 1 + > Documentation/devicetree/bindings/phy/fsl,imx8-pcie-phy.yaml | 4 +- > arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 55 ++++++++++++++++++++++ > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 46 ++++++++++++++++++- > drivers/pci/controller/dwc/pci-imx6.c | 19 +++++++- > drivers/phy/freescale/phy-fsl-imx8m-pcie.c | 205 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------- > drivers/reset/reset-imx7.c | 1 + > 7 files changed, 286 insertions(+), 45 deletions(-) > > [PATCH v2 1/7] reset: imx7: Add the iMX8MP PCIe PHY PERST support > [PATCH v2 2/7] dt-binding: phy: Add iMX8MP PCIe PHY binding > [PATCH v2 3/7] phy: freescale: imx8m-pcie: Add iMX8MP PCIe PHY > [PATCH v2 4/7] dt-bindings: imx6q-pcie: Add iMX8MP PCIe compatible > [PATCH v2 5/7] arm64: dts: imx8mp: add the iMX8MP PCIe support > [PATCH v2 6/7] arm64: dts: imx8mp-evk: Add PCIe support > [PATCH v2 7/7] PCI: imx6: Add the iMX8MP PCIe support > Richard, Thanks for working on this! Do you plan on submitting another version soon? I've tried to test this with an imx8mp board I'm bringing up and while the host controller enumerates I fail to get a link to a device. It's very likely I am missing something as this series depends on the IMX8MP blk-ctrl and gpc series which I also can't cleanly apply. Lucas just submitted a 'consolidated i.MX8MP HSIO/MEDIA/HDMI blk-ctrl series' [1] yet I can't find a repo/branch that applies to either. Perhaps you have a git repo somewhere I can look at while we wait for imx8mp blk-ctl/gpc to settle and you to submit a v3? Best Regards, Tim [1] https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=629586