Re: [PATCH 2/2] crypto: mediatek - update DT binding documentation

[Matthias Brugger]

On 26/05/17 11:43, Ryder Lee wrote:
This patch removes unnecessary clock in binding file.

Signed-off-by: Ryder Lee <ryder....@mediatek.com>
---

In the driver clocks are get by name, so this change does not break 
backwards compatibility.

Reviewed-by: Matthias Brugger <matthias....@gmail.com>

  Documentation/devicetree/bindings/crypto/mediatek-crypto.txt | 8 +++-----
  1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/crypto/mediatek-crypto.txt 
b/Documentation/devicetree/bindings/crypto/mediatek-crypto.txt
index c204725..450da36 100644
--- a/Documentation/devicetree/bindings/crypto/mediatek-crypto.txt
+++ b/Documentation/devicetree/bindings/crypto/mediatek-crypto.txt
@@ -6,8 +6,7 @@ Required properties:
  - interrupts: Should contain the five crypto engines interrupts in numeric
        order. These are global system and four descriptor rings.
  - clocks: the clock used by the core
-- clock-names: the names of the clock listed in the clocks property. These are
-       "ethif", "cryp"
+- clock-names: Must contain "cryp".
  - power-domains: Must contain a reference to the PM domain.
  
  
@@ -20,8 +19,7 @@ Example:
                             <GIC_SPI 84 IRQ_TYPE_LEVEL_LOW>,
                             <GIC_SPI 91 IRQ_TYPE_LEVEL_LOW>,
                             <GIC_SPI 97 IRQ_TYPE_LEVEL_LOW>;
-               clocks = <&topckgen CLK_TOP_ETHIF_SEL>,
-                        <&ethsys CLK_ETHSYS_CRYPTO>;
-               clock-names = "ethif","cryp";
+               clocks = <&ethsys CLK_ETHSYS_CRYPTO>;
+               clock-names = "cryp";
                power-domains = <&scpsys MT2701_POWER_DOMAIN_ETH>;
        };