| Message ID | 20171214173402.19074-2-srinivas.kandagatla@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Thu, Dec 14, 2017 at 05:33:48PM +0000, srinivas.kandagatla@linaro.org wrote: > From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > > This patch add dt bindings for Qualcomm APR bus driver > > Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > --- > .../devicetree/bindings/soc/qcom/qcom,apr.txt | 28 ++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt > > diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt > new file mode 100644 > index 000000000000..4e93213ae98d > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt > @@ -0,0 +1,28 @@ > +Qualcomm APR (Asynchronous Packet Router) binding > + > +This binding describes the Qualcomm APR. APR is a IPC protocol for > +communication between Application processor and QDSP. APR is mainly > +used for audio/voice services on the QDSP. > + > +- compatible: > + Usage: required > + Value type: <stringlist> > + Definition: must be "qcom,apr-<SOC-NAME>" example: "qcom,apr-msm8996" <soc>-apr is the more standard order. With that, Reviewed-by: Rob Herring <robh@kernel.org>
On 16/12/17 17:27, Rob Herring wrote: > On Thu, Dec 14, 2017 at 05:33:48PM +0000, srinivas.kandagatla@linaro.org wrote: >> From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >> >> This patch add dt bindings for Qualcomm APR bus driver >> >> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >> --- >> .../devicetree/bindings/soc/qcom/qcom,apr.txt | 28 ++++++++++++++++++++++ >> 1 file changed, 28 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt >> >> diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt >> new file mode 100644 >> index 000000000000..4e93213ae98d >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt >> @@ -0,0 +1,28 @@ >> +Qualcomm APR (Asynchronous Packet Router) binding >> + >> +This binding describes the Qualcomm APR. APR is a IPC protocol for >> +communication between Application processor and QDSP. APR is mainly >> +used for audio/voice services on the QDSP. >> + >> +- compatible: >> + Usage: required >> + Value type: <stringlist> >> + Definition: must be "qcom,apr-<SOC-NAME>" example: "qcom,apr-msm8996" > > <soc>-apr is the more standard order. With that, Yes, it makes sense, will do that in next version. > > Reviewed-by: Rob Herring <robh@kernel.org> Thanks for reviewed-by tag. Rgrds, Srini >
On Thu 14 Dec 09:33 PST 2017, srinivas.kandagatla@linaro.org wrote: > From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > > This patch add dt bindings for Qualcomm APR bus driver > > Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > --- > .../devicetree/bindings/soc/qcom/qcom,apr.txt | 28 ++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt > > diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt > new file mode 100644 > index 000000000000..4e93213ae98d > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt > @@ -0,0 +1,28 @@ > +Qualcomm APR (Asynchronous Packet Router) binding > + > +This binding describes the Qualcomm APR. APR is a IPC protocol for > +communication between Application processor and QDSP. APR is mainly > +used for audio/voice services on the QDSP. > + > +- compatible: > + Usage: required > + Value type: <stringlist> > + Definition: must be "qcom,apr-<SOC-NAME>" example: "qcom,apr-msm8996" This is not the only apr on msm8996 and some platform seems to have 3-4 aprs. I'm therefor hesitant to use the compatible to pick the static list of services available on the particular firmware. If this scheme is followed we at least would need to rename this qcom,msm8996-apr-audio-svc But I think it would be preferable to go with qcom,apr-v2 and then list the static services as child nodes. > + > + > +- qcom,smd-channel: > + Usage: required > + Value type: <string> > + Definition: standard SMD property specifying the SMD channel used for > + communication with the APR on QDSP. > + Should be "apr_audio_svc". This is not the only APR channel, but for apr itself this doesn't matter and might as well be qcom,glink-channels; so perhaps we can omit this from this document? > + Described in soc/qcom/qcom,smd.txt > + > += EXAMPLE > +The following example represents a QDSP based sound card on a MSM8996 device > +which uses apr as communication between Apps and QDSP. > + > + apr { > + compatible = "qcom,apr-msm8996"; > + qcom,smd-channels = "apr_audio_svc"; > + }; Regards, Bjorn
Thanks for review comments! On 03/01/18 00:35, Bjorn Andersson wrote: > On Thu 14 Dec 09:33 PST 2017, srinivas.kandagatla@linaro.org wrote: > >> From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >> >> This patch add dt bindings for Qualcomm APR bus driver >> >> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >> --- >> .../devicetree/bindings/soc/qcom/qcom,apr.txt | 28 ++++++++++++++++++++++ >> 1 file changed, 28 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt >> >> diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt >> new file mode 100644 >> index 000000000000..4e93213ae98d >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt >> @@ -0,0 +1,28 @@ >> +Qualcomm APR (Asynchronous Packet Router) binding >> + >> +This binding describes the Qualcomm APR. APR is a IPC protocol for >> +communication between Application processor and QDSP. APR is mainly >> +used for audio/voice services on the QDSP. >> + >> +- compatible: >> + Usage: required >> + Value type: <stringlist> >> + Definition: must be "qcom,apr-<SOC-NAME>" example: "qcom,apr-msm8996" > > This is not the only apr on msm8996 and some platform seems to have 3-4 > aprs. I'm therefor hesitant to use the compatible to pick the static > list of services available on the particular firmware. > > If this scheme is followed we at least would need to rename this > qcom,msm8996-apr-audio-svc > > But I think it would be preferable to go with qcom,apr-v2 and then list > the static services as child nodes. Am okay doing it, Only issue I have is the dsp state. If the dsp AVCS is not ready yet then we would be chatting on the service without the services being up on remote side? Chances of this seems to be slim but we need to confirm this before we make a call on this. > >> + >> + >> +- qcom,smd-channel: >> + Usage: required >> + Value type: <string> >> + Definition: standard SMD property specifying the SMD channel used for >> + communication with the APR on QDSP. >> + Should be "apr_audio_svc". > > This is not the only APR channel, but for apr itself this doesn't matter > and might as well be qcom,glink-channels; so perhaps we can omit this > from this document? Yes, I agree with you. will remove this in next version. > >> + Described in soc/qcom/qcom,smd.txt >> + >> += EXAMPLE >> +The following example represents a QDSP based sound card on a MSM8996 device >> +which uses apr as communication between Apps and QDSP. >> + >> + apr { >> + compatible = "qcom,apr-msm8996"; >> + qcom,smd-channels = "apr_audio_svc"; >> + }; > > Regards, > Bjorn >
diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt new file mode 100644 index 000000000000..4e93213ae98d --- /dev/null +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.txt @@ -0,0 +1,28 @@ +Qualcomm APR (Asynchronous Packet Router) binding + +This binding describes the Qualcomm APR. APR is a IPC protocol for +communication between Application processor and QDSP. APR is mainly +used for audio/voice services on the QDSP. + +- compatible: + Usage: required + Value type: <stringlist> + Definition: must be "qcom,apr-<SOC-NAME>" example: "qcom,apr-msm8996" + + +- qcom,smd-channel: + Usage: required + Value type: <string> + Definition: standard SMD property specifying the SMD channel used for + communication with the APR on QDSP. + Should be "apr_audio_svc". + Described in soc/qcom/qcom,smd.txt + += EXAMPLE +The following example represents a QDSP based sound card on a MSM8996 device +which uses apr as communication between Apps and QDSP. + + apr { + compatible = "qcom,apr-msm8996"; + qcom,smd-channels = "apr_audio_svc"; + };