dp: use 64bit integers for period calculation

for some huge bitrates there was an overflow in 32bit
numbers in period calculations, i.e:

dev->period = 1000000 * (uint64_t)sink_get_min_free_space(sink)

for 192000-32-7 format it was 5824000000, more than maxint32bit

Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>

Author: marcinszkudlinski

src/audio/module_adapter/module_adapter.c

@@ -139,10 +139,13 @@ static void module_adapter_calculate_dp_period(struct comp_dev *dev)
 	unsigned int period = UINT32_MAX;
 
 	for (int i = 0; i < mod->num_of_sinks; i++) {
-		/* calculate time required the module to provide OBS data portion - a period */
-		unsigned int sink_period = 1000000 * sink_get_min_free_space(mod->sinks[i]) /
-					   (sink_get_frame_bytes(mod->sinks[i]) *
-					   sink_get_rate(mod->sinks[i]));
+		/* calculate time required the module to provide OBS data portion - a period
+		 * use 64bit integers to avoid overflows
+		 */
+		unsigned int sink_period =
+			(uint64_t)1000000 * (uint64_t)sink_get_min_free_space(mod->sinks[i]) /
+			((uint64_t)sink_get_frame_bytes(mod->sinks[i]) *
+			(uint64_t)sink_get_rate(mod->sinks[i]));
 		/* note the minimal period for the module */
 		if (period > sink_period)
 			period = sink_period;

src/audio/src/src_ipc4.c

@@ -105,9 +105,12 @@ int src_set_params(struct processing_module *mod, struct sof_sink *sink)
 		 * according to OBS size and data rate
 		 * as SRC uses period value to calculate its internal buffers,
 		 * it must be done here, right after setting sink parameters
+		 *
+		 * calculate period using 64bit integer to avoid overflows
 		 */
-		dev->period = 1000000 * sink_get_min_free_space(sink) /
-			      (sink_get_frame_bytes(sink) * sink_get_rate(sink));
+		dev->period = (uint64_t)1000000 * (uint64_t)sink_get_min_free_space(sink) /
+		      ((uint64_t)sink_get_frame_bytes(sink) * (uint64_t)sink_get_rate(sink));
+
 		/* align down period to LL cycle time */
 		dev->period /= LL_TIMER_PERIOD_US;
 		dev->period *= LL_TIMER_PERIOD_US;