On Thu, Sep 05, 2024 at 04:12:52PM +0200, Jerome Brunet wrote:
This adds a sample rate definition for 12kHz, 24kHz and 128kHz.
Admittedly, just a few drivers are currently using these sample rates but there is enough of a recurrence to justify adding a definition for them and remove some custom rate constraint code while at it.
The new definitions are not added to the interval definitions, such as SNDRV_PCM_RATE_8000_44100, because it would silently add new supported rates to drivers that may or may not support them. For sure the drivers have not been tested for these new rates so it is better to leave them out of interval definitions.
That being said, the added rates are multiples of well know rates families, it is very likely that a lot of devices out there actually supports them.
Signed-off-by: Jerome Brunet jbrunet@baylibre.com
Almost wonder if a comment with the SNDRV_PCM_RATE_8000_xxx defines might also be an idea to warn they don't include all the rates, although it is I guess easily seen from the define itself so not sure if it might be over kill. But I am happy either way.
Reviewed-by: Charles Keepax ckeepax@opensource.cirrus.com
Thanks, Charles