Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 10 Feb 2021 13:03:22 -0600
From:   Rob Herring <robh@kernel.org>
To:     kostap@marvell.com
Cc:     linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, vladimir.vid@sartura.hr,
        tmn505@gmail.com, luka.kovacic@sartura.hr,
        sebastian.hesselbarth@gmail.com, gregory.clement@bootlin.com,
        andrew@lunn.ch, vkoul@kernel.org, kishon@ti.com,
        miquel.raynal@bootlin.com, mw@semihalf.com, jaz@semihalf.com,
        nadavh@marvell.com, stefanc@marvell.com, bpeled@marvell.com
Subject: Re: [PATCH v5 1/5] Documentation/bindings: phy: update references to
 cp11x
Message-ID: <20210210190322.GA2549599@robh.at.kernel.org>
References: <20210210131604.28806-1-kostap@marvell.com>
 <20210210131604.28806-2-kostap@marvell.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210210131604.28806-2-kostap@marvell.com>
Precedence: bulk

On Wed, Feb 10, 2021 at 03:16:00PM +0200, kostap@marvell.com wrote:
> From: Grzegorz Jaszczyk <jaz@semihalf.com>
> 
> The cp11x references in dts has changed, reflect it in comphy
> documentation.

Who cares. Examples are just examples.

If you really want to improve the binding, convert it to schema.

> Signed-off-by: Grzegorz Jaszczyk <jaz@semihalf.com>
> Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>
> ---
>  Documentation/devicetree/bindings/phy/phy-mvebu-comphy.txt | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/phy/phy-mvebu-comphy.txt b/Documentation/devicetree/bindings/phy/phy-mvebu-comphy.txt
> index 8c60e6985950..5ffd0f55d010 100644
> --- a/Documentation/devicetree/bindings/phy/phy-mvebu-comphy.txt
> +++ b/Documentation/devicetree/bindings/phy/phy-mvebu-comphy.txt
> @@ -42,22 +42,22 @@ Required properties (child nodes):
>  
>  Examples:
>  
> -	cpm_comphy: phy@120000 {
> +	CP11X_LABEL(comphy): phy@120000 {
>  		compatible = "marvell,comphy-cp110";
>  		reg = <0x120000 0x6000>;
> -		marvell,system-controller = <&cpm_syscon0>;
> -		clocks = <&CP110_LABEL(clk) 1 5>, <&CP110_LABEL(clk) 1 6>,
> -			 <&CP110_LABEL(clk) 1 18>;
> +		marvell,system-controller = <&CP11X_LABEL(syscon0)>;
> +		clocks = <&CP11X_LABEL(clk) 1 5>, <&CP11X_LABEL(clk) 1 6>,
> +			 <&CP11X_LABEL(clk) 1 18>;
>  		clock-names = "mg_clk", "mg_core_clk", "axi_clk";
>  		#address-cells = <1>;
>  		#size-cells = <0>;
>  
> -		cpm_comphy0: phy@0 {
> +		CP11X_LABEL(comphy0): phy@0 {
>  			reg = <0>;
>  			#phy-cells = <1>;
>  		};
>  
> -		cpm_comphy1: phy@1 {
> +		CP11X_LABEL(comphy1): phy@1 {
>  			reg = <1>;
>  			#phy-cells = <1>;
>  		};
> -- 
> 2.17.1
>