[5.10.y-cip,31/39] dt-bindings: Fix errors in 'if' schemas

Message ID	20220330185454.10887-32-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive)
State	Accepted
Delegated to:	Pavel Machek
Headers	show Return-Path: <prabhakar.mahadev-lad.rj@bp.renesas.com> ip: 210.160.252.172, mailfrom: prabhakar.mahadev-lad.rj@bp.renesas.com) From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> To: cip-dev@lists.cip-project.org, Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>, Pavel Machek <pavel@denx.de> Cc: Biju Das <biju.das.jz@bp.renesas.com> Subject: [PATCH 5.10.y-cip 31/39] dt-bindings: Fix errors in 'if' schemas Date: Wed, 30 Mar 2022 19:54:46 +0100 Message-Id: <20220330185454.10887-32-prabhakar.mahadev-lad.rj@bp.renesas.com> In-Reply-To: <20220330185454.10887-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20220330185454.10887-1-prabhakar.mahadev-lad.rj@bp.renesas.com>
Series	Add SD/eMMC support for Renesas RZ/G2L SoC \| expand [5.10.y-cip,00/39] Add SD/eMMC support for Renesas RZ/G2L SoC [5.10.y-cip,01/39] mmc: renesas_sdhi: probe into TMIO after SCC parameters have been setup [5.10.y-cip,02/39] mmc: renesas_sdhi: populate SCC pointer at the proper place [5.10.y-cip,03/39] mmc: renesas_sdhi: simplify reset routine a little [5.10.y-cip,04/39] mmc: renesas_sdhi: clear TAPEN when resetting, too [5.10.y-cip,05/39] mmc: renesas_sdhi: merge the SCC reset functions [5.10.y-cip,06/39] mmc: renesas_sdhi: remove superfluous SCLKEN [5.10.y-cip,07/39] mmc: renesas_sdhi: improve HOST_MODE usage [5.10.y-cip,08/39] mmc: renesas_sdhi: don't hardcode SDIF values [5.10.y-cip,09/39] mmc: renesas_sdhi: sort includes [5.10.y-cip,10/39] mmc: tmio: set max_busy_timeout [5.10.y-cip,11/39] mmc: tmio: add hook for custom busy_wait calculation [5.10.y-cip,12/39] mmc: renesas_sdhi: populate hook for longer busy_wait [5.10.y-cip,13/39] mmc: renesas_internal_dmac: add pre_req and post_req support [5.10.y-cip,14/39] mmc: tmio: Add data timeout error detection [5.10.y-cip,15/39] mmc: renesas_sdhi: Add a condition of cmd/data timeout for retune [5.10.y-cip,16/39] mmc: tmio: support custom irq masks [5.10.y-cip,17/39] mmc: renesas_sdhi: use custom mask for TMIO_MASK_ALL [5.10.y-cip,18/39] mmc: tmio: abort DMA before reset [5.10.y-cip,19/39] mmc: tmio: restore bus width when resetting [5.10.y-cip,20/39] mmc: renesas_sdhi: break SCC reset into own function [5.10.y-cip,21/39] mmc: renesas_sdhi: do hard reset if possible [5.10.y-cip,22/39] mmc: tmio: always flag retune when resetting and a card is present [5.10.y-cip,23/39] mmc: tmio: always restore irq register [5.10.y-cip,24/39] mmc: tmio: reenable card irqs after the reset callback [5.10.y-cip,25/39] mmc: tmio: reinit card irqs in reset routine [5.10.y-cip,26/39] clk: renesas: rzg2l: Add SDHI clk mux support [5.10.y-cip,27/39] clk: renesas: rzg2l: Add missing kerneldoc for resets [5.10.y-cip,28/39] clk: renesas: rzg2l: Check return value of pm_genpd_init() [5.10.y-cip,29/39] clk: renesas: rzg2l: propagate return value of_genpd_add_provider_simple() [5.10.y-cip,30/39] clk: renesas: r9a07g044: Add SDHI clock and reset entries [5.10.y-cip,31/39] dt-bindings: Fix errors in 'if' schemas [5.10.y-cip,32/39] dt-bindings: Drop redundant minItems/maxItems [5.10.y-cip,33/39] dt-bindings: mmc: renesas,sdhi: Fix dtbs-check warning [5.10.y-cip,34/39] dt-bindings: mmc: renesas,sdhi: Document RZ/G2L bindings [5.10.y-cip,35/39] dt-bindings: mmc: renesas,sdhi: Add optional SDnH clock [5.10.y-cip,36/39] dt-bindings: mmc: renesas,sdhi: Rename RZ/G2L clocks [5.10.y-cip,37/39] arm64: dts: renesas: r9a07g044: Add SDHI nodes [5.10.y-cip,38/39] arm64: dts: renesas: rzg2l-smarc-som: Enable eMMC on SMARC platform [5.10.y-cip,39/39] arm64: dts: renesas: rzg2l-smarc: Enable microSD on SMARC platform

Message ID

20220330185454.10887-32-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive)

State

Accepted

Delegated to:

Pavel Machek

Headers

From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
To: cip-dev@lists.cip-project.org,
	Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>,
	Pavel Machek <pavel@denx.de>
Cc: Biju Das <biju.das.jz@bp.renesas.com>
Subject: [PATCH 5.10.y-cip 31/39] dt-bindings: Fix errors in 'if' schemas
Date: Wed, 30 Mar 2022 19:54:46 +0100
Message-Id: <20220330185454.10887-32-prabhakar.mahadev-lad.rj@bp.renesas.com>
In-Reply-To: <20220330185454.10887-1-prabhakar.mahadev-lad.rj@bp.renesas.com>
References: <20220330185454.10887-1-prabhakar.mahadev-lad.rj@bp.renesas.com>

Series

Add SD/eMMC support for Renesas RZ/G2L SoC | expand

Commit Message

Lad Prabhakar March 30, 2022, 6:54 p.m. UTC

From: Rob Herring <robh@kernel.org>

commit 9183908e70e913d2db052588172968da55d82af5 upstream.

Properties in if/then schemas weren't getting checked by the meta-schemas.
Enabling meta-schema checks finds several errors.

The use of an 'items' schema (as opposed to the list form) is wrong in
some cases as it applies to all entries. 'contains' is the correct schema
to use in the case of multiple entries.

Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Chen-Yu Tsai <wens@csie.org>
Cc: Eric Anholt <eric@anholt.net>
Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Ray Jui <rjui@broadcom.com>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Ulf Hansson <ulf.hansson@linaro.org>
Cc: Kishon Vijay Abraham I <kishon@ti.com>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: linux-crypto@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linux-leds@vger.kernel.org
Cc: linux-mmc@vger.kernel.org
Cc: linux-gpio@vger.kernel.org
Acked-by: Maxime Ripard <mripard@kernel.org>
Acked-By: Vinod Koul <vkoul@kernel.org>
Reviewed-by: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Rob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/20210202205544.24812-3-robh@kernel.org
[PL: Only applied changes to renesas,sdhi.yaml]
Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
index 6bbf29b5c239..6c13703b31db 100644
--- a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
+++ b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
@@ -123,7 +123,7 @@  required:
 if:
   properties:
     compatible:
-      items:
+      contains:
         enum:
           - renesas,sdhi-r7s72100
           - renesas,sdhi-r7s9210

[5.10.y-cip,31/39] dt-bindings: Fix errors in 'if' schemas

Commit Message

Patch