[PATCH] phy: freescale: imx8qm-hsio: fix NULL pointer dereference

[Hongxing Zhu]

> -----Original Message-----
> From: Thomas Richard <thomas.richard at bootlin.com>
> Sent: 2026年1月15日 0:50
> To: Vinod Koul <vkoul at kernel.org>; Neil Armstrong
> <neil.armstrong at linaro.org>; Shawn Guo <shawnguo at kernel.org>; Sascha
> Hauer <s.hauer at pengutronix.de>; Pengutronix Kernel Team
> <kernel at pengutronix.de>; Fabio Estevam <festevam at gmail.com>; Frank Li
> <frank.li at nxp.com>; Hongxing Zhu <hongxing.zhu at nxp.com>
> Cc: Thomas Petazzoni <thomas.petazzoni at bootlin.com>;
> linux-phy at lists.infradead.org; imx at lists.linux.dev;
> linux-arm-kernel at lists.infradead.org; linux-kernel at vger.kernel.org; Thomas
> Richard <thomas.richard at bootlin.com>
> Subject: [PATCH] phy: freescale: imx8qm-hsio: fix NULL pointer dereference
> 
> [You don't often get email from thomas.richard at bootlin.com. Learn why this is
> important at https://aka.ms/LearnAboutSenderIdentification ]
> 
> During the probe the refclk_pad pointer is set to NULL if the
> 'fsl,refclk-pad-mode' property is not defined in the devicetree node. But in
> imx_hsio_configure_clk_pad() this pointer is unconditionally used which could
> result in a NULL pointer dereference. So check the pointer before to use it.
> 
> Fixes: 82c56b6dd24f ("phy: freescale: imx8qm-hsio: Add i.MX8QM HSIO PHY
> driver support")
> Signed-off-by: Thomas Richard <thomas.richard at bootlin.com>
Thanks for your patch.
Reviewed-by: Richard Zhu <hongxing.zhu at nxp.com>

Best Regards
Richard Zhu
> ---
>  drivers/phy/freescale/phy-fsl-imx8qm-hsio.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/phy/freescale/phy-fsl-imx8qm-hsio.c
> b/drivers/phy/freescale/phy-fsl-imx8qm-hsio.c
> index 977d21d753a5..279b8ac7822d 100644
> --- a/drivers/phy/freescale/phy-fsl-imx8qm-hsio.c
> +++ b/drivers/phy/freescale/phy-fsl-imx8qm-hsio.c
> @@ -251,7 +251,7 @@ static void imx_hsio_configure_clk_pad(struct phy
> *phy)
>         struct imx_hsio_lane *lane = phy_get_drvdata(phy);
>         struct imx_hsio_priv *priv = lane->priv;
> 
> -       if (strncmp(priv->refclk_pad, "output", 6) == 0) {
> +       if (priv->refclk_pad && strncmp(priv->refclk_pad, "output", 6)
> + == 0) {
>                 pll = true;
>                 regmap_update_bits(priv->misc, HSIO_CTRL0,
>                                    HSIO_IOB_A_0_TXOE |
> HSIO_IOB_A_0_M1M0_MASK,
> 
> ---
> base-commit: c537e12daeecaecdcd322c56a5f70659d2de7bde
> change-id:
> 20260114-phy-fsl-imx8qm-hsio-fix-null-pointer-dereference-67b110635f77
> 
> Best regards,
> --
> Thomas Richard <thomas.richard at bootlin.com>