[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1C73976A-9039-4F7B-959A-4A1E1840EF68@theobroma-systems.com>
Date: Tue, 12 Mar 2019 14:01:42 +0100
From: Christoph Müllner
<christoph.muellner@...obroma-systems.com>
To: Heiko Stuebner <heiko@...ech.de>
Cc: robh+dt@...nel.org, mark.rutland@....com, shawn.lin@...k-chips.com,
Philipp Tomsich <philipp.tomsich@...obroma-systems.com>,
Kishon Vijay Abraham I <kishon@...com>,
Brian Norris <briannorris@...omium.org>,
Klaus Goger <klaus.goger@...obroma-systems.com>,
Enric Balletbo i Serra <enric.balletbo@...labora.com>,
Douglas Anderson <dianders@...omium.org>,
Viresh Kumar <viresh.kumar@...aro.org>,
Ezequiel Garcia <ezequiel@...labora.com>,
Lin Huang <hl@...k-chips.com>, Randy Li <ayaka@...lik.info>,
Tony Xie <tony.xie@...k-chips.com>,
Vicente Bergas <vicencb@...il.com>,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
linux-rockchip@...ts.infradead.org
Subject: Re: [PATCH v2 2/4] dt-bindings: phy: Add a new property
drive-impedance-ohm for RK's emmc PHY
> On 12.03.2019, at 13:52, Heiko Stuebner <heiko@...ech.de> wrote:
>
> Hi Christoph,
>
> Am Donnerstag, 7. März 2019, 09:41:54 CET schrieb Christoph Muellner:
>> This patch documents the new proprty drive-impedance-ohm for
>> Rockchip's eMMC PHY node.
>>
>> Signed-off-by: Christoph Muellner <christoph.muellner@...obroma-systems.com>
>> Signed-off-by: Philipp Tomsich <philipp.tomsich@...obroma-systems.com>
>> ---
>> Documentation/devicetree/bindings/phy/rockchip-emmc-phy.txt | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/phy/rockchip-emmc-phy.txt b/Documentation/devicetree/bindings/phy/rockchip-emmc-phy.txt
>> index e3ea55763b0a..5f5d999dacc3 100644
>> --- a/Documentation/devicetree/bindings/phy/rockchip-emmc-phy.txt
>> +++ b/Documentation/devicetree/bindings/phy/rockchip-emmc-phy.txt
>> @@ -13,6 +13,9 @@ specified by name:
>
> you might want to adapt the section headline as well, as right now it
> talks about "Optional clocks", but should likely become the generic
> "Optional properties" as in other bindings (including moving the clock-
> binding.txt reference down into the clock elements)
Thank's for spotting this.
Will do for v3.
Thanks,
Christoph
>
>
> Heiko
>
>> (because most boards can get basic functionality without having
>> access to it), it is strongly suggested.
>> - clocks: Should have a phandle to the card clock exported by the SDHCI driver.
>> + - drive-impedance-ohm: Specifies the drive impedance in Ohm.
>> + Possible values are 33, 40, 50, 66 and 100.
>> + If not set, the default value of 50 will be applied.
>>
>> Example:
>>
>>
>
>
>
>
Powered by blists - more mailing lists