On Thu, Apr 22, 2021 at 03:04:17PM +0200, Erez Geva wrote:

> diff --git a/pmc.c b/pmc.c
> index a767c8a..00d6014 100644
> --- a/pmc.c
> +++ b/pmc.c
> @@ -335,6 +335,11 @@ static void pmc_show(struct ptp_message *msg, FILE *fp)
>               fprintf(fp, "TIMESCALE_PROPERTIES "
>                       IFMT "ptpTimescale %d", mtd->val & PTP_TIMESCALE ? 1 : 
> 0);
>               break;
> +     case TLV_MASTER_ONLY:
> +             mtd = (struct management_tlv_datum *) mgt->data;
> +             fprintf(fp, "MASTER_ONLY "
> +                     IFMT "masterOnly %d", mtd->val);
> +             break;

These switch/case tables are in numeric order.

Could you please put this entry between TLV_VERSION_NUMBER (0x200C)
and TLV_DELAY_MECHANISM (0x6000)?

Ditto for the other tables.

>       case TLV_TIME_STATUS_NP:
>               tsn = (struct time_status_np *) mgt->data;
>               fprintf(fp, "TIME_STATUS_NP "

Thanks,
Richard


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

Reply via email to