On 21/10/2022 04:00, Padmanabhan Rajanbabu wrote:
It's not clear to me why the solution here is to move the configuration to
the
sound card rather than to improve the I2S driver to be able to cope with whatever the restrictions are on the PSR in these systems - it seems more cumbersome for system integrators, especially since you've not documented the issues or how to configure it. Could you expand on what the
constraints
are here and why it's not possible for the driver to figure things out
(given
some quirk information)?
Thank you for reviewing the patch.
In Samsung I2S CPU controller, to derive the frame clock, we are supposed to configure the PSR and RFS internal dividers. i.e.
OPCLK -> PSR -> RCLK -> RFS -> Frame clock
Where: OPCLK - Operational clock PSR - Operational clock prescaler RCLK - Root Clock (derived from OPCLK based on PSR) RFS - Root frequency selection (divider) Frame clock - Sample frequency (derived from RCLK based on RFS)
Ultimately,
PSR = OPCLK / Frame clock / RFS
Unlike other platforms utilizing Samsung CPU DAI, FSD SoC has a limitation on operational clock, where the clock frequency is fixed (66 MHz) and cannot be modified.
Assuming that an userspace application wants perform playback @44100 Hz and assuming that RFS divider value is configured as 256, the PSR value will yield to
66 MHz / 44.1 KHz / 256 = 5
However if HW uses PSR = 5 to derive the frame clock from operational clock, then
RCLK = OPCLK / PSR = 66 MHz / 5 = 13.2 MHz Frame clock = RCLK / RFS = 13.2 MHz / 256 = 51562 Hz
The actual frame clock derived based on PSR is now different from what user application has intended. The situation did not improve even if the RFS is swept throughout the entire valid range.
We can overcome this scenario to an extent if we can get a flexibility to Configure both PSR as well as RFS.
i.e. to achieve frame clock of 44100 Hz, if PSR = 23 and RFS = 64 then frame clock = 66 MHz / 23 / 64 = 44836 Hz
Although the sample frequency is not precise, it is very much closer to the Intended frequency, when compared to that of the existing solution. Since this scenario is specific to FSD SoC and has no changes in the Samsung I2S CPU DAI, the configuration is being done from the sound card of FSD SoC during hw_params.
Please let me know if you think this scenario can be approached in any other way possible, rather than configuring from sound card.
Entire new driver for this, instead of improving existing Samsung drivers... no, it is no the way. If you followed this approach you would send 20 drivers for each "specific" quirk difference.
Best regards, Krzysztof