Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

kbk — Fri, 26 Feb 2021 15:52:03 GMT

Thanks I'm using segger studio as well. Wish segger had a share project to archive feature...

Kevin

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

Leo Yang — Fri, 26 Feb 2021 15:49:47 GMT

Hi, kbk

I use SEGGER studio to program, right-click the variable => Go To Definition.

It will be much more easier to trace code. : >

Here's the functions I use:

void read_all_cb(ret_code_t result, void *p_user_data) {
    if (result != NRF_SUCCESS) {
        NRF_LOG_WARNING("read_all_cb - error: %d", (int)result);
        return;
    }
    if (DEBUG)
        NRF_LOG_RAW_INFO("Reading......\n");

    float x_gscale_val, y_gscale_val, z_gscale_val;

    x_gscale_val = (float)((int16_t)((((uint16_t)m_buffer[1]) << 8) | m_buffer[0])) / (16 * 1024) + 0.43;
    y_gscale_val = (float)((int16_t)((((uint16_t)m_buffer[3]) << 8) | m_buffer[2])) / (16 * 1024) + 0.4;
    z_gscale_val = (float)((int16_t)((((uint16_t)m_buffer[5]) << 8) | m_buffer[4])) / (16 * 1024) - 0.2;

    if (timer_counter < RECORDING_MAX_TIME_STEP) {
        NRF_LOG_RAW_INFO("%d: ", timer_counter);
        NRF_LOG_RAW_INFO(NRF_LOG_FLOAT_MARKER ", ", NRF_LOG_FLOAT(x_gscale_val));
        NRF_LOG_RAW_INFO(NRF_LOG_FLOAT_MARKER ", ", NRF_LOG_FLOAT(y_gscale_val));
        NRF_LOG_RAW_INFO(NRF_LOG_FLOAT_MARKER "\n", NRF_LOG_FLOAT(z_gscale_val));
    }
    if(timer_counter != RECORDING_MAX_TIME_STEP){
        z_raw_val[timer_counter] = z_gscale_val;
        timer_counter += 1;
    }
}
static void read_all(void) {
    // [these structures have to be "static" - they cannot be placed on stack
    //  since the transaction is scheduled and these structures most likely
    //  will be referred after this function returns]
    static nrf_twi_mngr_transfer_t const transfers[] =
#ifdef LIS2DH12
        {LIS2DH12_READ_XYZ(&m_buffer[0])};
#else
        {ADXL345_READ_XYZ(&m_buffer[0])};
#endif
    static nrf_twi_mngr_transaction_t NRF_TWI_MNGR_BUFFER_LOC_IND transaction =
        {
            .callback = read_all_cb,
            .p_user_data = NULL,
            .p_transfers = transfers,
            .number_of_transfers = sizeof(transfers) / sizeof(transfers[0])};

    APP_ERROR_CHECK(nrf_twi_mngr_schedule(&m_nrf_twi_mngr, &transaction));

    // Signal on LED that something is going on.
    // bsp_board_led_invert(READ_ALL_INDICATOR);

    if (DEBUG)
        NRF_LOG_RAW_INFO("Read end\n\n");
}

static void bsp_config(void) {
    uint32_t err_code;

    err_code = app_timer_init();
    APP_ERROR_CHECK(err_code);

    APP_ERROR_CHECK(err_code);
}

// TWI (with transaction manager) initialization.
static void twi_config(void) {
    uint32_t err_code;

    nrf_drv_twi_config_t const config = {
        .scl = SCL_PIN,
        .sda = SDA_PIN,
        .frequency = NRF_DRV_TWI_FREQ_100K,
        .interrupt_priority = APP_IRQ_PRIORITY_LOWEST,
        .clear_bus_init = false};

    err_code = nrf_twi_mngr_init(&m_nrf_twi_mngr, &config);
    APP_ERROR_CHECK(err_code);
}

static void lfclk_config(void) {
    uint32_t err_code;

    err_code = nrf_drv_clock_init();
    APP_ERROR_CHECK(err_code);

    nrf_drv_clock_lfclk_request(NULL);
}

void timer_handler(void *p_context) {
    read_all();
}

void read_init(void) {
    ret_code_t err_code;

    err_code = app_timer_create(&m_timer, APP_TIMER_MODE_REPEATED, timer_handler);
    APP_ERROR_CHECK(err_code);

    err_code = app_timer_start(m_timer, APP_TIMER_TICKS(20), NULL);
    APP_ERROR_CHECK(err_code);
}

void log_init(void) {
    ret_code_t err_code;

    err_code = NRF_LOG_INIT(NULL);
    APP_ERROR_CHECK(err_code);

    NRF_LOG_DEFAULT_BACKENDS_INIT();
}

