[PATCH 2/2] Documentation: bindings: add description of phy for sdhci-of-arasan
soren.brinkmann at xilinx.com
Sun Sep 13 19:47:02 PDT 2015
On Fri, 2015-09-11 at 04:55PM +0800, Shawn Lin wrote:
> This patch adds phys and phy-names for sdhci-of-arasan as optional
> properties, and details the example as well.
> Signed-off-by: Shawn Lin <shawn.lin at rock-chips.com>
> Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 10 ++++++++--
> 1 file changed, 8 insertions(+), 2 deletions(-)
> diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt
> index da541c3..0264d5f 100644
> --- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt
> +++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt
> @@ -1,11 +1,12 @@
> Device Tree Bindings for the Arasan SDHCI Controller
> - The bindings follow the mmc, clock and interrupt bindings. Only
> - deviations are documented here.
> + The bindings follow the mmc, clock, interrupt and phy bindings.
> + Only deviations are documented here.
>  Documentation/devicetree/bindings/mmc/mmc.txt
>  Documentation/devicetree/bindings/clock/clock-bindings.txt
>  Documentation/devicetree/bindings/interrupt-controller/interrupts.txt
> +  Documentation/devicetree/bindings/phy/phy-bindings.txt
> Required Properties:
> - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' or
> @@ -16,6 +17,9 @@ Required Properties:
> - interrupts: Interrupt specifier
> - interrupt-parent: Phandle for the interrupt controller that services
> interrupts for this device.
> +Optional Properties:
> + - phys: From PHY bindings: Phandle for the Generic PHY for arasan.
> + - phy-names: MUST be "phy_arasan".
This might be a dumb question, but, is the external phy actually
optional? Or is it mandatory for certain implementations of the IP.
In the latter case, it should probably be a mandatory property depending
on the compatible string matching an implementation that requires an
external phy (accordingly, the implementation might have to treat it
that way too).
More information about the Linux-rockchip