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