-
Notifications
You must be signed in to change notification settings - Fork 321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
topology2: intel: bt-ssp-config: use cardinal clock as source #8214
Conversation
All existing SSP-based topologies use the audio cardinal clock, *EXCEPT* Bluetooth related ones. This doesn't make much sense, let's use the same clock source for all SSPs. Signed-off-by: Pierre-Louis Bossart <[email protected]>
@kv2019i @ranj063 @naveen-manohar @sathyap-chrome @sathya-nujella @Vamshigopal @RDharageswari please chime in |
Looks like the build is broken, there's no way a topology2 change could cause a build issue in the fuzzer https://github.com/thesofproject/sof/actions/runs/6223015308/job/16888062822?pr=8214
@kv2019i @lgirdwood known issue? |
@plbossart doh, that's my bad. this is a regression caused by #8163 . let me file a bug |
Added @yongzhi1 , this would need to be checked on platform. |
bclk_freq 256000/128000 and 1536000, can cardinal clk support above values? |
24576000/256000 = 96 all integer multiples so no issues. What will not work is 2,400,000 Hz (10.24 ratio) |
All existing SSP-based topologies use the audio cardinal clock, EXCEPT Bluetooth related ones. This doesn't make much sense, let's use the same clock source for all SSPs.