commit e506bea33a663946760df49c663093770246dd08
author Tasnim <tasnim.ouni@st.com> Fri Jan 12 16:59:50 2024 +0100
committer Tasnim <tasnim.ouni@st.com> Fri Jan 12 17:06:06 2024 +0100
tree 126a2d84e9b0a582b7956bbdc484ee356ab1e5a0
parent f7b885efb42dec5919af054e7b31d95d38170397

[HAL][PWR] Invoke macro UNUSED() in HAL_PWR_ConfigAttributes() to avoid compilation warnings due to unused parameter

Src/stm32l5xx_hal_pwr.c

diff --git a/Src/stm32l5xx_hal_pwr.c b/Src/stm32l5xx_hal_pwr.c
index 8bbf047..8310c5f 100644
--- a/Src/stm32l5xx_hal_pwr.c
+++ b/Src/stm32l5xx_hal_pwr.c
@@ -674,7 +674,6 @@
 void HAL_PWR_ConfigAttributes(uint32_t Item, uint32_t Attributes)
 {
   /* Check the parameters */
-  assert_param(IS_PWR_ITEMS_ATTRIBUTES(Item));
   assert_param(IS_PWR_ATTRIBUTES(Attributes));
 
   /* Privilege/non-privilege attribute */
@@ -694,6 +693,9 @@
 
 #if defined (__ARM_FEATURE_CMSE) && (__ARM_FEATURE_CMSE == 3U)
 
+  /* Check the parameters */
+  assert_param(IS_PWR_ITEMS_ATTRIBUTES(Item));
+
   /* Secure/non-secure attribute */
   if ((Attributes & PWR_SEC) == PWR_SEC)
   {
@@ -708,6 +710,11 @@
     /* do nothing */
   }
 
+#else
+
+  /* Prevent unused argument(s) compilation warning */
+  UNUSED(Item);
+
 #endif /* __ARM_FEATURE_CMSE */
 }
 
@@ -729,9 +736,6 @@
     return HAL_ERROR;
   }
 
-  /* Check the parameters */
-  assert_param(IS_PWR_ITEMS_ATTRIBUTES(Item));
-
   /* Get privilege or non-privilege attribute */
   if (READ_BIT(PWR->PRIVCFGR, PWR_PRIVCFGR_PRIV) != 0x00U)
   {
@@ -744,6 +748,9 @@
 
 #if defined (__ARM_FEATURE_CMSE) && (__ARM_FEATURE_CMSE == 3U)
 
+  /* Check the parameters */
+  assert_param(IS_PWR_ITEMS_ATTRIBUTES(Item));
+
   /* Get the secure or non-secure attribute state */
   if ((PWR_S->SECCFGR & Item) == Item)
   {
@@ -754,6 +761,11 @@
     attributes |= PWR_NSEC;
   }
 
+#else
+
+  /* Prevent unused argument(s) compilation warning */
+  UNUSED(Item);
+
 #endif /* __ARM_FEATURE_CMSE */
 
   /* return value */