commit 930dbd32a91d0dff8e1c61857880063cf405387f
author Philipp Wagner <phw@lowrisc.org> Thu Jun 17 17:50:26 2021 +0100
committer Philipp Wagner <mail@philipp-wagner.com> Fri Jun 18 20:24:47 2021 +0100
tree e54a591ca6fd52292be1315b8fb289ed67b7aa88
parent 82bf88f04d74c1d8b023b11dd2bd4a8b43ad4a73

[pwrmgr/dif] Pass input argument by value

The function `dif_pwrmgr_irq_restore_all()` has an input argument
`snapshot`, which is passed as pointer, but used as normal value. It's
an integer, so we can easily pass it by value.

Change triggered by a GCC 11 warning about a maybe unitialized variable.

Signed-off-by: Philipp Wagner <phw@lowrisc.org>

sw/device/lib/dif/dif_pwrmgr.c

diff --git a/sw/device/lib/dif/dif_pwrmgr.c b/sw/device/lib/dif/dif_pwrmgr.c
index b106f67..d94f6a1 100644
--- a/sw/device/lib/dif/dif_pwrmgr.c
+++ b/sw/device/lib/dif/dif_pwrmgr.c
@@ -585,12 +585,12 @@
 }
 
 dif_pwrmgr_result_t dif_pwrmgr_irq_restore_all(
-    const dif_pwrmgr_t *pwrmgr, const dif_pwrmgr_irq_snapshot_t *snapshot) {
-  if (pwrmgr == NULL || snapshot == NULL) {
+    const dif_pwrmgr_t *pwrmgr, dif_pwrmgr_irq_snapshot_t snapshot) {
+  if (pwrmgr == NULL) {
     return kDifPwrmgrBadArg;
   }
 
   mmio_region_write32(pwrmgr->params.base_addr, PWRMGR_INTR_ENABLE_REG_OFFSET,
-                      *snapshot);
+                      snapshot);
   return kDifPwrmgrOk;
 }