Merge pull request #14776 from affrinpinhero-2356/i2c_longTime_Mem_Solve

0xc0170 · web-flow · commit 8188f5f5ab59 · 2021-06-29T11:30:02.000+02:00
driver/i2c: STM32: I2C memory usage and time delay in read-write solved.
diff --git a/targets/TARGET_STM/TARGET_STM32F0/i2c_device.c b/targets/TARGET_STM/TARGET_STM32F0/i2c_device.c
@@ -41,7 +41,13 @@ uint32_t i2c_get_pclk(I2CName i2c)
                 pclk = HSI_VALUE;
                 break;
         }
-    } else {
+    } 
+#if defined I2C2_BASE
+    else if (i2c == I2C_2) {
+        pclk = HAL_RCC_GetSysClockFreq();
+    }
+#endif
+    else {
         // should not happend
         error("I2C: unknown instance");
     }
@@ -78,7 +84,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
     }
 
     else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance*/
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32F0/i2c_device.h b/targets/TARGET_STM/TARGET_STM32F0/i2c_device.h
@@ -48,10 +48,13 @@ extern "C" {
 #define I2CAPI_I2C1_CLKSRC RCC_I2C1CLKSOURCE_SYSCLK
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/TARGET_STM32F3/i2c_device.c b/targets/TARGET_STM/TARGET_STM32F3/i2c_device.c
@@ -122,7 +122,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
     }
 
     else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance*/
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32F3/i2c_device.h b/targets/TARGET_STM/TARGET_STM32F3/i2c_device.h
@@ -62,10 +62,13 @@ extern "C" {
 #define I2CAPI_I2C3_CLKSRC RCC_I2C3CLKSOURCE_SYSCLK
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/TARGET_STM32F7/i2c_device.c b/targets/TARGET_STM/TARGET_STM32F7/i2c_device.c
@@ -133,7 +133,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
                 break;
         }
     } else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance*/
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32F7/i2c_device.h b/targets/TARGET_STM/TARGET_STM32F7/i2c_device.h
@@ -57,10 +57,13 @@ extern "C" {
 #define I2CAPI_I2C4_CLKSRC RCC_I2C4CLKSOURCE_PCLK1
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/TARGET_STM32G0/i2c_device.c b/targets/TARGET_STM/TARGET_STM32G0/i2c_device.c
@@ -78,7 +78,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
                 break;
         }
     } else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance*/
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32G0/i2c_device.h b/targets/TARGET_STM/TARGET_STM32G0/i2c_device.h
@@ -47,10 +47,13 @@ extern "C" {
 #define I2CAPI_I2C2_CLKSRC RCC_I2C2CLKSOURCE_SYSCLK
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/TARGET_STM32G4/i2c_device.c b/targets/TARGET_STM/TARGET_STM32G4/i2c_device.c
@@ -133,7 +133,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
                 break;
         }
     } else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance*/
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32G4/i2c_device.h b/targets/TARGET_STM/TARGET_STM32G4/i2c_device.h
@@ -29,7 +29,7 @@ extern "C" {
 #define TIMING_VAL_DEFAULT_CLK_100KHZ  0xC0311319  // Standard mode with Rise Time = 400ns and Fall Time = 100ns
 #define TIMING_VAL_DEFAULT_CLK_400KHZ  0x10B1102E  // Fast mode with Rise Time = 250ns and Fall Time = 100ns
 #define TIMING_VAL_DEFAULT_CLK_1MHZ    0x00710B1E  // Fast mode Plus with Rise Time = 60ns and Fall Time = 100ns
-#define I2C_PCLK_DEF                170000000   // 170 MHz
+#define I2C_PCLK_DEF                160000000   // 160 MHz
 
 /*  Define IP version */
 #define I2C_IP_VERSION_V2
@@ -43,11 +43,13 @@ extern "C" {
 #define I2CAPI_I2C4_CLKSRC RCC_I2C4CLKSOURCE_SYSCLK
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
-
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 #endif // DEVICE_I2C
 
 #ifdef __cplusplus
diff --git a/targets/TARGET_STM/TARGET_STM32H7/i2c_device.c b/targets/TARGET_STM/TARGET_STM32H7/i2c_device.c
@@ -116,7 +116,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
                 break;
         }
     } else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance*/
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32H7/i2c_device.h b/targets/TARGET_STM/TARGET_STM32H7/i2c_device.h
@@ -43,10 +43,13 @@ extern "C" {
 #define I2CAPI_I2C4_CLKSRC RCC_I2C4CLKSOURCE_D3PCLK1
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/TARGET_STM32L0/i2c_device.c b/targets/TARGET_STM/TARGET_STM32L0/i2c_device.c
@@ -103,7 +103,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
                 break;
         }
     } else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance*/
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32L0/i2c_device.h b/targets/TARGET_STM/TARGET_STM32L0/i2c_device.h
@@ -54,10 +54,13 @@ extern "C" {
 #define I2CAPI_I2C4_CLKSRC RCC_I2C4CLKSOURCE_SYSCLK
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/TARGET_STM32L4/i2c_device.c b/targets/TARGET_STM/TARGET_STM32L4/i2c_device.c
@@ -168,7 +168,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
                 break;
         }
     } else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance*/
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32L4/i2c_device.h b/targets/TARGET_STM/TARGET_STM32L4/i2c_device.h
@@ -57,10 +57,13 @@ extern "C" {
 #define I2CAPI_I2C4_CLKSRC RCC_I2C4CLKSOURCE_SYSCLK
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/TARGET_STM32L5/i2c_device.c b/targets/TARGET_STM/TARGET_STM32L5/i2c_device.c
@@ -142,7 +142,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
     }
 
     else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance */
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32L5/i2c_device.h b/targets/TARGET_STM/TARGET_STM32L5/i2c_device.h
@@ -43,10 +43,13 @@ extern "C" {
 #define I2CAPI_I2C4_CLKSRC RCC_I2C4CLKSOURCE_SYSCLK
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/TARGET_STM32WB/i2c_device.c b/targets/TARGET_STM/TARGET_STM32WB/i2c_device.c
@@ -140,7 +140,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
     }
 
     else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered. 
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target. 
+        Enabling this may impact performance */
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32WB/i2c_device.h b/targets/TARGET_STM/TARGET_STM32WB/i2c_device.h
@@ -24,7 +24,7 @@
 extern "C" {
 #endif
 
-/*  Define I2C Device */
+/* Define I2C Device */
 #if DEVICE_I2C
 
 /*  Define IP version */
@@ -50,10 +50,13 @@ extern "C" {
 #define I2CAPI_I2C3_CLKSRC RCC_I2C3CLKSOURCE_SYSCLK
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/TARGET_STM32WL/i2c_device.c b/targets/TARGET_STM/TARGET_STM32WL/i2c_device.c
@@ -125,7 +125,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)
     }
 
     else {
+        /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered.
+        User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target.
+        Enabling this may impact performance */
+        MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
         tim = i2c_compute_timing(pclk, hz);
+#endif
     }
     return tim;
 }
