[Letux-kernel] [PATCH 10/18] MIPS: DTS: jz4780: fix uart dmas as reported by dtbscheck

[H. Nikolaus Schaller]

> Am 09.04.2022 um 15:16 schrieb Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>:
> 
> On 09/04/2022 15:07, H. Nikolaus Schaller wrote:
>> 
>> 
>>> Am 09.04.2022 um 13:18 schrieb Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>:
>>> 
>>> On 08/04/2022 20:37, H. Nikolaus Schaller wrote:
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10030000: 'dmas' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10030000: 'dma-names' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10031000: 'dmas' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10031000: 'dma-names' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10032000: 'dmas' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10032000: 'dma-names' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10033000: 'dmas' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10033000: 'dma-names' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10034000: 'dmas' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: serial at 10034000: 'dma-names' is a required property
>>>> 	From schema: Documentation/devicetree/bindings/serial/ingenic,uart.yaml
>>>> arch/mips/boot/dts/ingenic/ci20.dtb: i2c at 10050000: 'dmas' is a required property
>>> 
>>> All these warnings are the same two warnings...
>> 
>> See my earlier explanation that without them you can't verify by just reading commit message
>> and diff that all existing warnings have been addressed.
> 
> Which does not make sense and there is no need... Automation does it
> (see Rob's tools). Don't make human life more difficult...

Ok, you are right. If you apply this patch and then run dtbscheck again, there would be
a warning left over.

But may I honestly ask why you review the commits and read the commit message at all?
You could simply ignore it... And it would be easier for both of us to leave it completely
to Rob's tools :)

BR and thanks,
Nikolaus