On Sun, 3 Dec 2023 at 00:39, Richard Cochran <richardcoch...@gmail.com> wrote:
> @@ -306,13 +307,15 @@ static void do_set_action(struct pmc *pmc, int action,
> int index, char *str)
> "duration %hu "
> "NOTIFY_PORT_STATE %3s "
> "NOTIFY_TIME_SYNC %3s "
> - "NOTIFY_PARENT_DATA_SET %3s ",
> + "NOTIFY_PARENT_DATA_SET %3s "
> + "NOTIFY_CMLDS %3s ",
> &sen.duration,
> onoff_port_state,
> onoff_time_status,
> - onoff_parent_data_set);
> - if (cnt != 4) {
> - fprintf(stderr, "%s SET needs 4 values\n",
> + onoff_parent_data_set,
> + onoff_cmlds);
> + if (cnt != 5) {
> + fprintf(stderr, "%s SET needs 5 values\n",
Doing cnt != 4 && cnt != 5 should just work as sscanf() should stop
parsing when it doesn't find NOTIFY_CMLDS and leave onoff_cmlds as
"off". Obviously this won't scale as more events are added but it'd
keep syntax backwards-compatible for scripts.
Best regards
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel
