On Sun, Oct 17, 2021 at 11:31 AM Stefan Wahren [off-list ref] wrote:

Hi Rob,

Am 17.10.21 um 17:29 schrieb Rob Herring:

quoted

On Sun, 17 Oct 2021 15:06:36 +0200, Stefan Wahren wrote:

quoted

This converts the VCHIQ bindings to YAML format.

Signed-off-by: Stefan Wahren <redacted>
---
 .../bindings/soc/bcm/brcm,bcm2835-vchiq.txt        | 17 --------
 .../bindings/soc/bcm/brcm,bcm2835-vchiq.yaml       | 47 ++++++++++++++++++++++
 2 files changed, 47 insertions(+), 17 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.txt
 create mode 100644 Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.yaml

Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/patch/1542230


mailbox@7e00b840: compatible: Additional items are not allowed ('brcm,bcm2835-vchiq' was unexpected)
      arch/arm64/boot/dts/broadcom/bcm2837-rpi-3-a-plus.dt.yaml
      arch/arm64/boot/dts/broadcom/bcm2837-rpi-3-b.dt.yaml
      arch/arm64/boot/dts/broadcom/bcm2837-rpi-3-b-plus.dt.yaml
      arch/arm64/boot/dts/broadcom/bcm2837-rpi-cm3-io3.dt.yaml
      arch/arm/boot/dts/bcm2836-rpi-2-b.dt.yaml
      arch/arm/boot/dts/bcm2837-rpi-3-a-plus.dt.yaml
      arch/arm/boot/dts/bcm2837-rpi-3-b.dt.yaml
      arch/arm/boot/dts/bcm2837-rpi-3-b-plus.dt.yaml
      arch/arm/boot/dts/bcm2837-rpi-cm3-io3.dt.yaml

mailbox@7e00b840: compatible: ['brcm,bcm2836-vchiq', 'brcm,bcm2835-vchiq'] is too long
      arch/arm64/boot/dts/broadcom/bcm2837-rpi-3-a-plus.dt.yaml
      arch/arm64/boot/dts/broadcom/bcm2837-rpi-3-b.dt.yaml
      arch/arm64/boot/dts/broadcom/bcm2837-rpi-3-b-plus.dt.yaml
      arch/arm64/boot/dts/broadcom/bcm2837-rpi-cm3-io3.dt.yaml
      arch/arm/boot/dts/bcm2836-rpi-2-b.dt.yaml
      arch/arm/boot/dts/bcm2837-rpi-3-a-plus.dt.yaml
      arch/arm/boot/dts/bcm2837-rpi-3-b.dt.yaml
      arch/arm/boot/dts/bcm2837-rpi-3-b-plus.dt.yaml
      arch/arm/boot/dts/bcm2837-rpi-cm3-io3.dt.yaml

i'm confused patch 1/2 ( "ARM: dts: bcm283x: Drop fallback compatible
for vchiq" ) of this series should prevent this warning. Was it applied
before running dtbs_check?

No, so you can ignore this.

Rob