Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752625AbcL2KZo (ORCPT ); Thu, 29 Dec 2016 05:25:44 -0500 Received: from regular1.263xmail.com ([211.150.99.134]:36360 "EHLO regular1.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752380AbcL2KZm (ORCPT ); Thu, 29 Dec 2016 05:25:42 -0500 X-263anti-spam: KSV:0; X-MAIL-GRAY: 0 X-MAIL-DELIVERY: 1 X-KSVirus-check: 0 X-ABS-CHECKED: 4 X-RL-SENDER: randy.li@rock-chips.com X-FST-TO: xzy.xu@rock-chips.com X-SENDER-IP: 103.29.142.67 X-LOGIN-NAME: randy.li@rock-chips.com X-UNIQUE-TAG: <72a5d63080f62617c1abafbd734a7772> X-ATTACHMENT-NUM: 0 X-DNS-TYPE: 0 Subject: Re: [PATCH] Revert "mmc: dw_mmc-rockchip: add runtime PM support" To: Shawn Lin , linux-mmc@vger.kernel.org References: <1482939288-14950-1-git-send-email-ayaka@soulik.info> <7e37b8ad-39d0-3a94-f75a-2166ae2caf10@samsung.com> <7fb90b9e-75c3-0184-4f0a-d4113899b860@rock-chips.com> Cc: Jaehoon Chung , Randy Li , ulf.hansson@linaro.org, heiko@sntech.de, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, xzy.xu@rock-chips.com From: Randy Li Organization: Fuzhou Rockchip Message-ID: <04b667d9-2591-51ff-e024-047bbb6e17c3@rock-chips.com> Date: Thu, 29 Dec 2016 18:25:22 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <7fb90b9e-75c3-0184-4f0a-d4113899b860@rock-chips.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4939 Lines: 151 On 12/29/2016 03:25 PM, Shawn Lin wrote: > On 2016/12/29 15:13, Jaehoon Chung wrote: >> On 12/29/2016 12:02 PM, Jaehoon Chung wrote: >>> 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. >> >> To Shawn, >> >> Could you check this? If you have rk3288.. >> If it's not working fine, it needs to revert this patch until finding >> the problem. >> > > Hrmm.....as that patchset was tested based on rk3288 and rk3368, so I > need to know which board Randy are using now and could you share some Sorry, XZY has asked me about this in the morning and I answer him that I would give a feedback at home, so I didn't notice this mail. The board is Firefly reload. but the reporter told me that Firefly release also have the same problem. > log? > > I will have a look at it. > > >> Best Regards, >> Jaehoon Chung >> >>> >>> Best Regards, >>> Jaehoon Chung >>> >>>> >>>> Signed-off-by: Randy Li >>>> --- >>>> 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 >>>> #include >>>> #include >>>> -#include >>>> #include >>>> >>>> #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, >>>> }, >>>> }; >>>> >>>> >>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> >>> . >>> >> >> >> >> > > -- Randy Li The third produce department =========================================================================== This email message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message. [Fuzhou Rockchip Electronics, INC. China mainland] ===========================================================================