[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 at kernel.org> wrote:
>>
>> On Fri, 03 Dec 2021 21:37:37 +0530, Srinivasa Rao Mandadapu wrote:
>>> From: Srinivasa Rao Mandadapu <srivasam at 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 at codeaurora.org>
>>> Co-developed-by: Venkata Prasad Potturu <potturu at codeaurora.org>
>>> Signed-off-by: Venkata Prasad Potturu <potturu at 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 at 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
>