bindings/python/icsneopy/device/chipid.cpp

@@ -111,7 +111,7 @@ void init_chipid(pybind11::module_& m) {
 		.value("RAD_GALAXY_2_ZMPCHIP_ID", ChipID::RAD_GALAXY_2_ZMPCHIP_ID)
 		.value("NewDevice59_MCHIP", ChipID::NewDevice59_MCHIP)
 		.value("RADMoon2_Z7010_ZYNQ", ChipID::RADMoon2_Z7010_ZYNQ)
-		.value("neoVIFIRE2_Core_SG4", ChipID::neoVIFIRE2_Core_SG4)
+		.value("neoVIFIRE2_CORE_SG4", ChipID::neoVIFIRE2_CORE_SG4)
 		.value("RADBMS_MCHIP", ChipID::RADBMS_MCHIP)
 		.value("RADMoon2_ZL_MCHIP", ChipID::RADMoon2_ZL_MCHIP)
 		.value("RADGigastar_USBZ_Z7010_ZYNQ", ChipID::RADGigastar_USBZ_Z7010_ZYNQ)

include/icsneo/device/chipid.h

@@ -110,7 +110,7 @@ enum class ChipID : uint8_t {
 	RAD_GALAXY_2_ZMPCHIP_ID = 102,
 	NewDevice59_MCHIP = 103,
 	RADMoon2_Z7010_ZYNQ = 104,
-	neoVIFIRE2_Core_SG4 = 105,
+	neoVIFIRE2_CORE_SG4 = 105,
 	RADBMS_MCHIP = 106,
 	RADMoon2_ZL_MCHIP = 107,
 	RADGigastar_USBZ_Z7010_ZYNQ = 108,

include/icsneo/device/device.h

@@ -162,14 +162,6 @@ public:
 
 	bool hasBootloader() { return !!getBootloader(); }
 
-	virtual bool supportsSwVersionValidate() const {
-		return true;
-	}
-
-	void setBootloaderVersion(const HardwareInfo::Version& version) {
-		bootloaderVersion = version;
-	}
-
 	static std::string SerialNumToString(uint32_t serial);
 	static uint32_t SerialStringToNum(const std::string& serial);
 	static bool SerialStringIsNumeric(const std::string& serial);
@@ -990,7 +982,6 @@ protected:
 	LEDState ledState;
 	void updateLEDState();
 
-	std::optional<HardwareInfo::Version> bootloaderVersion = std::nullopt;
 
 private:
 	neodevice_t data;

include/icsneo/device/tree/neovifire2/neovifire2.h

@@ -98,15 +98,13 @@ public:
 	}
 
 	CoreChipVariant getCoreChipVariant() {
-		if(!bootloaderVersion.has_value()) {
 		const auto& hardwareInfo = getHardwareInfo(std::chrono::milliseconds(1000));
 		if(!hardwareInfo) {
 			chipVariant = CoreChipVariant::Invalid;
 			return chipVariant;
 		}
-			setBootloaderVersion(hardwareInfo->bootloaderVersion);
+		const auto& bootloaderVersion = hardwareInfo->bootloaderVersion;
-		}
+		if(bootloaderVersion.major >= CORE_SG4_BL_MAJOR_VERSION_CUTOFF) {
-		if(bootloaderVersion->major >= CORE_SG4_BL_MAJOR_VERSION_CUTOFF) {
 			chipVariant = CoreChipVariant::Core_SG4;
 		} else {
 			chipVariant = CoreChipVariant::Core;
@@ -124,7 +122,7 @@ public:
 		static std::vector<ChipInfo> chipsSG4 = {
 			{ChipID::neoVIFIRE2_MCHIP, true, "MCHIP", "fire2_mchip_ief", 0, FirmwareType::IEF},
 			{ChipID::neoVIFIRE2_ZYNQ, true, "ZCHIP", "fire2_zchip_ief", 1, FirmwareType::IEF},
-			{ChipID::neoVIFIRE2_Core_SG4, true, "Core", "fire2_core_sg4", 2, FirmwareType::IEF}
+			{ChipID::neoVIFIRE2_CORE_SG4, true, "Core", "fire2_core_sg4", 2, FirmwareType::IEF}
 		};
 		
 		if(chipVariant == CoreChipVariant::Core_SG4) {
@@ -139,7 +137,7 @@ public:
 		pipeline.add<EnterBootloaderPhase>()
 				.add<FlashPhase>(ChipID::neoVIFIRE2_MCHIP, BootloaderCommunication::RED);
 		if(chipVariant == CoreChipVariant::Core_SG4) {
-			pipeline.add<FlashPhase>(ChipID::neoVIFIRE2_Core_SG4, BootloaderCommunication::REDCore, false, false);
+			pipeline.add<FlashPhase>(ChipID::neoVIFIRE2_CORE_SG4, BootloaderCommunication::REDCore, false, false);
 		} else {
 			pipeline.add<FlashPhase>(ChipID::neoVIFIRE2_Core, BootloaderCommunication::REDCore, false, false);
 		}
@@ -149,10 +147,6 @@ public:
 		return pipeline;
 	}
 
-	bool supportsSwVersionValidate() const override {
-		return bootloaderVersion.has_value() && (bootloaderVersion->major > 4 || (bootloaderVersion->major == 4 && bootloaderVersion->minor >= 3));
-	}
-
 	std::vector<VersionReport> getChipVersions(bool refreshComponents = true) override {
 		if(chipVariant == CoreChipVariant::Invalid) {
 			getCoreChipVariant();

include/icsneo/platform/servd.h

@@ -18,24 +18,27 @@ class Servd : public Driver {
 public:
 	static void Find(std::vector<FoundDevice>& foundDevices);
 	static bool Enabled();
-	Servd(const device_eventhandler_t& err, neodevice_t& forDevice, const Address& address);
+	Servd(const device_eventhandler_t& err, neodevice_t& forDevice, const std::unordered_set<std::string>& availableDrivers);
 	~Servd() override;
 	bool open() override;
 	bool isOpen() override;
 	bool close() override;
+	bool faa(const std::string& key, int32_t inc, int32_t& orig);
 	bool enableCommunication(bool enable, bool& sendMsg) override;
 	driver_finder_t getFinder() override { return Servd::Find; }
 
 private:
-	void read();
+	void alive();
-	void write();
+	void read(Address&& address);
+	void write(Address&& address);
 	neodevice_t& device;
+	std::thread aliveThread; // makes sure the client and server are healthy
 	std::thread writeThread;
 	std::thread readThread;
 	Socket messageSocket;
 	bool opened = false;
 	bool comEnabled = false;
-	std::unique_ptr<Socket> dataSocket;
+	std::string driver;
 };
 
 }

include/icsneo/platform/socket.h

@@ -73,12 +73,11 @@ public:
 	using SocketHandleType = int;
 	#endif
 	
-	template<class... Args>
+	Socket() {
-	Socket(Args&&... args) {
 		#ifdef _WIN32
 		static WSA wsa;
 		#endif
-		mFD = socket(std::forward<Args>(args)...);
+		mFD = socket(AF_INET, SOCK_DGRAM, 0);
 	}
 	
 	~Socket() {
@@ -103,10 +102,6 @@ public:
 		#endif
 	}
 	
-	bool connect(const Address& to) {
-		return ::connect(mFD, (sockaddr*)&to.sockaddr(), sizeof(sockaddr_in)) != -1;
-	}
-	
 	bool bind(const Address& at) {
 		return ::bind(mFD, (sockaddr*)&at.sockaddr(), sizeof(sockaddr_in)) != -1;
 	}
@@ -146,14 +141,6 @@ public:
 		return true;
 	}
 	
-	bool send(const void* buffer, size_t size) {
-		auto sent = ::send(mFD, (const char*)buffer, (int)size, 0);
-		if(sent == -1) {
-			return false;
-		}
-		return (size_t)sent == size;
-	}
-	
 	bool recvfrom(void* buffer, size_t& size, Address& from) {
 		sockaddr_in addr;
 		socklen_t addLen = sizeof(addr);
@@ -176,8 +163,8 @@ public:
 	}
 
 	template<typename REQ, typename RES>
-	bool transceive(REQ&& request, RES&& response, const std::chrono::milliseconds& timeout) {
+	bool transceive(const Address& to, REQ&& request, RES&& response, const std::chrono::milliseconds& timeout) {
-		if(!send(request.data(), request.size())) {
+		if(!sendto(request.data(), request.size(), to)) {
 			return false;
 		}
 		bool hasData;

platform/servd.cpp

@@ -6,7 +6,7 @@
 
 using namespace icsneo;
 
-#define SERVD_VERSION 2
+#define SERVD_VERSION 1
 
 static const Address SERVD_ADDRESS = Address("127.0.0.1", 26741);
 static const std::string SERVD_VERSION_STR = std::to_string(SERVD_VERSION);
@@ -41,17 +41,20 @@ std::vector<std::string> split(const std::string_view& str, char delim = ' ') {
 }
 
 void Servd::Find(std::vector<FoundDevice>& found) {
-	Socket socket(AF_INET, SOCK_DGRAM, 0);
+	Socket socket;
-	socket.connect(SERVD_ADDRESS);
 	if(!socket.set_nonblocking()) {
 		EventManager::GetInstance().add(APIEvent::Type::ServdNonblockError, APIEvent::Severity::Error);
 		return;
 	}
+	if(!socket.bind(Address("127.0.0.1", 0))) {
+		EventManager::GetInstance().add(APIEvent::Type::ServdBindError, APIEvent::Severity::Error);
+		return;
+	}
 	std::string response;
 
 	response.resize(512);
 	const std::string version_request = SERVD_VERSION_STR + " version";
-	if(!socket.transceive(version_request, response, std::chrono::milliseconds(5000))) {
+	if(!socket.transceive(SERVD_ADDRESS, version_request, response, std::chrono::milliseconds(5000))) {
 		EventManager::GetInstance().add(APIEvent::Type::ServdTransceiveError, APIEvent::Severity::Error);
 		return;
 	}
@@ -63,39 +66,46 @@ void Servd::Find(std::vector<FoundDevice>& found) {
 
 	response.resize(512);
 	const std::string find_request = SERVD_VERSION_STR + " find";
-	if(!socket.transceive(find_request, response, std::chrono::milliseconds(5000))) {
+	if(!socket.transceive(SERVD_ADDRESS, find_request, response, std::chrono::milliseconds(5000))) {
 		EventManager::GetInstance().add(APIEvent::Type::ServdTransceiveError, APIEvent::Severity::Error);
 		return;
 	}
 	const auto lines = split(response, '\n');
 	for(auto&& line : lines) {
 		const auto cols = split(line, ' ');
-		if(cols.size() < 3) {
+		if(cols.size() < 2) {
 			EventManager::GetInstance().add(APIEvent::Type::ServdInvalidResponseError, APIEvent::Severity::Error);
 			continue;
 		}
 		const auto& serial = cols[0];
-		const auto& ip = cols[1];
+		std::unordered_set<std::string> drivers;
-		uint16_t port = 0;
+		for (size_t i = 1; i < cols.size(); ++i) {
-		try {
+			drivers.emplace(cols[i]);
-			port = static_cast<uint16_t>(std::stoi(cols[2]));
-		} catch (const std::exception&) {
-			EventManager::GetInstance().add(APIEvent::Type::ServdInvalidResponseError, APIEvent::Severity::Error);
-			continue;
 		}
-		Address address(ip.c_str(), port);
 		auto& newFound = found.emplace_back();
 		std::copy(serial.begin(), serial.end(), newFound.serial);
 		newFound.makeDriver = [=](device_eventhandler_t err, neodevice_t& forDevice) {
-			return std::make_unique<Servd>(err, forDevice, address);
+			return std::make_unique<Servd>(err, forDevice, drivers);
 		};
 	}
 }
 
-Servd::Servd(const device_eventhandler_t& err, neodevice_t& forDevice, const Address& address) :
+Servd::Servd(const device_eventhandler_t& err, neodevice_t& forDevice, const std::unordered_set<std::string>& availableDrivers) :
-	Driver(err), device(forDevice), messageSocket(AF_INET, SOCK_DGRAM, 0) {
+	Driver(err), device(forDevice) {
-	messageSocket.connect(address);
 	messageSocket.set_nonblocking();
+	messageSocket.bind(Address("127.0.0.1", 0));
+	if(availableDrivers.count("dxx")) {
+		driver = "dxx"; // prefer USB over Ethernet
+	} else if(availableDrivers.count("cab")) {
+		driver = "cab"; // prefer CAB over TCP
+	} else if(availableDrivers.count("tcp")) {
+		driver = "tcp";
+	} else if(availableDrivers.count("vcp")) {
+		driver = "vcp";
+	} else {
+		// just take the first driver
+		driver = *availableDrivers.begin();
+	}
 }
 
 Servd::~Servd() {
@@ -103,31 +113,21 @@ Servd::~Servd() {
 }
 
 bool Servd::open() {
-	const std::string request = SERVD_VERSION_STR + " open";
+	const std::string request = SERVD_VERSION_STR + " open " + std::string(device.serial) + " " + driver;
 	std::string response;
 	response.resize(512);
-	if(!messageSocket.transceive(request, response, std::chrono::milliseconds(5000))) {
+	if(!messageSocket.transceive(SERVD_ADDRESS, request, response, std::chrono::milliseconds(5000))) {
 		EventManager::GetInstance().add(APIEvent::Type::ServdTransceiveError, APIEvent::Severity::Error);
 		return false;
 	}
 	const auto tokens = split(response);
-	if(tokens.size() != 2) {
+	if(tokens.size() != 4) {
 		EventManager::GetInstance().add(APIEvent::Type::ServdInvalidResponseError, APIEvent::Severity::Error);
 		return false;
 	}
-	dataSocket = std::make_unique<Socket>(AF_INET, SOCK_STREAM, 0);
+	aliveThread = std::thread(&Servd::alive, this);
-	const auto& ip = tokens[0];
+	readThread = std::thread(&Servd::read, this, Address{tokens[2].c_str(), (uint16_t)std::stol(tokens[3].c_str())});
-	uint16_t port = 0;
+	writeThread = std::thread(&Servd::write, this, Address{tokens[0].c_str(), (uint16_t)std::stol(tokens[1].c_str())});
-	try {
-		port = static_cast<uint16_t>(std::stoi(tokens[1]));
-	} catch (const std::exception&) {
-		EventManager::GetInstance().add(APIEvent::Type::ServdInvalidResponseError, APIEvent::Severity::Error);
-		return false;
-	}
-	Address address(ip.c_str(), port);
-	dataSocket->connect(address);
-	readThread = std::thread(&Servd::read, this);
-	writeThread = std::thread(&Servd::write, this);
 	opened = true;
 	return true;
 }
@@ -138,6 +138,9 @@ bool Servd::isOpen() {
 
 bool Servd::close() {
 	setIsClosing(true);
+	if(aliveThread.joinable()) {
+		aliveThread.join();
+	}
 	if(readThread.joinable()) {
 		readThread.join();
 	}
@@ -145,16 +148,8 @@ bool Servd::close() {
 		writeThread.join();
 	}
 	if(isOpen()) {
-		Address localAddress;
+		const std::string request = SERVD_VERSION_STR + " close " + std::string(device.serial);
-		dataSocket->address(localAddress);
+		messageSocket.sendto(request.data(), request.size(), SERVD_ADDRESS);
-		const std::string request = SERVD_VERSION_STR + " close " + localAddress.ip() + " " + std::to_string(localAddress.port());
-		std::string response;
-		response.resize(1);
-		if(!messageSocket.transceive(request, response, std::chrono::milliseconds(5000))) {
-			EventManager::GetInstance().add(APIEvent::Type::ServdTransceiveError, APIEvent::Severity::Error);
-			return false;
-		}
-		dataSocket.reset();
 	}
 	opened = false;
 	setIsClosing(false);
@@ -164,13 +159,13 @@ bool Servd::close() {
 bool Servd::enableCommunication(bool enable, bool& sendMsg) {
 	const std::string serialString(device.serial);
 	{
-		const std::string request = SERVD_VERSION_STR + " lock com 1000";
+		const std::string request = SERVD_VERSION_STR + " lock " + serialString + " com 1000";
 		std::string response;
 		response.resize(1);
 		bool locked = false;
 		const auto timeout = std::chrono::steady_clock::now() + std::chrono::seconds(1);
 		do {
-			if(!messageSocket.transceive(request, response, std::chrono::milliseconds(5000))) {
+			if(!messageSocket.transceive(SERVD_ADDRESS, request, response, std::chrono::milliseconds(5000))) {
 				return false;
 			}
 			locked = response == "1" ? true : false;
@@ -186,10 +181,10 @@ bool Servd::enableCommunication(bool enable, bool& sendMsg) {
 	}
 	uint64_t com = 0;
 	{
-		const std::string request = SERVD_VERSION_STR + " load com";
+		const std::string request = SERVD_VERSION_STR + " load " + serialString + " com";
 		std::string response;
 		response.resize(20);
-		if(!messageSocket.transceive(request, response, std::chrono::milliseconds(5000))) {
+		if(!messageSocket.transceive(SERVD_ADDRESS, request, response, std::chrono::milliseconds(5000))) {
 			EventManager::GetInstance().add(APIEvent::Type::ServdTransceiveError, APIEvent::Severity::Error);
 			return false;
 		}
@@ -207,20 +202,16 @@ bool Servd::enableCommunication(bool enable, bool& sendMsg) {
 	}
 	if(comEnabled != enable) {
 		com += enable ? 1 : -1;
-		const std::string request = SERVD_VERSION_STR + " store com " + std::to_string(com);
+		const std::string request = SERVD_VERSION_STR + " store " + serialString  + " com " + std::to_string(com);
-		std::string response;
+		if(!messageSocket.sendto(request.data(), request.size(), SERVD_ADDRESS)) {
-		response.resize(1);
-		if(!messageSocket.transceive(request, response, std::chrono::milliseconds(5000))) {
 			EventManager::GetInstance().add(APIEvent::Type::ServdSendError, APIEvent::Severity::Error);
 			return false;
 		}
 	}
 	comEnabled = enable;
 	{
-		const std::string request = SERVD_VERSION_STR + " unlock com";
+		const std::string request = SERVD_VERSION_STR + " unlock " + serialString + " com";
-		std::string response;
+		if(!messageSocket.sendto(request.data(), request.size(), SERVD_ADDRESS)) {
-		response.resize(1);
-		if(!messageSocket.transceive(request, response, std::chrono::milliseconds(5000))) {
 			EventManager::GetInstance().add(APIEvent::Type::ServdSendError, APIEvent::Severity::Error);
 			return false;
 		}
@@ -228,11 +219,78 @@ bool Servd::enableCommunication(bool enable, bool& sendMsg) {
 	return true;
 }
 
-void Servd::read() {
+void Servd::alive() {
-	std::vector<uint8_t> buf(2 * 1024 * 1024);
+	Socket socket;
+	socket.set_nonblocking();
+	socket.bind(Address("127.0.0.1", 0));
+	const std::string statusRequest = SERVD_VERSION_STR + " status " + std::string(device.serial);
+	std::string statusResponse;
+	statusResponse.resize(8);
+	while(!isDisconnected() && !isClosing()) {
+		if(!socket.sendto(statusRequest.data(), statusRequest.size(), {"127.0.0.1", 26741})) {
+			EventManager::GetInstance().add(APIEvent::Type::ServdSendError, APIEvent::Severity::Error);
+			setIsDisconnected(true);
+			return;
+		}
+		bool hasData;
+		if(!socket.poll(std::chrono::milliseconds(2000), hasData)) {
+			EventManager::GetInstance().add(APIEvent::Type::ServdPollError, APIEvent::Severity::Error);
+			setIsDisconnected(true);
+			return;
+		}
+		if(!hasData) {
+			EventManager::GetInstance().add(APIEvent::Type::ServdNoDataError, APIEvent::Severity::Error);
+			setIsDisconnected(true);
+			return;
+		}
+		size_t statusResponseSize = statusResponse.size();
+		if(!socket.recv(statusResponse.data(), statusResponseSize)) {
+			EventManager::GetInstance().add(APIEvent::Type::ServdRecvError, APIEvent::Severity::Error);
+			setIsDisconnected(true);
+			return;
+		}
+		statusResponse.resize(statusResponseSize);
+		if(statusRequest == "closed") {
+			EventManager::GetInstance().add(APIEvent::Type::DeviceDisconnected, APIEvent::Severity::Error);
+			setIsDisconnected(true);
+			return;
+		}
+		if(statusResponse != "open") {
+			EventManager::GetInstance().add(APIEvent::Type::ServdInvalidResponseError, APIEvent::Severity::Error);
+			setIsDisconnected(true);
+			return;
+		}
+		std::this_thread::sleep_for(std::chrono::milliseconds(1000));
+	}
+}
+
+void Servd::read(Address&& address) {
+	Socket socket;
+	socket.set_nonblocking();
+	socket.set_reuse(true);
+	#ifdef _WIN32
+	if(!socket.bind(Address("127.0.0.1", address.port()))) {
+		EventManager::GetInstance().add(APIEvent::Type::ServdBindError, APIEvent::Severity::Error);
+		setIsDisconnected(true);
+		return;
+	}
+	#else
+	if(!socket.bind(Address(address.ip().c_str(), address.port()))) {
+		EventManager::GetInstance().add(APIEvent::Type::ServdBindError, APIEvent::Severity::Error);
+		setIsDisconnected(true);
+		return;
+	}
+	#endif
+	if(!socket.join_multicast("127.0.0.1", address.ip())) {
+		EventManager::GetInstance().add(APIEvent::Type::ServdJoinMulticastError, APIEvent::Severity::Error);
+		setIsDisconnected(true);
+		return;
+	}
+
+	std::vector<uint8_t> buf(65535);
 	while(!isDisconnected() && !isClosing()) {
 		bool hasData;
-		if(!dataSocket->poll(std::chrono::milliseconds(100), hasData)) {
+		if(!socket.poll(std::chrono::milliseconds(100), hasData)) {
 			EventManager::GetInstance().add(APIEvent::Type::ServdPollError, APIEvent::Severity::Error);
 			setIsDisconnected(true);
 			return;
@@ -241,7 +299,7 @@ void Servd::read() {
 			continue;
 		}
 		size_t bufSize = buf.size();
-		if(!dataSocket->recv(buf.data(), bufSize)) {
+		if(!socket.recv(buf.data(), bufSize)) {
 			EventManager::GetInstance().add(APIEvent::Type::ServdRecvError, APIEvent::Severity::Error);
 			setIsDisconnected(true);
 			return;
@@ -250,14 +308,16 @@ void Servd::read() {
 	}
 }
 
-void Servd::write() {
+void Servd::write(Address&& address) {
+	Socket socket;
+	socket.bind(Address("127.0.0.1", 0));
 	WriteOperation writeOp;
 	while(!isDisconnected() && !isClosing()) {
 		if(!writeQueue.wait_dequeue_timed(writeOp, std::chrono::milliseconds(100))) {
 			continue;
 		}
 		if(!isClosing()) {
-			if(!dataSocket->send(writeOp.bytes.data(), writeOp.bytes.size())) {
+			if(!socket.sendto(writeOp.bytes.data(), writeOp.bytes.size(), address)) {
 				EventManager::GetInstance().add(APIEvent::Type::ServdSendError, APIEvent::Severity::Error);
 				setIsDisconnected(true);
 				return;