[{"id":1773683,"web_url":"http://patchwork.ozlabs.org/comment/1773683/","msgid":"","list_archive_url":null,"date":"2017-09-22T14:42:45","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","submitter":{"id":8006,"url":"http://patchwork.ozlabs.org/api/people/8006/","name":"Jaehoon Chung","email":"jh80.chung@samsung.com"},"content":"Dear JJ,\n\nOn 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\n> This series brings support for HS200 and UHS modes to the mmc core.\n> It has been tested with the hsmmc driver on several platforms (DRA7,\n> AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some modifications\n> are required in the host driver to take advantage of this (voltage switching,\n> tuning). The changes to the host driver will be posted a another series as this\n> one is already long enough. \n> \n> The series starts with a small refactoring of th sd/mmc startup. The first 5 commits\n> are mostly moving code around with little or no functionnal change.\n> \n> Then the notion of \"mode\" is introduced. Until now, this information wasn't\n> kept in struct mmc. Only the clock and a flag for ddr was kept. Later the mode\n> information will be used to select the clock frequency, the ddr flag and the\n> tuning procedure. It will be also be check against the host capabilities.\n> \n> Then comes the big refactoring job in:\n> \"mmc: refactor MMC startup to make it easier to support new modes\" and\n> \"mmc: refactor SD startup to make it easier to support new modes\"\n> Since the number of modes is increasing, it makes sense to try them in a more\n> organized way. those commits use a list of supported modes and iterate through\n> them to find the best working one. It also allows to switch more easilly from\n> one mode to another (switching from HS200 to DDR52 to access boot partitions for example)\n> \n> Then there are a couple of new callback added to:\n> - send the initialization stream (74 clock cycles)\n> - wait while the card is busy (used during UHS voltage switching). checking is\n> done on dat0.\n> - select the IO voltage\n> \n> Then Power cycle is added. Without power cycle, if a UHS card fails to enumerate in\n> UHS mode, it can't fall back to high speed mode and card enumeration will fail.\n> Also in case of a reset (as opposed to a power on), it ensures that the SDCard\n> is in clean state before re-doing the initialization.\n> \n> And finally the last commits add the support for HS200 and UHS.\n> \n> With this in place and the required changes in the HSMMC host driver (including\n> DMA), we observe significant improvements in the performances on a DRA72 evm:\n> eMMC HS200: 124 MB/s\n> eMMC DDR52: 78 MB/s\n> sd SDR104: 71 MB/s\n> sd SDR50: 44 MB/s\n\nI have one question. \n\nDid you test about accessing Boot partition of eMMC?\nAs i know, boot partition doesn't support the HS200 mode.\n\nWhen i checked your patches, there is no the function of mode changing for accessing boot partition.\nIf i missed it, let me know, plz.\n\nBest Regards,\nJaehoon Chung\n\n> \n> cheers,\n> \n> Jean-Jacques\n> \n> changes since v1:\n> Changes take in account all the comment made during the first round of review.\n> But it also include a rework of the handling of the voltage supplies and 2 new\n> helper functions that can be used by the host driver during tuning and dts\n> parsing.\n> \n> Here is a detailed list:\n> * rebase on u-boot/master.\n> * add the description for the new functions/members\n> * fix the typos found in the reviews\n> * limit the new features to DM_MMC only\n> * fix the new checkpatch warnings/errors (is it now more strict ?)\n> * get the voltage regulators from dts (main and IO-lines)\n> * remove the set_vdd callback and do the job in the mmc core\n> * add a send_init_stream callback to perform the 74 clock cycle\n> sequence after power up\n> * do a full power cycle (if possible) before the initialization. This ensures\n> that the sd/MMC is in valid knwon state. Also it allows to check if the power\n> cycling works as expected.\n> * disable the UHS modes is power cycling is not available/working. This ensures\n> that we won't be stuck if the UHS initialization fails.\n> * select the appropriate signal voltage when selecting a mode (MMC only) \n> * add a helper function to parse the generic dt bindings\n> * add a helper function to send the tuning command\n> \n> Jean-Jacques Hiblot (17):\n> mmc: dm: get the IO-line and main voltage regulators from the dts\n> mmc: split mmc_startup()\n> mmc: move the MMC startup for version above v4.0 in a separate\n> function\n> mmc: make ext_csd part of struct mmc\n> mmc: add a function to read and test the ext csd (mmc >= 4)\n> mmc: introduce mmc modes\n> mmc: Add a function to dump the mmc capabilities\n> mmc: use mmc modes to select the correct bus speed\n> cmd: mmc: display the mode name and current bus speed in the mmc info\n> mmc: refactor SD startup to make it easier to support new modes\n> mmc: refactor MMC startup to make it easier to support new modes\n> mmc: Add a new callback function to perform the 74 clocks cycle\n> sequence\n> mmc: Add support for UHS modes\n> mmc: disable UHS modes if Vcc cannot be switched on and off\n> mmc: Change mode when switching to a boot partition\n> mmc: use the right voltage level for MMC DDR and HS200 modes\n> mmc: add a library function to send tuning command\n> \n> Kishon Vijay Abraham I (9):\n> mmc: make mmc_set_ios() return status\n> mmc: Enable signal voltage to be selected from mmc core\n> mmc: add power cyle support in mmc core\n> mmc: add a new mmc parameter to disable mmc clock\n> mmc: disable the mmc clock during power off\n> mmc: Add a execute_tuning() callback to the mmc operations.\n> mmc: add HS200 support in MMC core\n> mmc: Retry some MMC cmds on failure\n> dm: mmc: Add a library function to parse generic dt binding\n> \n> cmd/mmc.c | 3 +-\n> drivers/mmc/Kconfig | 23 +\n> drivers/mmc/fsl_esdhc.c | 2 +-\n> drivers/mmc/mmc-uclass.c | 87 +++\n> drivers/mmc/mmc.c | 1511 ++++++++++++++++++++++++++++++++++------------\n> include/mmc.h | 176 +++++-\n> 6 files changed, 1417 insertions(+), 385 deletions(-)\n>","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xzGRw0MsMz9t49\n\tfor ;\n\tSat, 23 Sep 2017 00:43:07 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 640EBC22095; Fri, 22 Sep 2017 14:43:01 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 0A4F6C2206C;\n\tFri, 22 Sep 2017 14:42:59 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid E157AC2206C; Fri, 22 Sep 2017 14:42:57 +0000 (UTC)","from mailout3.samsung.com (mailout3.samsung.com [203.254.224.33])\n\tby lists.denx.de (Postfix) with ESMTPS id A1D71C21F55\n\tfor ; Fri, 22 Sep 2017 14:42:56 +0000 (UTC)","from epcas1p3.samsung.com (unknown [182.195.41.47])\n\tby mailout3.samsung.com (KnoxPortal) with ESMTP id\n\t20170922144247epoutp03f0953394ffdd2dd02468df7f93873e31~mth5lvYTG1546315463epoutp03U;\n\tFri, 22 Sep 2017 14:42:47 +0000 (GMT)","from epsmges2p1.samsung.com (unknown [182.195.40.68]) by\n\tepcas1p3.samsung.com (KnoxPortal) with ESMTP id\n\t20170922144246epcas1p35b943b109f44f2b02ef6a9c134018345~mth5W7kBL1303213032epcas1p3V;\n\tFri, 22 Sep 2017 14:42:46 +0000 (GMT)","from epcas2p1.samsung.com ( [182.195.41.53]) by\n\tepsmges2p1.samsung.com (Symantec Messaging Gateway) with SMTP id\n\tE3.C9.10950.66125C95; Fri, 22 Sep 2017 23:42:46 +0900 (KST)","from epsmgms2p2new.samsung.com (unknown [182.195.42.143]) by\n\tepcas2p2.samsung.com (KnoxPortal) with ESMTP id\n\t20170922144246epcas2p2fcdd3415472f178d158893d662422d09~mth4yMfQa2614726147epcas2p2j;\n\tFri, 22 Sep 2017 14:42:46 +0000 (GMT)","from epmmp2 ( [203.254.227.17]) by epsmgms2p2new.samsung.com\n\t(Symantec Messaging Gateway) with SMTP id D4.1F.10338.56125C95;\n\tFri, 22 Sep 2017 23:42:46 +0900 (KST)","from [10.113.62.216] by mmp2.samsung.com (Oracle Communications\n\tMessaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTPA id\n\t<0OWO00D04RJ9PG10@mmp2.samsung.com>;\n\tFri, 22 Sep 2017 23:42:45 +0900 (KST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-5.0 required=5.0 tests=RCVD_IN_DNSWL_HI,\n\tRCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tSPF_HELO_PASS autolearn=unavailable\n\tautolearn_force=no version=3.4.0","X-AuditID":"b6c32a45-f79466d000002ac6-aa-59c521667b81","To":"Jean-Jacques Hiblot , trini@konsulko.com,\n\tkishon@ti.com, sjg@chromium.org","From":"Jaehoon Chung ","Message-id":"","Date":"Fri, 22 Sep 2017 23:42:45 +0900","User-Agent":"Mozilla/5.0 (X11; Linux i686; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-version":"1.0","In-reply-to":"<1506004213-22620-1-git-send-email-jjhiblot@ti.com>","Content-language":"en-US","X-Brightmail-Tracker":["H4sIAAAAAAAAA+NgFprIJsWRmVeSWpSXmKPExsWy7bCmqW6a4tFIg+ZTehYtJycxWVx42sNm\n\tcXTPPWaLb1u2MVpMnbSZ3eLt3k52BzaP2Q0XWTx2zrrL7vHqwCp2j7N3djB6HL+xnSmANSrV\n\tJiM1MSW1SCE1Lzk/JTMv3VbJOzjeOd7UzMBQ19DSwlxJIS8xN9VWycUnQNctMwfoBCWFssSc\n\tUqBQQGJxsZK+nU1RfmlJqkJGfnGJrVK0oaGRnqGBuZ6RkZGeiXGslZEpUElCasa9k5vZCjbr\n\tVWx49Z6pgfGGchcjJ4eEgInExTXLmCBsMYkL99azdTFycQgJ7GCU6Ny9iR3C+c4o8fbqdRaY\n\tjv49e6CqNjBKrJzRwwjh3GOUOHHkCitIlbCAm8Sty4+BbA4OEYEEid7WUpAws4C+xPPTa5hB\n\tbDYBHYnt344zgZTwCthJ3PxXDhJmEVCVuNV6lQ3EFhUIk1j56xUjiM0rICjxY/I9sBs4BRwk\n\tTi9byQwxUlPixZdJLBC2uERz600oW15i85q3zCCnSQicYZNY93sd1AMuEqemnWOEsIUlXh3f\n\twg5hS0s8W7WREaKhnVHi1o99bBBOB6PEwZ97WSGqjCVOdTUyQazgk+g4/Jcd5AMJAV6JjjYh\n\tiBIPiTkPVkPD1FHifCfIk6AAmswocX71KbYJjPKzkHw0C8kXs5B8MQvJFwsYWVYxiqUWFOem\n\tpxYbFRjqFSfmFpfmpesl5+duYgQnRy3XHYwzzvkcYhTgYFTi4TU4eDhSiDWxrLgy9xCjBAez\n\tkgjvJvmjkUK8KYmVValF+fFFpTmpxYcYTYEhPpFZSjQ5H5i480riDU0sDUzMzIzMzSyASU+c\n\tt37btQghgfTEktTs1NSC1CKYPiYOTqkGRlUf+UX31dvOVhrOP+C95WtV0Wo/o4ncX7TPX7Zr\n\teju7rumLcpLZ0rDcJpuPe4/GyVYf8A9L4zfx2tbqLrDObP6H7bztCWoyISLKihc5z09MX3/9\n\tQ2fLu/cthdsvuBRESrGcXSvDsqyE+3HOpwPhGnMesQoHuWQlvm8r046bFC77NuuZql+YEktx\n\tRqKhFnNRcSIAqnIXOqQDAAA=","H4sIAAAAAAAAA+NgFnrGLMWRmVeSWpSXmKPExsVy+t9jQd00xaORBg8WsVu0nJzEZHHhaQ+b\n\txdE995gtvm3ZxmgxddJmdou3ezvZHdg8ZjdcZPHYOesuu8erA6vYPc7e2cHocfzGdqYA1igu\n\tm5TUnMyy1CJ9uwSujHsnN7MVbNar2PDqPVMD4w3lLkZODgkBE4n+PXvYuhi5OIQE1jFKXJm/\n\tEsp5wChx9tg2ZpAqYQE3iVuXH7OC2CICCRL/t1xhA7GZBfQlnp9ewwzRMJVR4t2/z4wgCTYB\n\tHYnt344zdTFycPAK2Enc/FcOEmYRUJW41XoVrFdUIEyiv/kv2HxeAUGJH5PvsYDYnAIOEqeX\n\trWQGaWUWUJeYMiUXYpW4RHPrTRYIW15i85q3zBMYBWYh6Z6F0DELSccsJB0LGFlWMUqmFhTn\n\tpucWGxUY5aWW6xUn5haX5qXrJefnbmIEhv62w1r9OxgfL4k/xCjAwajEw2tw8HCkEGtiWXFl\n\t7iFGCQ5mJRHeTfJHI4V4UxIrq1KL8uOLSnNSiw8xSnOwKInzZvbNiBQSSE8sSc1OTS1ILYLJ\n\tMnFwSjUwCks2Cx3x/Lv+2WKm2k2vl8ztXiMVbXl6392gy+xxVdWbFzVXxupl3d11Xc8sXrAu\n\t9F3Xhit1Fz5s0smpk0sNW/xmTdhKPT7uUhd1/so7ezyjNjswXJi187zBrI1zpFVWhqZrPLVd\n\t9WZK1rzjbWFprGzKhibb3igvluJRtzuj/0H47717dxoZlViKMxINtZiLihMB1O6A0XkCAAA="],"X-CMS-MailID":"20170922144246epcas2p2fcdd3415472f178d158893d662422d09","X-Msg-Generator":"CA","X-Sender-IP":"182.195.42.143","X-Local-Sender":"=?utf-8?b?7KCV7J6s7ZuIG1RpemVuIFBsYXRmb3JtIExhYihTL1c=?=\n\t=?utf-8?b?7IS87YSwKRvsgrzshLHsoITsnpAbU2VuaW9yIEVuZ2luZWVy?=","X-Global-Sender":"=?utf-8?q?Jaehoon_Chung=1BTizen_Platform_Lab=2E=1BSamsun?=\n\t=?utf-8?q?g_Electronics=1BSenior_Engineer?=","X-Sender-Code":"=?utf-8?q?C10=1BTELE=1BC10V8111?=","CMS-TYPE":"102P","DLP-Filter":"Pass","X-CFilter-Loop":"Reflected","X-CMS-RootMailID":"20170921143028epcas3p3fce32811279a7e0bf9941e4bbc621ddd","X-RootMTR":"20170921143028epcas3p3fce32811279a7e0bf9941e4bbc621ddd","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1778215,"web_url":"http://patchwork.ozlabs.org/comment/1778215/","msgid":"<969b6bb5-9ded-87e8-d653-b39acc5a6285@ti.com>","list_archive_url":null,"date":"2017-10-02T09:15:28","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","submitter":{"id":70508,"url":"http://patchwork.ozlabs.org/api/people/70508/","name":"Jean-Jacques Hiblot","email":"jjhiblot@ti.com"},"content":"On 22/09/2017 16:42, Jaehoon Chung wrote:\n> Dear JJ,\n>\n> On 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\n>> This series brings support for HS200 and UHS modes to the mmc core.\n>> It has been tested with the hsmmc driver on several platforms (DRA7,\n>> AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some modifications\n>> are required in the host driver to take advantage of this (voltage switching,\n>> tuning). The changes to the host driver will be posted a another series as this\n>> one is already long enough.\n>>\n>> The series starts with a small refactoring of th sd/mmc startup. The first 5 commits\n>> are mostly moving code around with little or no functionnal change.\n>>\n>> Then the notion of \"mode\" is introduced. Until now, this information wasn't\n>> kept in struct mmc. Only the clock and a flag for ddr was kept. Later the mode\n>> information will be used to select the clock frequency, the ddr flag and the\n>> tuning procedure. It will be also be check against the host capabilities.\n>>\n>> Then comes the big refactoring job in:\n>> \"mmc: refactor MMC startup to make it easier to support new modes\" and\n>> \"mmc: refactor SD startup to make it easier to support new modes\"\n>> Since the number of modes is increasing, it makes sense to try them in a more\n>> organized way. those commits use a list of supported modes and iterate through\n>> them to find the best working one. It also allows to switch more easilly from\n>> one mode to another (switching from HS200 to DDR52 to access boot partitions for example)\n>>\n>> Then there are a couple of new callback added to:\n>> - send the initialization stream (74 clock cycles)\n>> - wait while the card is busy (used during UHS voltage switching). checking is\n>> done on dat0.\n>> - select the IO voltage\n>>\n>> Then Power cycle is added. Without power cycle, if a UHS card fails to enumerate in\n>> UHS mode, it can't fall back to high speed mode and card enumeration will fail.\n>> Also in case of a reset (as opposed to a power on), it ensures that the SDCard\n>> is in clean state before re-doing the initialization.\n>>\n>> And finally the last commits add the support for HS200 and UHS.\n>>\n>> With this in place and the required changes in the HSMMC host driver (including\n>> DMA), we observe significant improvements in the performances on a DRA72 evm:\n>> eMMC HS200: 124 MB/s\n>> eMMC DDR52: 78 MB/s\n>> sd SDR104: 71 MB/s\n>> sd SDR50: 44 MB/s\n> I have one question.\n>\n> Did you test about accessing Boot partition of eMMC?\n\n> As i know, boot partition doesn't support the HS200 mode.\n>\n> When i checked your patches, there is no the function of mode changing for accessing boot partition.\n> If i missed it, let me know, plz.\nYes I did test the boot partitions. HS200 is not used in this case.\nThe key function is mmc_boot_part_access_chk(). It checks if the current \nmode is supported by the partition. If not, it'll reselect a new mode.\n\nJean-Jacques\n\n\n>\n> Best Regards,\n> Jaehoon Chung\n>\n>> cheers,\n>>\n>> Jean-Jacques\n>>\n>> changes since v1:\n>> Changes take in account all the comment made during the first round of review.\n>> But it also include a rework of the handling of the voltage supplies and 2 new\n>> helper functions that can be used by the host driver during tuning and dts\n>> parsing.\n>>\n>> Here is a detailed list:\n>> * rebase on u-boot/master.\n>> * add the description for the new functions/members\n>> * fix the typos found in the reviews\n>> * limit the new features to DM_MMC only\n>> * fix the new checkpatch warnings/errors (is it now more strict ?)\n>> * get the voltage regulators from dts (main and IO-lines)\n>> * remove the set_vdd callback and do the job in the mmc core\n>> * add a send_init_stream callback to perform the 74 clock cycle\n>> sequence after power up\n>> * do a full power cycle (if possible) before the initialization. This ensures\n>> that the sd/MMC is in valid knwon state. Also it allows to check if the power\n>> cycling works as expected.\n>> * disable the UHS modes is power cycling is not available/working. This ensures\n>> that we won't be stuck if the UHS initialization fails.\n>> * select the appropriate signal voltage when selecting a mode (MMC only)\n>> * add a helper function to parse the generic dt bindings\n>> * add a helper function to send the tuning command\n>> \n>> Jean-Jacques Hiblot (17):\n>> mmc: dm: get the IO-line and main voltage regulators from the dts\n>> mmc: split mmc_startup()\n>> mmc: move the MMC startup for version above v4.0 in a separate\n>> function\n>> mmc: make ext_csd part of struct mmc\n>> mmc: add a function to read and test the ext csd (mmc >= 4)\n>> mmc: introduce mmc modes\n>> mmc: Add a function to dump the mmc capabilities\n>> mmc: use mmc modes to select the correct bus speed\n>> cmd: mmc: display the mode name and current bus speed in the mmc info\n>> mmc: refactor SD startup to make it easier to support new modes\n>> mmc: refactor MMC startup to make it easier to support new modes\n>> mmc: Add a new callback function to perform the 74 clocks cycle\n>> sequence\n>> mmc: Add support for UHS modes\n>> mmc: disable UHS modes if Vcc cannot be switched on and off\n>> mmc: Change mode when switching to a boot partition\n>> mmc: use the right voltage level for MMC DDR and HS200 modes\n>> mmc: add a library function to send tuning command\n>>\n>> Kishon Vijay Abraham I (9):\n>> mmc: make mmc_set_ios() return status\n>> mmc: Enable signal voltage to be selected from mmc core\n>> mmc: add power cyle support in mmc core\n>> mmc: add a new mmc parameter to disable mmc clock\n>> mmc: disable the mmc clock during power off\n>> mmc: Add a execute_tuning() callback to the mmc operations.\n>> mmc: add HS200 support in MMC core\n>> mmc: Retry some MMC cmds on failure\n>> dm: mmc: Add a library function to parse generic dt binding\n>>\n>> cmd/mmc.c | 3 +-\n>> drivers/mmc/Kconfig | 23 +\n>> drivers/mmc/fsl_esdhc.c | 2 +-\n>> drivers/mmc/mmc-uclass.c | 87 +++\n>> drivers/mmc/mmc.c | 1511 ++++++++++++++++++++++++++++++++++------------\n>> include/mmc.h | 176 +++++-\n>> 6 files changed, 1417 insertions(+), 385 deletions(-)\n>>\n>","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ti.com header.i=@ti.com header.b=\"mOQOmBIB\";\n\tdkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y5Gkj4R72z9sDB\n\tfor ;\n\tMon, 2 Oct 2017 20:16:45 +1100 (AEDT)","by lists.denx.de (Postfix, from userid 105)\n\tid 6728AC21EB4; Mon, 2 Oct 2017 09:16:44 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 58313C21CB3;\n\tMon, 2 Oct 2017 09:16:40 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 01AE5C21CB3; Mon, 2 Oct 2017 09:16:38 +0000 (UTC)","from fllnx209.ext.ti.com (fllnx209.ext.ti.com [198.47.19.16])\n\tby lists.denx.de (Postfix) with ESMTPS id 56B3AC21C41\n\tfor ; Mon, 2 Oct 2017 09:16:38 +0000 (UTC)","from dlelxv90.itg.ti.com ([172.17.2.17])\n\tby fllnx209.ext.ti.com (8.15.1/8.15.1) with ESMTP id v929FZ3E017107; \n\tMon, 2 Oct 2017 04:15:35 -0500","from DFLE100.ent.ti.com (dfle100.ent.ti.com [10.64.6.21])\n\tby dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id v929FUMZ007823; \n\tMon, 2 Oct 2017 04:15:30 -0500","from DFLE109.ent.ti.com (10.64.6.30) by DFLE100.ent.ti.com\n\t(10.64.6.21) with Microsoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.845.34;\n\tMon, 2 Oct 2017 04:15:30 -0500","from dlep33.itg.ti.com (157.170.170.75) by DFLE109.ent.ti.com\n\t(10.64.6.30) with Microsoft SMTP Server (version=TLS1_0,\n\tcipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.845.34 via Frontend\n\tTransport; Mon, 2 Oct 2017 04:15:30 -0500","from [172.22.129.181] (ileax41-snat.itg.ti.com [10.172.224.153])\n\tby dlep33.itg.ti.com (8.14.3/8.13.8) with ESMTP id v929FS7B020437;\n\tMon, 2 Oct 2017 04:15:29 -0500"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=T_DKIM_INVALID\n\tautolearn=unavailable autolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ti.com;\n\ts=ti-com-17Q1; t=1506935735;\n\tbh=e6iX5FbUxOmZvMMGMz8DhQ9ieqDvwy4/EXO17PDp5tg=;\n\th=Subject:To:CC:References:From:Date:In-Reply-To;\n\tb=mOQOmBIBxh1YG2DljYbx/Hz0+FauyQcLIHx6KjIWpvu8mOU/0bdYv1mVKg8d95r4U\n\tCeq8yvBRghw/vNMvhpho4Es7oG621ptW9rlIESgmkBOGvTggogKX/swQdmI6diGWFK\n\tKq/K2YpE0NHJxxkTbTybcjr/59EC4eH1AM4xEXHw=","To":"Jaehoon Chung , ,\n\t, ","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>\n\t","From":"Jean-Jacques Hiblot ","Message-ID":"<969b6bb5-9ded-87e8-d653-b39acc5a6285@ti.com>","Date":"Mon, 2 Oct 2017 11:15:28 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","X-EXCLAIMER-MD-CONFIG":"e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1786848,"web_url":"http://patchwork.ozlabs.org/comment/1786848/","msgid":"<79cd1066-41e3-17e8-1f3f-460f765056f8@gmail.com>","list_archive_url":null,"date":"2017-10-14T14:51:32","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","submitter":{"id":1124,"url":"http://patchwork.ozlabs.org/api/people/1124/","name":"Marek Vasut","email":"marek.vasut@gmail.com"},"content":"On 10/02/2017 11:15 AM, Jean-Jacques Hiblot wrote:\n> \n> \n> On 22/09/2017 16:42, Jaehoon Chung wrote:\n>> Dear JJ,\n>>\n>> On 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\n>>> This series brings support for HS200 and UHS modes to the mmc core.\n>>> It has been tested with the hsmmc driver on several platforms (DRA7,\n>>> AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some\n>>> modifications\n>>> are required in the host driver to take advantage of this (voltage\n>>> switching,\n>>> tuning). The changes to the host driver will be posted a another\n>>> series as this\n>>> one is already long enough.\n>>>\n>>> The series starts with a small refactoring of th sd/mmc startup. The\n>>> first 5 commits\n>>> are mostly moving code around with little or no functionnal change.\n>>>\n>>> Then the notion of \"mode\" is introduced. Until now, this information\n>>> wasn't\n>>> kept in struct mmc. Only the clock and a flag for ddr was kept. Later\n>>> the mode\n>>> information will be used to select the clock frequency, the ddr flag\n>>> and the\n>>> tuning procedure. It will be also be check against the host\n>>> capabilities.\n>>>\n>>> Then comes the big refactoring job in:\n>>> \"mmc: refactor MMC startup to make it easier to support new modes\" and\n>>> \"mmc: refactor SD startup to make it easier to support new modes\"\n>>> Since the number of modes is increasing, it makes sense to try them\n>>> in a more\n>>> organized way. those commits use a list of supported modes and\n>>> iterate through\n>>> them to find the best working one. It also allows to switch more\n>>> easilly from\n>>> one mode to another (switching from HS200 to DDR52 to access boot\n>>> partitions for example)\n>>>\n>>> Then there are a couple of new callback added to:\n>>> - send the initialization stream (74 clock cycles)\n>>> - wait while the card is busy (used during UHS voltage switching).\n>>> checking is\n>>>    done on dat0.\n>>> - select the IO voltage\n>>>\n>>> Then Power cycle is added. Without power cycle, if a UHS card fails\n>>> to enumerate in\n>>> UHS mode, it can't fall back to high speed mode and card enumeration\n>>> will fail.\n>>> Also in case of a reset (as opposed to a power on), it ensures that\n>>> the SDCard\n>>> is in clean state before re-doing the initialization.\n>>>\n>>> And finally the last commits add the support for HS200 and UHS.\n>>>\n>>> With this in place and the required changes in the HSMMC host driver\n>>> (including\n>>> DMA), we observe significant improvements in the performances on a\n>>> DRA72 evm:\n>>> eMMC HS200: 124 MB/s\n>>> eMMC DDR52: 78 MB/s\n>>> sd   SDR104: 71 MB/s\n>>> sd   SDR50: 44 MB/s\n>> I have one question.\n>>\n>> Did you test about accessing Boot partition of eMMC?\n> \n>> As i know, boot partition doesn't support the HS200 mode.\n>>\n>> When i checked your patches, there is no the function of mode changing\n>> for accessing boot partition.\n>> If i missed it, let me know, plz.\n> Yes I did test the boot partitions. HS200 is not used in this case.\n> The key function is mmc_boot_part_access_chk(). It checks if the current\n> mode is supported by the partition. If not, it'll reselect a new mode.\n\nIs anything new happening on the HS200 front ?","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"QFdMzknw\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3yDnbm21z6z9t2l\n\tfor ;\n\tSun, 15 Oct 2017 01:51:46 +1100 (AEDT)","by lists.denx.de (Postfix, from userid 105)\n\tid 473CAC21F30; Sat, 14 Oct 2017 14:51:40 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 8B055C21C57;\n\tSat, 14 Oct 2017 14:51:38 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 8A6EFC21C57; Sat, 14 Oct 2017 14:51:36 +0000 (UTC)","from mail-wm0-f44.google.com (mail-wm0-f44.google.com\n\t[74.125.82.44])\n\tby lists.denx.de (Postfix) with ESMTPS id 1EC4BC21C45\n\tfor ; Sat, 14 Oct 2017 14:51:35 +0000 (UTC)","by mail-wm0-f44.google.com with SMTP id u138so26058070wmu.5\n\tfor ; Sat, 14 Oct 2017 07:51:35 -0700 (PDT)","from [192.168.1.4] (ip-86-49-107-50.net.upcbroadband.cz.\n\t[86.49.107.50]) by smtp.gmail.com with ESMTPSA id\n\tt43sm2498723edh.22.2017.10.14.07.51.33\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tSat, 14 Oct 2017 07:51:33 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=FREEMAIL_FROM,\n\tRCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=subject:to:cc:references:from:message-id:date:user-agent\n\t:mime-version:in-reply-to:content-language:content-transfer-encoding; \n\tbh=BSO/Hn5fWXpkXA7U/4xdh1Dn6zYdXWKUyzRPlPiBL+U=;\n\tb=QFdMzknwsbIitCc6mSp6Xd+BkWLRA0RvJh0nNseFV/5zAh+I/T6OTB8+aEjR4guEEE\n\ta1Kn+pYWlhbUgBfbY72T6Bz+eNXGk/0QzRZ+3Nt9NXufjmhRAdD8Gcr4XX6DF+sktP7z\n\t+Ux1ogm1oda3V3o9G0rRDIAhusfsnKflQM4JBw7RIXwGTcGxAxLxjFWEsR/V/h0t4Fdo\n\thLw0Srn+KfCsiCumw4wF7J6DbAE/5XYWTvXw8jnJRu/5zqe1euMXaqjgUn/gdvcCGmlx\n\t4+comlKWFLuwzTFraKxd5c23vDd8cH5iEGF8fO1D3dGY0ogmHWb2O3f9gEMPWpSpXsVK\n\tCAYA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:subject:to:cc:references:from:message-id:date\n\t:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=BSO/Hn5fWXpkXA7U/4xdh1Dn6zYdXWKUyzRPlPiBL+U=;\n\tb=KdJfwouJ0JhPdsDYym5q4s3AKD2XzD5eOW7WFCNNYMejM27JJ7s6bZ+gIcsnxNlcNU\n\tPx7TGpZM0U8aUZu5GD0BQjSPbVUTeS98pG2q6CNsVWAyAKFm7+FoXopf/aStVMrXj/U9\n\tHKdUBKsNrV70FxOpZlqqKTa+b2+yEejGRVA+VjwwF0hmNfZ8mc0heOTDX+41UAGQK+QV\n\tE78FT9PUMGManu6rsLBAzDuJhNuRYsEO3AgcqGCuZVMyuBSuEs3xkParca6urdHzCzmi\n\tmOrvsABKN29LFN0u8N+XeRxvSE+7/yYazNbZkRgVvWCoMHmHZC10jZ/CgqhVMWYVjUoP\n\tKYRQ==","X-Gm-Message-State":"AMCzsaVOXfJfxXyRn8th12oVbdmLcN6VAK0KVJJtBJnB0L/zslrlEOZK\n\tVPmLJurFGtLSxT7qR4P8nC8I3j6w","X-Google-Smtp-Source":"AOwi7QAJh0knjJHWC12VxID7UIoDBGcsozfaJp7ymvV+AXVhX+dtxI+yOHyuGbOp4ih/VSD6/6NS9A==","X-Received":"by 10.80.195.4 with SMTP id a4mr6350905edb.142.1507992694377;\n\tSat, 14 Oct 2017 07:51:34 -0700 (PDT)","To":"Jean-Jacques Hiblot ,\n\tJaehoon Chung , trini@konsulko.com,\n\tkishon@ti.com, sjg@chromium.org","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>\n\t\n\t<969b6bb5-9ded-87e8-d653-b39acc5a6285@ti.com>","From":"Marek Vasut ","Message-ID":"<79cd1066-41e3-17e8-1f3f-460f765056f8@gmail.com>","Date":"Sat, 14 Oct 2017 16:51:32 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<969b6bb5-9ded-87e8-d653-b39acc5a6285@ti.com>","Content-Language":"en-US","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1789551,"web_url":"http://patchwork.ozlabs.org/comment/1789551/","msgid":"<029b7b42-0b0e-40af-03da-2b9990a5564e@ti.com>","list_archive_url":null,"date":"2017-10-18T13:19:16","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","submitter":{"id":70508,"url":"http://patchwork.ozlabs.org/api/people/70508/","name":"Jean-Jacques Hiblot","email":"jjhiblot@ti.com"},"content":"On 14/10/2017 16:51, Marek Vasut wrote:\n> On 10/02/2017 11:15 AM, Jean-Jacques Hiblot wrote:\n>>\n>> On 22/09/2017 16:42, Jaehoon Chung wrote:\n>>> Dear JJ,\n>>>\n>>> On 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\n>>>> This series brings support for HS200 and UHS modes to the mmc core.\n>>>> It has been tested with the hsmmc driver on several platforms (DRA7,\n>>>> AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some\n>>>> modifications\n>>>> are required in the host driver to take advantage of this (voltage\n>>>> switching,\n>>>> tuning). The changes to the host driver will be posted a another\n>>>> series as this\n>>>> one is already long enough.\n>>>>\n>>>> The series starts with a small refactoring of th sd/mmc startup. The\n>>>> first 5 commits\n>>>> are mostly moving code around with little or no functionnal change.\n>>>>\n>>>> Then the notion of \"mode\" is introduced. Until now, this information\n>>>> wasn't\n>>>> kept in struct mmc. Only the clock and a flag for ddr was kept. Later\n>>>> the mode\n>>>> information will be used to select the clock frequency, the ddr flag\n>>>> and the\n>>>> tuning procedure. It will be also be check against the host\n>>>> capabilities.\n>>>>\n>>>> Then comes the big refactoring job in:\n>>>> \"mmc: refactor MMC startup to make it easier to support new modes\" and\n>>>> \"mmc: refactor SD startup to make it easier to support new modes\"\n>>>> Since the number of modes is increasing, it makes sense to try them\n>>>> in a more\n>>>> organized way. those commits use a list of supported modes and\n>>>> iterate through\n>>>> them to find the best working one. It also allows to switch more\n>>>> easilly from\n>>>> one mode to another (switching from HS200 to DDR52 to access boot\n>>>> partitions for example)\n>>>>\n>>>> Then there are a couple of new callback added to:\n>>>> - send the initialization stream (74 clock cycles)\n>>>> - wait while the card is busy (used during UHS voltage switching).\n>>>> checking is\n>>>>    done on dat0.\n>>>> - select the IO voltage\n>>>>\n>>>> Then Power cycle is added. Without power cycle, if a UHS card fails\n>>>> to enumerate in\n>>>> UHS mode, it can't fall back to high speed mode and card enumeration\n>>>> will fail.\n>>>> Also in case of a reset (as opposed to a power on), it ensures that\n>>>> the SDCard\n>>>> is in clean state before re-doing the initialization.\n>>>>\n>>>> And finally the last commits add the support for HS200 and UHS.\n>>>>\n>>>> With this in place and the required changes in the HSMMC host driver\n>>>> (including\n>>>> DMA), we observe significant improvements in the performances on a\n>>>> DRA72 evm:\n>>>> eMMC HS200: 124 MB/s\n>>>> eMMC DDR52: 78 MB/s\n>>>> sd   SDR104: 71 MB/s\n>>>> sd   SDR50: 44 MB/s\n>>> I have one question.\n>>>\n>>> Did you test about accessing Boot partition of eMMC?\n>>> As i know, boot partition doesn't support the HS200 mode.\n>>>\n>>> When i checked your patches, there is no the function of mode changing\n>>> for accessing boot partition.\n>>> If i missed it, let me know, plz.\n>> Yes I did test the boot partitions. HS200 is not used in this case.\n>> The key function is mmc_boot_part_access_chk(). It checks if the current\n>> mode is supported by the partition. If not, it'll reselect a new mode.\n> Is anything new happening on the HS200 front ?\nI've been busy lately but I'll send the v3  in a few days that adresses \nall the comments done by Jaheoon and Simon.\n\nJean-Jacques\n>","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ti.com header.i=@ti.com header.b=\"QPiC5IP3\";\n\tdkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3yHCMT0sJHz9t5R\n\tfor ;\n\tThu, 19 Oct 2017 00:19:32 +1100 (AEDT)","by lists.denx.de (Postfix, from userid 105)\n\tid C1D53C21EBC; Wed, 18 Oct 2017 13:19:25 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 08F68C21E66;\n\tWed, 18 Oct 2017 13:19:23 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid D74B1C21E66; Wed, 18 Oct 2017 13:19:21 +0000 (UTC)","from lelnx194.ext.ti.com (lelnx194.ext.ti.com [198.47.27.80])\n\tby lists.denx.de (Postfix) with ESMTPS id 4D934C21E10\n\tfor ; Wed, 18 Oct 2017 13:19:21 +0000 (UTC)","from dlelxv90.itg.ti.com ([172.17.2.17])\n\tby lelnx194.ext.ti.com (8.15.1/8.15.1) with ESMTP id v9IDJI8G016438; \n\tWed, 18 Oct 2017 08:19:18 -0500","from DFLE107.ent.ti.com (dfle107.ent.ti.com [10.64.6.28])\n\tby dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id v9IDJIgU019003; \n\tWed, 18 Oct 2017 08:19:18 -0500","from DFLE111.ent.ti.com (10.64.6.32) by DFLE107.ent.ti.com\n\t(10.64.6.28) with Microsoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.845.34;\n\tWed, 18 Oct 2017 08:19:18 -0500","from dflp33.itg.ti.com (10.64.6.16) by DFLE111.ent.ti.com\n\t(10.64.6.32) with Microsoft SMTP Server (version=TLS1_0,\n\tcipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.845.34 via Frontend\n\tTransport; Wed, 18 Oct 2017 08:19:18 -0500","from [172.22.150.180] (ileax41-snat.itg.ti.com [10.172.224.153])\n\tby dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id v9IDJG1B010689;\n\tWed, 18 Oct 2017 08:19:17 -0500"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=T_DKIM_INVALID\n\tautolearn=unavailable autolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ti.com;\n\ts=ti-com-17Q1; t=1508332758;\n\tbh=Onzvar85vxPfN4cbieE2CfxqMhoddz7YW/hfYEAb6x0=;\n\th=Subject:To:CC:References:From:Date:In-Reply-To;\n\tb=QPiC5IP32uLc4HuZA//LsqkfNuUTxOMBnIsc+Hbbm9AT+Ev9mYTKggbdnJj8Sy8A+\n\tft25BDGQxwgmC2/UnFM05C/E5csx8qbm9VRY0pK5s3sUwvrhVwNdFvRKX8kFdlcPCX\n\tXD9dZcVEhnZTItrM5mid3HozAyFEUIO2iFHDaPDg=","To":"Marek Vasut , Jaehoon Chung\n\t, ,\n\t, ","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>\n\t\n\t<969b6bb5-9ded-87e8-d653-b39acc5a6285@ti.com>\n\t<79cd1066-41e3-17e8-1f3f-460f765056f8@gmail.com>","From":"Jean-Jacques Hiblot ","Message-ID":"<029b7b42-0b0e-40af-03da-2b9990a5564e@ti.com>","Date":"Wed, 18 Oct 2017 15:19:16 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.4.0","MIME-Version":"1.0","In-Reply-To":"<79cd1066-41e3-17e8-1f3f-460f765056f8@gmail.com>","Content-Language":"en-US","X-EXCLAIMER-MD-CONFIG":"e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1789566,"web_url":"http://patchwork.ozlabs.org/comment/1789566/","msgid":"<031f07c4-b0b0-376d-41dc-7358949702ab@gmail.com>","list_archive_url":null,"date":"2017-10-18T13:26:06","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","submitter":{"id":1124,"url":"http://patchwork.ozlabs.org/api/people/1124/","name":"Marek Vasut","email":"marek.vasut@gmail.com"},"content":"On 10/18/2017 03:19 PM, Jean-Jacques Hiblot wrote:\n> \n> \n> On 14/10/2017 16:51, Marek Vasut wrote:\n>> On 10/02/2017 11:15 AM, Jean-Jacques Hiblot wrote:\n>>>\n>>> On 22/09/2017 16:42, Jaehoon Chung wrote:\n>>>> Dear JJ,\n>>>>\n>>>> On 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\n>>>>> This series brings support for HS200 and UHS modes to the mmc core.\n>>>>> It has been tested with the hsmmc driver on several platforms (DRA7,\n>>>>> AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some\n>>>>> modifications\n>>>>> are required in the host driver to take advantage of this (voltage\n>>>>> switching,\n>>>>> tuning). The changes to the host driver will be posted a another\n>>>>> series as this\n>>>>> one is already long enough.\n>>>>>\n>>>>> The series starts with a small refactoring of th sd/mmc startup. The\n>>>>> first 5 commits\n>>>>> are mostly moving code around with little or no functionnal change.\n>>>>>\n>>>>> Then the notion of \"mode\" is introduced. Until now, this information\n>>>>> wasn't\n>>>>> kept in struct mmc. Only the clock and a flag for ddr was kept. Later\n>>>>> the mode\n>>>>> information will be used to select the clock frequency, the ddr flag\n>>>>> and the\n>>>>> tuning procedure. It will be also be check against the host\n>>>>> capabilities.\n>>>>>\n>>>>> Then comes the big refactoring job in:\n>>>>> \"mmc: refactor MMC startup to make it easier to support new modes\" and\n>>>>> \"mmc: refactor SD startup to make it easier to support new modes\"\n>>>>> Since the number of modes is increasing, it makes sense to try them\n>>>>> in a more\n>>>>> organized way. those commits use a list of supported modes and\n>>>>> iterate through\n>>>>> them to find the best working one. It also allows to switch more\n>>>>> easilly from\n>>>>> one mode to another (switching from HS200 to DDR52 to access boot\n>>>>> partitions for example)\n>>>>>\n>>>>> Then there are a couple of new callback added to:\n>>>>> - send the initialization stream (74 clock cycles)\n>>>>> - wait while the card is busy (used during UHS voltage switching).\n>>>>> checking is\n>>>>>     done on dat0.\n>>>>> - select the IO voltage\n>>>>>\n>>>>> Then Power cycle is added. Without power cycle, if a UHS card fails\n>>>>> to enumerate in\n>>>>> UHS mode, it can't fall back to high speed mode and card enumeration\n>>>>> will fail.\n>>>>> Also in case of a reset (as opposed to a power on), it ensures that\n>>>>> the SDCard\n>>>>> is in clean state before re-doing the initialization.\n>>>>>\n>>>>> And finally the last commits add the support for HS200 and UHS.\n>>>>>\n>>>>> With this in place and the required changes in the HSMMC host driver\n>>>>> (including\n>>>>> DMA), we observe significant improvements in the performances on a\n>>>>> DRA72 evm:\n>>>>> eMMC HS200: 124 MB/s\n>>>>> eMMC DDR52: 78 MB/s\n>>>>> sd   SDR104: 71 MB/s\n>>>>> sd   SDR50: 44 MB/s\n>>>> I have one question.\n>>>>\n>>>> Did you test about accessing Boot partition of eMMC?\n>>>> As i know, boot partition doesn't support the HS200 mode.\n>>>>\n>>>> When i checked your patches, there is no the function of mode changing\n>>>> for accessing boot partition.\n>>>> If i missed it, let me know, plz.\n>>> Yes I did test the boot partitions. HS200 is not used in this case.\n>>> The key function is mmc_boot_part_access_chk(). It checks if the current\n>>> mode is supported by the partition. If not, it'll reselect a new mode.\n>> Is anything new happening on the HS200 front ?\n> I've been busy lately but I'll send the v3  in a few days that adresses\n> all the comments done by Jaheoon and Simon.\n\nAwesome, thanks !\n\nIf you have a tree with those patches applied somewhere, that'd be nice\nas I'm working on adding HS200/SDR104 support into the uniphier-sd driver.\n\nAlso, any plan for HS400 ? :-)","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"WYbxJw31\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3yHCWJ4P8qz9t5Q\n\tfor ;\n\tThu, 19 Oct 2017 00:26:20 +1100 (AEDT)","by lists.denx.de (Postfix, from userid 105)\n\tid 47DA5C21EDC; Wed, 18 Oct 2017 13:26:14 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id BB3D0C21E66;\n\tWed, 18 Oct 2017 13:26:11 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 26AB8C21E66; Wed, 18 Oct 2017 13:26:10 +0000 (UTC)","from mail-wr0-f171.google.com (mail-wr0-f171.google.com\n\t[209.85.128.171])\n\tby lists.denx.de (Postfix) with ESMTPS id EA6A2C21E10\n\tfor ; Wed, 18 Oct 2017 13:26:08 +0000 (UTC)","by mail-wr0-f171.google.com with SMTP id k62so5016003wrc.9\n\tfor ; Wed, 18 Oct 2017 06:26:08 -0700 (PDT)","from [192.168.1.4] (ip-86-49-107-50.net.upcbroadband.cz.\n\t[86.49.107.50]) by smtp.gmail.com with ESMTPSA id\n\ta195sm9219684wme.34.2017.10.18.06.26.07\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 18 Oct 2017 06:26:07 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=FREEMAIL_FROM,\n\tRCVD_IN_MSPIKE_H2,\n\tT_DKIM_INVALID autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=subject:to:cc:references:from:message-id:date:user-agent\n\t:mime-version:in-reply-to:content-language:content-transfer-encoding; \n\tbh=r1wcDkP8KJ42saBwb59wVj4ilFahsaXZmD0FTq14rBU=;\n\tb=WYbxJw31OKEnM+/lMExbW951qdkNqVpaOBhNgTNzpeVBVsZXxthz6F5g6TW9HI+uYb\n\tuekSSfZdXS0OIlmIweq4/1D12u0h6g9ArdgUdInPtFifM60RaA+pg/w2s2HuUG9JCxHk\n\tkMcpOUgwMWm6uIkiG0yWeEBi47nq/LMcW0eFBEQbxwhKjfXnHi3TEtxddiUz5ZUefJAH\n\tvXK7+YgSQfrmb7GJbDHoh65zjwlAgsOc7utjjQwVtfeOM0/KbyOB2aa2fcEU8rT4/t40\n\tjS3Xj8SvQjdYaDZCNyV8edITydjeTLbAKSPtX9W8sB5MYPXx+V+WfmE8P6ArfBVV+Hvb\n\tgvRw==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:subject:to:cc:references:from:message-id:date\n\t:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=r1wcDkP8KJ42saBwb59wVj4ilFahsaXZmD0FTq14rBU=;\n\tb=pLacLNtZJ7BSmvlCwTKoIplFO5APftY4TlFL9DctUUjiL3FrIcDWpkbHEfUXTChop4\n\toRIKuJI2vp0lMYyKC20oq2TLompT4t1qqMJnS+BSAly01WzoWoplPo9NE0OEwmU6Cwwh\n\tZLLGYY7bTBgISPMAezOGbiK6t0DG52oliRAFCv3QJFnAjqJqBh4ZMf+yXDSNDm+J4KKs\n\twr7Ojmvpx7AT5TW6SK79IFWSQRlWwhDGcbenoRTbjJ1ZIUvTN/vuZi0X1wqcfquJYYZD\n\taNR9+pTL1TBrDu6UMj+e89xEHQDv1qDfuwIN0clxzm+LIHtWFVffIcw61OKM7gwCDpDY\n\tvVUg==","X-Gm-Message-State":"AMCzsaWUBCMLOK4NbGzqSbkLXb73Q6UQLThUG9fAdzM6Ibh0W3cNvDYe\n\tNY7pEzP4/cRC9551l1PqNO7XbaWv","X-Google-Smtp-Source":"ABhQp+T8syIMVBIJbLIE5cZdXjJ6YHMxJrDtiLLxSzwpOZLYSMKl1/ZWy5TfNUuwfk2hnVDk3UqHyA==","X-Received":"by 10.223.197.141 with SMTP id m13mr7101314wrg.203.1508333168122;\n\tWed, 18 Oct 2017 06:26:08 -0700 (PDT)","To":"Jean-Jacques Hiblot ,\n\tJaehoon Chung , trini@konsulko.com,\n\tkishon@ti.com, sjg@chromium.org","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>\n\t\n\t<969b6bb5-9ded-87e8-d653-b39acc5a6285@ti.com>\n\t<79cd1066-41e3-17e8-1f3f-460f765056f8@gmail.com>\n\t<029b7b42-0b0e-40af-03da-2b9990a5564e@ti.com>","From":"Marek Vasut ","Message-ID":"<031f07c4-b0b0-376d-41dc-7358949702ab@gmail.com>","Date":"Wed, 18 Oct 2017 15:26:06 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<029b7b42-0b0e-40af-03da-2b9990a5564e@ti.com>","Content-Language":"en-US","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1791330,"web_url":"http://patchwork.ozlabs.org/comment/1791330/","msgid":"","list_archive_url":null,"date":"2017-10-20T10:49:55","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","submitter":{"id":8006,"url":"http://patchwork.ozlabs.org/api/people/8006/","name":"Jaehoon Chung","email":"jh80.chung@samsung.com"},"content":"On 10/14/2017 11:51 PM, Marek Vasut wrote:\n> On 10/02/2017 11:15 AM, Jean-Jacques Hiblot wrote:\n>>\n>>\n>> On 22/09/2017 16:42, Jaehoon Chung wrote:\n>>> Dear JJ,\n>>>\n>>> On 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\n>>>> This series brings support for HS200 and UHS modes to the mmc core.\n>>>> It has been tested with the hsmmc driver on several platforms (DRA7,\n>>>> AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some\n>>>> modifications\n>>>> are required in the host driver to take advantage of this (voltage\n>>>> switching,\n>>>> tuning). The changes to the host driver will be posted a another\n>>>> series as this\n>>>> one is already long enough.\n>>>>\n>>>> The series starts with a small refactoring of th sd/mmc startup. The\n>>>> first 5 commits\n>>>> are mostly moving code around with little or no functionnal change.\n>>>>\n>>>> Then the notion of \"mode\" is introduced. Until now, this information\n>>>> wasn't\n>>>> kept in struct mmc. Only the clock and a flag for ddr was kept. Later\n>>>> the mode\n>>>> information will be used to select the clock frequency, the ddr flag\n>>>> and the\n>>>> tuning procedure. It will be also be check against the host\n>>>> capabilities.\n>>>>\n>>>> Then comes the big refactoring job in:\n>>>> \"mmc: refactor MMC startup to make it easier to support new modes\" and\n>>>> \"mmc: refactor SD startup to make it easier to support new modes\"\n>>>> Since the number of modes is increasing, it makes sense to try them\n>>>> in a more\n>>>> organized way. those commits use a list of supported modes and\n>>>> iterate through\n>>>> them to find the best working one. It also allows to switch more\n>>>> easilly from\n>>>> one mode to another (switching from HS200 to DDR52 to access boot\n>>>> partitions for example)\n>>>>\n>>>> Then there are a couple of new callback added to:\n>>>> - send the initialization stream (74 clock cycles)\n>>>> - wait while the card is busy (used during UHS voltage switching).\n>>>> checking is\n>>>> done on dat0.\n>>>> - select the IO voltage\n>>>>\n>>>> Then Power cycle is added. Without power cycle, if a UHS card fails\n>>>> to enumerate in\n>>>> UHS mode, it can't fall back to high speed mode and card enumeration\n>>>> will fail.\n>>>> Also in case of a reset (as opposed to a power on), it ensures that\n>>>> the SDCard\n>>>> is in clean state before re-doing the initialization.\n>>>>\n>>>> And finally the last commits add the support for HS200 and UHS.\n>>>>\n>>>> With this in place and the required changes in the HSMMC host driver\n>>>> (including\n>>>> DMA), we observe significant improvements in the performances on a\n>>>> DRA72 evm:\n>>>> eMMC HS200: 124 MB/s\n>>>> eMMC DDR52: 78 MB/s\n>>>> sd SDR104: 71 MB/s\n>>>> sd SDR50: 44 MB/s\n>>> I have one question.\n>>>\n>>> Did you test about accessing Boot partition of eMMC?\n>>\n>>> As i know, boot partition doesn't support the HS200 mode.\n>>>\n>>> When i checked your patches, there is no the function of mode changing\n>>> for accessing boot partition.\n>>> If i missed it, let me know, plz.\n>> Yes I did test the boot partitions. HS200 is not used in this case.\n>> The key function is mmc_boot_part_access_chk(). It checks if the current\n>> mode is supported by the partition. If not, it'll reselect a new mode.\n> \n> Is anything new happening on the HS200 front ?\n\nWill apply this patchset. Thanks!\n\nBest Regards,\nJaehoon Chung\n\n>","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3yJMyF6tHnz9t4b\n\tfor ;\n\tFri, 20 Oct 2017 21:50:13 +1100 (AEDT)","by lists.denx.de (Postfix, from userid 105)\n\tid BCC78C21FC1; Fri, 20 Oct 2017 10:50:08 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id E11BCC21F71;\n\tFri, 20 Oct 2017 10:50:04 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid E7F36C21F71; Fri, 20 Oct 2017 10:50:02 +0000 (UTC)","from mailout1.samsung.com (mailout1.samsung.com [203.254.224.24])\n\tby lists.denx.de (Postfix) with ESMTPS id A99D9C21ECA\n\tfor ; Fri, 20 Oct 2017 10:50:01 +0000 (UTC)","from epcas1p1.samsung.com (unknown [182.195.41.45])\n\tby mailout1.samsung.com (KnoxPortal) with ESMTP id\n\t20171020104956epoutp01b05a6f83c1bafdb4a4ad8abcdc94cc14~vQamXfjZm1711017110epoutp01L;\n\tFri, 20 Oct 2017 10:49:56 +0000 (GMT)","from epsmges1p3.samsung.com (unknown [182.195.40.68]) by\n\tepcas1p2.samsung.com (KnoxPortal) with ESMTP id\n\t20171020104956epcas1p2889082c2f6e32c99f2b59928dd2264a2~vQamBs12o2606726067epcas1p2M;\n\tFri, 20 Oct 2017 10:49:56 +0000 (GMT)","from epcas1p1.samsung.com ( [182.195.41.45]) by\n\tepsmges1p3.samsung.com (Symantec Messaging Gateway) with SMTP id\n\t68.C7.04214.4D4D9E95; Fri, 20 Oct 2017 19:49:56 +0900 (KST)","from epsmgms2p1new.samsung.com (unknown [182.195.42.142]) by\n\tepcas1p4.samsung.com (KnoxPortal) with ESMTP id\n\t20171020104955epcas1p444968c9861f4e405fca8784e457862f4~vQaleJrPf1938919389epcas1p4z;\n\tFri, 20 Oct 2017 10:49:55 +0000 (GMT)","from epmmp1.local.host ( [203.254.227.16]) by\n\tepsmgms2p1new.samsung.com (Symantec Messaging Gateway) with SMTP id\n\t7D.96.06995.3D4D9E95; Fri, 20 Oct 2017 19:49:55 +0900 (KST)","from [10.113.62.216] by mmp1.samsung.com (Oracle Communications\n\tMessaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTPA id\n\t<0OY400B9TBF7B5B0@mmp1.samsung.com>;\n\tFri, 20 Oct 2017 19:49:55 +0900 (KST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=SPF_HELO_PASS\n\tautolearn=unavailable autolearn_force=no version=3.4.0","X-AuditID":"b6c32a37-c8dff70000001076-e3-59e9d4d4b34e","To":"Marek Vasut , Jean-Jacques Hiblot\n\t, trini@konsulko.com, kishon@ti.com, sjg@chromium.org","From":"Jaehoon Chung ","Message-id":"","Date":"Fri, 20 Oct 2017 19:49:55 +0900","User-Agent":"Mozilla/5.0 (X11; Linux i686; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-version":"1.0","In-reply-to":"<79cd1066-41e3-17e8-1f3f-460f765056f8@gmail.com>","Content-language":"en-US","X-Brightmail-Tracker":["H4sIAAAAAAAAA+NgFmplk+LIzCtJLcpLzFFi42LZdlhTV/fKlZeRBj0HJSxaTk5isrjwtIfN\n\t4uiee8wW37ZsY7SYOmkzu8XbvZ3sDmwesxsusnjsnHWX3ePVgVXsHmfv7GD0OH5jO1MAa1Sq\n\tTUZqYkpqkUJqXnJ+SmZeuq2Sd3C8c7ypmYGhrqGlhbmSQl5ibqqtkotPgK5bZg7QCUoKZYk5\n\tpUChgMTiYiV9O5ui/NKSVIWM/OISW6VoQ0MjPUMDcz0jIyM9E+NYKyNToJKE1IwzLx8wF7yW\n\tqmhdbNHAuFK0i5GTQ0LAROLcm272LkYuDiGBHYwSXT/2MkE43xkl5n7/wg5TtaPtGgtEYjej\n\txMapl5hBEkIC9xglnjTlgNjCAm4Sty4/ZgUpEhFoZZTY9r2JFSTBLCAhseTQfTCbTUBHYvu3\n\t40wgNq+AncSxWbfAbBYBVYn1z96A2aICYRIrf71ihKgRlPgx+R4LiM0pYCuxYOMHNoiZmhIv\n\tvkxigbDFJZpbb0LZ8hKb17xlBjlCQuAEm8TfvbeZIF5wkWjp+AhlC0u8Or4F6DUOIFta4tJR\n\tW4j6dkaJWz/2sUE4HYwSB3/uZYVoMJY41dXIBLGBT+Ld1x5WiGZeiY42IYgSD4k5D1ZDzXeU\n\tON95HBqOq5kklq+8wDiBUW4WkodmIXliFpInZiF5YgEjyypGsdSC4tz01GLDAmO94sTc4tK8\n\tdL3k/NxNjOAkqGW+g3HDOZ9DjAIcjEo8vBsuvIgUYk0sK67MPcQowcGsJMJruP9lpBBvSmJl\n\tVWpRfnxRaU5q8SFGU2CAT2SWEk3OBybovJJ4QxNLAxMzI2BiszQ0VBLnFVt/LUJIID2xJDU7\n\tNbUgtQimj4mDU6qBUeWUtfC0nFd3S/LjuZ0VTLnUNkwUftwirJm51GvW1KI5H5xtrZ9Nn5ZU\n\tIqJuXhcru8Uov5yJe7N5kduX+NvBCwseZPu85U1/VlJrV1m7IHd19pwTE8OiPrTI5C94UHOq\n\t6bD58VYulymSosaSiq+jV978o9LdcuhNjh7L3uZNgV7Luqa6l09TYinOSDTUYi4qTgQA3Vao\n\tR5gDAAA=","H4sIAAAAAAAAA+NgFnrOLMWRmVeSWpSXmKPExsVy+t9jAd3LV15GGmzYqWvRcnISk8WFpz1s\n\tFkf33GO2+LZlG6PF1Emb2S3e7u1kd2DzmN1wkcVj56y77B6vDqxi9zh7Zwejx/Eb25kCWKO4\n\tbFJSczLLUov07RK4Ms68fMBc8FqqonWxRQPjStEuRk4OCQETiR1t11i6GLk4hAR2Mko8/DmH\n\tHcJ5wCixtuUJG0iVsICbxK3Lj1lBEiICrYwSO+7fBkswC0hILDl0nxWiYy2TxO07G1hAEmwC\n\tOhLbvx1nArF5Bewkjs26BWazCKhKrH/2BswWFQiT6G/+ywxRIyjxY/I9sF5OAVuJBRs/AC3g\n\tAFqgLjFlSi7ELnGJ5tabLBC2vMTmNW+ZJzAKzELSPQuhYxaSjllIOhYwsqxilEwtKM5Nzy02\n\tKjDMSy3XK07MLS7NS9dLzs/dxAgM/m2Htfp2MN5fEn+IUYCDUYmHN+Lci0gh1sSy4srcQ4wS\n\tHMxKIryG+19GCvGmJFZWpRblxxeV5qQWH2KU5mBREue9nXcsUkggPbEkNTs1tSC1CCbLxMEp\n\t1cAof39OSiDPh7xVv2SM2G9+OzJRku+KX4/Twm3LdX/31xpoMtXu7Kmy+B/uUqzw2aJY3cvB\n\tLbhHxmE/e/zMb/7bjBaxM5lPVA495H2qVCHQMuPQMY38RQ1NbcuX/G7dVmvf1Lt07aVXws29\n\tV6dq6r2J9r/nV+TwUWM9w8OM52f225S9Pyrnn6/EUpyRaKjFXFScCADxZEQDegIAAA=="],"X-CMS-MailID":"20171020104955epcas1p444968c9861f4e405fca8784e457862f4","X-Msg-Generator":"CA","X-Sender-IP":"182.195.42.142","X-Local-Sender":"=?utf-8?b?7KCV7J6s7ZuIG1RpemVuIFBsYXRmb3JtIExhYihTL1c=?=\n\t=?utf-8?b?7IS87YSwKRvsgrzshLHsoITsnpAbU2VuaW9yIEVuZ2luZWVy?=","X-Global-Sender":"=?utf-8?q?Jaehoon_Chung=1BTizen_Platform_Lab=2E=1BSamsun?=\n\t=?utf-8?q?g_Electronics=1BSenior_Engineer?=","X-Sender-Code":"=?utf-8?q?C10=1BTELE=1BC10V8111?=","CMS-TYPE":"101P","DLP-Filter":"Pass","X-CFilter-Loop":"Reflected","X-CMS-RootMailID":"20170921143028epcas3p3fce32811279a7e0bf9941e4bbc621ddd","X-RootMTR":"20170921143028epcas3p3fce32811279a7e0bf9941e4bbc621ddd","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>\n\t\n\t<969b6bb5-9ded-87e8-d653-b39acc5a6285@ti.com>\n\t<79cd1066-41e3-17e8-1f3f-460f765056f8@gmail.com>","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1791368,"web_url":"http://patchwork.ozlabs.org/comment/1791368/","msgid":"<9b9d0abe-d638-1040-15bf-8b5661200af3@samsung.com>","list_archive_url":null,"date":"2017-10-20T11:31:30","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","submitter":{"id":8006,"url":"http://patchwork.ozlabs.org/api/people/8006/","name":"Jaehoon Chung","email":"jh80.chung@samsung.com"},"content":"Dear JJ\n\nOn 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\n> This series brings support for HS200 and UHS modes to the mmc core.\n> It has been tested with the hsmmc driver on several platforms (DRA7,\n> AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some modifications\n> are required in the host driver to take advantage of this (voltage switching,\n> tuning). The changes to the host driver will be posted a another series as this\n> one is already long enough. \n> \n> The series starts with a small refactoring of th sd/mmc startup. The first 5 commits\n> are mostly moving code around with little or no functionnal change.\n> \n> Then the notion of \"mode\" is introduced. Until now, this information wasn't\n> kept in struct mmc. Only the clock and a flag for ddr was kept. Later the mode\n> information will be used to select the clock frequency, the ddr flag and the\n> tuning procedure. It will be also be check against the host capabilities.\n> \n> Then comes the big refactoring job in:\n> \"mmc: refactor MMC startup to make it easier to support new modes\" and\n> \"mmc: refactor SD startup to make it easier to support new modes\"\n> Since the number of modes is increasing, it makes sense to try them in a more\n> organized way. those commits use a list of supported modes and iterate through\n> them to find the best working one. It also allows to switch more easilly from\n> one mode to another (switching from HS200 to DDR52 to access boot partitions for example)\n> \n> Then there are a couple of new callback added to:\n> - send the initialization stream (74 clock cycles)\n> - wait while the card is busy (used during UHS voltage switching). checking is\n> done on dat0.\n> - select the IO voltage\n> \n> Then Power cycle is added. Without power cycle, if a UHS card fails to enumerate in\n> UHS mode, it can't fall back to high speed mode and card enumeration will fail.\n> Also in case of a reset (as opposed to a power on), it ensures that the SDCard\n> is in clean state before re-doing the initialization.\n> \n> And finally the last commits add the support for HS200 and UHS.\n> \n> With this in place and the required changes in the HSMMC host driver (including\n> DMA), we observe significant improvements in the performances on a DRA72 evm:\n> eMMC HS200: 124 MB/s\n> eMMC DDR52: 78 MB/s\n> sd SDR104: 71 MB/s\n> sd SDR50: 44 MB/s\n\nApplied to u-boot-mmc! Thanks!\nI will apply the patches relevant to this patchset. Sorry for late applying this.\n\nBest Regards,\nJaehoon Chung\n\n> \n> cheers,\n> \n> Jean-Jacques\n> \n> changes since v1:\n> Changes take in account all the comment made during the first round of review.\n> But it also include a rework of the handling of the voltage supplies and 2 new\n> helper functions that can be used by the host driver during tuning and dts\n> parsing.\n> \n> Here is a detailed list:\n> * rebase on u-boot/master.\n> * add the description for the new functions/members\n> * fix the typos found in the reviews\n> * limit the new features to DM_MMC only\n> * fix the new checkpatch warnings/errors (is it now more strict ?)\n> * get the voltage regulators from dts (main and IO-lines)\n> * remove the set_vdd callback and do the job in the mmc core\n> * add a send_init_stream callback to perform the 74 clock cycle\n> sequence after power up\n> * do a full power cycle (if possible) before the initialization. This ensures\n> that the sd/MMC is in valid knwon state. Also it allows to check if the power\n> cycling works as expected.\n> * disable the UHS modes is power cycling is not available/working. This ensures\n> that we won't be stuck if the UHS initialization fails.\n> * select the appropriate signal voltage when selecting a mode (MMC only) \n> * add a helper function to parse the generic dt bindings\n> * add a helper function to send the tuning command\n> \n> Jean-Jacques Hiblot (17):\n> mmc: dm: get the IO-line and main voltage regulators from the dts\n> mmc: split mmc_startup()\n> mmc: move the MMC startup for version above v4.0 in a separate\n> function\n> mmc: make ext_csd part of struct mmc\n> mmc: add a function to read and test the ext csd (mmc >= 4)\n> mmc: introduce mmc modes\n> mmc: Add a function to dump the mmc capabilities\n> mmc: use mmc modes to select the correct bus speed\n> cmd: mmc: display the mode name and current bus speed in the mmc info\n> mmc: refactor SD startup to make it easier to support new modes\n> mmc: refactor MMC startup to make it easier to support new modes\n> mmc: Add a new callback function to perform the 74 clocks cycle\n> sequence\n> mmc: Add support for UHS modes\n> mmc: disable UHS modes if Vcc cannot be switched on and off\n> mmc: Change mode when switching to a boot partition\n> mmc: use the right voltage level for MMC DDR and HS200 modes\n> mmc: add a library function to send tuning command\n> \n> Kishon Vijay Abraham I (9):\n> mmc: make mmc_set_ios() return status\n> mmc: Enable signal voltage to be selected from mmc core\n> mmc: add power cyle support in mmc core\n> mmc: add a new mmc parameter to disable mmc clock\n> mmc: disable the mmc clock during power off\n> mmc: Add a execute_tuning() callback to the mmc operations.\n> mmc: add HS200 support in MMC core\n> mmc: Retry some MMC cmds on failure\n> dm: mmc: Add a library function to parse generic dt binding\n> \n> cmd/mmc.c | 3 +-\n> drivers/mmc/Kconfig | 23 +\n> drivers/mmc/fsl_esdhc.c | 2 +-\n> drivers/mmc/mmc-uclass.c | 87 +++\n> drivers/mmc/mmc.c | 1511 ++++++++++++++++++++++++++++++++++------------\n> include/mmc.h | 176 +++++-\n> 6 files changed, 1417 insertions(+), 385 deletions(-)\n>","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3yJNtG1YV6z9sP1\n\tfor ;\n\tFri, 20 Oct 2017 22:31:48 +1100 (AEDT)","by lists.denx.de (Postfix, from userid 105)\n\tid 41E3DC21FC3; Fri, 20 Oct 2017 11:31:41 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id B33EFC21F71;\n\tFri, 20 Oct 2017 11:31:38 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid A6290C21F71; Fri, 20 Oct 2017 11:31:37 +0000 (UTC)","from mailout4.samsung.com (mailout4.samsung.com [203.254.224.34])\n\tby lists.denx.de (Postfix) with ESMTPS id A45DAC21ECA\n\tfor ; Fri, 20 Oct 2017 11:31:36 +0000 (UTC)","from epcas1p1.samsung.com (unknown [182.195.41.45])\n\tby mailout4.samsung.com (KnoxPortal) with ESMTP id\n\t20171020113131epoutp049509e7cb1199d89ebc9f7963dba8b0f4~vQ_5onTKg0795807958epoutp04v;\n\tFri, 20 Oct 2017 11:31:31 +0000 (GMT)","from epsmges1p3.samsung.com (unknown [182.195.40.65]) by\n\tepcas1p1.samsung.com (KnoxPortal) with ESMTP id\n\t20171020113131epcas1p19fe007255af7865bfab4966c1e1ff480~vQ_5ZJOtP0259902599epcas1p1H;\n\tFri, 20 Oct 2017 11:31:31 +0000 (GMT)","from epcas1p4.samsung.com ( [182.195.41.48]) by\n\tepsmges1p3.samsung.com (Symantec Messaging Gateway) with SMTP id\n\t60.9A.04214.39ED9E95; Fri, 20 Oct 2017 20:31:31 +0900 (KST)","from epsmgms2p1new.samsung.com (unknown [182.195.42.142]) by\n\tepcas1p3.samsung.com (KnoxPortal) with ESMTP id\n\t20171020113130epcas1p3b0cc82dba0bb5eac6c9ff0137697996a~vQ_5NFU9b0564205642epcas1p3z;\n\tFri, 20 Oct 2017 11:31:30 +0000 (GMT)","from epmmp1.local.host ( [203.254.227.16]) by\n\tepsmgms2p1new.samsung.com (Symantec Messaging Gateway) with SMTP id\n\tB4.28.06995.29ED9E95; Fri, 20 Oct 2017 20:31:30 +0900 (KST)","from [10.113.62.216] by mmp1.samsung.com (Oracle Communications\n\tMessaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTPA id\n\t<0OY400LL8DCI5810@mmp1.samsung.com>;\n\tFri, 20 Oct 2017 20:31:30 +0900 (KST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=SPF_HELO_PASS\n\tautolearn=unavailable autolearn_force=no version=3.4.0","X-AuditID":"b6c32a37-c75ff70000001076-47-59e9de933074","To":"Jean-Jacques Hiblot , trini@konsulko.com,\n\tkishon@ti.com, sjg@chromium.org","From":"Jaehoon Chung ","Message-id":"<9b9d0abe-d638-1040-15bf-8b5661200af3@samsung.com>","Date":"Fri, 20 Oct 2017 20:31:30 +0900","User-Agent":"Mozilla/5.0 (X11; Linux i686; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-version":"1.0","In-reply-to":"<1506004213-22620-1-git-send-email-jjhiblot@ti.com>","Content-language":"en-US","X-Brightmail-Tracker":["H4sIAAAAAAAAA02Se0hTURzHObt7XKXVcVr+MKhxocJoa3e+ZjQrilrkH1JQacm8uMMmuU12\n\tt2hRZJRpQ8mUotaDCAt8gJmP7KGZkVam9qAotVFqvsoe9C7Qdr0G/vc9v/P5nvP7/vjRlOqQ\n\tPIrOcriJy8FlM/JQacPdaJ2mNDCaqhucoA2HH5RIDI/fFcoN924FKMOPugZkOFFSqzCMNx1V\n\trJabzuQ+kZqu+18rTGMtFQpTZ18jMrW/vCZJkaWRlTbCWYhLTRyZTkuWw2pkNm0xrzXHxetY\n\tDZtoSGDUDs5OjMy65BTN+qzsYAuMejeX7QmWUjieZ5YnrXQ5PW6itjl5t5HZwbJ6LatL0Or1\n\tem1sTPoKfVwQySC2zgF9TrFmT1tZsywX9TI+FEIDjoXbHecUPhRKq3Ajgqrmn1LhQoV/Ivhy\n\tffF/qC93CInQTQSDdf3TjgCCG33dEoEKx+uh59mAzIdoOgJnQFGeRyhTeDkMd1RRgpbjZXDt\n\tR/sUrsRJUDBSjQQtxYugp+HzFDMXb4XyP2NIZMLgV2lgqqEQvBo6LpdT4pvRMPKtRCrqSDiU\n\t92paL4TaqnFK6A3wIzk8naiRiQnWwdW+j0jU4TDWXqcQ+gQ8H57eM4p8PoKeX81y8VCA4M7v\n\tpmlzDDz0HZSIP8yGj98LZaJZCQVHVCJigrNvKiWiXgPdR4WQwoBKEXRXPpQXowX+GYH8M0L4\n\tZ4TwzwhxAUkr0DySw9uthGdzYrQ8Z+c9Dqs202m/iqYWcGlCI7rSldyKMI2YWcorj0dSVTJu\n\tN++1tyKgKSZCyd4eTVUpLZx3L3E5zS5PNuFbUVxw4MepqLmZzuA6O9xmNjZRFxuvD65aIssy\n\tkcp51S+2q7CVc5NdhOQQ13+fhA6JykX70hwbLJ1tG4qg3PBds3Fz/YcoW3paJd01GpofX+8d\n\tqanZ7xxyb9tYu2QooI0o+zrwQh3+iTt/YGdHTO9kW9hfc0v48Fn2LV6Wfoq61G9hBxX8xN3n\n\t9YsJ31Bzo/Bk0+fR2cW203i8etUkN6fMWJR+scvqPeZTTFRb7nvf309mpLyNY5dSLp77B8rC\n\t1tqWAwAA","H4sIAAAAAAAAA+NgFnrKLMWRmVeSWpSXmKPExsVy+t9jAd1J915GGvzZaWHRcnISk8WFpz1s\n\tFkf33GO2+LZlG6PF1Emb2S3e7u1kd2DzmN1wkcVj56y77B6vDqxi9zh7Zwejx/Eb25kCWKO4\n\tbFJSczLLUov07RK4Ms4+NiqYoFtxbMk+1gbG20pdjJwcEgImEncanjF2MXJxCAnsZJTo27qY\n\tESQhJPCAUeL7NXsQW1jATeLW5cesILaIQILE/y1X2EBsZgF9ieen1zBDNE9llHj37zNYM5uA\n\tjsT2b8eZQGxeATuJjhfrweIsAqoSt7Z9YAaxRQXCJPqb/zJD1AhK/Jh8jwXE5hRwkDi9bCVQ\n\tnANogbrElCm5ELvEJZpbb7JA2PISm9e8ZZ7AKDALSfcshI5ZSDpmIelYwMiyilEytaA4Nz23\n\t2KjAMC+1XK84Mbe4NC9dLzk/dxMjMPC3Hdbq28F4f0n8IUYBDkYlHt6Icy8ihVgTy4orcw8x\n\tSnAwK4nwGu5/GSnEm5JYWZValB9fVJqTWnyIUZqDRUmc93besUghgfTEktTs1NSC1CKYLBMH\n\tp1QDY9vLGbY7mBe/kZV+zP3+y+v1X8WU3qvfSHA/3Xj1tmnF3D1fZ6zufp19p13gn7O9tO0L\n\t3ovidjbxN2q02k4KtW+tucFY2P4v68q8hfXqxqJaj06xelcde5cVJ6tyMuVkyZxlbI6+Adc2\n\t2m0LXTv7x8VNq9yKf/k/f5Spu/KQ78blM5dYVv6NDFdiKc5INNRiLipOBABAodYCeAIAAA=="],"X-CMS-MailID":"20171020113130epcas1p3b0cc82dba0bb5eac6c9ff0137697996a","X-Msg-Generator":"CA","X-Sender-IP":"182.195.42.142","X-Local-Sender":"=?utf-8?b?7KCV7J6s7ZuIG1RpemVuIFBsYXRmb3JtIExhYihTL1c=?=\n\t=?utf-8?b?7IS87YSwKRvsgrzshLHsoITsnpAbU2VuaW9yIEVuZ2luZWVy?=","X-Global-Sender":"=?utf-8?q?Jaehoon_Chung=1BTizen_Platform_Lab=2E=1BSamsun?=\n\t=?utf-8?q?g_Electronics=1BSenior_Engineer?=","X-Sender-Code":"=?utf-8?q?C10=1BTELE=1BC10V8111?=","CMS-TYPE":"101P","DLP-Filter":"Pass","X-CFilter-Loop":"Reflected","X-CMS-RootMailID":"20170921143028epcas3p3fce32811279a7e0bf9941e4bbc621ddd","X-RootMTR":"20170921143028epcas3p3fce32811279a7e0bf9941e4bbc621ddd","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1809119,"web_url":"http://patchwork.ozlabs.org/comment/1809119/","msgid":"","list_archive_url":null,"date":"2017-11-23T09:46:25","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and\n\tUHS\tmodes","submitter":{"id":72703,"url":"http://patchwork.ozlabs.org/api/people/72703/","name":"Siva Durga Prasad Paladugu","email":"sivadur@xilinx.com"},"content":"Hi Jaehoon,\r\n\r\n> -----Original Message-----\r\n> From: U-Boot [mailto:u-boot-bounces@lists.denx.de] On Behalf Of Jaehoon\r\n> Chung\r\n> Sent: Friday, October 20, 2017 5:02 PM\r\n> To: Jean-Jacques Hiblot ; trini@konsulko.com;\r\n> kishon@ti.com; sjg@chromium.org\r\n> Cc: u-boot@lists.denx.de\r\n> Subject: Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and\r\n> UHS modes\r\n> \r\n> Dear JJ\r\n> \r\n> On 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\r\n> > This series brings support for HS200 and UHS modes to the mmc core.\r\n> > It has been tested with the hsmmc driver on several platforms (DRA7,\r\n> > AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some\r\n> > modifications are required in the host driver to take advantage of\r\n> > this (voltage switching, tuning). The changes to the host driver will\r\n> > be posted a another series as this one is already long enough.\r\n> >\r\n> > The series starts with a small refactoring of th sd/mmc startup. The\r\n> > first 5 commits are mostly moving code around with little or no\r\n> functionnal change.\r\n> >\r\n> > Then the notion of \"mode\" is introduced. Until now, this information\r\n> > wasn't kept in struct mmc. Only the clock and a flag for ddr was kept.\r\n> > Later the mode information will be used to select the clock frequency,\r\n> > the ddr flag and the tuning procedure. It will be also be check against the\r\n> host capabilities.\r\n> >\r\n> > Then comes the big refactoring job in:\r\n> > \"mmc: refactor MMC startup to make it easier to support new modes\"\r\n> and\r\n> > \"mmc: refactor SD startup to make it easier to support new modes\"\r\n> > Since the number of modes is increasing, it makes sense to try them in\r\n> > a more organized way. those commits use a list of supported modes and\r\n> > iterate through them to find the best working one. It also allows to\r\n> > switch more easilly from one mode to another (switching from HS200 to\r\n> > DDR52 to access boot partitions for example)\r\n> >\r\n> > Then there are a couple of new callback added to:\r\n> > - send the initialization stream (74 clock cycles)\r\n> > - wait while the card is busy (used during UHS voltage switching). checking\r\n> is\r\n> > done on dat0.\r\n> > - select the IO voltage\r\n> >\r\n> > Then Power cycle is added. Without power cycle, if a UHS card fails to\r\n> > enumerate in UHS mode, it can't fall back to high speed mode and card\r\n> enumeration will fail.\r\n> > Also in case of a reset (as opposed to a power on), it ensures that\r\n> > the SDCard is in clean state before re-doing the initialization.\r\n> >\r\n> > And finally the last commits add the support for HS200 and UHS.\r\n> >\r\n> > With this in place and the required changes in the HSMMC host driver\r\n> > (including DMA), we observe significant improvements in the\r\n> performances on a DRA72 evm:\r\n> > eMMC HS200: 124 MB/s\r\n> > eMMC DDR52: 78 MB/s\r\n> > sd SDR104: 71 MB/s\r\n> > sd SDR50: 44 MB/s\r\n> \r\n> Applied to u-boot-mmc! Thanks!\r\n> I will apply the patches relevant to this patchset. Sorry for late applying this.\r\n\r\nI would like to test UHS and HS200 series on ZynqMP platform. \r\nIs it the branch \" testing-uhs-supporting\" that I should be using for testing it?\r\nPlease confirm.\r\n\r\nThanks,\r\nSiva\r\n\r\n> \r\n> Best Regards,\r\n> Jaehoon Chung\r\n> \r\n> >\r\n> > cheers,\r\n> >\r\n> > Jean-Jacques\r\n> >\r\n> > changes since v1:\r\n> > Changes take in account all the comment made during the first round of\r\n> review.\r\n> > But it also include a rework of the handling of the voltage supplies\r\n> > and 2 new helper functions that can be used by the host driver during\r\n> > tuning and dts parsing.\r\n> >\r\n> > Here is a detailed list:\r\n> > * rebase on u-boot/master.\r\n> > * add the description for the new functions/members\r\n> > * fix the typos found in the reviews\r\n> > * limit the new features to DM_MMC only\r\n> > * fix the new checkpatch warnings/errors (is it now more strict ?)\r\n> > * get the voltage regulators from dts (main and IO-lines)\r\n> > * remove the set_vdd callback and do the job in the mmc core\r\n> > * add a send_init_stream callback to perform the 74 clock cycle\r\n> > sequence after power up\r\n> > * do a full power cycle (if possible) before the initialization. This ensures\r\n> > that the sd/MMC is in valid knwon state. Also it allows to check if the\r\n> power\r\n> > cycling works as expected.\r\n> > * disable the UHS modes is power cycling is not available/working. This\r\n> ensures\r\n> > that we won't be stuck if the UHS initialization fails.\r\n> > * select the appropriate signal voltage when selecting a mode (MMC\r\n> > only)\r\n> > * add a helper function to parse the generic dt bindings\r\n> > * add a helper function to send the tuning command\r\n> >\r\n> > Jean-Jacques Hiblot (17):\r\n> > mmc: dm: get the IO-line and main voltage regulators from the dts\r\n> > mmc: split mmc_startup()\r\n> > mmc: move the MMC startup for version above v4.0 in a separate\r\n> > function\r\n> > mmc: make ext_csd part of struct mmc\r\n> > mmc: add a function to read and test the ext csd (mmc >= 4)\r\n> > mmc: introduce mmc modes\r\n> > mmc: Add a function to dump the mmc capabilities\r\n> > mmc: use mmc modes to select the correct bus speed\r\n> > cmd: mmc: display the mode name and current bus speed in the mmc\r\n> info\r\n> > mmc: refactor SD startup to make it easier to support new modes\r\n> > mmc: refactor MMC startup to make it easier to support new modes\r\n> > mmc: Add a new callback function to perform the 74 clocks cycle\r\n> > sequence\r\n> > mmc: Add support for UHS modes\r\n> > mmc: disable UHS modes if Vcc cannot be switched on and off\r\n> > mmc: Change mode when switching to a boot partition\r\n> > mmc: use the right voltage level for MMC DDR and HS200 modes\r\n> > mmc: add a library function to send tuning command\r\n> >\r\n> > Kishon Vijay Abraham I (9):\r\n> > mmc: make mmc_set_ios() return status\r\n> > mmc: Enable signal voltage to be selected from mmc core\r\n> > mmc: add power cyle support in mmc core\r\n> > mmc: add a new mmc parameter to disable mmc clock\r\n> > mmc: disable the mmc clock during power off\r\n> > mmc: Add a execute_tuning() callback to the mmc operations.\r\n> > mmc: add HS200 support in MMC core\r\n> > mmc: Retry some MMC cmds on failure\r\n> > dm: mmc: Add a library function to parse generic dt binding\r\n> >\r\n> > cmd/mmc.c | 3 +-\r\n> > drivers/mmc/Kconfig | 23 +\r\n> > drivers/mmc/fsl_esdhc.c | 2 +-\r\n> > drivers/mmc/mmc-uclass.c | 87 +++\r\n> > drivers/mmc/mmc.c | 1511\r\n> ++++++++++++++++++++++++++++++++++------------\r\n> > include/mmc.h | 176 +++++-\r\n> > 6 files changed, 1417 insertions(+), 385 deletions(-)\r\n> >\r\n> \r\n> _______________________________________________\r\n> U-Boot mailing list\r\n> U-Boot@lists.denx.de\r\n> https://lists.denx.de/listinfo/u-boot","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=xilinx.onmicrosoft.com\n\theader.i=@xilinx.onmicrosoft.com header.b=\"JpH0ZXW5\"; \n\tdkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=sivadur@xilinx.com; "],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3yjDxK1Dy8z9s7B\n\tfor ;\n\tThu, 23 Nov 2017 20:46:43 +1100 (AEDT)","by lists.denx.de (Postfix, from userid 105)\n\tid B42F0C21EEB; Thu, 23 Nov 2017 09:46:38 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 3EE11C21D76;\n\tThu, 23 Nov 2017 09:46:34 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid EBEC1C21D76; Thu, 23 Nov 2017 09:46:32 +0000 (UTC)","from NAM01-BY2-obe.outbound.protection.outlook.com\n\t(mail-by2nam01on0045.outbound.protection.outlook.com [104.47.34.45])\n\tby lists.denx.de (Postfix) with ESMTPS id DB464C21C4A\n\tfor ; Thu, 23 Nov 2017 09:46:31 +0000 (UTC)","from CO1PR02MB240.namprd02.prod.outlook.com (10.242.165.155) by\n\tCO1PR02MB239.namprd02.prod.outlook.com (10.242.165.153) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.260.4; Thu, 23 Nov 2017 09:46:26 +0000","from CO1PR02MB240.namprd02.prod.outlook.com\n\t([fe80::30b5:5db9:be16:fba0]) by\n\tCO1PR02MB240.namprd02.prod.outlook.com\n\t([fe80::30b5:5db9:be16:fba0%14]) with mapi id 15.20.0260.004;\n\tThu, 23 Nov 2017 09:46:26 +0000"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=RCVD_IN_MSPIKE_H2,\n\tSPF_HELO_PASS, T_DKIM_INVALID autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=xilinx.onmicrosoft.com; s=selector1-xilinx-com;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=iDYSLQDz5tdMb7IP6NkiNpeFP/FI6kDMEmwdbytUdiw=;\n\tb=JpH0ZXW5qnJYO0+Tyf43bV8DWUzuQh7jDBLbYgqIzC9E3k7Cr3Y1SLlLYx0wS/oeDLt4rvifA+l04DVC93UZID37tazA7NPN+GcOTV/tAwtxIyqnYCfFIFlmXxChQBZ1tG31HWKaZXLhEaNe1W5HMZTpkYJwc/eNjsLxbvuXubs=","From":"Siva Durga Prasad Paladugu ","To":"Jaehoon Chung , Jean-Jacques Hiblot\n\t, \"trini@konsulko.com\" ,\n\t\"kishon@ti.com\" , \"sjg@chromium.org\" ","Thread-Topic":"[U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","Thread-Index":"AQHTSZcMLCowZZ48IkqB+/hglUpYQ6Mh62HA","Date":"Thu, 23 Nov 2017 09:46:25 +0000","Message-ID":"","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>\n\t<9b9d0abe-d638-1040-15bf-8b5661200af3@samsung.com>","In-Reply-To":"<9b9d0abe-d638-1040-15bf-8b5661200af3@samsung.com>","Accept-Language":"en-GB, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=xilinx.onmicrosoft.com\n\theader.i=@xilinx.onmicrosoft.com header.b=\"JpH0ZXW5\"; \n\tdkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=sivadur@xilinx.com; "],"x-originating-ip":"[182.72.145.30]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; CO1PR02MB239;\n\t6:h/QaSSvZe91sKOPHAAIK7Z8StK2Gk4npzawdAwGBldZuo+sAupuhZoLoeJtwtjwwkc/aaAMkBL+xdIzmUNYJzrzG4aasPLp+gXP6Wk3KIkKqXRDogbMsdfCR4+PhpO14Iha1Y1G97Ykh5PBz+f7gZ2jsrHjfViJA1vqLfM8f9jtJzHwWFNS0Zm2/juGZNEXTmGwP5sfK7A5OqYGUskIS0BKCme4AGfuVnWWNKHQP87GEhgIIO0AypbkCBPIXeanvY7+ryZ6Th6T+sHPIKjxnoLT7soK47SEJeVAk46pvngFJLXHbaUU54eEbGq0CGWu133Hymi5L1O7D1O5Uf3amRbMm+ig/mQwdbEyYx7qSXVo=;\n\t5:cRIW/HTuXY39J6sv2esjAzMRbdzGemwER7D1XMzuoYSah46bDNYhbOmGyMP991Y/g8gmc6wbWj14qo1tGh3pIEf9LtbJj7wkCX5lpQnkvEA0iislAziBvCowbFnocPJSdpXXvSZ0nCwK6wCLaplvLUMU1jmm2qIvBOlBNyMRbuo=;\n\t24:mNr9dBcNdQoBt5P+U8QZGlPL+ngKCBl3Ky5Ix4n5bW7NPqi/5BObeK4ggOaPi7EsMzfYdi3BxYP4FwLR4v6xtaAJYcpIkesGzv8defn7Mes=;\n\t7:ODY29/wH098hmgA8GaLn/t1D+y817o8fKUc9c6fKYkvQbeD/bxkiB5KmeixAP9o9FXuJs2VCOzguE6IpDZsPJZC/od+dhHYNhEKKWN20IKkzF2R83RvfUlT8u+40KQAomqFiuDsScgZIa25T0AN54n+ZjAHvzpEZuEcAkrc7VG+iZZl4gjc+w6KMr1Xjq/keb0hdN+dvGQs7CF+q16jwferZQL7qA0OZIdqr1o33cfg63AA9RzoXZSleEw5YyY7M","x-ms-exchange-antispam-srfa-diagnostics":"SSOS;","x-ms-office365-filtering-ht":"Tenant","x-ms-office365-filtering-correlation-id":"f30bf702-9a31-4600-6d33-08d53257105e","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(4534020)(4602075)(7168020)(4627115)(201703031133081)(201702281549075)(48565401081)(5600024)(4604075)(2017052603199);\n\tSRVR:CO1PR02MB239; ","x-ms-traffictypediagnostic":"CO1PR02MB239:","x-microsoft-antispam-prvs":"","x-exchange-antispam-report-test":"UriScan:(190756311086443)(31051911155226)(16074681357397); ","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(10201501046)(3231022)(93006095)(93001095)(3002001)(100000703101)(100105400095)(6055026)(6041248)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123560025)(20161123555025)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:CO1PR02MB239; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:CO1PR02MB239; ","x-forefront-prvs":"05009853EF","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(376002)(366004)(39860400002)(346002)(199003)(189002)(13464003)(24454002)(54356999)(68736007)(4326008)(2906002)(33656002)(86362001)(2201001)(2501003)(14454004)(7696004)(6506006)(55236003)(3846002)(66066001)(6116002)(102836003)(5660300001)(6436002)(55016002)(53546010)(8936002)(229853002)(99286004)(81166006)(81156014)(53936002)(106356001)(110136005)(316002)(25786009)(107886003)(2950100002)(2900100001)(50986999)(105586002)(966005)(5250100002)(6246003)(478600001)(76176999)(101416001)(3280700002)(6306002)(7736002)(74316002)(9686003)(3660700001)(97736004)(189998001)(54906003)(305945005);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:CO1PR02MB239;\n\tH:CO1PR02MB240.namprd02.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; A:1; MX:3; LANG:en; ","received-spf":"None (protection.outlook.com: xilinx.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","MIME-Version":"1.0","X-OriginatorOrg":"xilinx.com","X-MS-Exchange-CrossTenant-Network-Message-Id":"f30bf702-9a31-4600-6d33-08d53257105e","X-MS-Exchange-CrossTenant-originalarrivaltime":"23 Nov 2017 09:46:25.6116\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"657af505-d5df-48d0-8300-c31994686c5c","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"CO1PR02MB239","Cc":"\"u-boot@lists.denx.de\" ,\n\tnofooter ","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and\n\tUHS\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1809123,"web_url":"http://patchwork.ozlabs.org/comment/1809123/","msgid":"<3442fc7a-bbf9-0fc7-9836-e5aca7792570@gmail.com>","list_archive_url":null,"date":"2017-11-23T09:51:17","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","submitter":{"id":1124,"url":"http://patchwork.ozlabs.org/api/people/1124/","name":"Marek Vasut","email":"marek.vasut@gmail.com"},"content":"On 11/23/2017 10:46 AM, Siva Durga Prasad Paladugu wrote:\n> Hi Jaehoon,\n> \n>> -----Original Message-----\n>> From: U-Boot [mailto:u-boot-bounces@lists.denx.de] On Behalf Of Jaehoon\n>> Chung\n>> Sent: Friday, October 20, 2017 5:02 PM\n>> To: Jean-Jacques Hiblot ; trini@konsulko.com;\n>> kishon@ti.com; sjg@chromium.org\n>> Cc: u-boot@lists.denx.de\n>> Subject: Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and\n>> UHS modes\n>>\n>> Dear JJ\n>>\n>> On 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\n>>> This series brings support for HS200 and UHS modes to the mmc core.\n>>> It has been tested with the hsmmc driver on several platforms (DRA7,\n>>> AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some\n>>> modifications are required in the host driver to take advantage of\n>>> this (voltage switching, tuning). The changes to the host driver will\n>>> be posted a another series as this one is already long enough.\n>>>\n>>> The series starts with a small refactoring of th sd/mmc startup. The\n>>> first 5 commits are mostly moving code around with little or no\n>> functionnal change.\n>>>\n>>> Then the notion of \"mode\" is introduced. Until now, this information\n>>> wasn't kept in struct mmc. Only the clock and a flag for ddr was kept.\n>>> Later the mode information will be used to select the clock frequency,\n>>> the ddr flag and the tuning procedure. It will be also be check against the\n>> host capabilities.\n>>>\n>>> Then comes the big refactoring job in:\n>>> \"mmc: refactor MMC startup to make it easier to support new modes\"\n>> and\n>>> \"mmc: refactor SD startup to make it easier to support new modes\"\n>>> Since the number of modes is increasing, it makes sense to try them in\n>>> a more organized way. those commits use a list of supported modes and\n>>> iterate through them to find the best working one. It also allows to\n>>> switch more easilly from one mode to another (switching from HS200 to\n>>> DDR52 to access boot partitions for example)\n>>>\n>>> Then there are a couple of new callback added to:\n>>> - send the initialization stream (74 clock cycles)\n>>> - wait while the card is busy (used during UHS voltage switching). checking\n>> is\n>>> done on dat0.\n>>> - select the IO voltage\n>>>\n>>> Then Power cycle is added. Without power cycle, if a UHS card fails to\n>>> enumerate in UHS mode, it can't fall back to high speed mode and card\n>> enumeration will fail.\n>>> Also in case of a reset (as opposed to a power on), it ensures that\n>>> the SDCard is in clean state before re-doing the initialization.\n>>>\n>>> And finally the last commits add the support for HS200 and UHS.\n>>>\n>>> With this in place and the required changes in the HSMMC host driver\n>>> (including DMA), we observe significant improvements in the\n>> performances on a DRA72 evm:\n>>> eMMC HS200: 124 MB/s\n>>> eMMC DDR52: 78 MB/s\n>>> sd SDR104: 71 MB/s\n>>> sd SDR50: 44 MB/s\n>>\n>> Applied to u-boot-mmc! Thanks!\n>> I will apply the patches relevant to this patchset. Sorry for late applying this.\n> \n> I would like to test UHS and HS200 series on ZynqMP platform. \n> Is it the branch \" testing-uhs-supporting\" that I should be using for testing it?\n> Please confirm.\n\nThat should do, plus look at the uniphier/renesas SDHI patches posted to\nthe ML adding UHS200 support, that should get you started with the\narasan IP too.","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"qK0z6Czg\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3yjF2j3fHXz9s7B\n\tfor ;\n\tThu, 23 Nov 2017 20:51:25 +1100 (AEDT)","by lists.denx.de (Postfix, from userid 105)\n\tid 2E81FC21E4C; Thu, 23 Nov 2017 09:51:24 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 60660C21D76;\n\tThu, 23 Nov 2017 09:51:21 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 1A3E0C21D76; Thu, 23 Nov 2017 09:51:20 +0000 (UTC)","from mail-wm0-f43.google.com (mail-wm0-f43.google.com\n\t[74.125.82.43])\n\tby lists.denx.de (Postfix) with ESMTPS id 8D25FC21C4A\n\tfor ; Thu, 23 Nov 2017 09:51:19 +0000 (UTC)","by mail-wm0-f43.google.com with SMTP id y80so15250987wmd.0\n\tfor ; Thu, 23 Nov 2017 01:51:19 -0800 (PST)","from [192.168.1.4] (ip-86-49-107-50.net.upcbroadband.cz.\n\t[86.49.107.50]) by smtp.gmail.com with ESMTPSA id\n\tv198sm2951074wmv.4.2017.11.23.01.51.17\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tThu, 23 Nov 2017 01:51:18 -0800 (PST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=FREEMAIL_FROM,\n\tRCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=subject:to:cc:references:from:message-id:date:user-agent\n\t:mime-version:in-reply-to:content-language:content-transfer-encoding; \n\tbh=tEIbGTYkJQaQRjtD2q+Wm3WBCW/wHgaV5L5FrSmzG3I=;\n\tb=qK0z6CzgfyfC+B/xcFOyqHCpIcg+8lxnIGnpYtKlz3Bbt+EosxFQiPdChDEiaikPdX\n\tdQeCcf6bErr/AJNUD/1kqG8yVZpaVd6O+uDIOTPCQGdvxI/OB5xXcc5EVz+d/oUhdqn2\n\tSxKuahkp5fetE5zH9hvPy/3TE4XCRY8qhQoHu6M/X1fEAaG6OCGAmgSNK+Jaa+pZPC3S\n\tGggbIiVMLrN49JOSfB0exXt4oVI3KWZyaVHROZZLtUeP4bbRm/u4UCNM1KumIG9zQCQU\n\tKaW0yOzltImH3nQOkM41FXI+o70xLKS35VKGY/uDPl5CvUGjJwslA84GDmxXcOVaqwZ2\n\tF/KA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:subject:to:cc:references:from:message-id:date\n\t:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=tEIbGTYkJQaQRjtD2q+Wm3WBCW/wHgaV5L5FrSmzG3I=;\n\tb=EsfRcfySjBlmOrr0GiCJh5CGsPKUNDYFuUymhjvwDXDCHVKe767ejt7kK0HUDf0lZE\n\ty4PvauJ82FYZsoR1akxWgZ1nCgH1DtocuMR4eWpIdgtGVw8rr+xvU5b3y9fU89k/9aSW\n\tbLTvDzyiLDQtlaN7K7GAZH+FRIR3b1wNQOXMcQGebalCcQzt3tMKqR/3Ib/4Bs6zIFVT\n\tQkDOfykjp0DcmdYyY+2VdwkkN6l5jct7d93QDjZsmgHKM5ne4EwIk47pVGWRNx+jHmk8\n\t//0iJ+oSrTTakYcn2qUdyH0KbeDPrZRIALRHnJuIVo5BK0ukpj+rcakZ6ZDkyb3WT5ZK\n\tpTNw==","X-Gm-Message-State":"AJaThX7gSbTowe9LKkxGrOAR+l15BnlEm6mQDi1xWFORSlObwWjYMSDB\n\tWuvYBT3+Sn5NV1rZqydAwto=","X-Google-Smtp-Source":"AGs4zMblhuShTFF6cF+46lqyZ82lOzgdFXsO3g0xVrv9g5e3Nxzo1UDm3KQpn6OxAHZkzcEOjAndqw==","X-Received":"by 10.28.142.20 with SMTP id q20mr6120136wmd.37.1511430679067;\n\tThu, 23 Nov 2017 01:51:19 -0800 (PST)","To":"Siva Durga Prasad Paladugu ,\n\tJaehoon Chung , Jean-Jacques Hiblot\n\t, \"trini@konsulko.com\" ,\n\t\"kishon@ti.com\" , \"sjg@chromium.org\" ","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>\n\t<9b9d0abe-d638-1040-15bf-8b5661200af3@samsung.com>\n\t","From":"Marek Vasut ","Message-ID":"<3442fc7a-bbf9-0fc7-9836-e5aca7792570@gmail.com>","Date":"Thu, 23 Nov 2017 10:51:17 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.4.0","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","Cc":"\"u-boot@lists.denx.de\" ,\n\tnofooter ","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1810474,"web_url":"http://patchwork.ozlabs.org/comment/1810474/","msgid":"<648b0042-90a2-81d9-d4e0-2fe893990802@samsung.com>","list_archive_url":null,"date":"2017-11-27T10:13:03","subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","submitter":{"id":8006,"url":"http://patchwork.ozlabs.org/api/people/8006/","name":"Jaehoon Chung","email":"jh80.chung@samsung.com"},"content":"On 11/23/2017 06:46 PM, Siva Durga Prasad Paladugu wrote:\n> Hi Jaehoon,\n> \n>> -----Original Message-----\n>> From: U-Boot [mailto:u-boot-bounces@lists.denx.de] On Behalf Of Jaehoon\n>> Chung\n>> Sent: Friday, October 20, 2017 5:02 PM\n>> To: Jean-Jacques Hiblot ; trini@konsulko.com;\n>> kishon@ti.com; sjg@chromium.org\n>> Cc: u-boot@lists.denx.de\n>> Subject: Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and\n>> UHS modes\n>>\n>> Dear JJ\n>>\n>> On 09/21/2017 11:29 PM, Jean-Jacques Hiblot wrote:\n>>> This series brings support for HS200 and UHS modes to the mmc core.\n>>> It has been tested with the hsmmc driver on several platforms (DRA7,\n>>> AM57x, AM437x, beaglebone black, Atmel SAMA5D3 xplained). Some\n>>> modifications are required in the host driver to take advantage of\n>>> this (voltage switching, tuning). The changes to the host driver will\n>>> be posted a another series as this one is already long enough.\n>>>\n>>> The series starts with a small refactoring of th sd/mmc startup. The\n>>> first 5 commits are mostly moving code around with little or no\n>> functionnal change.\n>>>\n>>> Then the notion of \"mode\" is introduced. Until now, this information\n>>> wasn't kept in struct mmc. Only the clock and a flag for ddr was kept.\n>>> Later the mode information will be used to select the clock frequency,\n>>> the ddr flag and the tuning procedure. It will be also be check against the\n>> host capabilities.\n>>>\n>>> Then comes the big refactoring job in:\n>>> \"mmc: refactor MMC startup to make it easier to support new modes\"\n>> and\n>>> \"mmc: refactor SD startup to make it easier to support new modes\"\n>>> Since the number of modes is increasing, it makes sense to try them in\n>>> a more organized way. those commits use a list of supported modes and\n>>> iterate through them to find the best working one. It also allows to\n>>> switch more easilly from one mode to another (switching from HS200 to\n>>> DDR52 to access boot partitions for example)\n>>>\n>>> Then there are a couple of new callback added to:\n>>> - send the initialization stream (74 clock cycles)\n>>> - wait while the card is busy (used during UHS voltage switching). checking\n>> is\n>>> done on dat0.\n>>> - select the IO voltage\n>>>\n>>> Then Power cycle is added. Without power cycle, if a UHS card fails to\n>>> enumerate in UHS mode, it can't fall back to high speed mode and card\n>> enumeration will fail.\n>>> Also in case of a reset (as opposed to a power on), it ensures that\n>>> the SDCard is in clean state before re-doing the initialization.\n>>>\n>>> And finally the last commits add the support for HS200 and UHS.\n>>>\n>>> With this in place and the required changes in the HSMMC host driver\n>>> (including DMA), we observe significant improvements in the\n>> performances on a DRA72 evm:\n>>> eMMC HS200: 124 MB/s\n>>> eMMC DDR52: 78 MB/s\n>>> sd SDR104: 71 MB/s\n>>> sd SDR50: 44 MB/s\n>>\n>> Applied to u-boot-mmc! Thanks!\n>> I will apply the patches relevant to this patchset. Sorry for late applying this.\n> \n> I would like to test UHS and HS200 series on ZynqMP platform. \n> Is it the branch \" testing-uhs-supporting\" that I should be using for testing it?\n> Please confirm.\n\nHi, i applied these patch into u-boot-mmc/master. So if you want to check, just use master branch.\nIf there are problem, let me know, plz. and send patches, plz.\nThen it's useful to me. \n\n(I will remove the testing-uhs-supporting\" branch, because it's already merged into master branch.)\n\nBest Regards,\nJaehoon Chung\n\n> \n> Thanks,\n> Siva\n> \n>>\n>> Best Regards,\n>> Jaehoon Chung\n>>\n>>>\n>>> cheers,\n>>>\n>>> Jean-Jacques\n>>>\n>>> changes since v1:\n>>> Changes take in account all the comment made during the first round of\n>> review.\n>>> But it also include a rework of the handling of the voltage supplies\n>>> and 2 new helper functions that can be used by the host driver during\n>>> tuning and dts parsing.\n>>>\n>>> Here is a detailed list:\n>>> * rebase on u-boot/master.\n>>> * add the description for the new functions/members\n>>> * fix the typos found in the reviews\n>>> * limit the new features to DM_MMC only\n>>> * fix the new checkpatch warnings/errors (is it now more strict ?)\n>>> * get the voltage regulators from dts (main and IO-lines)\n>>> * remove the set_vdd callback and do the job in the mmc core\n>>> * add a send_init_stream callback to perform the 74 clock cycle\n>>> sequence after power up\n>>> * do a full power cycle (if possible) before the initialization. This ensures\n>>> that the sd/MMC is in valid knwon state. Also it allows to check if the\n>> power\n>>> cycling works as expected.\n>>> * disable the UHS modes is power cycling is not available/working. This\n>> ensures\n>>> that we won't be stuck if the UHS initialization fails.\n>>> * select the appropriate signal voltage when selecting a mode (MMC\n>>> only)\n>>> * add a helper function to parse the generic dt bindings\n>>> * add a helper function to send the tuning command\n>>>\n>>> Jean-Jacques Hiblot (17):\n>>> mmc: dm: get the IO-line and main voltage regulators from the dts\n>>> mmc: split mmc_startup()\n>>> mmc: move the MMC startup for version above v4.0 in a separate\n>>> function\n>>> mmc: make ext_csd part of struct mmc\n>>> mmc: add a function to read and test the ext csd (mmc >= 4)\n>>> mmc: introduce mmc modes\n>>> mmc: Add a function to dump the mmc capabilities\n>>> mmc: use mmc modes to select the correct bus speed\n>>> cmd: mmc: display the mode name and current bus speed in the mmc\n>> info\n>>> mmc: refactor SD startup to make it easier to support new modes\n>>> mmc: refactor MMC startup to make it easier to support new modes\n>>> mmc: Add a new callback function to perform the 74 clocks cycle\n>>> sequence\n>>> mmc: Add support for UHS modes\n>>> mmc: disable UHS modes if Vcc cannot be switched on and off\n>>> mmc: Change mode when switching to a boot partition\n>>> mmc: use the right voltage level for MMC DDR and HS200 modes\n>>> mmc: add a library function to send tuning command\n>>>\n>>> Kishon Vijay Abraham I (9):\n>>> mmc: make mmc_set_ios() return status\n>>> mmc: Enable signal voltage to be selected from mmc core\n>>> mmc: add power cyle support in mmc core\n>>> mmc: add a new mmc parameter to disable mmc clock\n>>> mmc: disable the mmc clock during power off\n>>> mmc: Add a execute_tuning() callback to the mmc operations.\n>>> mmc: add HS200 support in MMC core\n>>> mmc: Retry some MMC cmds on failure\n>>> dm: mmc: Add a library function to parse generic dt binding\n>>>\n>>> cmd/mmc.c | 3 +-\n>>> drivers/mmc/Kconfig | 23 +\n>>> drivers/mmc/fsl_esdhc.c | 2 +-\n>>> drivers/mmc/mmc-uclass.c | 87 +++\n>>> drivers/mmc/mmc.c | 1511\n>> ++++++++++++++++++++++++++++++++++------------\n>>> include/mmc.h | 176 +++++-\n>>> 6 files changed, 1417 insertions(+), 385 deletions(-)\n>>>\n>>\n>> _______________________________________________\n>> U-Boot mailing list\n>> U-Boot@lists.denx.de\n>> https://lists.denx.de/listinfo/u-boot","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=samsung.com header.i=@samsung.com\n\theader.b=\"qNKAUjfF\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3yljLC0kWBz9s7F\n\tfor ;\n\tMon, 27 Nov 2017 21:13:22 +1100 (AEDT)","by lists.denx.de (Postfix, from userid 105)\n\tid AD5BFC21DE8; Mon, 27 Nov 2017 10:13:15 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 752CCC21D56;\n\tMon, 27 Nov 2017 10:13:11 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 33AB3C21D56; Mon, 27 Nov 2017 10:13:09 +0000 (UTC)","from mailout1.samsung.com (mailout1.samsung.com [203.254.224.24])\n\tby lists.denx.de (Postfix) with ESMTPS id 0852BC21C4C\n\tfor ; Mon, 27 Nov 2017 10:13:08 +0000 (UTC)","from epcas1p2.samsung.com (unknown [182.195.41.46])\n\tby mailout1.samsung.com (KnoxPortal) with ESMTP id\n\t20171127101304epoutp0191b266415f14a10a0c70df59087a4516~66bQsD3Wu2102521025epoutp01b;\n\tMon, 27 Nov 2017 10:13:04 +0000 (GMT)","from epsmges2p4.samsung.com (unknown [182.195.40.68]) by\n\tepcas1p3.samsung.com (KnoxPortal) with ESMTP id\n\t20171127101304epcas1p32b92ab19bb1024ce0df2391c7017784e~66bQNyNB61413614136epcas1p3B;\n\tMon, 27 Nov 2017 10:13:04 +0000 (GMT)","from epcas2p1.samsung.com ( [182.195.41.53]) by\n\tepsmges2p4.samsung.com (Symantec Messaging Gateway) with SMTP id\n\t76.3E.04319.035EB1A5; Mon, 27 Nov 2017 19:13:04 +0900 (KST)","from epsmgms2p2new.samsung.com (unknown [182.195.42.143]) by\n\tepcas2p2.samsung.com (KnoxPortal) with ESMTP id\n\t20171127101303epcas2p226afaf6993fd1ce119cdfe37389bdbd8~66bPaMNL12597125971epcas2p2J;\n\tMon, 27 Nov 2017 10:13:03 +0000 (GMT)","from epmmp2 ( [203.254.227.17]) by epsmgms2p2new.samsung.com\n\t(Symantec Messaging Gateway) with SMTP id BC.BC.03909.F25EB1A5;\n\tMon, 27 Nov 2017 19:13:03 +0900 (KST)","from [10.113.62.216] by mmp2.samsung.com (Oracle Communications\n\tMessaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTPA id\n\t<0P0200F09N1QS570@mmp2.samsung.com>;\n\tMon, 27 Nov 2017 19:13:03 +0900 (KST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=RCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL, SPF_HELO_PASS, T_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Filter":"OpenDKIM Filter v2.11.0 mailout1.samsung.com\n\t20171127101304epoutp0191b266415f14a10a0c70df59087a4516~66bQsD3Wu2102521025epoutp01b","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com;\n\ts=mail20170921; t=1511777585;\n\tbh=qmo1zjpqtq/DAwj7WBsjIxkAGa0utIP5LeiiOhpr8ys=;\n\th=Subject:To:Cc:From:Date:In-reply-to:References:From;\n\tb=qNKAUjfFgDpjQipDEhDv1GHRTg30dS9LpUxjy3IfOn1PhVVOZFjRVArW0Fr7y3Ocd\n\toy2j1sS6fL3bD0FapZHpOrROaipLm8JXlZBgDNNSNTiTcC47VQ7F+2XMDYMdqbeRNK\n\tAxiBXmGYwkiWaqZCohn4ePOjfBTqmje43Aeyrrq0=","X-AuditID":"b6c32a48-92fff700000010df-89-5a1be53079c7","To":"Siva Durga Prasad Paladugu , Jean-Jacques Hiblot\n\t, \"trini@konsulko.com\" ,\n\t\"kishon@ti.com\" , \"sjg@chromium.org\" ","From":"Jaehoon Chung ","Message-id":"<648b0042-90a2-81d9-d4e0-2fe893990802@samsung.com>","Date":"Mon, 27 Nov 2017 19:13:03 +0900","User-Agent":"Mozilla/5.0 (X11; Linux i686; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-version":"1.0","In-reply-to":"","Content-language":"en-US","X-Brightmail-Tracker":["H4sIAAAAAAAAA02Se0gUcRDH+e1uu6t5sZ2ag1IdmxVGXnfneW7RVVLJQf5hCZL2sEUXlby9\n\t6/YM6w8xUquzp1mUlY8w8NVTMXuIpdUhoaWRaJhkqKkgYZ5mRtGde4H/fWfmM8x8h6FxpYMM\n\tpjNEu2AT+UyW9CUa28IiwzXDIUma2sYgLq+9COPeD58lueaXLsTlPq0iuZmGRsRdKaqnuInm\n\tM9Q2ynQjt4swjb+ooUwd/U3I5Ox9jJmap34TcYuShM3pAp8q2FSCmGJJzRDTjOyu+OTtyZEG\n\tjTZcu5GLYlUibxaM7I7YuPCYjEz3HqzqKJ+Z5U7F8ZLEbtiy2WbJsguqdItkN7L7tFqdWquJ\n\tUut0OrU+4sAmXaQbOSSkV5dFWSu47FLXFJWLzq13IB8aGD20T5ZiDuRLK5kmBH/rXIQc/ETw\n\t9dMM+k99mxj0Ug8QTM8+o+RgAEF3V/U85c/shrLKcuQpBDBdCM5PFuMORNM4Ew/3f+z3MCSz\n\tHh7PODGPVjBb4OPkK8KjCWY1DDnrSI8OZBKgem4cycxSmL08MM/4MAeg8ELBPIMzYTDqKiJk\n\tHQQn8/u8eiXU103gnh2A+UDC7TuFuGxhB7iu9pGy9odxZwMl6xAYqXnotXkKQccNf7m5GEHV\n\ttUJMLkTASIUDkycsgdNtfyiPMWAUcLpAKSMmuPml1otHw7szTu+5rmNQ2zKGXUQrShYYKllg\n\tomSBiZIFJsoRUYOWCVbJnCZIOqteLfFmKUtMU6dYzI/Q/B+uMzWhF52xrYihEeunUMWGJCkX\n\t8UelY+ZWBDTOBiiWD7pTilT+2HHBZkm2ZWUKUiuKdB/8Eh4cmGJxf7VoT9bqN2r0BoMuysC5\n\t3y1IkdPYs1fJpPF24bAgWAXb/z6M9gnORRU9qw739JGxcw05052fC6a/d9es6n0X1h069NTa\n\ts4vI2Skeya/oDx37NRk4JA0lRtQPbPXTc773XNygJTHkedv3sre6yutW/YlPpWvzFo8+N+Ss\n\tsBjjD/5QvjbffbKm13jtSz8196ZTPFW8PCY7OiEmuuyzekN2y+8Tt5RTe+IKglhCSue163Cb\n\txP8DwIB9r50DAAA=","H4sIAAAAAAAAA+NgFnrFLMWRmVeSWpSXmKPExsVy+t9jQV39p9JRBptOylu0nJzEZHHhaQ+b\n\txd6DXxgtGnatYLP4tmUbo8XUSZvZLd7u7WR3YPeY3XCRxePVgVXsHmfv7GD0OH5jO5PH3s+/\n\tWQJYo7hsUlJzMstSi/TtErgyVs43L1hoUTHvy2f2BsZenS5GTg4JAROJ528fMoHYQgLrGCU6\n\t7/F0MXIB2Q8YJS50fGHrYuTgEBbwl5jf5wUSFxG4zCjx+NoLRpAGZoFgiZmPZ7JCNMxmkjg7\n\tdSM7SIJNQEdi+7fjYFN5Bewkrn48wgJiswioSjw5voYNxBYVCJPob/7LDFEjKPFj8j2wGk6B\n\tWImdl5awgixmFlCXmDIlF2KXuERz600WCFteYvOat8wTGAVmIemehdAxC0nHLCQdCxhZVjFK\n\tphYU56bnFhsVGOWllusVJ+YWl+al6yXn525iBMbAtsNa/TsYHy+JP8QowMGoxMNb4ScdJcSa\n\tWFZcmXuIUYKDWUmEV/YhUIg3JbGyKrUoP76oNCe1+BCjNAeLkjgvf/6xSCGB9MSS1OzU1ILU\n\tIpgsEwenVAOjleme81d958j7eS7R/m/reb3lh/S8uQHfZJMDt4rNr1xymnlKbF+T0o/cE6uZ\n\tHwaxFshYLJE7tkrmStcJhfTvMj7LXwjm3rMy5S+TvLG487Si+QzPyx58Cm9lupYYH1t6Xab2\n\tOW/s85B9t9U+HJdvYEgouxf15wPjuU/NM4O3eKmH84sWzFBRYinOSDTUYi4qTgQACGCv230C\n\tAAA="],"X-CMS-MailID":"20171127101303epcas2p226afaf6993fd1ce119cdfe37389bdbd8","X-Msg-Generator":"CA","CMS-TYPE":"102P","DLP-Filter":"Pass","X-CFilter-Loop":"Reflected","X-CMS-RootMailID":"20170921143028epcas3p3fce32811279a7e0bf9941e4bbc621ddd","X-RootMTR":"20170921143028epcas3p3fce32811279a7e0bf9941e4bbc621ddd","References":"\n\t<1506004213-22620-1-git-send-email-jjhiblot@ti.com>\n\t<9b9d0abe-d638-1040-15bf-8b5661200af3@samsung.com>\n\t","Cc":"\"u-boot@lists.denx.de\" ,\n\tnofooter ","Subject":"Re: [U-Boot] [PATCH v2 00/26] mmc: Add support for HS200 and UHS\n\tmodes","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]