linux-kernel - RE: [PATCH v3 04/14] PCI: dwc: Kconfig: Add iMX PCIe EP mode support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 26 Sep 2022 05:24:52 +0000
From:   Hongxing Zhu <hongxing.zhu@....com>
To:     Bjorn Helgaas <helgaas@...nel.org>
CC:     "l.stach@...gutronix.de" <l.stach@...gutronix.de>,
        "bhelgaas@...gle.com" <bhelgaas@...gle.com>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "lorenzo.pieralisi@....com" <lorenzo.pieralisi@....com>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        "kishon@...com" <kishon@...com>, "kw@...ux.com" <kw@...ux.com>,
        Frank Li <frank.li@....com>,
        "linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        dl-linux-imx <linux-imx@....com>
Subject: RE: [PATCH v3 04/14] PCI: dwc: Kconfig: Add iMX PCIe EP mode support

> -----Original Message-----
> From: Bjorn Helgaas <helgaas@...nel.org>
> Sent: 2022年9月23日 22:15
> To: Hongxing Zhu <hongxing.zhu@....com>
> Cc: l.stach@...gutronix.de; bhelgaas@...gle.com; robh+dt@...nel.org;
> lorenzo.pieralisi@....com; shawnguo@...nel.org; kishon@...com;
> kw@...ux.com; Frank Li <frank.li@....com>; linux-pci@...r.kernel.org;
> devicetree@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
> linux-kernel@...r.kernel.org; kernel@...gutronix.de; dl-linux-imx
> <linux-imx@....com>
> Subject: Re: [PATCH v3 04/14] PCI: dwc: Kconfig: Add iMX PCIe EP mode
> support
> 
> On Fri, Sep 23, 2022 at 02:06:50PM +0800, Richard Zhu wrote:
> > Since i.MX PCIe is one dual mode PCIe controller.
> 
> This is not a sentence.
Okay, would be changed.

> 
> > Add i.MX PCIe EP mode support, and split the PCIe modes to the Root
> > Complex mode and Endpoint mode.
> 
> Add blank lines between paragraphs or rewrap into a single paragraph that fills
> 75 columns.
Okay, would refine the commit log later.

> 
> I think you should split "[12/14] PCI: imx6: Add iMX8MM PCIe EP mode"
> into:
> 
>   - A patch that adds the generic endpoint infrastructure, e.g.,
>     imx6_pcie_ep_init(), imx6_pcie_ep_raise_irq(), imx6_add_pcie_ep().
> 
>   - A second patch that adds the i.MX8MM identifiers.
> 
> That way the i.MX8MM patch will be analogous to the i.MX8MQ and i.MX8MP
> patches.
> 
> Then you could squash this Kconfig patch into the generic endpoint
> infrastructure patch because this patch is what selects PCIE_DW_EP, which is
> what ensures that dw_pcie_ep_reset_bar(), dw_pcie_ep_raise_legacy_irq(), etc.,
> are available.
Good suggestion. Thanks a lot.
Would change it following this way.

> 
> > --- a/drivers/pci/controller/dwc/Kconfig
> > +++ b/drivers/pci/controller/dwc/Kconfig
> > @@ -92,10 +92,33 @@ config PCI_EXYNOS
> >  	  functions to implement the driver.
> >
> >  config PCI_IMX6
> > -	bool "Freescale i.MX6/7/8 PCIe controller"
> > +	bool
> > +
> > +config PCI_IMX6_HOST
> > +	bool "Freescale i.MX6/7/8 PCIe controller host mode"
> >  	depends on ARCH_MXC || COMPILE_TEST
> >  	depends on PCI_MSI_IRQ_DOMAIN
> >  	select PCIE_DW_HOST
> > +	select PCI_IMX6
> > +	help
> > +	  Enables support for the PCIe controller Root Complex mode in the
> > +	  iMX6/7/8 SoCs.
> 
> > +	  This controller can work either as EP or RC. In order to enable
> > +	  host-specific features PCIE_DW_HOST must be selected and in order
> > +	  to enable device-specific features PCIE_DW_EP must be selected.
> 
> I don't think these three lines are useful to the user.  They only describe what
> Kconfig does when PCI_IMX6_HOST is enabled, which is really an internal
> implementation detail.
Okay, would refine them later.

> 
> > +config PCI_IMX6_EP
> > +	bool "Freescale i.MX6/7/8 PCIe controller endpoint mode"
> > +	depends on ARCH_MXC || COMPILE_TEST
> > +	depends on PCI_ENDPOINT
> > +	select PCIE_DW_EP
> > +	select PCI_IMX6
> > +	help
> > +	  Enables support for the PCIe controller endpoint mode in the
> > +	  iMX6/7/8 SoCs.
> > +	  This controller can work either as EP or RC. In order to enable
> > +	  host-specific features PCIE_DW_HOST must be selected and in order
> > +	  to enable device-specific features PCIE_DW_EP must be selected.
> 
> Ditto.
Okay, would refine them later.

Best Regards
Richard Zhu

> 
> >  config PCIE_SPEAR13XX
> >  	bool "STMicroelectronics SPEAr PCIe controller"
> > --
> > 2.25.1
> >
> >
> > _______________________________________________
> > linux-arm-kernel mailing list
> > linux-arm-kernel@...ts.infradead.org
> > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists
> > .infradead.org%2Fmailman%2Flistinfo%2Flinux-arm-kernel&amp;data=05%7
> C0
> >
> 1%7Chongxing.zhu%40nxp.com%7Cc9b4cdb1fe6048bcbf2808da9d6e1245%7
> C686ea1
> >
> d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637995393327822891%7CUnk
> nown%7CTW
> >
> FpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXV
> CI6
> >
> Mn0%3D%7C3000%7C%7C%7C&amp;sdata=mtOA1T3wSvr4IRfbF8WpMlu%2B
> pMh6vQDPKab
> > 17Y4zdeE%3D&amp;reserved=0