Hi Chris > +static int usbhs_rza2_hardware_exit(struct platform_device *pdev) > +{ > + struct usbhs_priv *priv = usbhs_pdev_to_priv(pdev); > + > + if (priv->phy) { > + phy_put(priv->phy); > + priv->phy = NULL; > + } > + > + return 0; > +} phy_put() will do nothing if priv->phy was NULL. We can remove if() here ? Thank you for your help !! Best regards --- Kuninori Morimoto