Hi Yuvaraj, On 1 October 2013 12:03, Yuvaraj Kumar C D <yuvaraj.cd@xxxxxxxxx> wrote: > Exynos5250 contains one Synopsys AHCI SATA controller.The avalaible > ahci_platform driver is not sufficient to handle the AHCI PHY and PHY > clock related initialization. > +err_out: > + platform_set_drvdata(pdev, NULL); This is not necessary. Driver core clears it upon detach or failure. > + platform_device_put(ahci_pdev); > + return ret; > + } > + > + return 0; > +} > + > +static int exynos_ahci_remove(struct platform_device *pdev) > +{ > + struct exynos_ahci_priv *priv = platform_get_drvdata(pdev); > + struct platform_device *ahci_pdev = priv->ahci_pdev; > + > + platform_device_unregister(ahci_pdev); > + platform_set_drvdata(pdev, NULL); ditto -- With warm regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html