Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1548080

Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support"

Show key headers only | View raw

Hi Randy,

On 12/29/2016 12:34 AM, Randy Li wrote:
> This reverts commit f90142683f04bcb0729bf0df67a5e29562b725b9.
> It is reported that making RK3288 can't boot from eMMC/MMC.

Could you explain in more detail?
As you mentioned, this patch is making that RK3288 can't boot..then why?
Good way should be that finds the main reason and fixes it.
Not just revert.

Best Regards,
Jaehoon Chung

> 
> Signed-off-by: Randy Li <ayaka@soulik.info>
> ---
>  drivers/mmc/host/dw_mmc-rockchip.c | 41 +++-----------------------------------
>  1 file changed, 3 insertions(+), 38 deletions(-)
> 
> diff --git a/drivers/mmc/host/dw_mmc-rockchip.c b/drivers/mmc/host/dw_mmc-rockchip.c
> index 9a46e46..3189234 100644
> --- a/drivers/mmc/host/dw_mmc-rockchip.c
> +++ b/drivers/mmc/host/dw_mmc-rockchip.c
> @@ -14,7 +14,6 @@
>  #include <linux/mmc/dw_mmc.h>
>  #include <linux/of_address.h>
>  #include <linux/mmc/slot-gpio.h>
> -#include <linux/pm_runtime.h>
>  #include <linux/slab.h>
>  
>  #include "dw_mmc.h"
> @@ -327,7 +326,6 @@ static int dw_mci_rockchip_probe(struct platform_device *pdev)
>  {
>  	const struct dw_mci_drv_data *drv_data;
>  	const struct of_device_id *match;
> -	int ret;
>  
>  	if (!pdev->dev.of_node)
>  		return -ENODEV;
> @@ -335,49 +333,16 @@ static int dw_mci_rockchip_probe(struct platform_device *pdev)
>  	match = of_match_node(dw_mci_rockchip_match, pdev->dev.of_node);
>  	drv_data = match->data;
>  
> -	pm_runtime_get_noresume(&pdev->dev);
> -	pm_runtime_set_active(&pdev->dev);
> -	pm_runtime_enable(&pdev->dev);
> -	pm_runtime_set_autosuspend_delay(&pdev->dev, 50);
> -	pm_runtime_use_autosuspend(&pdev->dev);
> -
> -	ret = dw_mci_pltfm_register(pdev, drv_data);
> -	if (ret) {
> -		pm_runtime_disable(&pdev->dev);
> -		pm_runtime_set_suspended(&pdev->dev);
> -		pm_runtime_put_noidle(&pdev->dev);
> -		return ret;
> -	}
> -
> -	pm_runtime_put_autosuspend(&pdev->dev);
> -
> -	return 0;
> +	return dw_mci_pltfm_register(pdev, drv_data);
>  }
>  
> -static int dw_mci_rockchip_remove(struct platform_device *pdev)
> -{
> -	pm_runtime_get_sync(&pdev->dev);
> -	pm_runtime_disable(&pdev->dev);
> -	pm_runtime_put_noidle(&pdev->dev);
> -
> -	return dw_mci_pltfm_remove(pdev);
> -}
> -
> -static const struct dev_pm_ops dw_mci_rockchip_dev_pm_ops = {
> -	SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
> -				pm_runtime_force_resume)
> -	SET_RUNTIME_PM_OPS(dw_mci_runtime_suspend,
> -			   dw_mci_runtime_resume,
> -			   NULL)
> -};
> -
>  static struct platform_driver dw_mci_rockchip_pltfm_driver = {
>  	.probe		= dw_mci_rockchip_probe,
> -	.remove		= dw_mci_rockchip_remove,
> +	.remove		= dw_mci_pltfm_remove,
>  	.driver		= {
>  		.name		= "dwmmc_rockchip",
>  		.of_match_table	= dw_mci_rockchip_match,
> -		.pm		= &dw_mci_rockchip_dev_pm_ops,
> +		.pm		= &dw_mci_pltfm_pmops,
>  	},
>  };
>  
> 

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" Randy Li <ayaka@soulik.info> - 2016-12-28 16:40 +0100
  Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" Jaehoon Chung <jh80.chung@samsung.com> - 2016-12-30 04:00 +0100
  Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" Randy Li <randy.li@rock-chips.com> - 2016-12-30 03:50 +0100
  Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" Jaehoon Chung <jh80.chung@samsung.com> - 2016-12-29 04:10 +0100
    Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" ayaka <ayaka@soulik.info> - 2016-12-29 16:20 +0100
      Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" Jaehoon Chung <jh80.chung@samsung.com> - 2016-12-30 03:40 +0100
    Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" Shawn Lin <shawn.lin@rock-chips.com> - 2016-12-29 08:30 +0100
      Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" Randy Li <randy.li@rock-chips.com> - 2016-12-29 11:30 +0100
        Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" ayaka <ayaka@soulik.info> - 2016-12-29 14:10 +0100
          Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" Jaehoon Chung <jh80.chung@gmail.com> - 2016-12-29 15:10 +0100
    Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" Jaehoon Chung <jh80.chung@samsung.com> - 2016-12-29 08:50 +0100

csiph-web