void power_init(void) {
    ret_code_t err_code;

    err_code = nrf_pwr_mgmt_init();
    APP_ERROR_CHECK(err_code);
}

void gpio_config(void) {
    nrf_gpio_cfg_output(4);
    nrf_gpio_pin_set(8);
    nrf_delay_ms(30);

    nrf_gpio_cfg_output(4);
    nrf_gpio_pin_set(4);
    nrf_gpio_cfg_output(LED_1);
}

void setup(void) {
    log_init();
    // bsp = Board Support Package
    bsp_board_init(BSP_INIT_LEDS);

    // Start internal LFCLK XTAL oscillator - it is needed by BSP to handle
    // buttons with the use of APP_TIMER and for "read_all" ticks generation
    // (by RTC).
    lfclk_config();
    bsp_config();
    power_init();
    gpio_config();
    // Wait for LIS2D12 Turn-on time
    nrf_delay_ms(300);
}

Not all the functions in lis2dh12.h are used, I use LIS2DH12_READ_XYZ only.

Regards

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

kbk — Fri, 26 Feb 2021 14:59:49 GMT

Hello Leo

Thank you very much for taking the time to post this for me. I understand adding the two lis2dh12.files and I assume what you have here in the main.c code above is what I add to the main.c for the "twi_master_using_nrf_twi_mngr" project. Is that correct so far? What I don't understand is what is it that will call anything in those two added files... (I'm a terrible programmer) if you can help me further it would be very much appreciated.

Regards,

Kevin

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

Leo Yang — Fri, 26 Feb 2021 02:32:12 GMT

Hi, kbk. I basically use "twi_master_using_nrf_twi_mngr" project, and only edit some registers and values in "mma7660.h" and "mma7660.c" to init lis2dh12. I called these file "lis2dh12.h" and "lis2dh12.c".

Here's the code:

lis2dh12.h

/**
 * Copyright (c) 2015 - 2020, Nordic Semiconductor ASA
 *
 * All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without modification,
 * are permitted provided that the following conditions are met:
 *
 * 1. Redistributions of source code must retain the above copyright notice, this
 *    list of conditions and the following disclaimer.
 *
 * 2. Redistributions in binary form, except as embedded into a Nordic
 *    Semiconductor ASA integrated circuit in a product or a software update for
 *    such product, must reproduce the above copyright notice, this list of
 *    conditions and the following disclaimer in the documentation and/or other
 *    materials provided with the distribution.
 *
 * 3. Neither the name of Nordic Semiconductor ASA nor the names of its
 *    contributors may be used to endorse or promote products derived from this
 *    software without specific prior written permission.
 *
 * 4. This software, with or without modification, must only be used with a
 *    Nordic Semiconductor ASA integrated circuit.
 *
 * 5. Any software provided in binary form under this license must not be reverse
 *    engineered, decompiled, modified and/or disassembled.
 *
 * THIS SOFTWARE IS PROVIDED BY NORDIC SEMICONDUCTOR ASA "AS IS" AND ANY EXPRESS
 * OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
 * OF MERCHANTABILITY, NONINFRINGEMENT, AND FITNESS FOR A PARTICULAR PURPOSE ARE
 * DISCLAIMED. IN NO EVENT SHALL NORDIC SEMICONDUCTOR ASA OR CONTRIBUTORS BE
 * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
 * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
 * GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
 * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 *
 */
#ifndef LIS2DH12_H_
#define LIS2DH12_H_

#include "nrf_twi_mngr.h"

