address review comments

ds952811 · ds952811 · commit a04594e6efee · 2022-05-19T07:10:57.000Z
Signed-off-by: Dante Su &lt;dante.su@broadcom.com&gt;
diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -3080,7 +3080,7 @@ void PortsOrch::doPortTask(Consumer &consumer)
                         }
                         if (p.m_cap_lt < 1)
                         {
-                            SWSS_LOG_ERROR("%s: LT is not supported by the ASIC", alias.c_str());
+                            SWSS_LOG_WARN("%s: LT is not supported by the ASIC", alias.c_str());
                             // Don't retry
                             it = consumer.m_toSync.erase(it);
                             continue;
@@ -6128,8 +6128,6 @@ bool PortsOrch::getPortLinkTrainingRxStatus(const Port &port, sai_port_link_trai
 
     sai_attribute_t attr;
     attr.id = SAI_PORT_ATTR_LINK_TRAINING_RX_STATUS;
-    attr.value.u32 = 0;
-
     sai_status_t ret = sai_port_api->get_port_attribute(port.m_port_id, 1, &attr);
     if (ret != SAI_STATUS_SUCCESS)
     {

Original file line number	Diff line number	Diff line change
`@@ -3080,7 +3080,7 @@ void PortsOrch::doPortTask(Consumer &consumer)`
`3080`	`3080`	`}`
`3081`	`3081`	`if (p.m_cap_lt < 1)`
`3082`	`3082`	`{`
`3083`		`- SWSS_LOG_ERROR("%s: LT is not supported by the ASIC", alias.c_str());`
	`3083`	`+ SWSS_LOG_WARN("%s: LT is not supported by the ASIC", alias.c_str());`
`3084`	`3084`	`// Don't retry`
`3085`	`3085`	`it = consumer.m_toSync.erase(it);`
`3086`	`3086`	`continue;`
`@@ -6128,8 +6128,6 @@ bool PortsOrch::getPortLinkTrainingRxStatus(const Port &port, sai_port_link_trai`
`6128`	`6128`
`6129`	`6129`	`sai_attribute_t attr;`
`6130`	`6130`	`attr.id = SAI_PORT_ATTR_LINK_TRAINING_RX_STATUS;`
`6131`		`- attr.value.u32 = 0;`
`6132`		`-`
`6133`	`6131`	`sai_status_t ret = sai_port_api->get_port_attribute(port.m_port_id, 1, &attr);`
`6134`	`6132`	`if (ret != SAI_STATUS_SUCCESS)`
`6135`	`6133`	`{`