Revision history [back]

fc06600d: EtherCAT: General improvements

         { &hf_ecat_cmd,
           { "Command", "ecat.cmd",
-             FT_UINT8, BASE_HEX, VALS(EcCmdShort), 0x0, NULL, HFILL }
+            FT_UINT8, BASE_HEX, VALS(EcCmdLong), 0x0, NULL, HFILL }
         },

Field was changed to use the long value strings:
ecat.cmd == "Broadcast Read"

The sub cmd fields still use short name:
ecat.sub1.cmd == BRD still supported but ecat.sub1.cmd == "BRD" preferred.

packet-ethercat-datagram.c:

static const value_string EcCmdLong[] =
{
   {     0, "No operation" },
   {     1, "Auto Increment Physical Read" },
   {     2, "Auto Increment Physical Write" },
   {     3, "Auto Increment Physical ReadWrite" },
   {     4, "Configured address Physical Read" },
   {     5, "Configured address Physical Write" },
   {     6, "Configured address Physical ReadWrite" },
   {     7, "Broadcast Read" },
   {     8, "Broadcast Write" },
   {     9, "Broadcast ReadWrite" },
   {    10, "Logical Read" },
   {    11, "Logical Write" },
   {    12, "Logical ReadWrite" },
   {    13, "Auto Increment Physical Read Multiple Write" },
   {    14, "Configured Address Physical Read Multiple Write" },
   {   255, "EXT" },
   {   0, NULL }
};