Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Sun, 5 Jun 2022 16:59:44 +0200
From:   Andrew Lunn <andrew@lunn.ch>
To:     Rob Herring <robh@kernel.org>
Cc:     Piyush Malgujar <pmalgujar@marvell.com>, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org, krzysztof.kozlowski+dt@linaro.org,
        devicetree@vger.kernel.org, cchavva@marvell.com,
        deppel@marvell.com, "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>
Subject: Re: [PATCH v2 2/3] dt-bindings: net: cavium-mdio.txt: add
 clock-frequency attribute
Message-ID: <YpzE4A1MUYNbhKJo@lunn.ch>
References: <20220530125329.30717-1-pmalgujar@marvell.com>
 <20220530125329.30717-3-pmalgujar@marvell.com>
 <20220602150755.GA2323599-robh@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220602150755.GA2323599-robh@kernel.org>
Precedence: bulk

On Thu, Jun 02, 2022 at 10:07:55AM -0500, Rob Herring wrote:
> On Mon, May 30, 2022 at 05:53:27AM -0700, Piyush Malgujar wrote:
> > Add support to configure MDIO clock frequency via DTS
> > 
> > Signed-off-by: Damian Eppel <deppel@marvell.com>
> > Signed-off-by: Piyush Malgujar <pmalgujar@marvell.com>
> > ---
> >  Documentation/devicetree/bindings/net/cavium-mdio.txt | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/net/cavium-mdio.txt b/Documentation/devicetree/bindings/net/cavium-mdio.txt
> > index 020df08b8a30f4df80766bb90e100ae6210a777b..638c341966a80823b9eb2f33b947f38110907cc1 100644
> > --- a/Documentation/devicetree/bindings/net/cavium-mdio.txt
> > +++ b/Documentation/devicetree/bindings/net/cavium-mdio.txt
> > @@ -41,6 +41,9 @@ Properties:
> >  
> >  - reg: The PCI device and function numbers of the nexus device.
> >  
> > +- clock-frequency: MDIO bus clock frequency in Hz. It defaults to 3.125 MHz and
> > +		   and not to standard 2.5 MHz for Marvell Octeon family.

Hi Piyush

There is an ambiguity here in the English. It could be interpreted
that 2.5MHz is the standard for Marvell Octeon family. When in fact
802.3 c22 says it should be up to 2.5MHz.

    For Marvell Octeon family it defaults to 3.125 MHz and not to
    the 802.3 standard 2.5 MHz.

> 
> Already covered by mdio.yaml, so perhaps convert this to DT schema 
> format instead.

Hi Rob

Yes, this property is in mdio.yaml:

  clock-frequency:
    description:
      Desired MDIO bus clock frequency in Hz. Values greater than IEEE 802.3
      defined 2.5MHz should only be used when all devices on the bus support
      the given clock speed.

However, for some reason, this driver decides to break the standard
and defaults to 3.125MHz not 2.5MHz. So i would like that clearly
documented in the binding.

	   Andrew