sd_power_system_off() cause reboot

[gootoomoon]

I write this code to set my ble device power down,but it causes reboot without any error report.

                    
     GPIO_WAKEUP_BUTTON_CONFIG(LEFT_BUTTON_PIN_NO);
     GPIO_WAKEUP_BUTTON_CONFIG(RIGHT_BUTTON_PIN_NO);

     err_code = sd_power_system_off();

when I comment out this two lines

                   
       //GPIO_WAKEUP_BUTTON_CONFIG(LEFT_BUTTON_PIN_NO);
       //GPIO_WAKEUP_BUTTON_CONFIG(RIGHT_BUTTON_PIN_NO);

        err_code = sd_power_system_off();

power down worked,but could not wake up from this two button

I try to add

	err_code = app_button_enable();
        APP_ERROR_CHECK(err_code);

befor call sd_power_system_off() but fail too.

Is that right ? if I want wake it up by set the pin from high to low.

[Ole Morten]

If a wake-up condition is true when you enter system off, the chip will immediately wake back-up, which will appear as a reset, since the chip is always reset when waking from system off.

This means that before entering system off, you should make sure that no configured wakeup button is already low, and if it is, you may want to wait for it to be released before entering system off.

Also, it seems like you're using SDK 4.4.2, but if you have a second revision chip (nRF51822 revision FA or later), I'd strongly recommend you to upgrade to SDK 5.1.0 and S110 version 6.0.0.

[gootoomoon]

Yes I am using SDK 4.4.2 my chip now is not support the latest version of SDK

Should I PULL UP this pin before I entering system off?

I find that GPIO_WAKEUP_BUTTON_CONFIG do
{
GPIO_PIN_CONFIG(PIN_NO,
GPIO_PIN_CNF_DIR_Input,
GPIO_PIN_CNF_INPUT_Connect,
GPIO_PIN_CNF_PULL_Disabled,
GPIO_PIN_CNF_DRIVE_S0S1,
GPIO_PIN_CNF_SENSE_Low);
} while (0)

Do GPIO_PIN_CNF_PULL_Disabled will cause instable? Can I set is to PULL_UP?

[Ole Morten]

Yes, if you don't have external pull, you should definitely enable internal pull for your buttons.

You should make sure that you use the same chip in development as in production, and I suspect you may end up using second revision chips in production, and hence, you may want to upgrade your development boards as well.

[gootoomoon]

Thank you

[erriuga]

Was @Ole answer the correct answer here?