The embedded-i2c-sgp41's discuss from sensirion

sensirion / embedded-i2c-sgp41 Goto Github PK

View Code? Open in Web Editor NEW

The SGP41 is Sensirion’s new digital VOC (volatile organic compounds) and NOx sensor designed for easy integration into air purifiers or demand-controlled ventilation.

License: BSD 3-Clause "New" or "Revised" License

Makefile 1.59% C 89.59% C++ 8.83%

embedded-i2c-sgp41's Issues

Unused size parameter in sgp41_get_serial_number() function

In the sgp41_get_serial_number() function, the serial_number_size parameter is unused:

embedded-i2c-sgp41/sgp41_i2c.c

Lines 137 to 161 in f5e439f

 int16_t sgp41_get_serial_number(uint16_t* serial_number, 

 uint8_t serial_number_size) { 

 int16_t error; 

 uint8_t buffer[9]; 

 uint16_t offset = 0; 

 offset = sensirion_i2c_add_command_to_buffer(&buffer[0], offset, 0x3682); 

 error = sensirion_i2c_write_data(SGP41_I2C_ADDRESS, &buffer[0], offset); 

 if (error) { 

 return error; 

 } 

 sensirion_i2c_hal_sleep_usec(1000); 

 error = sensirion_i2c_read_data_inplace(SGP41_I2C_ADDRESS, &buffer[0], 6); 

 if (error) { 

 return error; 

 } 

 serial_number[0] = sensirion_common_bytes_to_uint16_t(&buffer[0]); 

 serial_number[1] = sensirion_common_bytes_to_uint16_t(&buffer[2]); 

 serial_number[2] = sensirion_common_bytes_to_uint16_t(&buffer[4]); 

 return NO_ERROR; 

 }

Also it's not mentioned as a parameter in the header:

embedded-i2c-sgp41/sgp41_i2c.h

Lines 117 to 126 in f5e439f

 /** 

  * sgp41_get_serial_number() - This command provides the decimal serial number 

  * of the SGP41 chip by returning 3x2 bytes. 

  * 

  * @param serial_number 48-bit unique serial number 

  * 

  * @return 0 on success, an error code otherwise 

  */ 

 int16_t sgp41_get_serial_number(uint16_t* serial_number, 

 uint8_t serial_number_size);

Not enough data received in sam21d boards

Hi, I'm been trying to get any reading from my sgp41 sensor with a seeeduino xiao and no success.
I'm using the example in this library. It gives the serial number but in the selfTest answers "not enough data received".
The board is based on the sam21d microcontroller and I've seen around that with esp boards the library has the same problem.
All the best,
Javi

Recommend Projects

sensirion / embedded-i2c-sgp41 Goto Github PK

embedded-i2c-sgp41's Issues

Unused size parameter in sgp41_get_serial_number() function

Not enough data received in sam21d boards

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	int16_t sgp41_get_serial_number(uint16_t* serial_number,
	uint8_t serial_number_size) {
	int16_t error;
	uint8_t buffer[9];
	uint16_t offset = 0;
	offset = sensirion_i2c_add_command_to_buffer(&buffer[0], offset, 0x3682);

	error = sensirion_i2c_write_data(SGP41_I2C_ADDRESS, &buffer[0], offset);
	if (error) {
	return error;
	}

	sensirion_i2c_hal_sleep_usec(1000);

	error = sensirion_i2c_read_data_inplace(SGP41_I2C_ADDRESS, &buffer[0], 6);
	if (error) {
	return error;
	}

	serial_number[0] = sensirion_common_bytes_to_uint16_t(&buffer[0]);
	serial_number[1] = sensirion_common_bytes_to_uint16_t(&buffer[2]);
	serial_number[2] = sensirion_common_bytes_to_uint16_t(&buffer[4]);

	return NO_ERROR;
	}

	/**
	* sgp41_get_serial_number() - This command provides the decimal serial number
	* of the SGP41 chip by returning 3x2 bytes.
	*
	* @param serial_number 48-bit unique serial number
	*
	* @return 0 on success, an error code otherwise
	*/
	int16_t sgp41_get_serial_number(uint16_t* serial_number,
	uint8_t serial_number_size);