Audio: Volume: Fix the ramping and ZC mode align

This patch fixes an issue where processing smaller parts of
the period can break the data align constraints. The frames
count value from zc_get() function or from cd->vol_ramp_frames
is rounded up to next align compatible frames count.

The failure with align constraints appeared as glitches in
audio with some 44.1 kHz family sample rates.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>

Author: singalsu

src/audio/volume/volume.c

@@ -556,6 +556,7 @@ static int volume_process(struct processing_module *mod,
 			  struct output_stream_buffer *output_buffers, int num_output_buffers)
 {
 	struct vol_data *cd = module_get_private_data(mod);
+	struct audio_stream *source = input_buffers[0].data;
 	uint32_t avail_frames = input_buffers[0].size;
 	uint32_t frames;
 	int64_t prev_sum = 0;
@@ -571,12 +572,27 @@ static int volume_process(struct processing_module *mod,
 			frames = avail_frames;
 		} else if (cd->ramp_type == SOF_VOLUME_LINEAR_ZC) {
 			/* with ZC ramping look for next ZC offset */
-			frames = cd->zc_get(input_buffers[0].data, cd->vol_ramp_frames, &prev_sum);
+			frames = cd->zc_get(source, cd->vol_ramp_frames, &prev_sum);
+			/* Align frames count to audio stream constraints. If it rounds to zero
+			 * round it up to smallest nonzero aligned frames count.
+			 */
+			frames = audio_stream_align_frames_round_nearest(source, frames);
+			if (!frames)
+				frames = audio_stream_align_frames_round_up(source, 1);
 		} else {
-			/* without ZC process max ramp chunk */
-			frames = cd->vol_ramp_frames;
+			/* During volume ramp align the number of frames used in this
+			 * gain step. Align up since with low rates this would typically
+			 * become zero.
+			 */
+			frames = audio_stream_align_frames_round_up(source, cd->vol_ramp_frames);
 		}
 
+		/* Cancel the gain step for ZC or smaller ramp step if it exceeds
+		 * the available frames.
+		 */
+		if (frames > avail_frames)
+			frames = avail_frames;
+
 		if (!cd->ramp_finished) {
 			volume_ramp(mod);
 			cd->vol_ramp_elapsed_frames += frames;