LED-Device updates (#1315)

* SPI Fix - Free allocated memory in case of invert

* Remove development statement

* UI updates

* Minor corrections

* Tpm2net Fix - Free allocated memory

* LGTM Finding

* Add config directory and user type to SysInfo

* WS281x - Ensure that device runs with root priviliges

* WS281x - Update DMA defauls as per rpi_ws281x recommendation

* Support Sysinfo for Qt < 5.10

* Address LGTM finding

libsrc/leddevice/dev_net/LedDeviceTpm2net.cpp

@@ -4,9 +4,15 @@ const ushort TPM2_DEFAULT_PORT = 65506;
 
 LedDeviceTpm2net::LedDeviceTpm2net(const QJsonObject &deviceConfig)
 	: ProviderUdp(deviceConfig)
+	, _tpm2_buffer(nullptr)
 {
 }
 
+LedDeviceTpm2net::~LedDeviceTpm2net()
+{
+	free (_tpm2_buffer);
+}
+
 LedDevice* LedDeviceTpm2net::construct(const QJsonObject &deviceConfig)
 {
 	return new LedDeviceTpm2net(deviceConfig);
@@ -25,6 +31,8 @@ bool LedDeviceTpm2net::init(const QJsonObject &deviceConfig)
 		_tpm2ByteCount = 3 * _ledCount;
 		_tpm2TotalPackets = (_tpm2ByteCount / _tpm2_max) + ((_tpm2ByteCount % _tpm2_max) != 0);
 
+		_tpm2_buffer = (uint8_t*) malloc(_tpm2_max+7);
+
 		isInitOK = true;
 	}
 	return isInitOK;
@@ -32,8 +40,6 @@ bool LedDeviceTpm2net::init(const QJsonObject &deviceConfig)
 
 int LedDeviceTpm2net::write(const std::vector<ColorRgb> &ledValues)
 {
-	uint8_t * tpm2_buffer = (uint8_t*) malloc(_tpm2_max+7);
-
 	int retVal = 0;
 
 	int _thisPacketBytes = 0;
@@ -48,23 +54,22 @@ int LedDeviceTpm2net::write(const std::vector<ColorRgb> &ledValues)
 			_thisPacketBytes = (_tpm2ByteCount - rawIdx < _tpm2_max) ? _tpm2ByteCount % _tpm2_max : _tpm2_max;
 //			                        is this the last packet?         ?    ^^ last packet          : ^^ earlier packets
 
-			tpm2_buffer[0] = 0x9c;	// Packet start byte
-			tpm2_buffer[1] = 0xda; // Packet type Data frame
-			tpm2_buffer[2] = (_thisPacketBytes >> 8) & 0xff; // Frame size high
-			tpm2_buffer[3] = _thisPacketBytes & 0xff; // Frame size low
-			tpm2_buffer[4] = _tpm2ThisPacket++; // Packet Number
-			tpm2_buffer[5] = _tpm2TotalPackets; // Number of packets
+			_tpm2_buffer[0] = 0x9c;	// Packet start byte
+			_tpm2_buffer[1] = 0xda; // Packet type Data frame
+			_tpm2_buffer[2] = (_thisPacketBytes >> 8) & 0xff; // Frame size high
+			_tpm2_buffer[3] = _thisPacketBytes & 0xff; // Frame size low
+			_tpm2_buffer[4] = _tpm2ThisPacket++; // Packet Number
+			_tpm2_buffer[5] = _tpm2TotalPackets; // Number of packets
 		}
 
-		tpm2_buffer [6 + rawIdx%_tpm2_max] = rawdata[rawIdx];
+		_tpm2_buffer [6 + rawIdx%_tpm2_max] = rawdata[rawIdx];
 
 //     is this the      last byte of last packet    ||   last byte of other packets
 		if ( (rawIdx == _tpm2ByteCount-1) || (rawIdx %_tpm2_max == _tpm2_max-1) )
 		{
-			tpm2_buffer [6 + rawIdx%_tpm2_max +1] = 0x36;		// Packet end byte
-			retVal &= writeBytes(_thisPacketBytes+7, tpm2_buffer);
+			_tpm2_buffer [6 + rawIdx%_tpm2_max +1] = 0x36;		// Packet end byte
+			retVal &= writeBytes(_thisPacketBytes+7, _tpm2_buffer);
 		}
 	}
-
 	return retVal;
 }

libsrc/leddevice/dev_net/LedDeviceTpm2net.h

@@ -18,6 +18,11 @@ public:
 	///
 	explicit LedDeviceTpm2net(const QJsonObject &deviceConfig);
 
+	///
+	/// @brief Destructor of the TPM2 LED-device
+	///
+	~LedDeviceTpm2net() override;
+
 	///
 	/// @brief Constructs the LED-device
 	///
