<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <p><font face="Helvetica, Arial, sans-serif">Dear Baruch Siach,</font></p>
    <p><b><font face="Helvetica, Arial, sans-serif">Your patch worked,
          some body can push it in upstream?</font></b></p>
    <p><font face="Helvetica, Arial, sans-serif">I remain blocked on:<br>
      </font></p>
    <p><font face="Helvetica, Arial, sans-serif">ethtool -m eth3<br>
        Cannot get module EEPROM information: Operation not supported<br>
      </font></p>
    <p><font face="Helvetica, Arial, sans-serif">[   22.265548] mvpp2
        f4000000.ethernet eth3: configuring for inband/2500base-x link
        mode<br>
        [   22.266897] IPv6: ADDRCONF(NETDEV_UP): eth3: link is not
        ready</font></p>
    <p><font face="Helvetica, Arial, sans-serif">I try do work:</font></p>
    <p><font face="Helvetica, Arial, sans-serif"><a
          class="moz-txt-link-freetext"
          href="https://www.fs.com/products/48929.html">https://www.fs.com/products/48929.html</a>
        Cisco GLC-LH-SM-20 Compatible 1000BASE-LX/LH SFP 1310nm 20km DOM
        Transceiver Module<br>
      </font></p>
    <p>But failed, maybe not dynamically down to 1G.</p>
    Cheers,<br>
    <br>
    <div class="moz-cite-prefix">On 11/01/18 07:23, Baruch Siach wrote:<br>
    </div>
    <blockquote type="cite" cite="mid:87o9b96nyl.fsf@tkos.co.il">
      <pre wrap="">Hi Alpha,

Please keep the Macchiato list on Cc

alpha_one_x86 writes:
</pre>
      <blockquote type="cite">
        <blockquote type="cite">
          <pre wrap="">If this is the case, what DT are you using? Note that mainline kernel 
armada-8040-mcbin.dts is meant for the DoubleShot, which has 88x3310 PHYs on 
eth0 and eth1 (marked U2 and U3 on the PCB). These PHYs are not assembled on 
the SingleShot. Instead, both interfaces are directly connected to SFP cages. 
So the error message (-19 == -ENODEV) is correct. You need to remove the PHYs 
description from the DT for the SingleShot.
</pre>
        </blockquote>
        <pre wrap="">I have Single Shot. And for vanilla I use armada-8040-mcbin.dts/dtb
Can you indicate me how change the dts? (maybe send me a patch), I see
multiple eth0, eth1, ... ref.
</pre>
      </blockquote>
      <pre wrap="">
The ethX reference is quite confusing indeed. When you see cpX_ethY it
refers to different Ethernet MAC instances on each CP. There are two
CP110 (cp0 and cp1), each with three Ethernet MACs.

For the Single Shot you can try the following completely untested patch
against v4.19.

diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
index 56fa44860909..3da9b4cd78a4 100644
--- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
+++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
@@ -227,22 +227,6 @@
        };
 };
 
-&cp0_xmdio {
-       status = "okay";
-
-       phy0: ethernet-phy@0 {
-               compatible = "ethernet-phy-ieee802.3-c45";
-               reg = <0>;
-               sfp = <&sfp_eth0>;
-       };
-
-       phy8: ethernet-phy@8 {
-               compatible = "ethernet-phy-ieee802.3-c45";
-               reg = <8>;
-               sfp = <&sfp_eth1>;
-       };
-};
-
 &cp0_ethernet {
        status = "okay";
 };
@@ -250,10 +234,11 @@
 &cp0_eth0 {
        status = "okay";
        /* Network PHY */
-       phy = <&phy0>;
        phy-mode = "10gbase-kr";
        /* Generic PHY, providing serdes lanes */
        phys = <&cp0_comphy4 0>;
+       managed = "in-band-status";
+       sfp = <&sfp_eth0>;
 };
 
 &cp0_sata0 {
@@ -288,10 +273,11 @@
 &cp1_eth0 {
        status = "okay";
        /* Network PHY */
-       phy = <&phy8>;
        phy-mode = "10gbase-kr";
        /* Generic PHY, providing serdes lanes */
        phys = <&cp1_comphy4 0>;
+       managed = "in-band-status";
+       sfp = <&sfp_eth1>;
 };
 
 &cp1_eth1 {

Hope this helps,
baruch

</pre>
    </blockquote>
    <br>
    <pre class="moz-signature" cols="72">-- 
alpha_one_x86/BRULE Herman <a class="moz-txt-link-rfc2396E" href="mailto:alpha_one_x86@first-world.info"><alpha_one_x86@first-world.info></a>
Main developer of Supercopier/Ultracopier/CatchChallenger, Esourcing and server management
IT, OS, technologies, research & development, security and business department</pre>
  </body>
</html>