[PATCH] phy: freescale: imx8m-pcie: Handle IMX8_PCIE_REFCLK_PAD_UNUSED

[Hongxing Zhu]

> -----Original Message-----
> From: Marek Vasut <marex at denx.de>
> Sent: 2022年3月14日 18:53
> To: Hongxing Zhu <hongxing.zhu at nxp.com>; linux-phy at lists.infradead.org
> Cc: Fabio Estevam <festevam at denx.de>; Kishon Vijay Abraham I
> <kishon at ti.com>; Marcel Ziswiler <marcel.ziswiler at toradex.com>;
> dl-linux-imx <linux-imx at nxp.com>; Peng Fan <peng.fan at nxp.com>; Shawn
> Guo <shawnguo at kernel.org>; Vinod Koul <vkoul at kernel.org>;
> linux-arm-kernel at lists.infradead.org
> Subject: Re: [PATCH] phy: freescale: imx8m-pcie: Handle
> IMX8_PCIE_REFCLK_PAD_UNUSED
> 
> On 3/14/22 04:29, Hongxing Zhu wrote:
> 
> [...]
> 
> >>>> Signed-off-by: Marek Vasut <marex at denx.de>
> >>> Hi Marek:
> >>> Thanks for your patch.
> >>> I assume that you had tested this IMX8_PCIE_REFCLK_PAD_UNUSED
> option.
> >>
> >> Yes
> >>
> >>> The CLK_N/P pads are not connected on the tested board, right?
> >>
> >> They are connected from MX8MP SoC to MiniPCIe slot.
> >> There is no external clock source like on the EVK, the MX8MP internal
> >> PLL is the clock source.
> >>
> > Got that, thanks.
> 
> btw. if you plan to resend the MX8MP PCIe patches (assuming they didn't get
> applied yet), CC me, I can provide a Tested-by .

Sure, I would CC you from next review cycle.
It's great that you can help. Thanks.

> 
> >>> BTW, it's better to add the Fixes tag into the commit log.
> >>
> >> Do you think this should be considered a bugfix ?
> > Emm...
> > It's reasonable to consider it as new added.
> > Okay, I'm fine with it. Thanks again for your patch.
> >
> > Reviewed-by: Richard Zhu <hongxing.zhu at nxp.com>
> 
> Let's wait for others' opinion, I'm really unsure whether this is a
> Fixes: or not.
Okay.

Best Regards
Richard Zhu