From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932233AbbJMLpx (ORCPT ); Tue, 13 Oct 2015 07:45:53 -0400 Received: from mailout3.samsung.com ([203.254.224.33]:39141 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752356AbbJMLpv (ORCPT ); Tue, 13 Oct 2015 07:45:51 -0400 X-AuditID: cbfee690-f794e6d0000014de-bd-561ceeec107b Message-id: <561CECB2.2090208@samsung.com> Date: Tue, 13 Oct 2015 17:06:18 +0530 From: Alim Akhtar User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-version: 1.0 To: Arnd Bergmann Cc: linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, JBottomley@odin.com, vinholikatti@gmail.com, amit.daniel@samsung.com, essuuj@gmail.com, devicetree@vger.kernel.org Subject: Re: [PATCH v3 11/13] Documentation: devicetree: ufs: Add DT bindings for exynos UFS host controller References: <1443686970-28104-1-git-send-email-alim.akhtar@samsung.com> <1443686970-28104-12-git-send-email-alim.akhtar@samsung.com> <1457698.WJ3mz2FHZZ@wuerfel> In-reply-to: <1457698.WJ3mz2FHZZ@wuerfel> Content-type: text/plain; charset=ISO-8859-1; format=flowed Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrGIsWRmVeSWpSXmKPExsWyRsSkTvfNO5kwg2UrTS0aroZY/J10jN1i /pFzrBbLLyxhsvi//jaLxeVdc9gsuq/vYLPYsbDKgcPj969JjB47Z91l9zj84wezR9+WVYwe nzfJBbBGcdmkpOZklqUW6dslcGXsO7SYuWCDbEXrtia2Bsaj4l2MHBwSAiYSd/+LdTFyApli EhfurWfrYuTiEBJYwSjx9ewfVoiEicTXlWuhErMYJZY9u8YC4TxglPjyYisLSBWvgJZEz7pu dhCbRUBV4tG7WWwgNpuAtsTd6VuYQLaJCkRIPL4gBFEuKPFj8j2wVhEBRYmpL54xg8xkFtjC KPHuxlWwhLBAnsTK1TPArhASWMoo8XK+BYjNKaAp0bX7I1gNs4C1xMpJ2xghbHmJzWvegg2S ELjFLnHtw1YmiIMEJL5NPsQC8bKsxKYDzBCfSUocXHGDZQKj2CwkN81CMnYWkrELGJlXMYqm FiQXFCelF5noFSfmFpfmpesl5+duYgRG3+l/zybsYLx3wPoQowAHoxIP74tImTAh1sSy4src Q4ymQFdMZJYSTc4HxnheSbyhsZmRhamJqbGRuaWZkjjva6mfwUIC6YklqdmpqQWpRfFFpTmp xYcYmTg4pRoYJz/gvH0nlKH29bN+Zm57m2MVnItV9doK/hi7r0+wui7PXRT6aOX2uNML7Cad 7ZFSXO9+oreIjUGW4T7jknPaZzY/VZrX17Kqbu7yw7pmc9coP/klYFjBd+uXWo3de74dd9c1 ivEoX9uzTqM0rrswOvLH/U2HPzwON1rMdHvplM/Vkx5t+HRjmhJLcUaioRZzUXEiAG2xA2m5 AgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrCIsWRmVeSWpSXmKPExsVy+t9jQd0372TCDHp61C0aroZY/J10jN1i /pFzrBbLLyxhsvi//jaLxeVdc9gsuq/vYLPYsbDKgcPj969JjB47Z91l9zj84wezR9+WVYwe nzfJBbBGNTDaZKQmpqQWKaTmJeenZOal2yp5B8c7x5uaGRjqGlpamCsp5CXmptoqufgE6Lpl 5gCdoqRQlphTChQKSCwuVtK3wzQhNMRN1wKmMULXNyQIrsfIAA0krGHM2HdoMXPBBtmK1m1N bA2MR8W7GDk5JARMJL6uXMsGYYtJXLi3Hsjm4hASmMUosezZNRYI5wGjxJcXW1lAqngFtCR6 1nWzg9gsAqoSj97NAutmE9CWuDt9C1MXIweHqECExOMLQhDlghI/Jt8DaxURUJSY+uIZM8hM ZoEtjBLvblwFSwgL5EmsXD2DFcQWEljKKPFyvgWIzSmgKdG1+yNYDbOAtcTKSdsYIWx5ic1r 3jJPYAQ6E2HHLCRls5CULWBkXsUokVqQXFCclJ5rlJdarlecmFtcmpeul5yfu4kRHOPPpHcw Ht7lfohRgINRiYf3RaRMmBBrYllxZe4hRgkOZiUR3qQWoBBvSmJlVWpRfnxRaU5q8SFGU2Ag TGSWEk3OB6afvJJ4Q2MTc1NjU0sTCxMzSyVx3huHGMKEBNITS1KzU1MLUotg+pg4OKUaGCe/ VPZNWvRk/oG/Oxca7asOfxXxyJOtmmP1MYd7uxsfXmP5uFI9NlHkudxt5clmoYvuTeP5cbX0 6JP+pinv1Tc/vLcuKuN80IYUrSe1M9YvTbbsWGN/bcZRU7abl/Wi926ccjlneuYbU8v2vrop x76s760+vSDzeTJDk5zs40aLpfZnVml2zj2uxFKckWioxVxUnAgASm9jggcDAAA= DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Arnd, On 10/01/2015 05:07 PM, Arnd Bergmann wrote: > On Thursday 01 October 2015 13:39:28 Alim Akhtar wrote: >> From: Seungwon Jeon >> >> This adds Exynos Universal Flash Storage (UFS) Host Controller DT bindings. >> >> Signed-off-by: Seungwon Jeon >> Signed-off-by: Alim Akhtar >> --- >> .../devicetree/bindings/ufs/ufs-exynos.txt | 93 ++++++++++++++++++++ >> 1 file changed, 93 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/ufs/ufs-exynos.txt >> >> diff --git a/Documentation/devicetree/bindings/ufs/ufs-exynos.txt b/Documentation/devicetree/bindings/ufs/ufs-exynos.txt >> new file mode 100644 >> index 0000000..00df72e >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/ufs/ufs-exynos.txt >> @@ -0,0 +1,93 @@ >> +* Exynos Universal Flash Storage (UFS) Host Controller >> + >> +UFSHC nodes are defined to describe on-chip UFS host controllers. >> +Each UFS controller instance should have its own node. >> + >> +Required properties: >> +- compatible : compatible name, contains "samsung,exynos7-ufs" >> +- interrupts : >> +- reg : > > This needs a list of all the register ranges, which order they are in > and what the respective strings must be. > will add the details >> +- clocks : List of phandle and clock specifier pairs >> +- clock-names : List of clock input name strings sorted in the same >> + order as the clocks property. > > You need to list the names you require here. Also the 'clock specifier' includes > the phandle, it's not a pair but just a list of specifiers. > ok will add >> +- ufs,pwr-attr-mode : specifies mode value for power mode change >> +- ufs,pwr-attr-lane : specifies lane count value for power mode change >> +- ufs,pwr-attr-gear : specifies gear count value for power mode change >> +- ufs,pwr-attr-hs-series : specifies HS rate series for power mode change >> +- ufs,pwr-local-l2-timer : specifies array of local UNIPRO L2 timer values >> + >> +- ufs,pwr-remote-l2-timer : specifies array of remote UNIPRO L2 timer values >> + >> +- ufs-rx-adv-fine-gran-sup_en : specifies support of fine granularity of MPHY, >> + this is a boolean property. >> +- ufs-rx-adv-fine-gran-step : specifies granularity steps of MPHY >> +- ufs-rx-adv-min-activate-time-cap : specifies rx advanced minimum activate time of MPHY >> +- ufs-pa-granularity : specifies Granularity for PA_TActivate and PA_Hibern8Time >> +- ufs-pa-tacctivate : specifies time wake-up remote M-RX >> +- ufs-pa-hibern8time : specifies minimum time to wait in HIBERN8 state > > These all require a specification of what the allowed values are and/or the units > for the numbers. > will update the specifications above >> + >> + clocks = <&core 0>, <&ref 0>, <&iface 0>; >> + clock-names = "core_clk", "ref_clk", "iface_clk"; > > Better rename them to "core", "ref" and "iface", no point requiring to > spell out "clk" here. > >> + ufs,pwr-attr-mode = "FAST"; > > A string is rather unusual here, what are the allowed values? Could you > use a boolean property instead? > will update the binding, supported modes are FAST, SLOW, FAST_auto etc, so kept the string for more readability >> + ufs,pwr-attr-lane = /bits/ 8 <2>; >> + ufs,pwr-attr-gear = /bits/ 8 <2>; > > Why the "/bits/ 8" ? > I am using of_property_read_u8() to read 8 bit value from property. > > Arnd >