diff --git a/targets/TARGET_STM/TARGET_STM32WL/i2c_device.h b/targets/TARGET_STM/TARGET_STM32WL/i2c_device.h
@@ -42,10 +42,13 @@ extern "C" {
 #define I2CAPI_I2C3_CLKSRC RCC_I2C3CLKSOURCE_SYSCLK
 
 uint32_t i2c_get_pclk(I2CName i2c);
-uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 uint32_t i2c_get_timing(I2CName i2c, int hz);
+
+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
+uint32_t i2c_compute_timing(uint32_t clock_src_freq, uint32_t i2c_freq);
 void i2c_compute_presc_scldel_sdadel(uint32_t clock_src_freq, uint32_t I2C_speed);
 uint32_t i2c_compute_scll_sclh(uint32_t clock_src_freq, uint32_t I2C_speed);
+#endif // MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO
 
 #endif // DEVICE_I2C
 
diff --git a/targets/TARGET_STM/i2c_api.c b/targets/TARGET_STM/i2c_api.c
diff --git a/targets/targets.json b/targets/targets.json

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,13 @@ uint32_t i2c_get_pclk(I2CName i2c)`
`41`	`41`	`pclk = HSI_VALUE;`
`42`	`42`	`break;`
`43`	`43`	`}`
`44`		`- } else {`
	`44`	`+ }`
	`45`	`+#if defined I2C2_BASE`
	`46`	`+ else if (i2c == I2C_2) {`
	`47`	`+ pclk = HAL_RCC_GetSysClockFreq();`
	`48`	`+ }`
	`49`	`+#endif`
	`50`	`+ else {`
`45`	`51`	`// should not happend`
`46`	`52`	`error("I2C: unknown instance");`
`47`	`53`	`}`
`@@ -78,7 +84,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)`
`78`	`84`	`}`
`79`	`85`
`80`	`86`	`else {`
	`87`	`+ /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered.`
	`88`	`+ User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target.`
	`89`	`+ Enabling this may impact performance*/`
	`90`	`+ MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);`
	`91`	`+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO`
`81`	`92`	`tim = i2c_compute_timing(pclk, hz);`
	`93`	`+#endif`
`82`	`94`	`}`
`83`	`95`	`return tim;`
`84`	`96`	`}`
Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)`
`122`	`122`	`}`
`123`	`123`
`124`	`124`	`else {`
	`125`	`+ /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered.`
	`126`	`+ User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target.`
	`127`	`+ Enabling this may impact performance*/`
	`128`	`+ MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);`
	`129`	`+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO`
`125`	`130`	`tim = i2c_compute_timing(pclk, hz);`
	`131`	`+#endif`
`126`	`132`	`}`
`127`	`133`	`return tim;`
`128`	`134`	`}`
Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)`
`133`	`133`	`break;`
`134`	`134`	`}`
`135`	`135`	`} else {`
	`136`	`+ /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered.`
	`137`	`+ User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target.`
	`138`	`+ Enabling this may impact performance*/`
	`139`	`+ MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);`
	`140`	`+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO`
`136`	`141`	`tim = i2c_compute_timing(pclk, hz);`
	`142`	`+#endif`
`137`	`143`	`}`
`138`	`144`	`return tim;`
`139`	`145`	`}`
Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,13 @@ uint32_t i2c_get_timing(I2CName i2c, int hz)`
`78`	`78`	`break;`
`79`	`79`	`}`
`80`	`80`	`} else {`
	`81`	`+ /* If MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO assert is triggered.`
	`82`	`+ User needs to enable I2C_TIMING_VALUE_ALGO in target.json for specific target.`
	`83`	`+ Enabling this may impact performance*/`
	`84`	`+ MBED_ASSERT(MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO);`
	`85`	`+#if MBED_CONF_TARGET_I2C_TIMING_VALUE_ALGO`
`81`	`86`	`tim = i2c_compute_timing(pclk, hz);`
	`87`	`+#endif`
`82`	`88`	`}`
`83`	`89`	`return tim;`
`84`	`90`	`}`