From: Adam Serbinski <adam@serbinski.com> To: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> Cc: Mark Brown <broonie@kernel.org>, Rob Herring <robh+dt@kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, Andy Gross <agross@kernel.org>, Mark Rutland <mark.rutland@arm.com>, Liam Girdwood <lgirdwood@gmail.com>, Patrick Lai <plai@codeaurora.org>, Banajit Goswami <bgoswami@codeaurora.org>, Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>, alsa-devel@alsa-project.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 6/6] ASoC: qdsp6: dt-bindings: Add q6afe pcm dt binding documentation Date: Wed, 12 Feb 2020 07:07:56 -0500 [thread overview] Message-ID: <d4e16b7944adbd8859c8287673da3417@serbinski.com> (raw) In-Reply-To: <b5c1328a-e3ca-826d-9ff0-f2bbce24ac22@linaro.org> On 2020-02-12 06:43, Srinivas Kandagatla wrote: > On 12/02/2020 11:01, Adam Serbinski wrote: >>>> >>>> + >>>> + - qcom,pcm-slot-mapping >>>> + Usage: required for pcm interface >>> >>> Are these not specific to 8k and 16k mode ? >>> We should probably list values for both modes here. >> >> No, this is just the offset that the audio sample is placed in with >> respect to a maximum of 4 slots, 16 bits wide, beginning with the sync >> pulse. > > > That's not true atleast by the QDSP documentation, > according to it we will use more slots to transfer at higher sample > rate. ex: > 16 kHz data can be transferred using 8 kHz samples in two > slots. > > Also there are 32 slots for each of 4 supported channels for PCM AFE > port. Ok, if that's the case, then it sounds like someone else is going to have to implement it. I have no way to test that kind of a configuration, so attempting to implement it would be futile. > > >> >> When switching between 8 and 16k sample rate, it is just the sync >> pulse rate that is changed. The audio sample will be delivered in the >> same slot, just at a different frequency.
WARNING: multiple messages have this Message-ID (diff)
From: Adam Serbinski <adam@serbinski.com> To: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> Cc: Mark Rutland <mark.rutland@arm.com>, devicetree@vger.kernel.org, alsa-devel@alsa-project.org, Banajit Goswami <bgoswami@codeaurora.org>, linux-arm-msm@vger.kernel.org, Patrick Lai <plai@codeaurora.org>, Takashi Iwai <tiwai@suse.com>, Liam Girdwood <lgirdwood@gmail.com>, Rob Herring <robh+dt@kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, Mark Brown <broonie@kernel.org>, Andy Gross <agross@kernel.org>, linux-kernel@vger.kernel.org Subject: Re: [alsa-devel] [PATCH v3 6/6] ASoC: qdsp6: dt-bindings: Add q6afe pcm dt binding documentation Date: Wed, 12 Feb 2020 07:07:56 -0500 [thread overview] Message-ID: <d4e16b7944adbd8859c8287673da3417@serbinski.com> (raw) In-Reply-To: <b5c1328a-e3ca-826d-9ff0-f2bbce24ac22@linaro.org> On 2020-02-12 06:43, Srinivas Kandagatla wrote: > On 12/02/2020 11:01, Adam Serbinski wrote: >>>> >>>> + >>>> + - qcom,pcm-slot-mapping >>>> + Usage: required for pcm interface >>> >>> Are these not specific to 8k and 16k mode ? >>> We should probably list values for both modes here. >> >> No, this is just the offset that the audio sample is placed in with >> respect to a maximum of 4 slots, 16 bits wide, beginning with the sync >> pulse. > > > That's not true atleast by the QDSP documentation, > according to it we will use more slots to transfer at higher sample > rate. ex: > 16 kHz data can be transferred using 8 kHz samples in two > slots. > > Also there are 32 slots for each of 4 supported channels for PCM AFE > port. Ok, if that's the case, then it sounds like someone else is going to have to implement it. I have no way to test that kind of a configuration, so attempting to implement it would be futile. > > >> >> When switching between 8 and 16k sample rate, it is just the sync >> pulse rate that is changed. The audio sample will be delivered in the >> same slot, just at a different frequency. _______________________________________________ Alsa-devel mailing list Alsa-devel@alsa-project.org https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2020-02-12 12:08 UTC|newest] Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <20200212015222.8229-1-adam@serbinski.com> 2020-02-12 1:52 ` [PATCH v3 1/6] ASoC: qdsp6: dt-bindings: Add q6afe pcm dt binding Adam Serbinski 2020-02-12 1:52 ` [alsa-devel] " Adam Serbinski 2020-02-12 1:52 ` [PATCH v3 2/6] ASoC: qdsp6: q6afe: add support to pcm ports Adam Serbinski 2020-02-12 1:52 ` [alsa-devel] " Adam Serbinski 2020-02-12 1:52 ` [PATCH v3 3/6] ASoC: qdsp6: q6afe-dai: add support to pcm port dais Adam Serbinski 2020-02-12 1:52 ` [alsa-devel] " Adam Serbinski 2020-02-12 9:52 ` Srinivas Kandagatla 2020-02-12 9:52 ` [alsa-devel] " Srinivas Kandagatla 2020-02-12 9:53 ` Srinivas Kandagatla 2020-02-12 9:53 ` [alsa-devel] " Srinivas Kandagatla 2020-02-12 1:52 ` [PATCH v3 4/6] ASoC: qdsp6: q6routing: add pcm port routing Adam Serbinski 2020-02-12 1:52 ` [alsa-devel] " Adam Serbinski 2020-02-12 1:52 ` [PATCH v3 5/6] ASoC: qcom: apq8096: add support for primary and quaternary I2S/PCM Adam Serbinski 2020-02-12 1:52 ` [alsa-devel] " Adam Serbinski 2020-02-12 1:52 ` [PATCH v3 6/6] ASoC: qdsp6: dt-bindings: Add q6afe pcm dt binding documentation Adam Serbinski 2020-02-12 1:52 ` [alsa-devel] " Adam Serbinski 2020-02-12 9:59 ` Srinivas Kandagatla 2020-02-12 9:59 ` [alsa-devel] " Srinivas Kandagatla 2020-02-12 11:01 ` Adam Serbinski 2020-02-12 11:01 ` [alsa-devel] " Adam Serbinski 2020-02-12 11:43 ` Srinivas Kandagatla 2020-02-12 11:43 ` [alsa-devel] " Srinivas Kandagatla 2020-02-12 12:07 ` Adam Serbinski [this message] 2020-02-12 12:07 ` Adam Serbinski
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=d4e16b7944adbd8859c8287673da3417@serbinski.com \ --to=adam@serbinski.com \ --cc=agross@kernel.org \ --cc=alsa-devel@alsa-project.org \ --cc=bgoswami@codeaurora.org \ --cc=bjorn.andersson@linaro.org \ --cc=broonie@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=lgirdwood@gmail.com \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=perex@perex.cz \ --cc=plai@codeaurora.org \ --cc=robh+dt@kernel.org \ --cc=srinivas.kandagatla@linaro.org \ --cc=tiwai@suse.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.