From 10f25ad7090c530edc2d34aa8815b287603b586b Mon Sep 17 00:00:00 2001
From: Kebo Liu <kebol@mellanox.com>
Date: Tue, 11 Aug 2020 20:51:12 +0300
Subject: [PATCH] Parse 'Extended Specification compliance' as independent
 field (#112)

Previously 'Extended Specification compliance' was added as part of "sfp_info_bulk", this will request all the vendors to update the related platform API implementation due to the width change of "sfp_info_bulk".

To avoid this, make it parsed separately, it will not impact vendors who haven't update the platform API implementation.
---
 sonic_platform_base/sonic_sfp/sff8436.py | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/sonic_platform_base/sonic_sfp/sff8436.py b/sonic_platform_base/sonic_sfp/sff8436.py
index c0ce2fd93..0b88dafa6 100644
--- a/sonic_platform_base/sonic_sfp/sff8436.py
+++ b/sonic_platform_base/sonic_sfp/sff8436.py
@@ -373,11 +373,6 @@ class sff8436InterfaceId(sffbase):
                 {'offset' : 3,
                  'type' : 'nested',
                  'decode' : specification_compliance},
-            'Extended Specification compliance':
-                {'offset' : 64,
-                 'size':1,
-                 'type' : 'enum',
-                 'decode' : ext_specification_compliance},
             'EncodingCodes':
                 {'offset':11,
                  'size':1,
@@ -456,6 +451,14 @@ class sff8436InterfaceId(sffbase):
                 'type': 'date'}
         }
 
+    sfp_ext_specification_compliance = {
+        'Extended Specification compliance':
+            {'offset' : 0,
+             'size':1,
+             'type' : 'enum',
+             'decode' : ext_specification_compliance}
+        }
+
     qsfp_dom_capability = {
         'Tx_power_support':
             {'offset': 0,
@@ -509,6 +512,9 @@ def parse_vendor_date(self, sn_raw_data, start_pos):
     def parse_vendor_oui(self, sn_raw_data, start_pos):
         return sffbase.parse(self, self.vendor_oui, sn_raw_data, start_pos)
 
+    def parse_ext_specification_compliance(self, sn_raw_data, start_pos):
+        return sffbase.parse(self, self.sfp_ext_specification_compliance, sn_raw_data, start_pos)
+
     def parse_qsfp_dom_capability(self, sn_raw_data, start_pos):
         return sffbase.parse(self, self.qsfp_dom_capability, sn_raw_data, start_pos)