MAX32520: Fix I2C slave address register offset. (#384)

Co-authored-by: Scheiffler <[email protected]>
MSDK-Commit: e5a58576da502c7b71cdf7e5b034c7d72b195d8c

Author: Jacob-Scheiffler

MAX/Libraries/CMSIS/Device/Maxim/MAX32520/Include/i2c_regs.h

@@ -5,37 +5,37 @@
  */
 
 /******************************************************************************
-* Copyright (C) 2022 Maxim Integrated Products, Inc., All Rights Reserved.
-*
-* Permission is hereby granted, free of charge, to any person obtaining a
-* copy of this software and associated documentation files (the "Software"),
-* to deal in the Software without restriction, including without limitation
-* the rights to use, copy, modify, merge, publish, distribute, sublicense,
-* and/or sell copies of the Software, and to permit persons to whom the
-* Software is furnished to do so, subject to the following conditions:
-*
-* The above copyright notice and this permission notice shall be included
-* in all copies or substantial portions of the Software.
-*
-* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
-* OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
-* IN NO EVENT SHALL MAXIM INTEGRATED BE LIABLE FOR ANY CLAIM, DAMAGES
-* OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
-* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
-* OTHER DEALINGS IN THE SOFTWARE.
-*
-* Except as contained in this notice, the name of Maxim Integrated
-* Products, Inc. shall not be used except as stated in the Maxim Integrated
-* Products, Inc. Branding Policy.
-*
-* The mere transfer of this software does not imply any licenses
-* of trade secrets, proprietary technology, copyrights, patents,
-* trademarks, maskwork rights, or any other form of intellectual
-* property whatsoever. Maxim Integrated Products, Inc. retains all
-* ownership rights.
-*
-******************************************************************************/
+ * Copyright (C) 2023 Maxim Integrated Products, Inc., All Rights Reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included
+ * in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
+ * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+ * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+ * IN NO EVENT SHALL MAXIM INTEGRATED BE LIABLE FOR ANY CLAIM, DAMAGES
+ * OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+ * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ * OTHER DEALINGS IN THE SOFTWARE.
+ *
+ * Except as contained in this notice, the name of Maxim Integrated
+ * Products, Inc. shall not be used except as stated in the Maxim Integrated
+ * Products, Inc. Branding Policy.
+ *
+ * The mere transfer of this software does not imply any licenses
+ * of trade secrets, proprietary technology, copyrights, patents,
+ * trademarks, maskwork rights, or any other form of intellectual
+ * property whatsoever. Maxim Integrated Products, Inc. retains all
+ * ownership rights.
+ *
+ ******************************************************************************/
 
 #ifndef LIBRARIES_CMSIS_DEVICE_MAXIM_MAX32520_INCLUDE_I2C_REGS_H_
 #define LIBRARIES_CMSIS_DEVICE_MAXIM_MAX32520_INCLUDE_I2C_REGS_H_
@@ -103,8 +103,9 @@ typedef struct {
     __IO uint32_t ckh;                  /**< <tt>\b 0x38:</tt> I2C CKH Register */
     __R  uint32_t rsv_0x3c;
     __IO uint32_t to;                   /**< <tt>\b 0x40:</tt> I2C TO Register */
-    __IO uint32_t sla;                  /**< <tt>\b 0x44:</tt> I2C SLA Register */
+    __R  uint32_t rsv_0x44;
     __IO uint32_t dma;                  /**< <tt>\b 0x48:</tt> I2C DMA Register */
+    __IO uint32_t sla;                  /**< <tt>\b 0x4C:</tt> I2C SLA Register */
 } mxc_i2c_regs_t;
 
 /* Register offsets for module I2C */
@@ -130,8 +131,8 @@ typedef struct {
 #define MXC_R_I2C_CKL                      ((uint32_t)0x00000034UL) /**< Offset from I2C Base Address: <tt> 0x0034</tt> */
 #define MXC_R_I2C_CKH                      ((uint32_t)0x00000038UL) /**< Offset from I2C Base Address: <tt> 0x0038</tt> */
 #define MXC_R_I2C_TO                       ((uint32_t)0x00000040UL) /**< Offset from I2C Base Address: <tt> 0x0040</tt> */
-#define MXC_R_I2C_SLA                      ((uint32_t)0x00000044UL) /**< Offset from I2C Base Address: <tt> 0x0044</tt> */
 #define MXC_R_I2C_DMA                      ((uint32_t)0x00000048UL) /**< Offset from I2C Base Address: <tt> 0x0048</tt> */
+#define MXC_R_I2C_SLA                      ((uint32_t)0x0000004CUL) /**< Offset from I2C Base Address: <tt> 0x004C</tt> */
 /**@} end of group i2c_registers */
 
 /**
@@ -502,6 +503,20 @@ typedef struct {
 
 /**@} end of group I2C_TO_Register */
 
+/**
+ * @ingroup  i2c_registers
+ * @defgroup I2C_DMA I2C_DMA
+ * @brief    DMA Register.
+ * @{
+ */
+#define MXC_F_I2C_DMA_TXEN_POS                         0 /**< DMA_TXEN Position */
+#define MXC_F_I2C_DMA_TXEN                             ((uint32_t)(0x1UL << MXC_F_I2C_DMA_TXEN_POS)) /**< DMA_TXEN Mask */
+
+#define MXC_F_I2C_DMA_RXEN_POS                         1 /**< DMA_RXEN Position */
+#define MXC_F_I2C_DMA_RXEN                             ((uint32_t)(0x1UL << MXC_F_I2C_DMA_RXEN_POS)) /**< DMA_RXEN Mask */
+
+/**@} end of group I2C_DMA_Register */
+
 /**
  * @ingroup  i2c_registers
  * @defgroup I2C_SLA I2C_SLA
@@ -520,20 +535,6 @@ typedef struct {
 
 /**@} end of group I2C_SLA_Register */
 
-/**
- * @ingroup  i2c_registers
- * @defgroup I2C_DMA I2C_DMA
- * @brief    DMA Register.
- * @{
- */
-#define MXC_F_I2C_DMA_TXEN_POS                         0 /**< DMA_TXEN Position */
-#define MXC_F_I2C_DMA_TXEN                             ((uint32_t)(0x1UL << MXC_F_I2C_DMA_TXEN_POS)) /**< DMA_TXEN Mask */
-
-#define MXC_F_I2C_DMA_RXEN_POS                         1 /**< DMA_RXEN Position */
-#define MXC_F_I2C_DMA_RXEN                             ((uint32_t)(0x1UL << MXC_F_I2C_DMA_RXEN_POS)) /**< DMA_RXEN Mask */
-
-/**@} end of group I2C_DMA_Register */
-
 #ifdef __cplusplus
 }
 #endif

MAX/Libraries/PeriphDrivers/Source/I2C/i2c_reva_es17.svd

@@ -1260,14 +1260,14 @@
         </fields>
       </register>
       <register>
-        <name>SLA</name>
-        <description>Slave Address Register.</description>
-        <addressOffset>0x44</addressOffset>
+        <name>DMA</name>
+        <description>DMA Register.</description>
+        <addressOffset>0x48</addressOffset>
         <fields>
           <field>
-            <name>SLA</name>
-            <description>Slave Address.</description>
-            <bitRange>[9:0]</bitRange>
+            <name>TXEN</name>
+            <description>TX channel enable.</description>
+            <bitRange>[0:0]</bitRange>
             <enumeratedValues>
               <enumeratedValue>
                 <name>dis</name>
@@ -1282,33 +1282,33 @@
             </enumeratedValues>
           </field>
           <field>
-            <name>EA</name>
-            <description>Extended Address Select. This bit selects whether the SLA contains a 7-bit or 10-bit address.</description>
-            <bitRange>[15:15]</bitRange>
+            <name>RXEN</name>
+            <description>RX channel enable.</description>
+            <bitRange>[1:1]</bitRange>
             <enumeratedValues>
               <enumeratedValue>
-                <name>7_bit_address</name>
-                <description>7-bit address select.</description>
+                <name>dis</name>
+                <description>Disable.</description>
                 <value>0</value>
               </enumeratedValue>
               <enumeratedValue>
-                <name>10_bit_address</name>
-                <description>10-bit address select.</description>
+                <name>en</name>
+                <description>Enable.</description>
                 <value>1</value>
               </enumeratedValue>
             </enumeratedValues>
           </field>
         </fields>
       </register>
       <register>
-        <name>DMA</name>
-        <description>DMA Register.</description>
-        <addressOffset>0x48</addressOffset>
+        <name>SLA</name>
+        <description>Slave Address Register.</description>
+        <addressOffset>0x4C</addressOffset>
         <fields>
           <field>
-            <name>TXEN</name>
-            <description>TX channel enable.</description>
-            <bitRange>[0:0]</bitRange>
+            <name>SLA</name>
+            <description>Slave Address.</description>
+            <bitRange>[9:0]</bitRange>
             <enumeratedValues>
               <enumeratedValue>
                 <name>dis</name>
@@ -1323,18 +1323,18 @@
             </enumeratedValues>
           </field>
           <field>
-            <name>RXEN</name>
-            <description>RX channel enable.</description>
-            <bitRange>[1:1]</bitRange>
+            <name>EA</name>
+            <description>Extended Address Select. This bit selects whether the SLA contains a 7-bit or 10-bit address.</description>
+            <bitRange>[15:15]</bitRange>
             <enumeratedValues>
               <enumeratedValue>
-                <name>dis</name>
-                <description>Disable.</description>
+                <name>7_bit_address</name>
+                <description>7-bit address select.</description>
                 <value>0</value>
               </enumeratedValue>
               <enumeratedValue>
-                <name>en</name>
-                <description>Enable.</description>
+                <name>10_bit_address</name>
+                <description>10-bit address select.</description>
                 <value>1</value>
               </enumeratedValue>
             </enumeratedValues>