[EXTERNAL PHY] Clarification needed on SAI_PORT_ATTR_SUPPORTED_FEC_MODE and SAI_PORT_ATTR_SUPPORTED_FEC_MODE_EXTENDED attributes

[SidharajU]

Hi,

I have a question about user expectations when the following attributes are called: SAI_PORT_ATTR_SUPPORTED_FEC_MODE SAI_PORT_ATTR_SUPPORTED_FEC_MODE_EXTENDED

FEC modes can be different based on the port speed. So does this attribute provide a list of all supported FECs for the configured speed or it should provide a list of all FEC modes supported by the port regardless of the configured speed? From external PHY perspective, A PHY device can be equivalent to a single port in certain cases. So a port capability can become a PHY capability.

[SidharajU]

Based on today's discussion in the OCP SAI meeting, its best to return all supported FEC modes by the chip. In certain cases, supported FEC modes should be known to the user before even configuring the speed so it is required to return all supported FEC modes. If there is a requirement to return speed based FEC modes, then we can add a separate attribute to support it.