On Sat, Apr 15, 2023 at 05:29:32PM -0700, Richard Cochran wrote:
> Based on Erez's review, I expanded Christopher's patch to remove the
> control field codes completely.  I've tested this with the following
> hardware, and nothing broke:
> 
> Intel I210
> Intel X550T
> LabX Titanium 411 AVB switch
> 
> Please give this a try with your favorite hardware time stamping
> device.  I'd like to know how many devices stop working when the
> controlField is clear.

Works fine with Xilinx Timer on a ZCU102 system.

Thanks,
Vincent

Compile issue after applying patch against master commit:

  commit 60d829b102be9b8b13357a0f7a93f3e02d44b4ce
  Author: Maciek Machnikowski <mac...@machnikowski.net>
  Date:   Tue Apr 11 15:16:47 2023 +0200

-----------
port.c: In function ‘port_tx_sync’:
port.c:1778:35: error: ‘CTL_FOLLOW_UP’ undeclared (first use in this function)
  fup->header.control            = CTL_FOLLOW_UP;
                                   ^
port.c:1778:35: note: each undeclared identifier is reported only once for each 
function it appears in
port.c: In function ‘process_delay_req’:
port.c:2129:35: error: ‘CTL_DELAY_RESP’ undeclared (first use in this function)
  msg->header.control            = CTL_DELAY_RESP;
                                   ^
port.c: In function ‘process_pdelay_req’:
port.c:2320:35: error: ‘CTL_OTHER’ undeclared (first use in this function)
  rsp->header.control            = CTL_OTHER;
                                   ^
port.c: In function ‘port_management_construct’:
port.c:3209:35: error: ‘CTL_MANAGEMENT’ undeclared (first use in this function)
  msg->header.control            = CTL_MANAGEMENT;



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

Reply via email to