On Fri, Sep 02, 2022 at 12:26:11PM +0530, SyncMonk Technologies wrote:
> @@ -1094,6 +1098,32 @@ struct clock *clock_create(enum clock_type type, 
> struct config *config,
>               return NULL;
>       }
>  
> +     /* Configure the Virtual Port. */
> +     vport_ifname = config_get_string(config, NULL, "vport_address");
> +     if (vport_ifname && vport_ifname[0]) {
> +             c->vport_if = interface_create(vport_ifname);
> +             if (config_set_section_int(config, interface_name(c->vport_if),
> +                                        "announceReceiptTimeout", 2)) {
> +                     return NULL;
> +             }
> +             if (config_set_section_int(config,
> +                                        interface_name(c->vport_if),
> +                                        "delay_mechanism",
> +                                        DM_NO_MECHANISM)) {
> +                     return NULL;
> +             }
> +             if (config_set_section_int(config,
> +                                        interface_name(c->vport_if),
> +                                        "network_transport",
> +                                        TRANS_VPORT)) {
> +                     return NULL;
> +             }
> +             if (config_set_section_int(config, interface_name(c->vport_if),
> +                                        "delay_filter_length", 1)) {
> +                     return NULL;
> +             }
> +     }

Why hard code these values?  Why not let the user decide?

Thanks,
Richard


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

Reply via email to