Skip to content

Commit

Permalink
[portsorch]: Remove SAI_PORT_ATTR_BIND_MODE (sonic-net#262)
Browse files Browse the repository at this point in the history
This attribute is removed in the SAI master.
Port can be part of .1D bridge and VLANs at the same time. Hence
bind mode is not necessary.
  • Loading branch information
Shuotian Cheng authored Jul 21, 2017
1 parent e521a35 commit 742852d
Showing 1 changed file with 0 additions and 14 deletions.
14 changes: 0 additions & 14 deletions orchagent/portsorch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1181,20 +1181,6 @@ bool PortsOrch::addVlanMember(Port vlan, Port port, string& tagging_mode)
SWSS_LOG_NOTICE("Set untagged port %s VLAN ID to %hu", port.m_alias.c_str(), vlan.m_vlan_id);
}

#if 0
// Set port bind mode
sai_attribute_t port_attr;
port_attr.id = SAI_PORT_ATTR_BIND_MODE;
port_attr.value.s32 = SAI_PORT_BIND_MODE_PORT;
status = sai_port_api->set_port_attribute(port.m_port_id, &port_attr);
if (status != SAI_STATUS_SUCCESS)
{
SWSS_LOG_ERROR("Failed to set port pid:%lx bind mode to port: %d",
port.m_port_id, status);
return false;
}
#endif

port.m_vlan_id = vlan.m_vlan_id;
port.m_port_vlan_id = vlan.m_vlan_id;
port.m_vlan_member_id = vlan_member_id;
Expand Down

0 comments on commit 742852d

Please sign in to comment.