soc: nordic: s2ram: Optimize s2am marking functions

adamkondraciuk · dkalowsk · commit f7fd6b8e5492 · 2025-07-01T10:55:35.000-10:00
Optimize `pm_s2ram_mark_check_and_clear()` function.

Signed-off-by: Adam Kondraciuk &lt;adam.kondraciuk@nordicsemi.no&gt;
diff --git a/soc/nordic/nrf54h/pm_s2ram.c b/soc/nordic/nrf54h/pm_s2ram.c
@@ -187,18 +187,16 @@ void pm_s2ram_mark_set(void)
 
 bool pm_s2ram_mark_check_and_clear(void)
 {
-	bool unretained_wake;
 	bool restore_valid;
 	uint32_t reset_reason = nrf_resetinfo_resetreas_local_get(NRF_RESETINFO);
 
 	if (reset_reason != NRF_RESETINFO_RESETREAS_LOCAL_UNRETAINED_MASK) {
 		return false;
 	}
-	unretained_wake = reset_reason & NRF_RESETINFO_RESETREAS_LOCAL_UNRETAINED_MASK;
 	nrf_resetinfo_resetreas_local_set(NRF_RESETINFO, 0);
 
 	restore_valid = nrf_resetinfo_restore_valid_check(NRF_RESETINFO);
 	nrf_resetinfo_restore_valid_set(NRF_RESETINFO, false);
 
-	return (unretained_wake & restore_valid) ? true : false;
+	return restore_valid;
 }

Original file line number	Diff line number	Diff line change
`@@ -187,18 +187,16 @@ void pm_s2ram_mark_set(void)`
`187`	`187`
`188`	`188`	`bool pm_s2ram_mark_check_and_clear(void)`
`189`	`189`	`{`
`190`		`- bool unretained_wake;`
`191`	`190`	`bool restore_valid;`
`192`	`191`	`uint32_t reset_reason = nrf_resetinfo_resetreas_local_get(NRF_RESETINFO);`
`193`	`192`
`194`	`193`	`if (reset_reason != NRF_RESETINFO_RESETREAS_LOCAL_UNRETAINED_MASK) {`
`195`	`194`	`return false;`
`196`	`195`	`}`
`197`		`- unretained_wake = reset_reason & NRF_RESETINFO_RESETREAS_LOCAL_UNRETAINED_MASK;`
`198`	`196`	`nrf_resetinfo_resetreas_local_set(NRF_RESETINFO, 0);`
`199`	`197`
`200`	`198`	`restore_valid = nrf_resetinfo_restore_valid_check(NRF_RESETINFO);`
`201`	`199`	`nrf_resetinfo_restore_valid_set(NRF_RESETINFO, false);`
`202`	`200`
`203`		`- return (unretained_wake & restore_valid) ? true : false;`
	`201`	`+ return restore_valid;`
`204`	`202`	`}`