From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Wed, 05 Aug 2015 10:43:40 +0200 (CEST) Received: from regular1.263xmail.com ([211.150.99.135]:55375 "EHLO regular1.263xmail.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S27008877AbbHEInfqRFLu (ORCPT ); Wed, 5 Aug 2015 10:43:35 +0200 Received: from shawn.lin?rock-chips.com (unknown [192.168.167.226]) by regular1.263xmail.com (Postfix) with SMTP id EEA6C19945; Wed, 5 Aug 2015 16:43:30 +0800 (CST) X-263anti-spam: KSV:0; X-MAIL-GRAY: 0 X-MAIL-DELIVERY: 1 X-KSVirus-check: 0 X-ABS-CHECKED: 4 X-ADDR-CHECKED: 0 Received: from [172.16.12.109] (localhost.localdomain [127.0.0.1]) by smtp.263.net (Postfix) with ESMTP id AB53A130441; Wed, 5 Aug 2015 16:43:20 +0800 (CST) X-RL-SENDER: shawn.lin@rock-chips.com X-FST-TO: srinivas.kandagatla@linaro.org X-SENDER-IP: 58.22.7.114 X-LOGIN-NAME: shawn.lin@rock-chips.com X-UNIQUE-TAG: X-ATTACHMENT-NUM: 0 X-SENDER: lintao@rock-chips.com X-DNS-TYPE: 0 Received: from [172.16.12.109] (unknown [58.22.7.114]) by smtp.263.net (Postfix) whith ESMTP id 177775IA2Z0; Wed, 05 Aug 2015 16:43:30 +0800 (CST) Subject: Re: [RFC PATCH v3 2/5] Documentation: synopsys-dw-mshc: add bindings for idmac and edmac To: Krzysztof Kozlowski , Ulf Hansson , Jaehoon Chung , Seungwon Jeon References: <1438762614-22154-1-git-send-email-shawn.lin@rock-chips.com> <1438762672-22243-1-git-send-email-shawn.lin@rock-chips.com> <55C1C8E9.1060808@samsung.com> Cc: shawn.lin@rock-chips.com, dianders@chromium.org, linux-mips@linux-mips.org, Arnd Bergmann , Alexandre Belloni , Alexey Brodkin , Stefan Agner , Zhou Wang , Kumar Gala , Alim Akhtar , Ian Campbell , Wang Long , Rob Herring , Chaotian Jing , Lukasz Majewski , Jun Nie , Catalin Marinas , Kevin Hao , Olof Johansson , Ray Jui , Govindraj Raja , linux-samsung-soc@vger.kernel.org, Heiko Stuebner , "Rafael J. Wysocki" , Geert Uytterhoeven , Vineet Gupta , Scott Branden , Anand Moon , linux-rockchip@lists.infradead.org, Tushar Behera , Pawel Moll , Will Deacon , Mischa Jonker , Ralf Baechle , Javier Martinez Canillas , Vincent Yang , Stephen Warren , devicetree@vger.kernel.org, Kukjin Kim , Tomeu Vizoso , Stephen Boyd , Russell King , Joachim Eastwood , Sjoerd Simons , Weijun Yang , Peter Griffin , linux-arm-kernel@lists.infradead.org, addy ke , Uwe Kleine-K?nig , Jean Delvare , Kevin Hilman , Andrew Bresticker , linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, Zhangfei Gao , Wei Xu , Andreas Faerber , Mark Rutland , Srinivas Kandagatla From: Shawn Lin Message-ID: <55C1CCA8.7040304@rock-chips.com> Date: Wed, 5 Aug 2015 16:43:20 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <55C1C8E9.1060808@samsung.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 48585 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: shawn.lin@rock-chips.com Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips 在 2015/8/5 16:27, Krzysztof Kozlowski 写道: > On 05.08.2015 17:17, Shawn Lin wrote: >> synopsys-dw-mshc supports three types of transfer mode. We add bindings >> and description for how to use them at runtime. Without idmac and edmac >> property, pio is the default transfer mode. Make sure that Idmac and emdac >> should not be used simultaneously. >> >> Signed-off-by: Shawn Lin >> --- >> >> Changes in v3: None >> Changes in v2: None >> >> .../devicetree/bindings/mmc/synopsys-dw-mshc.txt | 41 ++++++++++++++++++++++ >> 1 file changed, 41 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt b/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt >> index 346c609..30369cb 100644 >> --- a/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt >> +++ b/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt >> @@ -75,6 +75,25 @@ Optional properties: >> * vmmc-supply: The phandle to the regulator to use for vmmc. If this is >> specified we'll defer probe until we can find this regulator. >> >> +* supports-idmac: Enables support for internal DMAC block within the Synopsys >> + Designware Mobile Storage IP block. If supports-idmac property is present, then >> + we MUST NOT add supports-edmac property since we'd assume that dw-mshc IP is >> + integrated with only one type of dma master. >> + >> +* supports-edmac: Enables support for external DMAC block outside the Synopsys >> + Designware Mobile Storage IP block. If supports-edmac property is present, then >> + we MUST NOT add supports-idmac property since we'd assume that dw-mshc IP is >> + integrated with only one type of dma master. >> + >> + (Without "supports-idmac" and "supports-edmac", use PIO as default transfer mode) > Aren't you breaking here backward compatibility with existing DTB? > > Best regards, > Krzysztof Thanks, Krzysztof. I guess you mean that I should keep existing DTB w/o these two properties work fine using idmac? >> + >> +* dmas: List of DMA specifiers with the controller specific format as described >> + in the generic DMA client binding. This property should be combined with >> + supports-edmac. Refer to dma.txt for details. >> + >> +* dma-names: DMA request names. Must be "rx-tx". And This property should be >> + combined with supports-edmac. Refer to dma.txt for details. >> + >> Aliases: >> >> - All the MSHC controller nodes should be represented in the aliases node using >> @@ -95,6 +114,8 @@ board specific portions as listed below. >> #size-cells = <0>; >> }; >> >> +[board specific internal DMA resources] >> + >> dwmmc0@12200000 { >> clock-frequency = <400000000>; >> clock-freq-min-max = <400000 200000000>; >> @@ -106,4 +127,24 @@ board specific portions as listed below. >> bus-width = <8>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + supports-idmac; >> }; >> + >> +[board specific generic DMA request binding] >> + >> + dwmmc0@12200000 { >> + clock-frequency = <400000000>; >> + clock-freq-min-max = <400000 200000000>; >> + num-slots = <1>; >> + broken-cd; >> + fifo-depth = <0x80>; >> + card-detect-delay = <200>; >> + vmmc-supply = <&buck8>; >> + bus-width = <8>; >> + cap-mmc-highspeed; >> + cap-sd-highspeed; >> + supports-edmac; >> + dmas = <&pdma 12>; >> + dma-names = "rx-tx"; >> + }; >> + >> > > > -- Shawn Lin From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shawn Lin Subject: Re: [RFC PATCH v3 2/5] Documentation: synopsys-dw-mshc: add bindings for idmac and edmac Date: Wed, 5 Aug 2015 16:43:20 +0800 Message-ID: <55C1CCA8.7040304@rock-chips.com> References: <1438762614-22154-1-git-send-email-shawn.lin@rock-chips.com> <1438762672-22243-1-git-send-email-shawn.lin@rock-chips.com> <55C1C8E9.1060808@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <55C1C8E9.1060808@samsung.com> Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: To: Krzysztof Kozlowski , Ulf Hansson , Jaehoon Chung , Seungwon Jeon Cc: shawn.lin@rock-chips.com, dianders@chromium.org, linux-mips@linux-mips.org, Arnd Bergmann , Alexandre Belloni , Alexey Brodkin , Stefan Agner , Zhou Wang , Kumar Gala , Alim Akhtar , Ian Campbell , Wang Long , Rob Herring , Chaotian Jing , Lukasz Majewski , Jun Nie , Catalin Marinas , Kevin Hao , Olof Johansson , Ray Jui , Govindraj Raja List-Id: linux-mmc@vger.kernel.org =E5=9C=A8 2015/8/5 16:27, Krzysztof Kozlowski =E5=86=99=E9=81=93: > On 05.08.2015 17:17, Shawn Lin wrote: >> synopsys-dw-mshc supports three types of transfer mode. We add bindi= ngs >> and description for how to use them at runtime. Without idmac and ed= mac >> property, pio is the default transfer mode. Make sure that Idmac and= emdac >> should not be used simultaneously. >> >> Signed-off-by: Shawn Lin >> --- >> >> Changes in v3: None >> Changes in v2: None >> >> .../devicetree/bindings/mmc/synopsys-dw-mshc.txt | 41 ++++++++++= ++++++++++++ >> 1 file changed, 41 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.= txt b/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt >> index 346c609..30369cb 100644 >> --- a/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt >> +++ b/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt >> @@ -75,6 +75,25 @@ Optional properties: >> * vmmc-supply: The phandle to the regulator to use for vmmc. If t= his is >> specified we'll defer probe until we can find this regulator. >> =20 >> +* supports-idmac: Enables support for internal DMAC block within th= e Synopsys >> + Designware Mobile Storage IP block. If supports-idmac property is= present, then >> + we MUST NOT add supports-edmac property since we'd assume that dw= -mshc IP is >> + integrated with only one type of dma master. >> + >> +* supports-edmac: Enables support for external DMAC block outside t= he Synopsys >> + Designware Mobile Storage IP block. If supports-edmac property is= present, then >> + we MUST NOT add supports-idmac property since we'd assume that dw= -mshc IP is >> + integrated with only one type of dma master. >> + >> + (Without "supports-idmac" and "supports-edmac", use PIO as defaul= t transfer mode) > Aren't you breaking here backward compatibility with existing DTB? > > Best regards, > Krzysztof Thanks, Krzysztof. I guess you mean that I should keep existing DTB w/o these two=20 properties work fine using idmac? >> + >> +* dmas: List of DMA specifiers with the controller specific format = as described >> + in the generic DMA client binding. This property should be combin= ed with >> + supports-edmac. Refer to dma.txt for details. >> + >> +* dma-names: DMA request names. Must be "rx-tx". And This property = should be >> + combined with supports-edmac. Refer to dma.txt for details. >> + >> Aliases: >> =20 >> - All the MSHC controller nodes should be represented in the alias= es node using >> @@ -95,6 +114,8 @@ board specific portions as listed below. >> #size-cells =3D <0>; >> }; >> =20 >> +[board specific internal DMA resources] >> + >> dwmmc0@12200000 { >> clock-frequency =3D <400000000>; >> clock-freq-min-max =3D <400000 200000000>; >> @@ -106,4 +127,24 @@ board specific portions as listed below. >> bus-width =3D <8>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + supports-idmac; >> }; >> + >> +[board specific generic DMA request binding] >> + >> + dwmmc0@12200000 { >> + clock-frequency =3D <400000000>; >> + clock-freq-min-max =3D <400000 200000000>; >> + num-slots =3D <1>; >> + broken-cd; >> + fifo-depth =3D <0x80>; >> + card-detect-delay =3D <200>; >> + vmmc-supply =3D <&buck8>; >> + bus-width =3D <8>; >> + cap-mmc-highspeed; >> + cap-sd-highspeed; >> + supports-edmac; >> + dmas =3D <&pdma 12>; >> + dma-names =3D "rx-tx"; >> + }; >> + >> > > > --=20 Shawn Lin