From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753390AbbJNBxJ (ORCPT ); Tue, 13 Oct 2015 21:53:09 -0400 Received: from mailout1.w1.samsung.com ([210.118.77.11]:29886 "EHLO mailout1.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753019AbbJNBxG (ORCPT ); Tue, 13 Oct 2015 21:53:06 -0400 X-AuditID: cbfec7f5-f794b6d000001495-af-561db57e79d0 Subject: Re: [PATCH 3/3] ARM: dts: exynos5422-odroidxu3: Added UHS-I bus speed support To: Jaehoon Chung , Alim Akhtar References: <1444578364-1384-1-git-send-email-linux.amoon@gmail.com> <1444578364-1384-3-git-send-email-linux.amoon@gmail.com> <561B48B7.1040201@samsung.com> <561C4CEE.3050905@samsung.com> <561C7DF1.6000609@samsung.com> <561D9ACE.6050103@samsung.com> <561DACC6.9040209@samsung.com> <561DB290.5070903@samsung.com> Cc: Anand Moon , Kukjin Kim , Javier Martinez Canillas , Lukasz Majewski , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-samsung-soc@vger.kernel.org" , Linux Kernel , Douglas Anderson From: Krzysztof Kozlowski X-Enigmail-Draft-Status: N1110 Message-id: <561DB577.8080808@samsung.com> Date: Wed, 14 Oct 2015 10:52:55 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-version: 1.0 In-reply-to: <561DB290.5070903@samsung.com> Content-type: text/plain; charset=utf-8 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmplkeLIzCtJLcpLzFFi42I5/e/4Zd26rbJhBvdfC1osvVVtMf/IOVaL s8sOslm8ebuGyeLGrzZWi9cvDC36H79mtnjzcDOjxabH11gtLu+aw2Yx4/w+Jot1G2+xO/B4 zG64yOKxc9Zddo9NqzrZPDYvqffY0g/k9W1ZxejxeZNcAHsUl01Kak5mWWqRvl0CV8bTOQ9Y Ci7yV6x/eY6tgfEYTxcjB4eEgInE1DWhXYycQKaYxIV769m6GLk4hASWMkrsbfkG5XxhlHjc MJURpEFYIELixFUjkAYRAX+J48/XsYHYQgL3WST2rioGqWcWuMUs8bJxHSNIgk3AWGLz8iVs EBvkJHq7J7GA2LwCWhINL1+C1bAIqEos/rmTCcQWBZo/cUIDK0SNoMSPyffA6jkFtCX6VjWx gdzALKAuMWVKLkiYWUBeYvOat8wTGAVnIemYhVA1C0nVAkbmVYyiqaXJBcVJ6blGesWJucWl eel6yfm5mxghkfN1B+PSY1aHGAU4GJV4eA+slQ0TYk0sK67MPcQowcGsJMKbugQoxJuSWFmV WpQfX1Sak1p8iFGag0VJnHfmrvchQgLpiSWp2ampBalFMFkmDk6pBsa6+6IFfXIWdmt5ItXi 7t85+iY6mundW87tn46oB5bmrNw5TXW33pYftSmZp9YuOvogoeNrzYPrnIpz/DOtDkwr/vzl Nqt1uwDnh6n6Sx0Xp9WkK4jK7z3FGeKzkd+n2eXTl98BxkL7Hq3wq524+so1ZgmJDyKXIt0M 78e/Een3r1v/oi/T6p8SS3FGoqEWc1FxIgBtkzPRmAIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 14.10.2015 10:40, Jaehoon Chung wrote: > On 10/14/2015 10:15 AM, Krzysztof Kozlowski wrote: >> On 14.10.2015 10:06, Alim Akhtar wrote: >>> +Doug >>> Hello, >>> AFAIR, dw_mmc host controller does support UHS-I [1], specially SDR50 >>> and SDR104 modes. >>> >>> [1]: http://www.spinics.net/lists/linux-mmc/msg28186.html >>> >>> What I remember is, one need to set "broken-cd" property also in order >>> to make it work because of the vqmmc and vmmc connection on board. I >>> didn't find the link right now, but you can search on the web, there >>> was a long discussion about handling this. >>> Have not checked it recently, so not sure if this got broken somehow. >>> >> >> Please, don't top post. >> >> I am not a SD/MMC specialist (I do not feel enough confident in its >> internals) but the datasheet for 5422 does not mention UHS. However it >> mentions "High Speed DDR Mode with 200 MHz clock rate (HS400)". This >> does not look like UHS... > > You're right. It's not UHS mode. it mentions eMMC's HS400 mode. > UHS and HS400 are difference mode. > > eMMC mode are supported > >> https://www.sdcard.org/developers/overview/bus_speed/ >> This of course is not a definite proof that 5422 does not support UHS. I >> am just saying that I couldn't find any information that *it does*. > > Well, I think you want to know whether it supported or not. > Then you can find the information at User manual. > In mobile storage part of User manual, it described the overview. > Mobile storage host supports these specification: > - Secure Digital memory (SD memory version 3.0) > - Secure Digital I/O SDIO (SDIO version 3.0) > - etc... > > SD3.0 is supported UHS-I mode. > If user manual of exynos5422 is mentioned this specification, it should be supported UHS-I mode. Thanks! Indeed datasheet mentions that supported SD and SDIO is version 3.0 (not 3.01... which seems irrelevant because 3.0 introduced UHS-I, right?). Best regards, Krzysztof From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH 3/3] ARM: dts: exynos5422-odroidxu3: Added UHS-I bus speed support Date: Wed, 14 Oct 2015 10:52:55 +0900 Message-ID: <561DB577.8080808@samsung.com> References: <1444578364-1384-1-git-send-email-linux.amoon@gmail.com> <1444578364-1384-3-git-send-email-linux.amoon@gmail.com> <561B48B7.1040201@samsung.com> <561C4CEE.3050905@samsung.com> <561C7DF1.6000609@samsung.com> <561D9ACE.6050103@samsung.com> <561DACC6.9040209@samsung.com> <561DB290.5070903@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <561DB290.5070903@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org To: Jaehoon Chung , Alim Akhtar Cc: Anand Moon , Kukjin Kim , Javier Martinez Canillas , Lukasz Majewski , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-samsung-soc@vger.kernel.org" , Linux Kernel , Douglas Anderson List-Id: devicetree@vger.kernel.org On 14.10.2015 10:40, Jaehoon Chung wrote: > On 10/14/2015 10:15 AM, Krzysztof Kozlowski wrote: >> On 14.10.2015 10:06, Alim Akhtar wrote: >>> +Doug >>> Hello, >>> AFAIR, dw_mmc host controller does support UHS-I [1], specially SDR50 >>> and SDR104 modes. >>> >>> [1]: http://www.spinics.net/lists/linux-mmc/msg28186.html >>> >>> What I remember is, one need to set "broken-cd" property also in order >>> to make it work because of the vqmmc and vmmc connection on board. I >>> didn't find the link right now, but you can search on the web, there >>> was a long discussion about handling this. >>> Have not checked it recently, so not sure if this got broken somehow. >>> >> >> Please, don't top post. >> >> I am not a SD/MMC specialist (I do not feel enough confident in its >> internals) but the datasheet for 5422 does not mention UHS. However it >> mentions "High Speed DDR Mode with 200 MHz clock rate (HS400)". This >> does not look like UHS... > > You're right. It's not UHS mode. it mentions eMMC's HS400 mode. > UHS and HS400 are difference mode. > > eMMC mode are supported > >> https://www.sdcard.org/developers/overview/bus_speed/ >> This of course is not a definite proof that 5422 does not support UHS. I >> am just saying that I couldn't find any information that *it does*. > > Well, I think you want to know whether it supported or not. > Then you can find the information at User manual. > In mobile storage part of User manual, it described the overview. > Mobile storage host supports these specification: > - Secure Digital memory (SD memory version 3.0) > - Secure Digital I/O SDIO (SDIO version 3.0) > - etc... > > SD3.0 is supported UHS-I mode. > If user manual of exynos5422 is mentioned this specification, it should be supported UHS-I mode. Thanks! Indeed datasheet mentions that supported SD and SDIO is version 3.0 (not 3.01... which seems irrelevant because 3.0 introduced UHS-I, right?). Best regards, Krzysztof From mboxrd@z Thu Jan 1 00:00:00 1970 From: k.kozlowski@samsung.com (Krzysztof Kozlowski) Date: Wed, 14 Oct 2015 10:52:55 +0900 Subject: [PATCH 3/3] ARM: dts: exynos5422-odroidxu3: Added UHS-I bus speed support In-Reply-To: <561DB290.5070903@samsung.com> References: <1444578364-1384-1-git-send-email-linux.amoon@gmail.com> <1444578364-1384-3-git-send-email-linux.amoon@gmail.com> <561B48B7.1040201@samsung.com> <561C4CEE.3050905@samsung.com> <561C7DF1.6000609@samsung.com> <561D9ACE.6050103@samsung.com> <561DACC6.9040209@samsung.com> <561DB290.5070903@samsung.com> Message-ID: <561DB577.8080808@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 14.10.2015 10:40, Jaehoon Chung wrote: > On 10/14/2015 10:15 AM, Krzysztof Kozlowski wrote: >> On 14.10.2015 10:06, Alim Akhtar wrote: >>> +Doug >>> Hello, >>> AFAIR, dw_mmc host controller does support UHS-I [1], specially SDR50 >>> and SDR104 modes. >>> >>> [1]: http://www.spinics.net/lists/linux-mmc/msg28186.html >>> >>> What I remember is, one need to set "broken-cd" property also in order >>> to make it work because of the vqmmc and vmmc connection on board. I >>> didn't find the link right now, but you can search on the web, there >>> was a long discussion about handling this. >>> Have not checked it recently, so not sure if this got broken somehow. >>> >> >> Please, don't top post. >> >> I am not a SD/MMC specialist (I do not feel enough confident in its >> internals) but the datasheet for 5422 does not mention UHS. However it >> mentions "High Speed DDR Mode with 200 MHz clock rate (HS400)". This >> does not look like UHS... > > You're right. It's not UHS mode. it mentions eMMC's HS400 mode. > UHS and HS400 are difference mode. > > eMMC mode are supported > >> https://www.sdcard.org/developers/overview/bus_speed/ >> This of course is not a definite proof that 5422 does not support UHS. I >> am just saying that I couldn't find any information that *it does*. > > Well, I think you want to know whether it supported or not. > Then you can find the information at User manual. > In mobile storage part of User manual, it described the overview. > Mobile storage host supports these specification: > - Secure Digital memory (SD memory version 3.0) > - Secure Digital I/O SDIO (SDIO version 3.0) > - etc... > > SD3.0 is supported UHS-I mode. > If user manual of exynos5422 is mentioned this specification, it should be supported UHS-I mode. Thanks! Indeed datasheet mentions that supported SD and SDIO is version 3.0 (not 3.01... which seems irrelevant because 3.0 introduced UHS-I, right?). Best regards, Krzysztof