Re: [PATCH v2 04/10] phy: qcom-ufs-qmp-xx: Discard remove callback for drivers.

From: Kishon Vijay Abraham I
Date: Wed Oct 26 2016 - 16:21:01 EST




On Tuesday 18 October 2016 07:58 PM, Vivek Gautam wrote:
> remove() callback does a phy_exit() only and nothing else now.

remove callback calls a phy_power_off() ;-)

-Kishon

> The phy_exit() over the generic phy is called from the phy
> consumer, and phy provider driver should not explicitly need to
> call any phy_exit().
> So discard the remove callback for qcom-ufs phy platform drivers.
>
> Signed-off-by: Vivek Gautam <vivek.gautam@xxxxxxxxxxxxxx>
> Reviewed-by: Subhash Jadavani <subhashj@xxxxxxxxxxxxxx>
> ---
>
> No change since v1.
>
> drivers/phy/phy-qcom-ufs-qmp-14nm.c | 16 ----------------
> drivers/phy/phy-qcom-ufs-qmp-20nm.c | 16 ----------------
> drivers/phy/phy-qcom-ufs.c | 9 ---------
> 3 files changed, 41 deletions(-)
>
> diff --git a/drivers/phy/phy-qcom-ufs-qmp-14nm.c b/drivers/phy/phy-qcom-ufs-qmp-14nm.c
> index 6ee5149..a305016 100644
> --- a/drivers/phy/phy-qcom-ufs-qmp-14nm.c
> +++ b/drivers/phy/phy-qcom-ufs-qmp-14nm.c
> @@ -163,21 +163,6 @@ out:
> return err;
> }
>
> -static int ufs_qcom_phy_qmp_14nm_remove(struct platform_device *pdev)
> -{
> - struct device *dev = &pdev->dev;
> - struct phy *generic_phy = to_phy(dev);
> - struct ufs_qcom_phy *ufs_qcom_phy = get_ufs_qcom_phy(generic_phy);
> - int err = 0;
> -
> - err = ufs_qcom_phy_remove(generic_phy, ufs_qcom_phy);
> - if (err)
> - dev_err(dev, "%s: ufs_qcom_phy_remove failed = %d\n",
> - __func__, err);
> -
> - return err;
> -}
> -
> static const struct of_device_id ufs_qcom_phy_qmp_14nm_of_match[] = {
> {.compatible = "qcom,ufs-phy-qmp-14nm"},
> {},
> @@ -186,7 +171,6 @@ MODULE_DEVICE_TABLE(of, ufs_qcom_phy_qmp_14nm_of_match);
>
> static struct platform_driver ufs_qcom_phy_qmp_14nm_driver = {
> .probe = ufs_qcom_phy_qmp_14nm_probe,
> - .remove = ufs_qcom_phy_qmp_14nm_remove,
> .driver = {
> .of_match_table = ufs_qcom_phy_qmp_14nm_of_match,
> .name = "ufs_qcom_phy_qmp_14nm",
> diff --git a/drivers/phy/phy-qcom-ufs-qmp-20nm.c b/drivers/phy/phy-qcom-ufs-qmp-20nm.c
> index 770087a..2db1fbb 100644
> --- a/drivers/phy/phy-qcom-ufs-qmp-20nm.c
> +++ b/drivers/phy/phy-qcom-ufs-qmp-20nm.c
> @@ -219,21 +219,6 @@ out:
> return err;
> }
>
> -static int ufs_qcom_phy_qmp_20nm_remove(struct platform_device *pdev)
> -{
> - struct device *dev = &pdev->dev;
> - struct phy *generic_phy = to_phy(dev);
> - struct ufs_qcom_phy *ufs_qcom_phy = get_ufs_qcom_phy(generic_phy);
> - int err = 0;
> -
> - err = ufs_qcom_phy_remove(generic_phy, ufs_qcom_phy);
> - if (err)
> - dev_err(dev, "%s: ufs_qcom_phy_remove failed = %d\n",
> - __func__, err);
> -
> - return err;
> -}
> -
> static const struct of_device_id ufs_qcom_phy_qmp_20nm_of_match[] = {
> {.compatible = "qcom,ufs-phy-qmp-20nm"},
> {},
> @@ -242,7 +227,6 @@ MODULE_DEVICE_TABLE(of, ufs_qcom_phy_qmp_20nm_of_match);
>
> static struct platform_driver ufs_qcom_phy_qmp_20nm_driver = {
> .probe = ufs_qcom_phy_qmp_20nm_probe,
> - .remove = ufs_qcom_phy_qmp_20nm_remove,
> .driver = {
> .of_match_table = ufs_qcom_phy_qmp_20nm_of_match,
> .name = "ufs_qcom_phy_qmp_20nm",
> diff --git a/drivers/phy/phy-qcom-ufs.c b/drivers/phy/phy-qcom-ufs.c
> index f639a7c..b8f9286 100644
> --- a/drivers/phy/phy-qcom-ufs.c
> +++ b/drivers/phy/phy-qcom-ufs.c
> @@ -645,15 +645,6 @@ int ufs_qcom_phy_calibrate_phy(struct phy *generic_phy, bool is_rate_B)
> }
> EXPORT_SYMBOL_GPL(ufs_qcom_phy_calibrate_phy);
>
> -int ufs_qcom_phy_remove(struct phy *generic_phy,
> - struct ufs_qcom_phy *ufs_qcom_phy)
> -{
> - phy_power_off(generic_phy);
> -
> - return 0;
> -}
> -EXPORT_SYMBOL_GPL(ufs_qcom_phy_remove);
> -
> int ufs_qcom_phy_exit(struct phy *generic_phy)
> {
> struct ufs_qcom_phy *ufs_qcom_phy = get_ufs_qcom_phy(generic_phy);
>