applications: asset_tracker_v2: Explicitly disable PSM if not enabled

If the user sets CONFIG_MODEM_AUTO_REQUEST_POWER_SAVING_FEATURES=n,
it's reasonable to expect that PSM would not be requested by the
device.
However, the nRF9160 modem stores PSM request information to NVM,
so PSM must explicitly be disabled before the modem is put into
normal mode to be sure that PSM is not requested by the device.

This patch adds such logic to the modem module.

Signed-off-by: Jan Tore Guggedal <[email protected]>

Author: jtguggedal

applications/asset_tracker_v2/src/modules/modem_module.c

@@ -744,14 +744,19 @@ static int neighbor_cells_measurement_start(void)
 static int configure_low_power(void)
 {
 	int err;
+	bool enable = IS_ENABLED(CONFIG_MODEM_AUTO_REQUEST_POWER_SAVING_FEATURES);
 
-	err = lte_lc_psm_req(true);
+	err = lte_lc_psm_req(enable);
 	if (err) {
 		LOG_ERR("lte_lc_psm_req, error: %d", err);
 		return err;
 	}
 
-	LOG_DBG("PSM requested");
+	if (enable) {
+		LOG_DBG("PSM requested");
+	} else {
+		LOG_DBG("PSM disabled");
+	}
 
 	return 0;
 }
@@ -809,12 +814,10 @@ static int setup(void)
 		}
 	}
 
-	if (IS_ENABLED(CONFIG_MODEM_AUTO_REQUEST_POWER_SAVING_FEATURES)) {
-		err = configure_low_power();
-		if (err) {
-			LOG_ERR("configure_low_power, error: %d", err);
-			return err;
-		}
+	err = configure_low_power();
+	if (err) {
+		LOG_ERR("configure_low_power, error: %d", err);
+		return err;
 	}
 
 	err = modem_data_init();

doc/nrf/releases/release-notes-changelog.rst

@@ -64,6 +64,7 @@ nRF9160: Asset Tracker v2
 
 * Updated the code and documentation to use the acronym GNSS instead of GPS when not referring explicitly to the GPS system.
 * Added support for atmospheric pressure readings retrieved from the BME680 sensor on Thingy:91.
+* Fixed an issue where PSM could be requested from the network even though it was disabled in Kconfig.
 
 * Added: