[PATCH 2/3] PCI: imx6: Use the external clock as REF_CLK when needed for i.MX8MQ

[Manivannan Sadhasivam]

On Wed, Aug 12, 2026 at 02:49:08AM +0000, Hongxing Zhu wrote:
> > -----Original Message-----
> > From: Rudi Heitbaum <rudi at heitbaum.com>
> > Sent: Sunday, August 2, 2026 7:32 PM
> > To: Hongxing Zhu <hongxing.zhu at nxp.com>; Lucas Stach
> > <l.stach at pengutronix.de>; Frank Li <frank.li at nxp.com>; Sascha Hauer
> > <s.hauer at pengutronix.de>
> > Cc: Lorenzo Pieralisi <lpieralisi at kernel.org>; Krzysztof Wilczyński
> > <kwilczynski at kernel.org>; Manivannan Sadhasivam <mani at kernel.org>; Rob
> > Herring <robh at kernel.org>; Bjorn Helgaas <bhelgaas at google.com>;
> > Krzysztof Kozlowski <krzk+dt at kernel.org>; Conor Dooley
> > <conor+dt at kernel.org>; Fabio Estevam <festevam at gmail.com>; linux-
> > pci at vger.kernel.org; devicetree at vger.kernel.org; imx at lists.linux.dev; linux-
> > arm-kernel at lists.infradead.org; linux-kernel at vger.kernel.org;
> > rudi at heitbaum.com
> > Subject: [PATCH 2/3] PCI: imx6: Use the external clock as REF_CLK when
> > needed for i.MX8MQ
> > 
> > i.MX8MQ has two PCIe REF_CLK sources: an off-chip oscillator and the
> > internal PLL. The driver unconditionally selected the oscillator, which is what
> > the
> > 
> > 	/* TODO: This code assumes external oscillator is being used */
> > 
> > comment refers to. Boards wiring an oscillator to the pad describe it as the
> > "pcie_bus" clock, but nothing in the driver looks that clock up by name, so it
> > does not select anything.
> > 
> > Reuse the generic enable_ext_refclk flag, set when an "extref" clock is
> > supplied, to select between the two, as is already done for i.MX95 in commit
> > d8574ce57d76 ("PCI: imx6: Add external reference clock input mode
> > support").
> > 
> > Signed-off-by: Rudi Heitbaum <rudi at heitbaum.com>
> Sorry for the late reply.
> 
> The code changes look good to me.
> 
> However, I'm concerned about backward compatibility. This patch changes the
> default REF_CLK source from external oscillator to internal PLL when no
> "extref" clock is specified, which could break existing i.MX8MQ boards.
> 

This clocks is marked as 'optional' in the binding. So the driver should
continue to work with older DTs that don't specify this clock.

If this clock is truly required, then it should not be defined as optional in
the binding and the ABI break should be justified in both binding and driver
commit messages.

- Mani

-- 
மணிவண்ணன் சதாசிவம்