RE: Enable USB mass storage (SD card) only when USB connected

Hung Bui — Fri, 20 Sep 2024 10:43:42 GMT

Thanks for providing the solution. I would assume it's not related to the USB communication but more on how the SPI is handled before SystemOFF.

RE: Enable USB mass storage (SD card) only when USB connected

ax_shepherd — Thu, 19 Sep 2024 18:38:13 GMT

This post lead me to the solution.

pm_device_action_run(dev_spi.bus, PM_DEVICE_ACTION_SUSPEND);

RE: Enable USB mass storage (SD card) only when USB connected

ax_shepherd — Thu, 19 Sep 2024 13:49:15 GMT

The "Division by zero" error was due to something else that I missed, I now fixed that issue. I now have a "Card error on CMD0" error on the first wake up after initial boot where the data saving on SD works.

As a bad workaround I added sys_reboot(SYS_REBOOT_WARM); when the SD initialisation fails and this somewhat gives me the behaviour I expect but not really fixing the bug. It seems that just umounting the SD is not enough it also needs to me uninitialized somehow when USB MSC is enabled. With "CONFIG_USB_MASS_STORAGE=n" I can write to the SD after waking up from sys_poweroff without problem.

static void sd_card_write_work_handler(struct k_work *work) {
    LOG_INF("sd_card_write_work_handler called");
    // Lock the mutex to protect buffer access during SD card writing
    if (usb_connected == 0){
        k_mutex_lock(&buffer_mutex, K_FOREVER);
        if (sd_card_initialized == 0) {
            LOG_INF("Unmount disk...");
            turn_off_sd(0);
            int ret = sd_card_init();
            if (ret != 0) {
                LOG_ERR("Failed to initialize SD card");
                turn_off_sd(0);
                k_mutex_unlock(&buffer_mutex); // Unlock mutex before returning
                sys_reboot(SYS_REBOOT_WARM);
                //sys_poweroff();
                //return;
            }
            save_sd_init_config(1);
        }else{
            LOG_INF("SD card already initialised.");
        }

        struct tm datetime = get_datetime();
        char hwid[HW_ID_LEN] = "unknown";
        get_hwid(hwid);
        const char *ext = ".csv";
        char filename[32];
        snprintf(filename, sizeof(filename), "%s_%04d_%02d_%02d%s",
                hwid,
                datetime.tm_year + 1900,
                (datetime.tm_mon + 1),
                datetime.tm_mday,
                ext);
        LOG_INF("filename:%s", filename);

        for (int i = 0; i < record_count; i++) {
            size_t size = strlen(records[i]);
            //turn_buzzer_off();
            int ret = sd_card_open_write_close(filename, records[i], &size);
            if (ret) {
                LOG_ERR("Failed to write to file. Ret: %d", ret);
            } 
        }
        if (!motion_detected) {
            LOG_INF("Unmount disk...");
            unmount_sd_card();
            LOG_INF("Unmount done.");
            turn_off_sd(0);
            
            record_count = 0;
            if(!ble_initialized){
                // if (!usb_connected){
                LOG_INF("CONFIG acc_sensitivity:%f acc_sampling:%d usb_connected:%d usb_call_back_set:%d sd_card_initialized:%d", 
	            acc_sensitivity, acc_sampling, usb_connected, usb_call_back_set, sd_card_initialized);
                LOG_INF("Going to sleep...");
                save_usb_callback_config(0);
                k_msleep(100);
                sys_poweroff();
                // }else{
                //     LOG_INF("USB connected stay awake...");
                //     sd_card_init();
                //     sd_card_initialized = true;
                // }
            }else{
                LOG_INF("BLE enabled...");
            }
        } else {
            LOG_INF("Motion detected, staying awake");
        }
        // Unlock the mutex after finishing buffer access
        k_mutex_unlock(&buffer_mutex);
    }else{
        LOG_INF("CONFIG acc_sensitivity:%f acc_sampling:%d usb_connected:%d usb_call_back_set:%d sd_card_initialized:%d", 
	    acc_sensitivity, acc_sampling, usb_connected, usb_call_back_set, sd_card_initialized);
        LOG_INF("Writing new file disabled when usb connected.");
    }

}

