Use Winsock for Windows

include/message.h

@@ -3,14 +3,20 @@
 #include <stdbool.h>
 #include <stdint.h>
 
-#define PACKED __attribute__((__packed__))
+#if defined(_MSC_VER)  // Check if compiling with Microsoft Visual Studio
+#define PACKED(__DECL__) __pragma(pack(push, 1)) __DECL__ __pragma(pack(pop))
+#elif defined(__GNUC__)  // Check if compiling with GCC or any compiler defining __GNUC__, including Clang
+#define PACKED(__DECL__) __DECL__ __attribute__((packed))
+#else
+#error "Unsupported compiler"
+#endif
 
 static const uint8_t CLIENT_HOST = 9;
 static const uint8_t CLIENT_INDEX = 6;
 
 static const uint8_t PREAMBLE = 0x55;
 
-struct PACKED Header {
+PACKED( struct Header {
 
     // The preamble marks the start of a message and is always 0x55
     uint8_t preamble;
@@ -53,13 +59,13 @@ struct PACKED Header {
         };
     };
 
-};
+});
 
-struct PACKED Footer {
+PACKED( struct Footer {
     uint16_t crc;
-};
+});
 
-struct PACKED SetSystemLedReq {
+PACKED( struct SetSystemLedReq {
 
     struct Header header;
 
@@ -88,87 +94,87 @@ struct PACKED SetSystemLedReq {
 
     struct Footer footer;
 
-};
+});
 
-struct PACKED SetSystemLedResp {
+PACKED( struct SetSystemLedResp {
     struct Header header;
     uint8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED SubscribeAddNodeReq
+PACKED( struct SubscribeAddNodeReq
 {
     struct Header header;
     uint8_t hostbyte;
     uint32_t sub_vision; // always 0x03000000
     struct Footer footer;
-};
+});
 
-struct PACKED SubscribeAddNodeResp
+PACKED( struct SubscribeAddNodeResp
 {
     struct Header header;
     uint8_t retcode; // 0x00 or 0x50 are acceptable here
     uint8_t hostbyte;
     struct Footer footer;
-};
+});
 
-struct PACKED SubNodeResetReq
+PACKED( struct SubNodeResetReq
 {
     struct Header header;
     uint8_t hostbyte;
     struct Footer footer;
-};
+});
 
-struct PACKED SubNodeResetResp
+PACKED( struct SubNodeResetResp
 {
     struct Header header;
     uint8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED SetRobotModeReq
+PACKED( struct SetRobotModeReq
 {
     struct Header header;
     uint8_t mode;
     struct Footer footer;
-};
+});
 
-struct PACKED SetRobotModeResp
+PACKED( struct SetRobotModeResp
 {
     struct Header header;
     uint8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED ChassisSpeedModeReq
+PACKED( struct ChassisSpeedModeReq
 {
     struct Header header;
     float speed[3];
     struct Footer footer;
-};
+});
 
-struct PACKED ChassisSpeedModeResp
+PACKED( struct ChassisSpeedModeResp
 {
     struct Header header;
     int8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED SetChassisWheelSpeedReq
+PACKED( struct SetChassisWheelSpeedReq
 {
     struct Header header;
     int8_t wheel_speed[4];
     struct Footer footer;
-};
+});
 
-struct PACKED SetChassisWheelSpeedResp
+PACKED( struct SetChassisWheelSpeedResp
 {
     struct Header header;
     int8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED GimbalCtrlSpeedReq
+PACKED( struct GimbalCtrlSpeedReq
 {
     struct Header header;
     // Values between -360 and 360
@@ -183,53 +189,53 @@ struct PACKED GimbalCtrlSpeedReq
     // Always 0xDC
     uint8_t ctrl;
     struct Footer footer;
-};
+});
 
-struct PACKED GimbalCtrlSpeedResp
+PACKED( struct GimbalCtrlSpeedResp
 {
     struct Header header;
     int8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED SetWheelSpeedReq
+PACKED( struct SetWheelSpeedReq
 {
     struct Header header;
     int16_t wheel_speed[4];
     struct Footer footer;
-};
+});
 
-struct PACKED SetWheelSpeedResp
+PACKED( struct SetWheelSpeedResp
 {
     struct Header header;
     int8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED SdkHeartbeatReq {
+PACKED( struct SdkHeartbeatReq {
     struct Header header;
     struct Footer footer;
-};
+});
 
-struct PACKED SdkHeartbeatResp {
+PACKED( struct SdkHeartbeatResp {
     struct Header header;
     uint8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED SetSdkModeReq {
+PACKED( struct SetSdkModeReq {
     struct Header header;
     uint8_t enable;
     struct Footer footer;
-};
+});
 
-struct PACKED SetSdkModeResp {
+PACKED( struct SetSdkModeResp {
     struct Header header;
     uint8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED SetSdkConnectionReq {
+PACKED( struct SetSdkConnectionReq {
     struct Header header;
     uint8_t control;
     uint8_t host;
@@ -238,32 +244,32 @@ struct PACKED SetSdkConnectionReq {
     uint32_t ip_address;
     uint16_t port;
     struct Footer footer;
-};
+});
 
-struct PACKED SetSdkConnectionResp {
+PACKED( struct SetSdkConnectionResp {
     struct Header header;
     uint8_t retcode;
     uint8_t state;
     uint32_t config_ip;
     struct Footer footer;
-};
+});
 
-struct PACKED BlasterFireReq {
+PACKED( struct BlasterFireReq {
     struct Header header;
     struct {
         uint8_t type : 4;
         uint8_t times : 4;
     };
     struct Footer footer;
-};
+});
 
-struct PACKED BlasterFireResp {
+PACKED( struct BlasterFireResp {
     struct Header header;
     uint8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED StreamCtrlReq {
+PACKED( struct StreamCtrlReq {
     struct Header header;
     uint8_t ctrl;
     struct {
@@ -272,15 +278,15 @@ struct PACKED StreamCtrlReq {
     };
     uint8_t resolution;
     struct Footer footer;
-};
+});
 
-struct PACKED StreamCtrlResp {
+PACKED( struct StreamCtrlResp {
     struct Header header;
     uint8_t retcode;
     struct Footer footer;
-};
+});
 
-struct PACKED AddSubMsgReq {
+PACKED( struct AddSubMsgReq {
     struct Header header;
     uint8_t node_id;
     uint8_t msg_id;
@@ -290,9 +296,9 @@ struct PACKED AddSubMsgReq {
     uint64_t subject_uuid;
     uint16_t freq;
     struct Footer footer;
-};
+});
 
-struct PACKED AddSubMsgResp {
+PACKED( struct AddSubMsgResp {
     struct Header header;
     uint8_t retcode;
     uint8_t pub_node_id;
@@ -300,9 +306,9 @@ struct PACKED AddSubMsgResp {
     uint8_t ack_msg_id;
     uint16_t ack_err_uid_data;
     struct Footer footer;
-};
+});
 
-struct PACKED PushPeriodMsg {
+PACKED( struct PushPeriodMsg {
     struct Header header;
     uint8_t sub_mode;
     uint8_t msg_id;
@@ -312,9 +318,19 @@ struct PACKED PushPeriodMsg {
             float y;
             float z;
         } position;
+        struct {
+            uint16_t yaw_ground;
+            uint16_t pitch_ground;
+            uint16_t yaw;
+            uint16_t pitch;
+            struct {
+                uint8_t option_mode : 2;
+                uint8_t return_center : 1;
+            };
+        } gimbal;
     };
     struct Footer footer;
-};
+});
 
 union Request {
     struct Header header;

src/connection.c

@@ -4,10 +4,26 @@
 #include "connection.h"
 
 #include <stdlib.h>
+
+#ifdef _WIN32
+bool StartupWSA() {
+    static bool wsastarted = false;
+    if (!wsastarted) {
+        WORD		wVersionRequested;
+        WSADATA		wsaData;
+
+        wVersionRequested = MAKEWORD(2, 2);
+        if (WSAStartup(wVersionRequested, &wsaData) == 0) {
+            wsastarted = true;
+        }
+    }
+    return wsastarted;
+}
+#else
 #include <sys/socket.h>
 #include <sys/select.h>
 #include <netinet/in.h>
-#include <arpa/inet.h>
+#endif
 #include <string.h>
 #include <fcntl.h>
 #include <stdio.h>
@@ -82,44 +98,58 @@ int max_fd = -1;
 
 // TODO: Close the socket and return NULL on error
 struct Connection*
-connection_new(unsigned int source_port, const char* source_ip, unsigned int dest_port, const char* dest_ip)
+connection_new(uint16_t source_port, uint32_t source_ip, uint16_t dest_port, uint32_t dest_ip)
 {
     struct Connection* conn = malloc(sizeof(struct Connection));
     memset(conn, 0, sizeof(struct Connection));
 
-    // Request a UDP socket
-    conn->sockfd = socket(AF_INET, SOCK_DGRAM, 0);
-
     // Set the source address and port if they are provided
     if(source_port && source_ip) {
-       struct sockaddr_in loc_addr;
-       loc_addr.sin_family = AF_INET;
-       loc_addr.sin_port = htons(source_port);
-       loc_addr.sin_addr.s_addr = inet_addr(source_ip);
-       if(bind(conn->sockfd, (struct sockaddr*)&loc_addr, sizeof(loc_addr)) < 0)
-       {
-           perror("unable to bind local port");
-           exit(EXIT_FAILURE);
-       }
+       conn->local_addr.sin_family = AF_INET;
+       conn->local_addr.sin_port = htons(source_port);
+       conn->local_addr.sin_addr.s_addr = source_ip;
     }
 
-    // Make the socket non-blocking
-    int flags = fcntl(conn->sockfd, F_GETFL);
-    fcntl(conn->sockfd, F_SETFL, flags | O_NONBLOCK);
-
     // Set the address of the drone
     memset(&conn->remote_addr, 0, sizeof(conn->remote_addr));
     conn->addrlen = sizeof(conn->remote_addr);
     conn->remote_addr.sin_family = AF_INET;
     conn->remote_addr.sin_port = htons(dest_port);
-    conn->remote_addr.sin_addr.s_addr = inet_addr(dest_ip);
+    conn->remote_addr.sin_addr.s_addr = dest_ip;
+
+    return conn;
+}
+
+enum connection_error
+connection_connect(struct Connection* conn) {
+
+#ifdef _WIN32
+    StartupWSA();
+#endif
+
+    // Request a UDP socket
+    conn->sockfd = socket(AF_INET, SOCK_DGRAM, 0);
+
+    if(conn->local_addr.sin_addr.s_addr
+            && bind(conn->sockfd, (struct sockaddr*)&conn->local_addr, sizeof(conn->local_addr)) < 0)
+        return CONNECTION_LOCAL_BIND;
+ 
+    // Make the socket non-blocking
+#ifdef _WIN32
+    static unsigned long sock_opts = 1;
+    ioctlsocket(conn->sockfd, FIONBIO, &sock_opts);	//Non-Blocking
+#else
+    int flags = fcntl(conn->sockfd, F_GETFL);
+    fcntl(conn->sockfd, F_SETFL, flags | O_NONBLOCK);
+#endif
 
     // File descriptors are numbers that count up sequentially,
     // so save the last one as the greatest file descriptor.
     // This is needed for polling the sockets later.
     max_fd = conn->sockfd;
 
-    return conn;
+    return CONNECTION_NO_ERROR;
+
 }
 
 struct Connection*

src/connection.h

@@ -6,14 +6,25 @@
 #include <stddef.h>
 #include <netinet/in.h>
 
+#define ROBOT_DEFAULT_WIFI_ADDR 0X102A8C0
+
 struct Connection {
     int sockfd;
     socklen_t addrlen;
     struct sockaddr_in remote_addr;
+    struct sockaddr_in local_addr;
 };
 
 struct Connection*
-connection_new(unsigned int source_port, const char* source_ip, unsigned int dest_port, const char* dest_ip);
+connection_new(uint16_t source_port, uint32_t source_ip, uint16_t dest_port, uint32_t dest_ip);
+
+enum connection_error {
+    CONNECTION_NO_ERROR,
+    CONNECTION_LOCAL_BIND
+};
+
+enum connection_error
+connection_connect();
 
 struct Connection*
 connection_poll_ready(struct Client* client);

src/robo.c

@@ -4,6 +4,7 @@
 
 #include <string.h>
 #include <stdlib.h>
+#include <arpa/inet.h>
 
 #include <stdio.h>
 
@@ -30,6 +31,11 @@ struct RobotImp {
 
     float x, y, z;
 
+    uint16_t yaw_ground;
+    uint16_t pitch_ground;
+    uint16_t yaw;
+    uint16_t pitch;
+
     enum {
         CONNECTING_TO_PROXY_PORT,
         SETTING_SDK_CONNECTION,
@@ -39,6 +45,7 @@ struct RobotImp {
         SUSBCRIBING_SUBNODE,
         SETTING_MOVEMENT_MODE,
         SUBSCRIBING_POSITION_STATUS,
+        SUBSCRIBING_GIMBAL_STATUS,
         SENDING_HEARTBEAT,
         WAITING,
         READY,
@@ -113,7 +120,8 @@ robot_work(Robot robot) {
 
         case CONNECTING_TO_PROXY_PORT:
         {
-            robot->client->sdk_conn = connection_new(0, 0, 30030, "192.168.2.1");
+            robot->client->sdk_conn = connection_new(0, 0, 30030, ROBOT_DEFAULT_WIFI_ADDR);
+            connection_connect(robot->client->sdk_conn);
             robot->state = SETTING_SDK_CONNECTION;
             break;
         }
@@ -128,7 +136,8 @@ robot_work(Robot robot) {
 
         case CONNECTING_TO_DEVICE_PORT:
         {
-            robot->client->dev_conn = connection_new(10010, "192.168.2.24", 20020, "192.168.2.1");
+            if(connection_connect(robot->client->dev_conn))
+                robot->state = STOPPED;
             robot->state = SETTING_SDK_MODE;
             break;
         }
@@ -177,6 +186,14 @@ robot_work(Robot robot) {
         {
             add_sub_msg ( &req, robot->seq++, true, 20, 1, DDS_POSITION );
             req_send(robot->client->dev_conn, &req);
+            robot->state = SUBSCRIBING_GIMBAL_STATUS;
+            break;
+        }
+
+        case SUBSCRIBING_GIMBAL_STATUS:
+        {
+            add_sub_msg ( &req, robot->seq++, true, 21, 1, DDS_GIMBAL_POS );
+            req_send(robot->client->dev_conn, &req);
             robot->state = WAITING;
             break;
         }
@@ -263,6 +280,7 @@ robot_work(Robot robot) {
                 robot->state = STOPPED;
                 break;
             }
+            robot->client->dev_conn = connection_new(10010, resp.resp.sdkconn.config_ip, 20020, ROBOT_DEFAULT_WIFI_ADDR);
             robot->state = CONNECTING_TO_DEVICE_PORT;
             break;
         case SET_SDK_MODE_CMD:
@@ -309,10 +327,18 @@ robot_work(Robot robot) {
             robot->state = READY;
             break;
         case PUSH_PERIOD_MSG_CMD:
-            // assume it is a position status for now
-            robot->x = resp.resp.push.position.x;
-            robot->y = resp.resp.push.position.y;
-            robot->z = resp.resp.push.position.z;
+            switch(resp.resp.push.msg_id) {
+                case 20:
+                    robot->x = resp.resp.push.position.x;
+                    robot->y = resp.resp.push.position.y;
+                    robot->z = resp.resp.push.position.z;
+                    break;
+                case 21:
+                    robot->yaw_ground = resp.resp.push.gimbal.yaw_ground;
+                    robot->pitch_ground = resp.resp.push.gimbal.pitch_ground;
+                    robot->yaw = resp.resp.push.gimbal.yaw;
+                    robot->pitch = resp.resp.push.gimbal.pitch;
+            }
             break;
     }