From: Rob Herring <robh+dt@kernel.org>
To: Srinivasa Rao Mandadapu <srivasam@codeaurora.com>,
Srinivasa Rao Mandadapu <srivasam@codeaurora.org>
Cc: alsa-devel@alsa-project.org, bjorn.andersson@linaro.org,
judyhsiao@chromium.org, tiwai@suse.com,
linux-kernel@vger.kernel.org, bgoswami@codeaurora.org,
agross@kernel.org, plai@codeaurora.org,
Venkata Prasad Potturu <potturu@codeaurora.org>,
perex@perex.cz, devicetree@vger.kernel.org, lgirdwood@gmail.com,
linux-arm-msm@vger.kernel.org, swboyd@chromium.org,
rohitkr@codeaurora.org, broonie@kernel.org,
srinivas.kandagatla@linaro.org
Subject: Re: [PATCH v8 09/10] ASoC: dt-bindings: Add SC7280 lpass cpu bindings
Date: Fri, 3 Dec 2021 17:38:07 -0600 [thread overview]
Message-ID: <CAL_JsqKf4Y84+_PQqhwMEEiJNrR92urMUYSqYTEU0_c7fYnyhQ@mail.gmail.com> (raw)
In-Reply-To: <1638574455.248037.1043006.nullmailer@robh.at.kernel.org>
On Fri, Dec 3, 2021 at 5:34 PM Rob Herring <robh@kernel.org> wrote:
>
> On Fri, 03 Dec 2021 21:37:37 +0530, Srinivasa Rao Mandadapu wrote:
> > From: Srinivasa Rao Mandadapu <srivasam@codeaurora.org>
> >
> > Add bindings for sc7280 lpass cpu driver which supports
> > audio over i2s based speaker, soundwire based headset, msm dmics
> > and HDMI Port.
> >
> > Signed-off-by: Srinivasa Rao Mandadapu <srivasam@codeaurora.org>
> > Co-developed-by: Venkata Prasad Potturu <potturu@codeaurora.org>
> > Signed-off-by: Venkata Prasad Potturu <potturu@codeaurora.org>
> > ---
> > .../devicetree/bindings/sound/qcom,lpass-cpu.yaml | 70 +++++++++++++++++++---
> > 1 file changed, 62 insertions(+), 8 deletions(-)
> >
>
> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.example.dt.yaml: lpass@62d80000: reg: [[0, 1658351616, 0, 425984], [0, 1659895808, 0, 167936]] is too short
> From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml
Are you running the checks before you send out your patches. Because
it seems like you keep sending things with the same errors.
If there's errors, I'm not going to review this. If you need help
getting it to work, then ask.
And what's with your email setup? codeaurora.com bounces.
Rob
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh+dt@kernel.org>
To: Srinivasa Rao Mandadapu <srivasam@codeaurora.com>,
Srinivasa Rao Mandadapu <srivasam@codeaurora.org>
Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org,
bgoswami@codeaurora.org, lgirdwood@gmail.com,
Venkata Prasad Potturu <potturu@codeaurora.org>,
linux-arm-msm@vger.kernel.org, plai@codeaurora.org,
linux-kernel@vger.kernel.org, swboyd@chromium.org,
tiwai@suse.com, agross@kernel.org, rohitkr@codeaurora.org,
broonie@kernel.org, bjorn.andersson@linaro.org,
judyhsiao@chromium.org
Subject: Re: [PATCH v8 09/10] ASoC: dt-bindings: Add SC7280 lpass cpu bindings
Date: Fri, 3 Dec 2021 17:38:07 -0600 [thread overview]
Message-ID: <CAL_JsqKf4Y84+_PQqhwMEEiJNrR92urMUYSqYTEU0_c7fYnyhQ@mail.gmail.com> (raw)
In-Reply-To: <1638574455.248037.1043006.nullmailer@robh.at.kernel.org>
On Fri, Dec 3, 2021 at 5:34 PM Rob Herring <robh@kernel.org> wrote:
>
> On Fri, 03 Dec 2021 21:37:37 +0530, Srinivasa Rao Mandadapu wrote:
> > From: Srinivasa Rao Mandadapu <srivasam@codeaurora.org>
> >
> > Add bindings for sc7280 lpass cpu driver which supports
> > audio over i2s based speaker, soundwire based headset, msm dmics
> > and HDMI Port.
> >
> > Signed-off-by: Srinivasa Rao Mandadapu <srivasam@codeaurora.org>
> > Co-developed-by: Venkata Prasad Potturu <potturu@codeaurora.org>
> > Signed-off-by: Venkata Prasad Potturu <potturu@codeaurora.org>
> > ---
> > .../devicetree/bindings/sound/qcom,lpass-cpu.yaml | 70 +++++++++++++++++++---
> > 1 file changed, 62 insertions(+), 8 deletions(-)
> >
>
> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.example.dt.yaml: lpass@62d80000: reg: [[0, 1658351616, 0, 425984], [0, 1659895808, 0, 167936]] is too short
> From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.yaml
Are you running the checks before you send out your patches. Because
it seems like you keep sending things with the same errors.
If there's errors, I'm not going to review this. If you need help
getting it to work, then ask.
And what's with your email setup? codeaurora.com bounces.
Rob
next prev parent reply other threads:[~2021-12-03 23:38 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-03 16:07 [PATCH v8 00/10] Add support for audio on SC7280 based targets Srinivasa Rao Mandadapu
2021-12-03 16:07 ` [PATCH v8 01/10] ASoC: qcom: SC7280: Update config for building codec dma drivers Srinivasa Rao Mandadapu
2021-12-03 16:07 ` Srinivasa Rao Mandadapu
2021-12-03 16:07 ` [PATCH v8 02/10] ASoC: qcom: Move lpass_pcm_data structure to lpass header Srinivasa Rao Mandadapu
2021-12-03 16:07 ` Srinivasa Rao Mandadapu
2021-12-03 16:07 ` [PATCH v8 03/10] ASoC: qcom: lpass: Add dma fields for codec dma lpass interface Srinivasa Rao Mandadapu
2021-12-03 16:07 ` Srinivasa Rao Mandadapu
2021-12-03 16:07 ` [PATCH v8 04/10] ASoC: qcom: Add helper function to get dma control and lpaif handle Srinivasa Rao Mandadapu
2021-12-03 16:07 ` Srinivasa Rao Mandadapu
2021-12-03 16:07 ` [PATCH v8 05/10] ASoC: qcom: Add register definition for codec rddma and wrdma Srinivasa Rao Mandadapu
2021-12-03 16:07 ` Srinivasa Rao Mandadapu
2021-12-03 16:07 ` [PATCH v8 06/10] ASoC: qcom: Add regmap config support for codec dma driver Srinivasa Rao Mandadapu
2021-12-03 16:07 ` Srinivasa Rao Mandadapu
2021-12-03 16:07 ` [PATCH v8 07/10] ASoC: qcom: Add " Srinivasa Rao Mandadapu
2021-12-03 16:07 ` Srinivasa Rao Mandadapu
2021-12-03 16:07 ` [PATCH v8 08/10] ASoC: qcom: Add lpass CPU driver for codec dma control Srinivasa Rao Mandadapu
2021-12-03 16:07 ` Srinivasa Rao Mandadapu
2021-12-03 16:07 ` [PATCH v8 09/10] ASoC: dt-bindings: Add SC7280 lpass cpu bindings Srinivasa Rao Mandadapu
2021-12-03 16:07 ` Srinivasa Rao Mandadapu
2021-12-03 23:34 ` Rob Herring
2021-12-03 23:34 ` Rob Herring
2021-12-03 23:38 ` Rob Herring [this message]
2021-12-03 23:38 ` Rob Herring
2021-12-03 23:40 ` Trilok Soni
2021-12-03 23:40 ` Trilok Soni
2021-12-04 7:59 ` Srinivasa Rao Mandadapu
2021-12-04 7:59 ` Srinivasa Rao Mandadapu
2021-12-04 17:50 ` Rob Herring
2021-12-04 17:50 ` Rob Herring
2021-12-03 16:07 ` [PATCH v8 10/10] ASoC: qcom: lpass-sc7280: Add platform driver for lpass audio Srinivasa Rao Mandadapu
2021-12-03 16:07 ` 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=CAL_JsqKf4Y84+_PQqhwMEEiJNrR92urMUYSqYTEU0_c7fYnyhQ@mail.gmail.com \
--to=robh+dt@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=judyhsiao@chromium.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=perex@perex.cz \
--cc=plai@codeaurora.org \
--cc=potturu@codeaurora.org \
--cc=rohitkr@codeaurora.org \
--cc=srinivas.kandagatla@linaro.org \
--cc=srivasam@codeaurora.com \
--cc=srivasam@codeaurora.org \
--cc=swboyd@chromium.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: 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.