#ifdef __cplusplus
extern "C" {
#endif

#define SCL_PIN 5
#define SDA_PIN 2

/** I2C Device Address 8 bit format if SA0=0 -> 31 if SA0=1 -> 33 **/
#define LIS2DH12_ADDR 0x19U
#define LIS2DH12_I2C_ADD_L 0x31U
#define LIS2DH12_I2C_ADD_H 0x33U

#define LIS2DH12_OUT_X_L 0x28U
#define LIS2DH12_OUT_X_H 0x29U
#define LIS2DH12_OUT_Y_L 0x2AU
#define LIS2DH12_OUT_Y_H 0x2BU
#define LIS2DH12_OUT_Z_L 0x2CU
#define LIS2DH12_OUT_Z_H 0x2DU
#define LIS2DH12_FIFO_CTRL_REG 0x2EU

#define LIS2DH12_NUMBER_OF_REGISTERS 11

// The Alert bit (6) set signals that the register must be read again, since
// its value may be inaccurate (it was read at the same time as the device was
// attempting to update it).
// Applies to XOUT, YOUT, ZOUT registers.
#define LIS2DH12_DATA_IS_VALID(reg_data) (!((reg_data) & (1U << 6)))

// Gets acceleration (g) value (6-bit 2's complement) from register data.
// [use "/ 4" instead of ">> 2", as the result of right-shifting of a signed
//  type value is implementation-defined]
#define LIS2DH12_GET_ACC(reg_data) ((int8_t)((reg_data) << 2) / 4)

// Get orientation of the device (PoLa bits from the Tilt Status register).
#define LIS2DH12_GET_ORIENTATION(tilt_status) (tilt_status & 0x1C)
#define LIS2DH12_ORIENTATION_LEFT 0x04
#define LIS2DH12_ORIENTATION_RIGHT 0x08
#define LIS2DH12_ORIENTATION_DOWN 0x14
#define LIS2DH12_ORIENTATION_UP 0x18

#define LIS2DH12_TAP_DETECTED(tilt_status) (tilt_status & (1U << 5))
#define LIS2DH12_SHAKE_DETECTED(tilt_status) (tilt_status & (1U << 7))

extern uint8_t NRF_TWI_MNGR_BUFFER_LOC_IND lis2dh12_xout_reg_addr;

#define LIS2DH12_READ(p_reg_addr, p_buffer, byte_cnt)                       \
    NRF_TWI_MNGR_WRITE(LIS2DH12_ADDR, p_reg_addr, 1, NRF_TWI_MNGR_NO_STOP), \
        NRF_TWI_MNGR_READ(LIS2DH12_ADDR, p_buffer, byte_cnt, 0)

#define LIS2DH12_READ_XYZ(p_buffer) \
    LIS2DH12_READ(&lis2dh12_xout_reg_addr, p_buffer, 6)

#define LIS2DH12_INIT_TRANSFER_COUNT 4

extern nrf_twi_mngr_transfer_t const
    lis2dh12_init_transfers[LIS2DH12_INIT_TRANSFER_COUNT];

#ifdef __cplusplus
}
#endif

#endif // LIS2DH12_H_

lis2dh12.c

/**
 * Copyright (c) 2015 - 2020, Nordic Semiconductor ASA
 *
 * All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without modification,
 * are permitted provided that the following conditions are met:
 *
 * 1. Redistributions of source code must retain the above copyright notice, this
 *    list of conditions and the following disclaimer.
 *
 * 2. Redistributions in binary form, except as embedded into a Nordic
 *    Semiconductor ASA integrated circuit in a product or a software update for
 *    such product, must reproduce the above copyright notice, this list of
 *    conditions and the following disclaimer in the documentation and/or other
 *    materials provided with the distribution.
 *
 * 3. Neither the name of Nordic Semiconductor ASA nor the names of its
 *    contributors may be used to endorse or promote products derived from this
 *    software without specific prior written permission.
 *
 * 4. This software, with or without modification, must only be used with a
 *    Nordic Semiconductor ASA integrated circuit.
 *
 * 5. Any software provided in binary form under this license must not be reverse
 *    engineered, decompiled, modified and/or disassembled.
 *
 * THIS SOFTWARE IS PROVIDED BY NORDIC SEMICONDUCTOR ASA "AS IS" AND ANY EXPRESS
 * OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
 * OF MERCHANTABILITY, NONINFRINGEMENT, AND FITNESS FOR A PARTICULAR PURPOSE ARE
 * DISCLAIMED. IN NO EVENT SHALL NORDIC SEMICONDUCTOR ASA OR CONTRIBUTORS BE
 * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
 * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
 * GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
 * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 *
 */
/** @file
 * @defgroup nrf_twi_master_example main.c
 * @{
 * @ingroup nrf_twi_example
 * @brief TWI Example Application main file.
 *
 * This file contains the source code for a sample application using TWI.
 *
 * @image html example_board_setup_a.jpg "Use board setup A for this example."
 */


#include "lis2dh12.h"

uint8_t NRF_TWI_MNGR_BUFFER_LOC_IND lis2dh12_xout_reg_addr = (LIS2DH12_OUT_X_L | 0x80);

static uint8_t NRF_TWI_MNGR_BUFFER_LOC_IND CTRL_REG1[] = { 0x20, 0x57 };
// set data rate to 100 Hz

static uint8_t NRF_TWI_MNGR_BUFFER_LOC_IND CTRL_REG4[] = { 0x23, 0x04 };
// set full-scale to +/- 2g

static uint8_t NRF_TWI_MNGR_BUFFER_LOC_IND CTRL_REG5[] = { 0x24, 0x40 };
// set FIFO enable bit

