2017-06-01 02:30:38

by Ryder Lee

Subject: [PATCH v1 0/2] remove mediatek ethif clock in crypto driver

This patch series only remove redundant clock setting in mediatek crypto driver.

Changes since v1:
-rework the commit message.

Ryder Lee (2):
crypto: mediatek - remove redundant clock setting
dt-bindings: crypto: remove mediatek ethif clock

Documentation/devicetree/bindings/crypto/mediatek-crypto.txt | 8 +++-----
drivers/crypto/mediatek/mtk-platform.c | 10 +---------
drivers/crypto/mediatek/mtk-platform.h | 2 --
3 files changed, 4 insertions(+), 16 deletions(-)

--
1.9.1

2017-06-01 02:30:22

by Ryder Lee

Subject: [PATCH v1 2/2] dt-bindings: crypto: remove mediatek ethif clock

This patch removes the parent clock 'ethif' in bindings, since we don't
need to control the parent of a clock in current clock framework.

Moreover, the clocks are get by name in the driver, thus this change
does not break backwards compatibility.

Signed-off-by: Ryder Lee <[email protected]>
Reviewed-by: Matthias Brugger <[email protected]>
---
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>;
};
--
1.9.1

2017-06-07 21:38:17

by Rob Herring (Arm)

Subject: Re: [PATCH v1 2/2] dt-bindings: crypto: remove mediatek ethif clock

On Thu, Jun 01, 2017 at 10:30:22AM +0800, Ryder Lee wrote:
> This patch removes the parent clock 'ethif' in bindings, since we don't
> need to control the parent of a clock in current clock framework.
>
> Moreover, the clocks are get by name in the driver, thus this change
> does not break backwards compatibility.
>
> Signed-off-by: Ryder Lee <[email protected]>
> Reviewed-by: Matthias Brugger <[email protected]>
> ---
> Documentation/devicetree/bindings/crypto/mediatek-crypto.txt | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)

Acked-by: Rob Herring <[email protected]>

2017-06-10 04:19:11

by Herbert Xu

Subject: Re: [PATCH v1 0/2] remove mediatek ethif clock in crypto driver

On Thu, Jun 01, 2017 at 10:30:20AM +0800, Ryder Lee wrote:
> This patch series only remove redundant clock setting in mediatek crypto driver.
>
> Changes since v1:
> -rework the commit message.

All applied. Thanks.
--
Email: Herbert Xu <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt