04-11-2019 12:10 PM - last edited on 04-11-2019 06:59 PM by handytech
When testing the api, I noticed the following errors:
+++ bmp280_defs.h 2019-04-09 18:44:19.878657105 +0200 @@ -302,7 +302,7 @@ #define BMP280_ST_DIG_T2_MIN UINT16_C(22000) #define BMP280_ST_DIG_T2_MAX UINT16_C(30000) #define BMP280_ST_DIG_T3_MIN INT16_C(-3000) -#define BMP280_ST_DIG_T3_MAX INT16_C(-1000) +#define BMP280_ST_DIG_T3_MAX INT16_C(1000) #define BMP280_ST_DIG_P1_MIN UINT16_C(30000) #define BMP280_ST_DIG_P1_MAX UINT16_C(42000) #define BMP280_ST_DIG_P2_MIN INT16_C(-12970)
@@ -356,8 +356,8 @@ /*! @brief Macros holding the bound values for the pressure in hPa */ -#define BMP280_ST_PLAUSIBLE_PRESS_MIN UINT8_C(900) -#define BMP280_ST_PLAUSIBLE_PRESS_MAX UINT8_C(1100) +#define BMP280_ST_PLAUSIBLE_PRESS_MIN UINT16_C(900) +#define BMP280_ST_PLAUSIBLE_PRESS_MAX UINT16_C(1100) /*! @brief Macros holding integer resolution for pressure in hpa and temperature in C measurement */
Furthermore, in the example pressure.c it can be seen that it makes sense to create a single function from the two functions
'bmp280_get_comp_temp _ ??? ()' 'bmp280_get_comp_pres _ ??? ()'
otherwise the variable 't_fine' will not be set and the result will be wrong.
greetings
Solved! Go to Solution.
04-11-2019 07:09 PM
Thank-you for your report and suggestions, we have forwarded this information to the appropriate team.