DellEMC SFP Refactor

[aravindmani-1]

Why I did it

Since SFP refactor has common API's, redundant platform API has to be removed.

How I did it

Removed redundant SFP API in DellEMC platforms.

How to verify it

Execute "sfpshow eeprom -d", verify syslogs pertaining to pmon, OIR tests etc..
z9332f_ut.txt

Note:
Please merge this PR only when sonic-platform-common submodule is updated with sonic_xcvr package in sonic-buildimage repo.

Which release branch to backport (provide reason below if selected)

• 201811
• 201911
• 202006
• 202012
• 202106

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

[lgtm-com]

This pull request introduces 5 alerts and fixes 3 when merging 356a885 into fcff3f3 - view on LGTM.com

new alerts:

• 5 for Syntax error

fixed alerts:

• 1 for Unused local variable
• 1 for Unused import
• 1 for Variable defined multiple times

[prgeor]

@aravindmani-1 please test the case where a user plugs-out and plugs-in a different sfp type

1. QSFP+/QSFP28 -> QSFP-DD
2. QSFP-DD -> QSFP+/QSFP28

Ensure the APIs created by the xcvr_api_factory.py is correctly matches to the type of sfp module inserted.

[aravindmani-1]

@aravindmani-1 please test the case where a user plugs-out and plugs-in a different sfp type

1. QSFP+/QSFP28 -> QSFP-DD
2. QSFP-DD -> QSFP+/QSFP28

Ensure the APIs created by the xcvr_api_factory.py is correctly matches to the type of sfp module inserted.

yes. with the OIR testing, QSFP-DD to QSFP28/QSFP+ and vice versa has been tested, it is working fine.
OIR_Test.txt

[lgtm-com]

This pull request fixes 3 alerts when merging 19e4b49 into e2bffdf - view on LGTM.com

fixed alerts:

• 1 for Unused local variable
• 1 for Unused import
• 1 for Variable defined multiple times

[lgtm-com]

This pull request fixes 3 alerts when merging fde2d68 into b8eb9f2 - view on LGTM.com

fixed alerts:

• 1 for Unused local variable
• 1 for Unused import
• 1 for Variable defined multiple times

[lgtm-com]

This pull request fixes 3 alerts when merging 529f3c0 into db0b789 - view on LGTM.com

fixed alerts:

• 1 for Unused local variable
• 1 for Unused import
• 1 for Variable defined multiple times

[aravindmani-1]

@prgeor @qiluo-msft Could you please help to cherry pick this PR to 202111 branch?.

[judyjoseph]

@aravindmani-1 Please raise a separate PR for 202111 as there are conflicts