static uint8_t NRF_TWI_MNGR_BUFFER_LOC_IND FIFO_CTRL[] = { 0x2E, 0x80 };
// set FIFO to stream mode

nrf_twi_mngr_transfer_t const lis2dh12_init_transfers[LIS2DH12_INIT_TRANSFER_COUNT] =
{
    NRF_TWI_MNGR_WRITE(LIS2DH12_ADDR, CTRL_REG1, sizeof(CTRL_REG1), 0),
    NRF_TWI_MNGR_WRITE(LIS2DH12_ADDR, CTRL_REG4, sizeof(CTRL_REG4), 0),
    NRF_TWI_MNGR_WRITE(LIS2DH12_ADDR, CTRL_REG5, sizeof(CTRL_REG5), 0),
    NRF_TWI_MNGR_WRITE(LIS2DH12_ADDR, FIFO_CTRL, sizeof(FIFO_CTRL), 0)
};

and main.c

void setup(void) {
    log_init();
    // bsp = Board Support Package
    bsp_board_init(BSP_INIT_LEDS);

    // Start internal LFCLK XTAL oscillator - it is needed by BSP to handle
    // buttons with the use of APP_TIMER and for "read_all" ticks generation
    // (by RTC).
    lfclk_config();
    bsp_config();
    power_init();
    gpio_config();
    // Wait for LIS2D12 Turn-on time
    nrf_delay_ms(300);
}

int main(void) {
    setup();

    twi_config();
    read_init();
    
    // Initialize sensor (lis2dh12 only).
    APP_ERROR_CHECK(nrf_twi_mngr_perform(&m_nrf_twi_mngr, NULL, lis2dh12_init_transfers,
        LIS2DH12_INIT_TRANSFER_COUNT, NULL));

    ......
}

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

kbk — Thu, 25 Feb 2021 15:40:30 GMT

Hello Leo. Can you please share your code for the lis2hh12 and your nrf52 please?

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

Leo Yang — Tue, 29 Sep 2020 16:53:13 GMT

Finally, I figured it out. I forgot to set the FIFO enable bit in CTRL_REG5....

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

haakonsh — Mon, 28 Sep 2020 07:32:10 GMT

You need to scope the I/O lines and the sensor's power supply...

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

Leo Yang — Thu, 24 Sep 2020 15:28:23 GMT

I tried the code above, but still not working.

I'll keep trying. If I find something new, I 'll update the ticket.

Thanks for your help, I really appreciate it. : )

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

hmolesworth — Thu, 24 Sep 2020 14:58:01 GMT

High Drive '1' increases the current available from an i/o pin by turning on a second FET connecting the pin to Vdd internally in the nRF52. The LIS2DH12 doesn't normally require high drive, but as I mentioned 10uF is too large for normal use on an i/o pin as the LIS2DH12 is a low-power part. Try this:

 //nrf_gpio_cfg_output(8);
 nrf_gpio_pin_set(8);
 nrf_gpio_cfg(8, NRF_GPIO_PIN_DIR_OUTPUT, NRF_GPIO_PIN_INPUT_DISCONNECT, NRF_GPIO_PIN_NOPULL, NRF_GPIO_PIN_S0H1, NRF_GPIO_PIN_NOSENSE);
 nrf_delay_ms(30);

Low drive with 10uF may cause too slow a power rise time for the LIS2DH12 to correctly start up

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

Leo Yang — Thu, 24 Sep 2020 14:28:31 GMT

Hi hmolesworth,

I reordered the gpio operation as you mentioned, but I still get '0' from X_OUT_L.

    nrf_gpio_cfg_output(8);
    nrf_gpio_pin_set(8);
    nrf_delay_ms(300);

    nrf_gpio_cfg_output(4);
    nrf_gpio_pin_set(4);

I tried to print all the value received from LIS2DH12 (X_OUT_L, X_OUT_H, Y_OUT_L, Y_OUT_H, Z_OUT_L, Z_OUT_H).

However, I can only read either '0' or '255'.

Here's the result:

I also read the WHO_AM_I(0x0F) register, and I get a value(0x51) inconsistent with the value I shall get(0x33).

==> I would suggest reducing to 1uF or less but in any case set P0.08 to S0H1 drive.

Maybe to change the capacitor is not that easy, because I'm using a custom board which is just bigger than my thumb.

==> I would suggest reducing to 1uF or less but in any case set P0.08 to S0H1 drive.

nrf_gpio_cfg_output() function set the pin to S0S1, will this be a problem?

What is the difference between Standard '0', standard '1' and Standard '0', high drive '1'?

Sorry, I'm really a newbie...

