diff --git a/packages/base/any/onlp/src/sff/module/inc/sff/8472.h b/packages/base/any/onlp/src/sff/module/inc/sff/8472.h
index 6b7a1161..20b52d70 100644
--- a/packages/base/any/onlp/src/sff/module/inc/sff/8472.h
+++ b/packages/base/any/onlp/src/sff/module/inc/sff/8472.h
@@ -968,7 +968,6 @@ _sff8472_media_sfp28_cr(const uint8_t* idprom)
     /* module should be sfp */
     if (!SFF8472_MODULE_SFP(idprom)) return 0;
 
-    if (idprom[2] != SFF8472_CONN_NOSEP) return 0;
     if ((idprom[3] & SFF8472_CC3_INF_1X_CU_PASSIVE) == 0) return 0;
     if (idprom[12] == 0xFF) return 1;
 
diff --git a/packages/base/any/onlp/src/sff/module/src/sff.c b/packages/base/any/onlp/src/sff/module/src/sff.c
index bb916944..281c4450 100644
--- a/packages/base/any/onlp/src/sff/module/src/sff.c
+++ b/packages/base/any/onlp/src/sff/module/src/sff.c
@@ -472,6 +472,7 @@ sff_eeprom_parse_standard__(sff_eeprom_t* se, uint8_t* eeprom)
                     se->info.length = se->eeprom[146];
                     break;
                 case SFF_SFP_TYPE_SFP:
+                case SFF_SFP_TYPE_SFP28:
                     se->info.length = se->eeprom[18];
                     break;
                 default: