Fix DMA & NVIC driver warnigns. RevB DMA driver had incorrect pointer…

… syntax. NVIC driver had an extern inside a function.
analogdevicesinc · Oct 13, 2024 · 58d6aca · 58d6aca
1 parent 2e34472
commit 58d6aca
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/Libraries/PeriphDrivers/Source/DMA/dma_revb.c b/Libraries/PeriphDrivers/Source/DMA/dma_revb.c
@@ -1,6 +1,6 @@
 /******************************************************************************
  *
- * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
+ * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by
  * Analog Devices, Inc.),
  * Copyright (C) 2023-2024 Analog Devices, Inc.
  *
@@ -197,7 +197,7 @@ int MXC_DMA_RevB_SetSrcDst(mxc_dma_srcdst_t srcdst)
 //TODO(ADI): Necessary?
 int MXC_DMA_RevB_GetSrcDst(mxc_dma_srcdst_t *srcdst)
 {
-    if (CHECK_HANDLE(srcdst.ch)) {
+    if (CHECK_HANDLE(srcdst->ch)) {
         srcdst->source = (void *)dma_resource[srcdst->ch].regs->src;
         srcdst->dest = (void *)dma_resource[srcdst->ch].regs->dst;
         srcdst->len = (dma_resource[srcdst->ch].regs->cnt) & ~MXC_F_DMA_CNTRLD_EN;
@@ -231,7 +231,7 @@ int MXC_DMA_RevB_SetSrcReload(mxc_dma_srcdst_t srcdst)
 //TODO(ADI): Necessary?
 int MXC_DMA_RevB_GetSrcReload(mxc_dma_srcdst_t *srcdst)
 {
-    if (CHECK_HANDLE(srcdst.ch)) {
+    if (CHECK_HANDLE(srcdst->ch)) {
         srcdst->source = (void *)dma_resource[srcdst->ch].regs->srcrld;
         srcdst->dest = (void *)dma_resource[srcdst->ch].regs->dstrld;
         srcdst->len = (dma_resource[srcdst->ch].regs->cntrld) & ~MXC_F_DMA_CNTRLD_EN;

diff --git a/Libraries/PeriphDrivers/Source/DMA/dma_revb.h b/Libraries/PeriphDrivers/Source/DMA/dma_revb.h
@@ -1,6 +1,6 @@
 /******************************************************************************
  *
- * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
+ * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by
  * Analog Devices, Inc.),
  * Copyright (C) 2023-2024 Analog Devices, Inc.
  *
@@ -47,7 +47,7 @@ int MXC_DMA_RevB_DisableInt(int ch);
 int MXC_DMA_RevB_Start(int ch);
 int MXC_DMA_RevB_Stop(int ch);
 mxc_dma_ch_regs_t *MXC_DMA_RevB_GetCHRegs(int ch);
-void MXC_DMA_RevB_Handler();
+void MXC_DMA_RevB_Handler(void);
 int MXC_DMA_RevB_MemCpy(void *dest, void *src, int len, mxc_dma_complete_cb_t callback);
 int MXC_DMA_RevB_DoTransfer(mxc_dma_config_t config, mxc_dma_srcdst_t firstSrcDst,
                             mxc_dma_trans_chain_t callback);

diff --git a/Libraries/PeriphDrivers/Source/SYS/nvic_table.c b/Libraries/PeriphDrivers/Source/SYS/nvic_table.c
@@ -1,6 +1,6 @@
 /******************************************************************************
  *
- * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
+ * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by
  * Analog Devices, Inc.),
  * Copyright (C) 2023-2024 Analog Devices, Inc.
  *
@@ -35,15 +35,15 @@ __attribute__((aligned(512)))
 #endif
 static void (*ramVectorTable[MXC_IRQ_COUNT])(void);
 
-void NVIC_SetRAM(void)
-{
 #if defined(__ICCARM__)
     extern void (*const __isr_vector[])(void);
 #else
     /* should be defined in starup_<device>.S */
     extern uint32_t __isr_vector[MXC_IRQ_COUNT];
 #endif
 
+void NVIC_SetRAM(void)
+{
     memcpy(&ramVectorTable, &__isr_vector, sizeof(ramVectorTable));
     SCB->VTOR = (uint32_t)&ramVectorTable;
 }