Re: [PATCH 03/18] MIPS: DTS: jz4780: fix tcu timer as reported by dtbscheck
From: Krzysztof Kozlowski
Date: Sat Apr 09 2022 - 07:11:59 EST
On 08/04/2022 20:37, H. Nikolaus Schaller wrote:
> arch/mips/boot/dts/ingenic/ci20.dtb: timer@10002000: compatible: 'oneOf' conditional failed, one must be fixed:
> ['ingenic,jz4780-tcu', 'ingenic,jz4770-tcu', 'simple-mfd'] is too long
> 'ingenic,jz4780-tcu' is not one of ['ingenic,jz4740-tcu', 'ingenic,jz4725b-tcu', 'ingenic,jz4760-tcu', 'ingenic,x1000-tcu']
> 'simple-mfd' was expected
> 'ingenic,jz4760-tcu' was expected
Trim it a bit...
> From schema: Documentation/devicetree/bindings/timer/ingenic,tcu.yaml
You need to explain this. You're changing the effective compatible of
the device and doing so based only on schema warning does not look
enough. Please write real reason instead of this fat warning, e.g. that
both devices are actually compatible and this has no real effect except
schema checks.
Best regards,
Krzysztof