Re: [PATCH v8 09/10] ASoC: dt-bindings: Add SC7280 lpass cpu bindings

[Trilok Soni]

On 12/3/2021 3:38 PM, Rob Herring wrote:
> On Fri, Dec 3, 2021 at 5:34 PM Rob Herring <robh@xxxxxxxxxx> wrote:
> > On Fri, 03 Dec 2021 21:37:37 +0530, Srinivasa Rao Mandadapu wrote:
> > > From: Srinivasa Rao Mandadapu <srivasam@xxxxxxxxxxxxxx>
> > >
> > > 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@xxxxxxxxxxxxxx>
> > > Co-developed-by: Venkata Prasad Potturu <potturu@xxxxxxxxxxxxxx>
> > > Signed-off-by: Venkata Prasad Potturu <potturu@xxxxxxxxxxxxxx>
> > > ---
> > >   .../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.

codeaurora.com is definitely a wrong domain. It should be codeaurora.org 
instead.

> Rob