Fix comment errors and update proposal doc

Signed-off-by: Rich Sugarman <rsugarman@nvidia.com>

Author: rdsugarmannv

doc/SAI-Proposal-Global-PTP-Configuration.md

@@ -19,21 +19,28 @@ In addition, many existing switches have a configuration model which allows glob
 
 ## Technical Specification
 
-### New Switch PTP Configuration Type
+### Updated description of existing PORT PTP type - the main comment is new
 ```c
-typedef struct _sai_switch_ptp_mode_t
+/**
+ * @brief PTP mode
+ * 
+ * These modes can be used at the port and switch level.
+ * All ports use the value set at the switch level unless explicitly configured
+ * at the port level to a value other than SAI_PORT_PTP_MODE_NONE.
+ */
+typedef enum _sai_port_ptp_mode_t
 {
-    /** None - per-port configuration is used to switch on PTP for a port */
-    SAI_SWITCH_PTP_MODE_NONE,
+    /** No special processing for PTP packets */
+    SAI_PORT_PTP_MODE_NONE,
 
-    /** One-step - all ports use one-step PTP mode */
-    SAI_SWITCH_PTP_MODE_SINGLE_STEP_TIMESTAMP,
+    /** Single-step Timestamp mode for the PTP packets */
+    SAI_PORT_PTP_MODE_SINGLE_STEP_TIMESTAMP,
+
+    /** Two-step Timestamp mode for the PTP packets */
+    SAI_PORT_PTP_MODE_TWO_STEP_TIMESTAMP,
+
+} sai_port_ptp_mode_t;
 
-    /** Two-step - all ports use two-step PTP mode */
-    SAI_SWITCH_PTP_MODE_TWO_STEP_TIMESTAMP
-    
-} sai_switch_ptp_mode_t;
-```
 
 ### New Switch Attribute
 ```c
@@ -47,9 +54,9 @@ typedef enum _sai_switch_attr_t
      * Global PTP mode configuration for the switch.
      * Applies to all ports unless overridden by port-specific settings.
      *
-     * @type sai_switch_ptp_mode_t
+     * @type sai_port_ptp_mode_t
      * @flags CREATE_AND_SET
-     * @default SAI_SWITCH_PTP_MODE_NONE
+     * @default SAI_PORT_PTP_MODE_NONE
      */
     SAI_SWITCH_ATTR_PTP_MODE,
 
@@ -62,7 +69,7 @@ typedef enum _sai_switch_attr_t
 // Set global PTP configuration at switch level
 sai_attribute_t attr;
 attr.id = SAI_SWITCH_ATTR_PTP_MODE;
-attr.value.s32 = SAI_SWITCH_PTP_MODE_SINGLE_STEP_TIMESTAMP;
+attr.value.s32 = SAI_PORT_PTP_MODE_SINGLE_STEP_TIMESTAMP;
 
 sai_status_t status = sai_switch_api->set_switch_attribute(
     switch_id,

inc/saiport.h

@@ -395,7 +395,6 @@ typedef enum _sai_port_priority_flow_control_mode_t
 
 /**
  * @brief PTP mode
- * 
  * These modes can be used at the port and switch level.
  * All ports use the value set at the switch level unless explicitly configured
  * at the port level to a value other than SAI_PORT_PTP_MODE_NONE.