SEGGER J-Link V7.94e - Real time terminal output
SEGGER J-Link (unknown) V1.0, SN=801046272
Process: JLink.exe
[00:00:00.003,906] <inf> lis2dh: LIS2DH: int2 on gpio@842500.28
[00:00:00.006,134] <inf> lis2dh: fs=2, odr=0x3 lp_en=0x8 scale=9576
[00:00:00.007,598] <inf> pcf85063a: pcf85063a@51 is initialized!
*** Booting nRF Connect SDK v3.5.99-ncs1-1 ***
[00:00:00.160,797] <inf> main: Hi! DFU Starting nrf5340dk_nrf5340_cpuapp
[00:00:11.176,910] <inf> nvs_storage: set SD_INIT config 0 at id 7
[00:00:11.177,764] <inf> nvs_storage: set acc sensitivity config 1.200 at id 4
[00:00:11.178,527] <inf> nvs_storage: config: 4, acc_sensitivity: 1.200
[00:00:11.186,676] <inf> nvs_storage: set acc sampling config 1 at id 3
[00:00:11.187,377] <inf> nvs_storage: config: 3, acc_sampling: 1
[00:00:11.187,835] <inf> nvs_storage: The integer value is: 1
[00:00:11.188,354] <inf> nvs_storage: config: 7, SD_INIT: 0
[00:00:11.188,781] <inf> nvs_storage: The integer value is: 0
[00:00:12.189,270] <inf> main: RTC init...
[00:00:13.190,277] <inf> main: Enable USB...
[00:00:13.198,211] <inf> nvs_storage: set USB_CALLBACK config 1 at id 6
[00:00:14.198,791] <inf> main: CONFIG acc_sensitivity:1.200000 acc_sampling:1 usb_connected:0 usb_call_back_set:1 sd_card_initialized:0
[00:00:14.199,951] <inf> accelerometer: Setting triggers for motion detection...
[00:00:14.203,979] <inf> lis2dh: int2_ths=0x4c range_g=2 ums2=11767979
[00:00:14.206,329] <inf> lis2dh: int2_dur=0x0
[00:00:15.207,733] <dbg> main: main: Done.
[00:00:31.012,451] <inf> accelerometer: Motion detected, starting data collection
[00:00:31.012,969] <inf> accelerometer: Motion detected, starting data collection, usb_connected:0 sd_card_initialized:0
[00:00:36.014,221] <inf> accelerometer: Motion timer expired, stopping data collection
[00:00:36.014,739] <inf> accelerometer: Submit to queue...
[00:00:36.015,167] <inf> accelerometer: sd_card_write_work_handler called
[00:00:36.015,441] <inf> accelerometer: Unmount disk...
[00:00:36.204,864] <inf> nvs_storage: set SD_INIT config 1 at id 7
[00:00:36.206,634] <inf> accelerometer: filename:2BFE6027164802EE_2020_04_12.csv
[00:00:36.221,466] <inf> accelerometer: Unmount disk...
[00:00:36.221,710] <inf> accelerometer: Disk unmounted.
[00:00:36.227,752] <inf> nvs_storage: set SD_INIT config 0 at id 7
[00:00:36.228,210] <inf> accelerometer: Unmount done.
[00:00:36.228,576] <inf> accelerometer: CONFIG acc_sensitivity:1.200000 acc_sampling:1 usb_connected:0 usb_call_back_set:1 sd_card_initialized:0
[00:00:36.229,156] <inf> accelerometer: Going to sleep...
[00:00:36.235,229] <inf> nvs_storage: set USB_CALLBACK config 0 at id 6
[00:00:00.003,875] <inf> lis2dh: LIS2DH: int2 on gpio@842500.28
[00:00:00.006,042] <inf> lis2dh: fs=2, odr=0x3 lp_en=0x8 scale=9576
[00:00:00.007,476] <inf> pcf85063a: pcf85063a@51 is initialized!
*** Booting nRF Connect SDK v3.5.99-ncs1-1 ***
[00:00:00.014,892] <err> usb_msc: Storage init ERROR !!!! - Aborting USB init
[00:00:00.015,747] <inf> main: Hi! DFU Starting nrf5340dk_nrf5340_cpuapp
[00:00:11.031,616] <inf> nvs_storage: set SD_INIT config 0 at id 7
[00:00:11.032,165] <inf> nvs_storage: config: 4, acc_sensitivity: 1.200
[00:00:11.032,714] <inf> nvs_storage: config: 3, acc_sampling: 0
[00:00:11.033,172] <inf> nvs_storage: The integer value is: 0
[00:00:11.033,721] <inf> nvs_storage: config: 7, SD_INIT: 0
[00:00:11.034,149] <inf> nvs_storage: The integer value is: 0
[00:00:12.034,606] <inf> main: RTC init...
[00:00:13.035,614] <inf> main: Enable USB...
[00:00:13.043,518] <inf> nvs_storage: set USB_CALLBACK config 1 at id 6
[00:00:14.044,250] <inf> main: CONFIG acc_sensitivity:1.200000 acc_sampling:0 usb_connected:0 usb_call_back_set:1 sd_card_initialized:0
[00:00:14.045,410] <inf> accelerometer: Setting triggers for motion detection...
[00:00:14.049,438] <inf> lis2dh: int2_ths=0x4c range_g=2 ums2=11767979
[00:00:14.051,788] <inf> lis2dh: int2_dur=0x0
[00:00:14.053,710] <inf> accelerometer: Motion detected, starting data collection
[00:00:14.054,229] <inf> accelerometer: Motion detected, starting data collection, usb_connected:0 sd_card_initialized:0
[00:00:15.053,283] <dbg> main: main: Done.
[00:00:19.055,480] <inf> accelerometer: Motion timer expired, stopping data collection
[00:00:19.055,969] <inf> accelerometer: Submit to queue...
[00:00:19.056,396] <inf> accelerometer: sd_card_write_work_handler called
[00:00:19.056,701] <inf> accelerometer: Unmount disk...
[00:00:19.091,125] <err> sd: Card error on CMD0
[00:00:19.091,369] <err> sd_card: SD card init failed, please check if SD card inserted
[00:00:19.091,705] <err> accelerometer: Failed to initialize SD card
[00:00:00.003,875] <inf> lis2dh: LIS2DH: int2 on gpio@842500.28
[00:00:00.006,042] <inf> lis2dh: fs=2, odr=0x3 lp_en=0x8 scale=9576
[00:00:00.007,476] <inf> pcf85063a: pcf85063a@51 is initialized!

RE: Enable USB mass storage (SD card) only when USB connected

ax_shepherd — Thu, 19 Sep 2024 12:20:12 GMT

Hi Hung,
Thank you for the quick reply. Unfortunately, I do not have a DK. For sleep mode, I am using System OFF with sys_poweroff();, and I wake up from a GPIO interrupt triggered by the accelerometer.
I will check for the "Division by zero" error and update you as soon as I find the cause. However, the "usb_msc: Storage init ERROR" still occurs on wake-up.

RE: Enable USB mass storage (SD card) only when USB connected

Hung Bui — Thu, 19 Sep 2024 11:31:42 GMT

Hi Axel,

Can you reproduce the issue with a DK (use ramdisk)?

When you say going to sleep, is it System OFF sleep or System On Idle ?

The error is " Division by zero", have you checked if it's related to USB or something else ?