src: bugfix: rate for sink not set

[marcinszkudlinski]

Rate for sink was not set properly in src.c

[singalsu]

Thanks, to be sure I'll give this a try with #7547 . We don't have in CI test for this.

[kv2019i]

Looks good. I'll wait @singalsu for your ack on this.

[singalsu]

Unfortunately this is not fixing the regression that has happened. @marcinszkudlinski Is there a bug that you see that this fixes?

Probably OK to merge this but some other fixes are needed too.

[singalsu]

I wonder if every component should set both source and sink params from init IPC? E.g. Aria sets both in

sof/src/audio/aria/aria.c

Line 182 in 8f60180

source = list_first_item(&dev->bsource_list, struct comp_buffer, sink_list);

What is the recommendation for IPC4 modules @marcinszkudlinski ?

I'm getting with this patch a divide by zero exception in source_get_data_frames_available(). Apparently source_get_frame_bytes(source) returns zero. Doing "arecord -Dhw:0,2 -f dat -r 44100 -d 10 rec.wav" with topology from #7547.

I think the exception happens also without this patch but it was the 1st time able was able to catch it to trace.

[lgirdwood]

@wszypelt @lrudyX not expecting this to fail unless CI is making a different assumption to SRC for rates ?

[lrudyX]

@wszypelt @lrudyX not expecting this to fail unless CI is making a different assumption to SRC for rates ?

We have fail with LNL. Fails only with this platform.

[lgirdwood]

SOFCI TEST

[lgirdwood]

@marcinszkudlinski rerunning CI to rule out a false positive result.

[kv2019i]

Proceeding with merge, only known fails in CI report.