| Message ID | 20220530125329.30717-3-pmalgujar@marvell.com |
|---|---|
| State | New |
| Headers | show |
| Series | net: mdio: mdio-thunder: MDIO clock related changes for Marvell Octeon Family. | expand |
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
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. + - #address-cells: Must be <2>. - #size-cells: Must be <2>. @@ -64,6 +67,7 @@ Example: #address-cells = <1>; #size-cells = <0>; reg = <0x87e0 0x05003800 0x0 0x30>; + clock-frequency = <3125000>; ethernet-phy@0 { ... @@ -75,6 +79,7 @@ Example: #address-cells = <1>; #size-cells = <0>; reg = <0x87e0 0x05003880 0x0 0x30>; + clock-frequency = <3125000>; ethernet-phy@0 { ...