2019-11-18 07:45:26

by Anil Joy Varughese

[permalink] [raw]
Subject: [PATCH] bindings:phy Mark phy_clk binding as deprecated in Cadence Sierra Phy.

Updated the Sierra Phy binding doc to mark phy_clk as deprecated.

Signed-off-by: Anil Joy Varughese <[email protected]>
---
.../devicetree/bindings/phy/phy-cadence-sierra.txt | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
index 6e1b47b..9a42b46 100644
--- a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
+++ b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
@@ -5,7 +5,7 @@ Required properties:
- compatible: cdns,sierra-phy-t0
- clocks: Must contain an entry in clock-names.
See ../clocks/clock-bindings.txt for details.
-- clock-names: Must be "phy_clk"
+- clock-names: Must be "phy_clk". This is deprecated and should not be used with newer bindings.
- resets: Must contain an entry for each in reset-names.
See ../reset/reset.txt for details.
- reset-names: Must include "sierra_reset" and "sierra_apb".
--
1.7.1


2019-11-18 08:07:01

by Kishon Vijay Abraham I

[permalink] [raw]
Subject: Re: [PATCH] bindings:phy Mark phy_clk binding as deprecated in Cadence Sierra Phy.

Anil,

On 18/11/19 1:13 PM, Anil Joy Varughese wrote:
> Updated the Sierra Phy binding doc to mark phy_clk as deprecated.

This should also indicate why your are deprecating it.

Thanks
Kishon

>
> Signed-off-by: Anil Joy Varughese <[email protected]>
> ---
> .../devicetree/bindings/phy/phy-cadence-sierra.txt | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
> index 6e1b47b..9a42b46 100644
> --- a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
> +++ b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
> @@ -5,7 +5,7 @@ Required properties:
> - compatible: cdns,sierra-phy-t0
> - clocks: Must contain an entry in clock-names.
> See ../clocks/clock-bindings.txt for details.
> -- clock-names: Must be "phy_clk"
> +- clock-names: Must be "phy_clk". This is deprecated and should not be used with newer bindings.
> - resets: Must contain an entry for each in reset-names.
> See ../reset/reset.txt for details.
> - reset-names: Must include "sierra_reset" and "sierra_apb".
>

2019-11-18 09:04:31

by Anil Joy Varughese

[permalink] [raw]
Subject: RE: [PATCH] bindings:phy Mark phy_clk binding as deprecated in Cadence Sierra Phy.

Hi Kishon,

> -----Original Message-----
> From: Kishon Vijay Abraham I <[email protected]>
> Sent: Monday, November 18, 2019 1:32 PM
> To: Anil Joy Varughese <[email protected]>; [email protected];
> [email protected]; [email protected];
> [email protected]
> Cc: Milind Parab <[email protected]>; Rafal Ciepiela
> <[email protected]>
> Subject: Re: [PATCH] bindings:phy Mark phy_clk binding as deprecated in
> Cadence Sierra Phy.
>
> EXTERNAL MAIL
>
>
> Anil,
>
> On 18/11/19 1:13 PM, Anil Joy Varughese wrote:
> > Updated the Sierra Phy binding doc to mark phy_clk as deprecated.
>
> This should also indicate why your are deprecating it.
>
> Thanks
> Kishon

I am deprecating it since there is a new set of bindings (cmn_refclk, cmn_refclk1) announced by TI.

Will update and send new patch.

Thanks,
Anil



>
> >
> > Signed-off-by: Anil Joy Varughese <[email protected]>
> > ---
> > .../devicetree/bindings/phy/phy-cadence-sierra.txt | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
> b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
> > index 6e1b47b..9a42b46 100644
> > --- a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
> > +++ b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.txt
> > @@ -5,7 +5,7 @@ Required properties:
> > - compatible: cdns,sierra-phy-t0
> > - clocks: Must contain an entry in clock-names.
> > See ../clocks/clock-bindings.txt for details.
> > -- clock-names: Must be "phy_clk"
> > +- clock-names: Must be "phy_clk". This is deprecated and should not
> be used with newer bindings.
> > - resets: Must contain an entry for each in reset-names.
> > See ../reset/reset.txt for details.
> > - reset-names: Must include "sierra_reset" and "sierra_apb".
> >