Re: [PATCH v9 3/3] PCI: Remove device links to PHY

[Vladimir Oltean]

Hi Bjorn,

On Fri, Sep 11, 2026 at 05:12:09PM -0500, Bjorn Helgaas wrote:
> [+cc Felix, author of device link leak fixes]
> 
> On Mon, Aug 10, 2026 at 03:43:15PM +0300, Vladimir Oltean wrote:
> > This is practically a full revert of commit
> > 7a4db656a635 ("PCI: dra7xx: Create functional dependency between PCIe and PHY")
> > and a partial revert of the device link pieces from commits
> > dfb80534692d ("PCI: cadence: Add generic PHY support to host and EP drivers")
> > 49229238ab47 ("PCI: keystone: Cleanup PHY handling")
> 
> Can we split this into three commits (dra7xx, cadence, keystone)?
> This combined commit is applied on the pci/controller/misc branch, and
> then we have conflicts between that branch and
> pci/controller/dwc-dra7xx.
> 
> I think if we split this into three separate commits, we can apply the
> dra7xx one on pci/controller/dwc-dra7xx and drop Felix's device link
> leak fixes below so we don't clutter the git history with conflict
> resolutions or irrelevant commits:
> 
>   https://git.kernel.org/cgit/linux/kernel/git/pci/pci.git/commit/?id=2b9a036b568c ("PCI: dra7xx: Fix device link leak when devm_phy_get() fails")
> 
>   https://git.kernel.org/cgit/linux/kernel/git/pci/pci.git/commit/?id=bdfdf654b4b2 ("PCI: dra7xx: Fix device links leak when dra7xx_pcie_enable_phy() fails")

Isn't this patch already merged in pci/next as
https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git/commit/?id=9a9d18547ddc97a73d5d90cff4d5aef90bd8c15d
?