Add enum for input/output/monitoring/serial ports statuses.

cmd.c

@@ -197,7 +197,11 @@ bool parse_command(struct videohub_data *d, char *cmd) {
 			snprintf(s_port->port[port_num].name, sizeof(s_port->port[port_num].name), "%s", port_data);
 			break;
 		case PARSE_STATUS:
-			snprintf(s_port->port[port_num].status, sizeof(s_port->port[port_num].status), "%s", port_data);
+			s_port->port[port_num].status = S_UNKNOWN;
+			if (streq("BNC", port_data))          s_port->port[port_num].status = S_BNC;
+			else if (streq("Optical", port_data)) s_port->port[port_num].status = S_OPTICAL;
+			else if (streq("RS422", port_data))   s_port->port[port_num].status = S_RS422;
+			else if (streq("None", port_data))    s_port->port[port_num].status = S_NONE;
 			break;
 		case PARSE_DIR:
 			s_port->port[port_num].direction = DIR_AUTO;

data.h

@@ -28,6 +28,14 @@ struct device_desc {
 	unsigned int	num_video_processing_units;
 };
 
+enum port_status {
+	S_UNKNOWN,
+	S_NONE,
+	S_BNC,
+	S_OPTICAL,
+	S_RS422,
+};
+
 enum port_lock {
 	PORT_UNLOCKED,
 	PORT_LOCKED,
@@ -46,7 +54,7 @@ struct port {
 	// The statuses (actually they are connection types) are:
 	//  BNC, Optical or None /missing port/ - for input/output
 	//  RS422, None                         - for serial ports
-	char			status[8];
+	enum port_status	status;
 	// For serial ports.
 	// The values are:
 	//    control - In (Workstation)

display.c

@@ -26,12 +26,14 @@ static void printf_line(int len) {
 	printf("\n");
 }
 
-static char format_status(char *status) {
-	if (!strlen(status))          return ' ';
-	if (streq(status, "BNC"))     return 'B';
-	if (streq(status, "Optical")) return 'o';
-	if (streq(status, "None"))    return 'x';
-	if (streq(status, "RS422"))   return '4'; // For serial ports
+static char format_status(enum port_status status) {
+	switch (status) {
+	case S_UNKNOWN: return ' ';
+	case S_BNC    : return 'B';
+	case S_OPTICAL: return 'o';
+	case S_NONE   : return 'x';
+	case S_RS422  : return '4'; // For serial ports
+	}
 	return '?';
 }