From: Rob Herring <robh@kernel.org>
To: Srinivasa Rao Mandadapu <srivasam@codeaurora.org>
Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org,
bgoswami@codeaurora.org,
V Sujith Kumar Reddy <vsujithk@codeaurora.org>,
linux-arm-msm@vger.kernel.org, plai@codeaurora.org,
tiwai@suse.com, agross@kernel.org, lgirdwood@gmail.com,
broonie@kernel.org, srinivas.kandagatla@linaro.org,
rohitkr@codeaurora.org, bjorn.andersson@linaro.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v6 2/5] ASoC: dt-bindings: Add dt binding for lpass hdmi
Date: Wed, 23 Sep 2020 14:41:54 -0600 [thread overview]
Message-ID: <20200923204154.GA1263348@bogus> (raw)
In-Reply-To: <1600409084-29093-3-git-send-email-srivasam@codeaurora.org>
On Fri, Sep 18, 2020 at 11:34:41AM +0530, Srinivasa Rao Mandadapu wrote:
> From: V Sujith Kumar Reddy <vsujithk@codeaurora.org>
>
> Adds bindings for lpass hdmi interface
> which can support audio path over dp.
>
> Signed-off-by: Srinivasa Rao <srivasam@codeaurora.org>
> Signed-off-by: V Sujith Kumar Reddy <vsujithk@codeaurora.org>
> ---
> .../devicetree/bindings/sound/qcom,lpass-cpu.yaml | 74 +++++++++++++++-------
> 1 file changed, 52 insertions(+), 22 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml b/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml
> index 09c9bd2..f95ef70 100644
> --- a/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml
> +++ b/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml
> @@ -24,9 +24,10 @@ properties:
> - qcom,sc7180-lpass-cpu
>
> reg:
> - maxItems: 1
> + maxItems: 2
> description: LPAIF core registers
> -
> + reg-names:
> + maxItems: 2
> clocks:
> minItems: 3
> maxItems: 6
> @@ -36,15 +37,16 @@ properties:
> maxItems: 6
>
> interrupts:
> - maxItems: 1
> + maxItems: 2
> description: LPAIF DMA buffer interrupt
> -
> + interrupt-names:
> + maxItems: 2
> qcom,adsp:
> $ref: /schemas/types.yaml#/definitions/phandle
> description: Phandle for the audio DSP node
>
> iommus:
> - maxItems: 1
> + maxItems: 2
> description: Phandle to apps_smmu node with sid mask
>
> power-domains:
> @@ -60,10 +62,12 @@ properties:
> const: 0
>
> patternProperties:
> - "(^mi2s-[0-9a-f]$|mi2s)":
> + "^dai-link@[0-9a-f]$":
> type: object
> - description: Required properties for each DAI
> -
> + description: |
> + LPASS CPU dai node for each I2S device. Bindings of each node
> + depends on the specific driver providing the functionality and
> + properties.
> properties:
> reg:
> maxItems: 1
> @@ -85,9 +89,11 @@ patternProperties:
> required:
> - compatible
> - reg
> + - reg-names
> - clocks
> - clock-names
> - interrupts
> + - interrupt-names
> - '#sound-dai-cells'
>
> additionalProperties: false
> @@ -134,13 +140,32 @@ allOf:
> then:
> properties:
> clock-names:
> - items:
> - - const: pcnoc-sway-clk
> - - const: audio-core
> - - const: mclk0
> - - const: pcnoc-mport-clk
> - - const: mi2s-bit-clk0
> - - const: mi2s-bit-clk1
> + oneOf:
> + - items: #for I2S
> + - const: pcnoc-sway-clk
> + - const: audio-core
> + - const: mclk0
> + - const: pcnoc-mport-clk
> + - const: mi2s-bit-clk0
> + - const: mi2s-bit-clk1
> + - items: #for HDMI
> + - const: pcnoc-sway-clk
> + - const: audio-core
> + - const: pcnoc-mport-clk
> + reg-names:
> + anyOf:
> + - items: #for I2S and HDMI
> + - const: lpass-hdmiif
> + - const: lpass-lpaif
It would be a bit cleaner if you switch the order. Then you are just
adding on to the end.
Otherwise,
Reviewed-by: Rob Herring <robh@kernel.org>
> + - items: #for I2S
> + - const: lpass-lpaif
> + interrupt-names:
> + anyOf:
> + - items: #for I2S and HDMI
> + - const: lpass-irq-lpaif
> + - const: lpass-irq-hdmi
> + - items: #for I2S
> + - const: lpass-irq-lpaif
> required:
> - iommus
> - power-domains
> @@ -152,12 +177,15 @@ examples:
> soc {
> #address-cells = <2>;
> #size-cells = <2>;
> - lpass@62f00000 {
> + lpass@62d80000 {
> compatible = "qcom,sc7180-lpass-cpu";
>
> - reg = <0 0x62f00000 0 0x29000>;
> -
> - iommus = <&apps_smmu 0x1020 0>;
> + reg = <0 0x62d87000 0 0x68000>,
> + <0 0x62f00000 0 0x29000>;
> + reg-names = "lpass-hdmiif",
> + "lpass-lpaif";
> + iommus = <&apps_smmu 0x1020 0>,
> + <&apps_smmu 0x1032 0>;
> power-domains = <&lpass_hm 0>;
>
> clocks = <&gcc 131>,
> @@ -171,14 +199,16 @@ examples:
> "mclk0", "pcnoc-mport-clk",
> "mi2s-bit-clk0", "mi2s-bit-clk1";
>
> - interrupts = <0 160 1>;
> -
> + interrupts = <0 160 1>,
> + <0 268 1>;
> + interrupt-names = "lpass-irq-lpaif",
> + "lpass-irq-hdmi";
> #sound-dai-cells = <1>;
>
> #address-cells = <1>;
> #size-cells = <0>;
> /* Optional to set different MI2S SD lines */
> - mi2s-primary@0 {
> + dai-link@0 {
> reg = <MI2S_PRIMARY>;
> qcom,playback-sd-lines = <1>;
> qcom,capture-sd-lines = <0>;
> --
> Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc.,
> is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.
>
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Srinivasa Rao Mandadapu <srivasam@codeaurora.org>
Cc: agross@kernel.org, bjorn.andersson@linaro.org,
lgirdwood@gmail.com, broonie@kernel.org, plai@codeaurora.org,
bgoswami@codeaurora.org, perex@perex.cz, tiwai@suse.com,
srinivas.kandagatla@linaro.org, rohitkr@codeaurora.org,
linux-arm-msm@vger.kernel.org, alsa-devel@alsa-project.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
V Sujith Kumar Reddy <vsujithk@codeaurora.org>
Subject: Re: [PATCH v6 2/5] ASoC: dt-bindings: Add dt binding for lpass hdmi
Date: Wed, 23 Sep 2020 14:41:54 -0600 [thread overview]
Message-ID: <20200923204154.GA1263348@bogus> (raw)
In-Reply-To: <1600409084-29093-3-git-send-email-srivasam@codeaurora.org>
On Fri, Sep 18, 2020 at 11:34:41AM +0530, Srinivasa Rao Mandadapu wrote:
> From: V Sujith Kumar Reddy <vsujithk@codeaurora.org>
>
> Adds bindings for lpass hdmi interface
> which can support audio path over dp.
>
> Signed-off-by: Srinivasa Rao <srivasam@codeaurora.org>
> Signed-off-by: V Sujith Kumar Reddy <vsujithk@codeaurora.org>
> ---
> .../devicetree/bindings/sound/qcom,lpass-cpu.yaml | 74 +++++++++++++++-------
> 1 file changed, 52 insertions(+), 22 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml b/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml
> index 09c9bd2..f95ef70 100644
> --- a/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml
> +++ b/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml
> @@ -24,9 +24,10 @@ properties:
> - qcom,sc7180-lpass-cpu
>
> reg:
> - maxItems: 1
> + maxItems: 2
> description: LPAIF core registers
> -
> + reg-names:
> + maxItems: 2
> clocks:
> minItems: 3
> maxItems: 6
> @@ -36,15 +37,16 @@ properties:
> maxItems: 6
>
> interrupts:
> - maxItems: 1
> + maxItems: 2
> description: LPAIF DMA buffer interrupt
> -
> + interrupt-names:
> + maxItems: 2
> qcom,adsp:
> $ref: /schemas/types.yaml#/definitions/phandle
> description: Phandle for the audio DSP node
>
> iommus:
> - maxItems: 1
> + maxItems: 2
> description: Phandle to apps_smmu node with sid mask
>
> power-domains:
> @@ -60,10 +62,12 @@ properties:
> const: 0
>
> patternProperties:
> - "(^mi2s-[0-9a-f]$|mi2s)":
> + "^dai-link@[0-9a-f]$":
> type: object
> - description: Required properties for each DAI
> -
> + description: |
> + LPASS CPU dai node for each I2S device. Bindings of each node
> + depends on the specific driver providing the functionality and
> + properties.
> properties:
> reg:
> maxItems: 1
> @@ -85,9 +89,11 @@ patternProperties:
> required:
> - compatible
> - reg
> + - reg-names
> - clocks
> - clock-names
> - interrupts
> + - interrupt-names
> - '#sound-dai-cells'
>
> additionalProperties: false
> @@ -134,13 +140,32 @@ allOf:
> then:
> properties:
> clock-names:
> - items:
> - - const: pcnoc-sway-clk
> - - const: audio-core
> - - const: mclk0
> - - const: pcnoc-mport-clk
> - - const: mi2s-bit-clk0
> - - const: mi2s-bit-clk1
> + oneOf:
> + - items: #for I2S
> + - const: pcnoc-sway-clk
> + - const: audio-core
> + - const: mclk0
> + - const: pcnoc-mport-clk
> + - const: mi2s-bit-clk0
> + - const: mi2s-bit-clk1
> + - items: #for HDMI
> + - const: pcnoc-sway-clk
> + - const: audio-core
> + - const: pcnoc-mport-clk
> + reg-names:
> + anyOf:
> + - items: #for I2S and HDMI
> + - const: lpass-hdmiif
> + - const: lpass-lpaif
It would be a bit cleaner if you switch the order. Then you are just
adding on to the end.
Otherwise,
Reviewed-by: Rob Herring <robh@kernel.org>
> + - items: #for I2S
> + - const: lpass-lpaif
> + interrupt-names:
> + anyOf:
> + - items: #for I2S and HDMI
> + - const: lpass-irq-lpaif
> + - const: lpass-irq-hdmi
> + - items: #for I2S
> + - const: lpass-irq-lpaif
> required:
> - iommus
> - power-domains
> @@ -152,12 +177,15 @@ examples:
> soc {
> #address-cells = <2>;
> #size-cells = <2>;
> - lpass@62f00000 {
> + lpass@62d80000 {
> compatible = "qcom,sc7180-lpass-cpu";
>
> - reg = <0 0x62f00000 0 0x29000>;
> -
> - iommus = <&apps_smmu 0x1020 0>;
> + reg = <0 0x62d87000 0 0x68000>,
> + <0 0x62f00000 0 0x29000>;
> + reg-names = "lpass-hdmiif",
> + "lpass-lpaif";
> + iommus = <&apps_smmu 0x1020 0>,
> + <&apps_smmu 0x1032 0>;
> power-domains = <&lpass_hm 0>;
>
> clocks = <&gcc 131>,
> @@ -171,14 +199,16 @@ examples:
> "mclk0", "pcnoc-mport-clk",
> "mi2s-bit-clk0", "mi2s-bit-clk1";
>
> - interrupts = <0 160 1>;
> -
> + interrupts = <0 160 1>,
> + <0 268 1>;
> + interrupt-names = "lpass-irq-lpaif",
> + "lpass-irq-hdmi";
> #sound-dai-cells = <1>;
>
> #address-cells = <1>;
> #size-cells = <0>;
> /* Optional to set different MI2S SD lines */
> - mi2s-primary@0 {
> + dai-link@0 {
> reg = <MI2S_PRIMARY>;
> qcom,playback-sd-lines = <1>;
> qcom,capture-sd-lines = <0>;
> --
> Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc.,
> is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.
>
next prev parent reply other threads:[~2020-09-23 20:43 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-18 6:04 [PATCH v6 0/5] Qualcomm's lpass-hdmi ASoC driver to support audio over dp port Srinivasa Rao Mandadapu
2020-09-18 6:04 ` [PATCH v6 1/5] ASoC: Add sc7180-lpass binding header hdmi define Srinivasa Rao Mandadapu
2020-09-18 6:04 ` [PATCH v6 2/5] ASoC: dt-bindings: Add dt binding for lpass hdmi Srinivasa Rao Mandadapu
2020-09-23 20:41 ` Rob Herring [this message]
2020-09-23 20:41 ` Rob Herring
2020-09-18 6:04 ` [PATCH v6 3/5] Asoc:qcom:lpass-cpu:Update dts property read API Srinivasa Rao Mandadapu
2020-09-21 21:20 ` Srinivas Kandagatla
2020-09-22 11:08 ` Mark Brown
2020-09-22 11:08 ` Mark Brown
2020-09-22 11:22 ` Srinivas Kandagatla
2020-09-22 11:22 ` Srinivas Kandagatla
2020-09-22 11:43 ` Mark Brown
2020-09-22 11:43 ` Mark Brown
2020-09-22 12:53 ` Rohit Kumar
2020-09-22 12:53 ` Rohit Kumar
2020-09-22 13:57 ` Srinivas Kandagatla
2020-09-22 13:57 ` Srinivas Kandagatla
2020-09-18 6:04 ` [PATCH v6 4/5] ASoC: qcom: Add support for lpass hdmi driver Srinivasa Rao Mandadapu
2020-09-21 21:20 ` Srinivas Kandagatla
2020-09-23 8:41 ` Srinivasa Rao Mandadapu
2020-09-18 6:04 ` [PATCH v6 5/5] ASoC: qcom: sc7180: Add support for audio over DP Srinivasa Rao Mandadapu
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=20200923204154.GA1263348@bogus \
--to=robh@kernel.org \
--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=plai@codeaurora.org \
--cc=rohitkr@codeaurora.org \
--cc=srinivas.kandagatla@linaro.org \
--cc=srivasam@codeaurora.org \
--cc=tiwai@suse.com \
--cc=vsujithk@codeaurora.org \
/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: link
Be 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.