On Fri, Dec 15, 2023 at 12:55:08AM +0800, Jianhua Lu wrote:
On Thu, Dec 14, 2023 at 03:56:52PM +0000, Mark Brown wrote:
On Thu, Dec 14, 2023 at 11:51:50PM +0800, Jianhua Lu wrote:
Currently, qcom TDM setup need to read hw_params, if we want to move it to machine driver, we must hardcode some params, but it will reduce reduce readability.
What makes you say that TDM setup needs to read hw_params?
qcom_snd_tdm_hw_params function read PCM_FORMAT to set slot_width value, read channels to set rx_mask value.
A large part of the purpose of doing TDM configuration is to fix the slot width and assign which slots are in use by this interface - the TDM configuration is a constraint on what hardware paramters can be set and should always be followed regardless of what is being done with the audio stream. If you're just trying to configure the sample size for DSP modes then that shouldn't go through the TDM configuration API, that's just normal hw_params() so should be done directly. Possibly the hardware doesn't support manual TDM configuration?