From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chaotian Jing Subject: Re: [PATCH v5 1/7] mmc: dt-bindings: add Mediatek MMC bindings Date: Thu, 11 Jun 2015 09:33:51 +0800 Message-ID: <1433986431.14592.2.camel@mhfsdcap03> References: <1433903088-14407-1-git-send-email-chaotian.jing@mediatek.com> <1433903088-14407-2-git-send-email-chaotian.jing@mediatek.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+glpam-linux-mediatek=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Ulf Hansson Cc: Mark Rutland , James Liao , Arnd Bergmann , srv_heupstream , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Hongzhou Yang , Catalin Marinas , Bin Zhang =?UTF-8?Q?=28=E7=AB=A0=E6=96=8C=29?= , linux-mmc , Chris Ball , Will Deacon , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Rob Herring , linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Sascha Hauer , Matthias Brugger , "Joe.C" , Eddie Huang , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" List-Id: devicetree@vger.kernel.org Dear Ulf, Thanks for your review! please see my comment: On Wed, 2015-06-10 at 14:02 +0200, Ulf Hansson wrote: > On 10 June 2015 at 04:24, Chaotian Jing wrote: > > Document the device-tree binding of Mediatek MMC host > > > > Signed-off-by: Chaotian Jing > > --- > > Documentation/devicetree/bindings/mmc/mtk-sd.txt | 32 ++++++++++++++++++++++++ > > 1 file changed, 32 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/mmc/mtk-sd.txt > > > > diff --git a/Documentation/devicetree/bindings/mmc/mtk-sd.txt b/Documentation/devicetree/bindings/mmc/mtk-sd.txt > > new file mode 100644 > > index 0000000..a1adfa4 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/mmc/mtk-sd.txt > > @@ -0,0 +1,32 @@ > > +* MTK MMC controller > > + > > +The MTK MSDC can act as a MMC controller > > +to support MMC, SD, and SDIO types of memory cards. > > + > > +This file documents differences between the core properties in mmc.txt > > +and the properties used by the msdc driver. > > + > > +Required properties: > > +- compatible: Should be "mediatek,mt8173-mmc","mediatek,mt8135-mmc" > > +- interrupts: Should contain MSDC interrupt number > > +- clocks: MSDC source clock, HCLK > > +- clock-names: "source", "hclk" > > According to the mmc driver, hclk is treated as an optional clock. > Therefore I think you should list it under an "Optional properties" > section instead. > I will make the hclk to mandatory at next driver revision. if there was no hclk at dtsi, then the host would probe fail. > > +- pinctrl-names: should be "default", "state_uhs" > > +- pinctrl-0: should contain default/high speed pin ctrl > > +- pinctrl-1: should contain uhs mode pin ctrl > > +- vmmc-supply: power to the Core > > +- vqmmc-supply: power to the IO > > + > > +Examples: > > +mmc0: mmc@11230000 { > > + compatible = "mediatek,mt8173-mmc", "mediatek,mt8135-mmc"; > > + reg = <0 0x11230000 0 0x108>; > > + interrupts = ; > > + vmmc-supply = <&mt6397_vemc_3v3_reg>; > > + vqmmc-supply = <&mt6397_vio18_reg>; > > + clocks = <&pericfg CLK_PERI_MSDC30_0>, <&topckgen CLK_TOP_MSDC50_0_H_SEL>; > > + clock-names = "source", "hclk"; > > + pinctrl-names = "default", "state_uhs"; > > + pinctrl-0 = <&mmc0_pins_default>; > > + pinctrl-1 = <&mmc0_pins_uhs>; > > +}; > > -- > > 1.8.1.1.dirty > > > > Kind regards > Uffe