Minor networking fixes

arch/risc-v/src/mpfs/mpfs_ethernet.c

@@ -3390,6 +3390,10 @@ static int mpfs_phyinit(struct mpfs_ethmac_s *priv)
     {
       ret = ksz9477_i2c_init(bus, KSZ9477_PORT_SGMII);
     }
+  else
+    {
+      ret = -EINVAL;
+    }
 
 #endif
 

drivers/net/ksz9477.c

@@ -640,6 +640,14 @@ int ksz9477_init(ksz9477_port_t master_port)
       return ret ? ret : -EINVAL;
     }
 
+  /* Errata 16: SGMII registers are not initialized by hardware reset
+   * To ensure clean environment, reset the switch now.
+   */
+
+  regval16 = SGMII_CONTROL_SOFT_RESET;
+  ret = ksz9477_sgmii_write_indirect(KSZ9477_SGMII_CONTROL,
+                                     &regval16, 1);
+
   /* Check that indirect access to PHY MMD works.
    * Write LED mode to single-LED mode and verify access by
    * reading back the value.
@@ -721,13 +729,25 @@ int ksz9477_init(ksz9477_port_t master_port)
       ret = ksz9477_sgmii_write_indirect(KSZ9477_SGMII_AUTONEG_CONTROL,
                                          &regval16, 1);
 
+      if (ret != OK)
+        {
+          nerr("Failed to set SGMII port into PHY mode, ret %d\n", ret);
+          return ret ? ret : -EINVAL;
+        }
+
       /* Write to autonegotiation advertisement register activates the new
        * setting. Advertise only full duplex.
        */
 
       regval16 = SGMII_AUTONEG_ADVERTISE_FD;
       ret = ksz9477_sgmii_write_indirect(KSZ9477_SGMII_AUTONEG_ADVERTISE,
                                          &regval16, 1);
+
+      if (ret != OK)
+        {
+          nerr("Failed to set autoneg, ret %d\n", ret);
+          return ret ? ret : -EINVAL;
+        }
     }
 
   /* Configure the static port-based VLANs */
@@ -747,6 +767,12 @@ int ksz9477_init(ksz9477_port_t master_port)
               g_port_vlan_config[i]);
     }
 
+  if (ret != OK)
+    {
+      nerr("Failed to configure VLANs, ret %d\n", ret);
+      return ret ? ret : -EINVAL;
+    }
+
 #endif
 
 #ifdef CONFIG_NET_KSZ9477_PORT_SNIFF
@@ -760,6 +786,12 @@ int ksz9477_init(ksz9477_port_t master_port)
               g_port_mirror_config[i]);
     }
 
+  if (ret != OK)
+    {
+      nerr("Failed to configure sniffer port, ret %d\n", ret);
+      return ret ? ret : -EINVAL;
+    }
+
 #endif
 
   return ret;

drivers/net/ksz9477_reg.h

@@ -100,6 +100,10 @@
 
 /* Register bit definitions */
 
+/* KSZ9477_SGMII_CONTROL */
+
+#define SGMII_CONTROL_SOFT_RESET          (1 << 15)
+
 /* KSZ9477_ID2, KSZ9477_ID1 */
 
 #define KSZ9477_ID                         0x9477