@@ -48,6 +53,8 @@ private:
 	int _tpm2ByteCount;
 	int _tpm2TotalPackets;
 	int _tpm2ThisPacket;
+
+	uint8_t * _tpm2_buffer;
 };
 
 #endif // LEDEVICETPM2NET_H

libsrc/leddevice/dev_rpi_pwm/LedDeviceWS281x.cpp

@@ -1,4 +1,10 @@
 #include "LedDeviceWS281x.h"
+#include <utils/SysInfo.h>
+
+// Constants
+namespace {
+const bool verbose = false;
+} //End of constants
 
 LedDeviceWS281x::LedDeviceWS281x(const QJsonObject &deviceConfig)
 	: LedDevice(deviceConfig)
@@ -75,19 +81,26 @@ int LedDeviceWS281x::open()
 	int retval = -1;
 	_isDeviceReady = false;
 
-	// Try to open the LedDevice
-
-	ws2811_return_t rc = ws2811_init(&_led_string);
-	if ( rc != WS2811_SUCCESS )
+	if (!SysInfo::isUserAdmin())
 	{
-		QString errortext = QString ("Failed to open. Error message: %1").arg( ws2811_get_return_t_str(rc) );
+		QString errortext = QString ("Hyperion must run with \"root\" privileges for this device. Current user is: \"%1\"").arg(SysInfo::userName());
 		this->setInError( errortext );
 	}
 	else
 	{
-		// Everything is OK, device is ready
-		_isDeviceReady = true;
-		retval = 0;
+		// Try to open the LedDevice
+		ws2811_return_t rc = ws2811_init(&_led_string);
+		if ( rc != WS2811_SUCCESS )
+		{
+			QString errortext = QString ("Failed to open. Error message: %1").arg( ws2811_get_return_t_str(rc) );
+			this->setInError( errortext );
+		}
+		else
+		{
+			// Everything is OK, device is ready
+			_isDeviceReady = true;
+			retval = 0;
+		}
 	}
 	return retval;
 }
@@ -138,3 +151,22 @@ int LedDeviceWS281x::write(const std::vector<ColorRgb> &ledValues)
 
 	return ws2811_render(&_led_string) ? -1 : 0;
 }
+
+QJsonObject LedDeviceWS281x::discover(const QJsonObject& /*params*/)
+{
+	QJsonObject devicesDiscovered;
+	devicesDiscovered.insert("ledDeviceType", _activeDeviceType);
+
+	QJsonArray deviceList;
+
+	if (SysInfo::isUserAdmin())
+	{
+		//Indicate the general availability of the device, if hyperion is run under root
+		deviceList << QJsonObject ({{"found",true}});
+		devicesDiscovered.insert("devices", deviceList);
+	}
+
+	DebugIf(verbose,_log, "devicesDiscovered: [%s]", QString(QJsonDocument(devicesDiscovered).toJson(QJsonDocument::Compact)).toUtf8().constData());
+
+	return devicesDiscovered;
+}

libsrc/leddevice/dev_rpi_pwm/LedDeviceWS281x.h

@@ -29,6 +29,15 @@ public:
 	///
 	static LedDevice* construct(const QJsonObject &deviceConfig);
 
+	///
+	/// @brief Discover WS281x devices available (for configuration).
+	///
+	/// @param[in] params Parameters used to overwrite discovery default behaviour
+	///
+	/// @return A JSON structure holding a list of devices found
+	///
+	QJsonObject discover(const QJsonObject& params) override;
+
 protected:
 
 	///

libsrc/leddevice/dev_spi/ProviderSpi.cpp

@@ -144,12 +144,14 @@ int ProviderSpi::writeBytes(unsigned size, const uint8_t * data)
 		return -1;
 	}
 
+	uint8_t * newdata {nullptr};
+
 	_spi.tx_buf = __u64(data);
 	_spi.len    = __u32(size);
 
 	if (_spiDataInvert)
 	{
-		uint8_t * newdata = (uint8_t *)malloc(size);
+		newdata = static_cast<uint8_t *>(malloc(size));
 		for (unsigned i = 0; i<size; i++) {
 			newdata[i] = data[i] ^ 0xff;
 		}
@@ -159,6 +161,8 @@ int ProviderSpi::writeBytes(unsigned size, const uint8_t * data)
 	int retVal = ioctl(_fid, SPI_IOC_MESSAGE(1), &_spi);
 	ErrorIf((retVal < 0), _log, "SPI failed to write. errno: %d, %s", errno,  strerror(errno) );
 
+	free (newdata);
+
 	return retVal;
 }
 

libsrc/leddevice/schemas/schema-ws281x.json

@@ -17,7 +17,7 @@
 		"dma": {
 			"type": "integer",
 			"title":"edt_dev_spec_dmaNumber_title",
-			"default": 5,
+			"default": 10,
 			"propertyOrder" : 3
 		},
 		"pwmchannel": {