MIME-Version: 1.0
References: <1638547658-22032-1-git-send-email-srivasam@codeaurora.com>
 <1638547658-22032-10-git-send-email-srivasam@codeaurora.com> <1638574455.248037.1043006.nullmailer@robh.at.kernel.org>
In-Reply-To: <1638574455.248037.1043006.nullmailer@robh.at.kernel.org>
From:   Rob Herring <robh+dt@kernel.org>
Date:   Fri, 3 Dec 2021 17:38:07 -0600
Message-ID: <CAL_JsqKf4Y84+_PQqhwMEEiJNrR92urMUYSqYTEU0_c7fYnyhQ@mail.gmail.com>
Subject: Re: [PATCH v8 09/10] ASoC: dt-bindings: Add SC7280 lpass cpu bindings
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
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

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