From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753652AbdBVAzq (ORCPT ); Tue, 21 Feb 2017 19:55:46 -0500 Received: from lucky1.263xmail.com ([211.157.147.130]:54576 "EHLO lucky1.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753114AbdBVAzi (ORCPT ); Tue, 21 Feb 2017 19:55:38 -0500 X-263anti-spam: KSV:0; X-MAIL-GRAY: 1 X-MAIL-DELIVERY: 0 X-KSVirus-check: 0 X-ABS-CHECKED: 4 X-RL-SENDER: shawn.lin@rock-chips.com X-FST-TO: linux-kernel@vger.kernel.org X-SENDER-IP: 58.22.7.114 X-LOGIN-NAME: shawn.lin@rock-chips.com X-UNIQUE-TAG: <0172ab8987f7d85728168bcda66dff45> X-ATTACHMENT-NUM: 0 X-DNS-TYPE: 0 Subject: Re: [RFC PATCH 0/4] mmc: core: Provide CMD5 awake and partial_init support To: Ritesh Harjani , adrian.hunter@intel.com References: <1487577792-12510-1-git-send-email-riteshh@codeaurora.org> Cc: ulf.hansson@linaro.org, linux-mmc@vger.kernel.org, shawn.lin@rock-chips.com, devicetree@vger.kernel.org, andy.gross@linaro.org, linux-arm-msm@vger.kernel.org, georgi.djakov@linaro.org, alex.lemberg@sandisk.com, mateusz.nowak@intel.com, Yuliy.Izrailov@sandisk.com, asutoshd@codeaurora.org, david.griego@linaro.org, stummala@codeaurora.org, venkatg@codeaurora.org, pramod.gurav@linaro.org, jeremymc@redhat.com, linux-kernel@vger.kernel.org From: Shawn Lin Message-ID: Date: Wed, 22 Feb 2017 08:55:16 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <1487577792-12510-1-git-send-email-riteshh@codeaurora.org> Content-Type: text/plain; charset=gbk; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Ritesh, On 2017/2/20 16:03, Ritesh Harjani wrote: > As per JEDEC spec - CMD5 can be used to awake from sleep mode for emmc. > This patch series provide CMD5(awake) + mmc_partial_init support to resume > mmc card device. This is mainly to reduce the resume time. > For modem eMMC devices which support vccq to be 1v8, you patch seem to work fine but that didn't work for some eMMCs that only support 3v3 vccq, for instance, some older v4.41 eMMC devices. Also, the spec didn't clearly have a statement about the fact that the partial init should work if keeping vqmmc. So finally you should have to add a fallback method once the partial init doesn't work well. In the point, this policy is not just host specified, but also eMMC device specified. So I guess this is why you expose a new DT property here instead?! > This was tested on db410c (emmc with HS200 mode) and MS8996 (emmc with HS400ES) > based internal board. This patch reduced the resume time by ~50% on msm8996 > and ~11% on db410c. > > As of now this patch series provides a caps (MMC_CAP2_SLEEP_AWAKE) to enable this feature. > Since there is no dependency on host platform for this, we can enable this feature by > default as well. Thoughts? > > > Ritesh Harjani (4): > Documentation: mmc: add mmc-sleep-awake > mmc: core: add mmc-sleep-awake caps > mmc: mmc: add support for CMD5 awake > mmc: core: Implement mmc_partial_init during resume > > Documentation/devicetree/bindings/mmc/mmc.txt | 2 + > drivers/mmc/core/core.c | 13 +++ > drivers/mmc/core/core.h | 1 + > drivers/mmc/core/host.c | 2 + > drivers/mmc/core/mmc.c | 160 ++++++++++++++++++++++++-- > include/linux/mmc/card.h | 3 + > include/linux/mmc/host.h | 2 + > 7 files changed, 176 insertions(+), 7 deletions(-) > -- Best Regards Shawn Lin