From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7EEACC433F5 for ; Thu, 9 Dec 2021 10:07:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Cc:To:Subject:Message-ID:Date:From: In-Reply-To:References:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=cj4lxrAYlEhBJ5b2xFbNLmztgs/xwyLE8xj8LQTitt8=; b=pvo76X32jtu2CW 5GYKLka0XluiIAqCkhK5g1pIs+bqvJT70hjMmUhgMwPAx8g/kbzA71OHU2WqwJ7K4TG/TIR9hTmmg 3rGnxAxXt1k/0dj4qYxdGQygQGslyulxL7lvOP+av/ITfSEROWJVC/xh48PKl8qI7NKbECrlbaJzj aJut9w7Gi4ktSGps5UAw+RvEbjBCwsx6CYD+ku70wVesDWEhjPEy8WeI9+/JhDjHSVGnhyGAc5eL4 ut+sDxKBnTQ15lbvhVzMSDkuUjPTmgNg2OUk92kZ2pPOxa63y8d+9ViKDe8DZSGg4rUSdEFZFRgwu D408H2fHsYC7OoaCaHHg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mvGJd-00FWbo-Rn; Thu, 09 Dec 2021 10:06:18 +0000 Received: from mail-lf1-x130.google.com ([2a00:1450:4864:20::130]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mvGJY-00FWZb-HO for linux-arm-kernel@lists.infradead.org; Thu, 09 Dec 2021 10:06:14 +0000 Received: by mail-lf1-x130.google.com with SMTP id bu18so11119959lfb.0 for ; Thu, 09 Dec 2021 02:06:11 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=F1owymgiCbwjtMOY3GwDKTTRv4aRJ4ddqlYjVacAxeY=; b=IEAlki6D4NihEbGixc0ZB8q1mWKYeVyBm450yjRD6H+nFTePjftgbNP1DSbjKJC/wn aA6G1S7zyTmWIk2wnFu1f0vt67RzKqsiulULcMDgUC+z69q+XtVhP3Eu0UC0yezAwjNB vX82i6L0vOPMOqpqTVSvYYCMeFCK7iTHxJMGDkkT5ZH+GnHs9SODYR5MYMhq1DGi4GzO jzkp592RVDVKnCu1VIpNL7vqWdE78LfoQ/N7ed/l18v1uQfKPEjZBRH2eRUjtNTKCBPA nWa9O+uPUfVM+lsrddwkRKNoYhpf4nu+XyMW4JWKq2CaEyoOmsYKC8jQgxIh6QFH8/oT jcsw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=F1owymgiCbwjtMOY3GwDKTTRv4aRJ4ddqlYjVacAxeY=; b=kvP7bZZ6MZiuALy+FIRfdRsdTqOIA844X/aYP22pB7bJBFUl/ND2rQa7rKsd6YGngy PRVptkMkOpVT85MBwdTE6PXXEyIQuGgKNiikkX6eXjLYRZNhqM4ELaYucAP6apWma0YL FYBWM2gavgABxsriVe2pdKIVHJp0UlwQqkdSRszt9Hi8CQc4flrBCvUCKaoCNQP03XFY +SUb71jlLrw2/m83JU3arH3twNNIgeqxN1nA7NWZJT9oWF/fXH3OstvGgcAzQ/CG9sKD DKTi1eNIF4cPK2nKjLduaXlI/sjuJBJDarF7K9zllsHgB9Sn1h3o+xIkRce/yG+1o/rs 3gpg== X-Gm-Message-State: AOAM532SfPK5SfmAi6k9hvur2Fj+0aBZDj3sWTdWzzX8DnXBhB3CBr7Z XXigpN27QWk16GRhjw0gP/cTim7D+Zmb1a/AppqR+w== X-Google-Smtp-Source: ABdhPJwOXRN/8lU2AYgTrJejBXDpm4hFI6pejaLy27VCQ/luFqzDGRy/0PN4sedVg8qGNEQznzbU3QCdxNwh1mp0znQ= X-Received: by 2002:a05:6512:10c4:: with SMTP id k4mr4875603lfg.373.1639044369640; Thu, 09 Dec 2021 02:06:09 -0800 (PST) MIME-Version: 1.0 References: <20211206174201.2297265-1-robh@kernel.org> In-Reply-To: <20211206174201.2297265-1-robh@kernel.org> From: Ulf Hansson Date: Thu, 9 Dec 2021 11:05:33 +0100 Message-ID: Subject: Re: [PATCH] dt-bindings: mmc: Add missing properties used in examples To: Rob Herring Cc: Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Chaotian Jing , Matthias Brugger , Kunihiko Hayashi , Masami Hiramatsu , Adrian Hunter , Linus Walleij , Wenbin Mei , Masahiro Yamada , devicetree@vger.kernel.org, Thierry Reding , linux-mmc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211209_020612_619373_5FF6F14C X-CRM114-Status: GOOD ( 23.48 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, 6 Dec 2021 at 18:42, Rob Herring wrote: > > When 'unevaluatedProperties' support is enabled, the following warnings > are generated in the mmc bindings: > > Documentation/devicetree/bindings/mmc/mtk-sd.example.dt.yaml: mmc@11230000: Unevaluated properties are not allowed ('reg', 'interrupts' were unexpected) > Documentation/devicetree/bindings/mmc/sdhci-am654.example.dt.yaml: mmc@4f80000: Unevaluated properties are not allowed ('sdhci-caps-mask' was unexpected) > Documentation/devicetree/bindings/mmc/socionext,uniphier-sd.example.dt.yaml: mmc@5a400000: Unevaluated properties are not allowed ('dma-names', 'dmas' were unexpected) > Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@80126000: Unevaluated properties are not allowed ('dmas', 'dma-names' were unexpected) > Documentation/devicetree/bindings/mmc/arasan,sdhci.example.dt.yaml: mmc@80420000: Unevaluated properties are not allowed ('resets' was unexpected) > Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@52007000: Unevaluated properties are not allowed ('interrupt-names' was unexpected) > Documentation/devicetree/bindings/clock/imx8qxp-lpcg.example.dt.yaml: mmc@5b010000: Unevaluated properties are not allowed ('power-domains' was unexpected) > > Add the missing properties as necessary. For pl18x, drop interrupt-names > as there isn't any use of it when there are 2 interrupts. > > Cc: Ulf Hansson > Cc: Shawn Guo > Cc: Sascha Hauer > Cc: Pengutronix Kernel Team > Cc: Fabio Estevam > Cc: NXP Linux Team > Cc: Chaotian Jing > Cc: Matthias Brugger > Cc: Kunihiko Hayashi > Cc: Masami Hiramatsu > Cc: Adrian Hunter > Cc: Linus Walleij > Cc: Wenbin Mei > Cc: Masahiro Yamada > Cc: linux-mmc@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-mediatek@lists.infradead.org > Signed-off-by: Rob Herring Applied for next, thanks! Kind regards Uffe > --- > Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml | 3 +++ > Documentation/devicetree/bindings/mmc/arm,pl18x.yaml | 9 ++++++++- > Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml | 3 +++ > Documentation/devicetree/bindings/mmc/mtk-sd.yaml | 6 ++++++ > Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 2 ++ > .../devicetree/bindings/mmc/socionext,uniphier-sd.yaml | 6 ++++++ > 6 files changed, 28 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml b/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml > index de6f076e0ece..83be9e93d221 100644 > --- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml > +++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml > @@ -118,6 +118,9 @@ properties: > phy-names: > const: phy_arasan > > + resets: > + maxItems: 1 > + > arasan,soc-ctl-syscon: > $ref: /schemas/types.yaml#/definitions/phandle > description: > diff --git a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml > index 47595cb483be..fbc866d9bb2f 100644 > --- a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml > +++ b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml > @@ -60,6 +60,14 @@ properties: > minItems: 1 > maxItems: 2 > > + dmas: > + maxItems: 2 > + > + dma-names: > + items: > + - const: rx > + - const: tx > + > power-domains: true > > resets: > @@ -213,7 +221,6 @@ examples: > arm,primecell-periphid = <0x10153180>; > reg = <0x52007000 0x1000>; > interrupts = <49>; > - interrupt-names = "cmd_irq"; > clocks = <&rcc 0>; > clock-names = "apb_pclk"; > resets = <&rcc 1>; > diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml > index 19621a2f8beb..8d5cef0d3039 100644 > --- a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml > +++ b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml > @@ -116,6 +116,9 @@ properties: > - const: ahb > - const: per > > + power-domains: > + maxItems: 1 > + > pinctrl-names: > oneOf: > - minItems: 3 > diff --git a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml > index 82768a807294..faf89b0c918f 100644 > --- a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml > +++ b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml > @@ -36,6 +36,9 @@ properties: > - const: mediatek,mt8195-mmc > - const: mediatek,mt8183-mmc > > + reg: > + maxItems: 1 > + > clocks: > description: > Should contain phandle for the clock feeding the MMC controller. > @@ -62,6 +65,9 @@ properties: > - const: axi_cg > - const: ahb_cg > > + interrupts: > + maxItems: 1 > + > pinctrl-names: > items: > - const: default > diff --git a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml > index 224303f5b913..9fbf16b3bc8d 100644 > --- a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml > +++ b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml > @@ -48,6 +48,8 @@ properties: > - const: clk_ahb > - const: clk_xin > > + sdhci-caps-mask: true > + > # PHY output tap delays: > # Used to delay the data valid window and align it to the sampling clock. > # Binding needs to be provided for each supported speed mode otherwise the > diff --git a/Documentation/devicetree/bindings/mmc/socionext,uniphier-sd.yaml b/Documentation/devicetree/bindings/mmc/socionext,uniphier-sd.yaml > index 56f9ff12742d..a586fad0a46b 100644 > --- a/Documentation/devicetree/bindings/mmc/socionext,uniphier-sd.yaml > +++ b/Documentation/devicetree/bindings/mmc/socionext,uniphier-sd.yaml > @@ -26,6 +26,12 @@ properties: > clocks: > maxItems: 1 > > + dmas: > + maxItems: 1 > + > + dma-names: > + const: rx-tx > + > reset-names: > description: | > There are three reset signals at maximum > -- > 2.32.0 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel