Bosch Sensortec Community

    cancel
    Showing results for 
    Search instead for 
    Did you mean: 

    BME280 linux_userspace.c delay bug

    BME280 linux_userspace.c delay bug

    Tom_in_MN
    New Poster

    In 

    BME280_driver/examples/linux_userspace.h  (on Github)

    there  is

    /*Calculate the minimum delay required between consecutive measurement based upon the sensor enabled
    * and the oversampling configuration. */
    req_delay = bme280_cal_meas_delay(&dev->settings);

     

    which is used as

    /* Wait for the measurement to complete and print data */
    dev->delay_us(req_delay, dev->intf_ptr);

     

     

    but in bme280.c the definition of bme280_cal_meas_delay() has the comment (my bold):

    /*!
    * @brief This API is used to calculate the maximum delay in milliseconds required for the
    * temperature/pressure/humidity(which ever at enabled) measurement to complete.
    */
    uint32_t bme280_cal_meas_delay(const struct bme280_settings *settings)

     

    The comment says this returns milliseconds and you are using the result directly in delay_us() which expects microseconds.

    So it seems the call to delay_us() should be

    dev->delay_us(1000*req_delay, dev->intf_ptr);

     

     

    If the comment above is correct.

    Thanks,

    Tom

     

     

     

    1 REPLY 1

    BSTRobin
    Community Moderator
    Community Moderator

    Hi Tom_in_MN,

    Thank for your kindly feedback. We will check it。

    Icon--AD-black-48x48Icon--address-consumer-data-black-48x48Icon--appointment-black-48x48Icon--back-left-black-48x48Icon--calendar-black-48x48Icon--center-alignedIcon--Checkbox-checkIcon--clock-black-48x48Icon--close-black-48x48Icon--compare-black-48x48Icon--confirmation-black-48x48Icon--dealer-details-black-48x48Icon--delete-black-48x48Icon--delivery-black-48x48Icon--down-black-48x48Icon--download-black-48x48Ic-OverlayAlertIcon--externallink-black-48x48Icon-Filledforward-right_adjustedIcon--grid-view-black-48x48IC_gd_Check-Circle170821_Icons_Community170823_Bosch_Icons170823_Bosch_Icons170821_Icons_CommunityIC-logout170821_Icons_Community170825_Bosch_Icons170821_Icons_CommunityIC-shopping-cart2170821_Icons_CommunityIC-upIC_UserIcon--imageIcon--info-i-black-48x48Icon--left-alignedIcon--Less-minimize-black-48x48Icon-FilledIcon--List-Check-grennIcon--List-Check-blackIcon--List-Cross-blackIcon--list-view-mobile-black-48x48Icon--list-view-black-48x48Icon--More-Maximize-black-48x48Icon--my-product-black-48x48Icon--newsletter-black-48x48Icon--payment-black-48x48Icon--print-black-48x48Icon--promotion-black-48x48Icon--registration-black-48x48Icon--Reset-black-48x48Icon--right-alignedshare-circle1Icon--share-black-48x48Icon--shopping-bag-black-48x48Icon-shopping-cartIcon--start-play-black-48x48Icon--store-locator-black-48x48Ic-OverlayAlertIcon--summary-black-48x48tumblrIcon-FilledvineIc-OverlayAlertwhishlist