On Mon, 2 May 2022 at 19:37, Andrew Lunn [off-list ref] wrote:

quoted

+     if (!memcmp(id.base.vendor_name, "HUAWEI          ", 16) &&
+         !memcmp(id.base.vendor_pn, "MA5671A         ", 16))
+             sfp->tx_fault_ignore = true;
+     else
+             sfp->tx_fault_ignore = false;
+
+     return 0;
+
      return 0;

Why do we need two return 0; Probably a merged conflict gone wrong.

Apologies for the oversight. Will submit a v2 shortly.

    Andrew

Matthew