On Wed, Jun 01, 2022 at 10:38:29AM +0530, SyncMonk Technologies wrote:
> As per IEEE1588-2019, Table 21 adding NO_MECHANISM support. In this
> case port does not implement the delay mechanism.

>  static struct config_enum delay_mech_enu[] = {
> -     { "Auto", DM_AUTO },
> -     { "E2E",  DM_E2E },
> -     { "P2P",  DM_P2P },
> +     { "Auto",         DM_AUTO },
> +     { "E2E",          DM_E2E },
> +     { "P2P",          DM_P2P },
> +     { "NO_MECHANISM", DM_NO_MECHANISM },

Please describe the new option in the man page.

>  static struct config_enum tsproc_enu[] = {
> -     { "filter",        TSPROC_FILTER        },
> -     { "raw",           TSPROC_RAW           },
> -     { "filter_weight", TSPROC_FILTER_WEIGHT },
> -     { "raw_weight",    TSPROC_RAW_WEIGHT    },
> +     { "filter",        TSPROC_FILTER                },
> +     { "raw",           TSPROC_RAW           },
> +     { "filter_weight", TSPROC_FILTER_WEIGHT },
> +     { "raw_weight",    TSPROC_RAW_WEIGHT    },
> +     { "no_delay",      TSPROC_NO_DELAY_MECHANISM    },
>       { NULL, 0 },

It's not clear to me why you are adding a new tsproc mode. If you set
the delay with "tsproc_set_delay(0)", I think you will get identical
behavior. There is the initial_delay option which does that.

-- 
Miroslav Lichvar



_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to