Thank you,

regards

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

hmolesworth — Wed, 23 Sep 2020 18:57:31 GMT

Looks like you do have the correct device address, I thought the TWI driver bit-shifted but I see now that it does not.

The 10uF capacitor is very large for a standard nRF52 port pin to drive; I would suggest reducing to 1uF or less but in any case set P0.08 to S0H1 drive. P0.08 must be driven high before P0.04 is driven high, as otherwise P0.04 provides phantom power via internal LIS2DH12 schottky clamp diode before real power is applied and that will cause the LIS2DH12 to behave badly. Allow a delay of (say) 30mSecs after raising P0.08 before setting P0.04 as an active high output to allow the huge 10uF capacitor to charge. This might correct the issue.

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

haakonsh — Wed, 23 Sep 2020 13:35:34 GMT

You need to scope the I/O lines and the accelerometer's power supply.

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

Leo Yang — Wed, 23 Sep 2020 12:25:17 GMT

Thanks for your reply, haakonsh!

Maybe the bit shift is kind of confusing. What I'm doing is to discard the LSB of the 8-bit address and make it into a 7-bit address. The LSB in 0x33 is a R/W bit.

0x33(0b 0011 0011) right shift ==> 0x19(0b 0001 1001)

The i2c slave address table in datasheet,

Documentation: https://www.mouser.tw/datasheet/2/389/dm00091513-1797743.pdf

I read the input like this, and I know SA0 is connected to high, so the slave address will be 0x19(first 7 bit).

    nrf_gpio_cfg_input(3, NRF_GPIO_PIN_PULLUP);
    int pin_status = nrf_gpio_pin_read(3);
    printf("%d: %d\n", 3, pin_status);

I guess my situation is more likely to be a hardware problem. I run the same code on NRF52DK + ADXL345, with only register and slave address change, it works fine. I can read non-zero value from ADXL345.

I pulled p0.04 high to enable I2C, and also pulled p0.08 high for power supply. Am I doing something wrong?

Thank you, regards

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

haakonsh — Wed, 23 Sep 2020 07:54:44 GMT

The driver will take care of the R/W bit. You need to decide whether to use address 0b11000 or 0b11001.

By diving into the driver you'll eventually end up in:

__STATIC_INLINE void nrf_twim_address_set(NRF_TWIM_Type * p_reg,
                                          uint8_t address)
{
    p_reg->ADDRESS = address;
}

Where 'address' has not been modified in any way from what you pass to nrf_drv_twi_tx(). The ADDRESS register description declares that the ADDRESS register takes a 7-bit address without any bit-shifting.

The R/W bit is added to the address depending on whether the TASKS_STARTRX or TASKS_STARTTX task was triggered.

For the future I suggest you add test points to all serial I/O lines, as a digital scope will give you a lot more information than a peripheral driver who's waiting for a reply from a slave who's not listening.

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

Leo Yang — Wed, 23 Sep 2020 04:35:24 GMT

Hi hmolesworth, thanks for your fast response!

I left shifted the slave address, with debug mode in Segger Embedded Studio, but I got stuck in the while loop.(line 93)

    err_code = nrf_drv_twi_tx(&m_twi, ACC_ADDR, CTRL_REG1, sizeof(CTRL_REG1), false);
    APP_ERROR_CHECK(err_code);
    while (m_xfer_done == false); // <== stuck here

It seems that "m_xfer_done" was always false which means twi_handler() never returned to NRF_DRV_TWI_EVT_DONE state.

I read the official documentation and it said a 7-bit address is needed.

I thought the slave address(0x33U >> 1) is correct, because the LED_1 still blinks.

regards

RE: Custom NRF52832 + LIS2DH12 accelerometer but always get '0'

hmolesworth — Tue, 22 Sep 2020 17:13:16 GMT

As you know, "If the SA0 pad is connected to the voltage supply, LSb is ‘1’ (address 0011001b), else if the SA0 pad is connected to ground, the LSb value is ‘0’ (address 0011000b)."

This line doesn't do what is intended:

#define ACC_ADDR (0x33U >> 1)

That '1' LS bit has just been discarded; this is often confusing with i2c, is the 7-bit address left- or right-shifted or not and the answer depends on the library. Check the function to see whether it actually expects an un-shifted address.

I would try this:

#define ACC_ADDR (0x33U << 1)

The red LED works fine off a CR2032, but to get enough voltage for blue and green LEDs when the battery is low try adding another i/o pin and driving a capacitor with a diode clamp to provide voltage boost; the nRF52832 4-output PWM circuit is ideal for this - red/green/blue/boost, a common application made easy.