> + 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. Andrew