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 6A644C433F5 for ; Wed, 25 May 2022 23:11:13 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:Message-Id:Cc:To :Subject:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=qCCWewFCElhJvLpUdlp4SDloQK1J1fMoS/Jp2Xstdks=; b=C7XXF7EvUa8oZHkv0nUVVrJxIf LYtLQ4yyNw8iRx+tGo3EhUIHRxBEPe8ViGFrnaeMg3gH6GlAbgBB/xfBKhibYprKLGWcVCxY5aHUW tvYXp5PDNPi6kGzYuDpbK8U3zOdXodxWJePnfsiBqtJSV26bpcDIHouTKF2fkUVNgzTHaWYlQ2bEf hviAF6+zh/FTsUA0e6r3IGtQbE5HuQJL4aLQW9P4hje2ce+fwmBH97E51Bh8pWz0iLtO3CxmVdjvS /ne3oYT+okUOmuv6RHjpl7hjSFZgX+N31jO1TEeXYPlIDPMDwyeLnof9WBeiN+/I8Qsy0IxHPZ1c9 yZ7RO+IA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nu09W-00CwEO-AF; Wed, 25 May 2022 23:10:54 +0000 Received: from aposti.net ([89.234.176.197]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nu09S-00CwD7-Mq for linux-mtd@lists.infradead.org; Wed, 25 May 2022 23:10:52 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=crapouillou.net; s=mail; t=1653520235; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=G94xiIYp4JaD6Z24NCSIZA2hrRe2VgRo6jmt+CqtRoA=; b=EvXdrcy6vMz4imZ/1j/gjWx4E9yc2khxmyi8Vah6S991CnSswjFfQZleYS9Cr0AYwbvlJ+ 6pXEwleMXHiZ5Wu2dKWzSpxzszkdh54BJ9NMN9Ir9rRZCLsZgopJo4iWEruq1ZVGr+Xy9e pGaNkYUA0kIthfhnzrRJuDqpyBhnLu0= Date: Thu, 26 May 2022 00:10:25 +0100 From: Paul Cercueil Subject: Re: [PATCH] dt-bindings: memory-controllers: ingenic: Split out child node properties To: Rob Herring Cc: Krzysztof Kozlowski , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-mtd@lists.infradead.org Message-Id: In-Reply-To: <20220525210140.2489866-1-robh@kernel.org> References: <20220525210140.2489866-1-robh@kernel.org> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220525_161050_943676_48B97AAA X-CRM114-Status: GOOD ( 14.74 ) X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1"; Format="flowed" Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org Hi Rob, Le mer., mai 25 2022 at 16:01:40 -0500, Rob Herring a = =E9crit : > Binding schemas which define child node properties such as memory > controllers with timing properties need a separate schema which can be > referenced from child device schemas. This is necessary for > unevaluatedProperties checks to work properly. > = > Move the ingenic,nemc child properties to its own file and reference > from ingenic,nand.yaml which describes a child NAND controller. > = > Signed-off-by: Rob Herring Acked-by: Paul Cercueil Cheers, -Paul > --- > .../ingenic,nemc-peripherals.yaml | 46 = > +++++++++++++++++++ > .../memory-controllers/ingenic,nemc.yaml | 32 ------------- > .../devicetree/bindings/mtd/ingenic,nand.yaml | 1 + > 3 files changed, 47 insertions(+), 32 deletions(-) > create mode 100644 = > Documentation/devicetree/bindings/memory-controllers/ingenic,nemc-periphe= rals.yaml > = > diff --git = > a/Documentation/devicetree/bindings/memory-controllers/ingenic,nemc-perip= herals.yaml = > b/Documentation/devicetree/bindings/memory-controllers/ingenic,nemc-perip= herals.yaml > new file mode 100644 > index 000000000000..b8ed52a44d57 > --- /dev/null > +++ = > b/Documentation/devicetree/bindings/memory-controllers/ingenic,nemc-perip= herals.yaml > @@ -0,0 +1,46 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: = > http://devicetree.org/schemas/memory-controllers/ingenic,nemc-peripherals= .yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Ingenic SoCs NAND / External Memory Controller (NEMC) = > devicetree bindings > + > +maintainers: > + - Paul Cercueil > + > +properties: > + reg: > + minItems: 1 > + maxItems: 255 > + > + ingenic,nemc-bus-width: > + $ref: /schemas/types.yaml#/definitions/uint32 > + enum: [8, 16] > + description: Specifies the bus width in bits. > + > + ingenic,nemc-tAS: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Address setup time in nanoseconds. > + > + ingenic,nemc-tAH: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Address hold time in nanoseconds. > + > + ingenic,nemc-tBP: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Burst pitch time in nanoseconds. > + > + ingenic,nemc-tAW: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Address wait time in nanoseconds. > + > + ingenic,nemc-tSTRV: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Static memory recovery time in nanoseconds. > + > +required: > + - reg > + > +additionalProperties: true > +... > diff --git = > a/Documentation/devicetree/bindings/memory-controllers/ingenic,nemc.yaml = > b/Documentation/devicetree/bindings/memory-controllers/ingenic,nemc.yaml > index 24f9e1982028..dd13a5106d6c 100644 > --- = > a/Documentation/devicetree/bindings/memory-controllers/ingenic,nemc.yaml > +++ = > b/Documentation/devicetree/bindings/memory-controllers/ingenic,nemc.yaml > @@ -39,38 +39,6 @@ properties: > patternProperties: > ".*@[0-9]+$": > type: object > - properties: > - reg: > - minItems: 1 > - maxItems: 255 > - > - ingenic,nemc-bus-width: > - $ref: /schemas/types.yaml#/definitions/uint32 > - enum: [8, 16] > - description: Specifies the bus width in bits. > - > - ingenic,nemc-tAS: > - $ref: /schemas/types.yaml#/definitions/uint32 > - description: Address setup time in nanoseconds. > - > - ingenic,nemc-tAH: > - $ref: /schemas/types.yaml#/definitions/uint32 > - description: Address hold time in nanoseconds. > - > - ingenic,nemc-tBP: > - $ref: /schemas/types.yaml#/definitions/uint32 > - description: Burst pitch time in nanoseconds. > - > - ingenic,nemc-tAW: > - $ref: /schemas/types.yaml#/definitions/uint32 > - description: Address wait time in nanoseconds. > - > - ingenic,nemc-tSTRV: > - $ref: /schemas/types.yaml#/definitions/uint32 > - description: Static memory recovery time in nanoseconds. > - > - required: > - - reg > = > required: > - compatible > diff --git a/Documentation/devicetree/bindings/mtd/ingenic,nand.yaml = > b/Documentation/devicetree/bindings/mtd/ingenic,nand.yaml > index 9de8ef6e59ca..8c272c842bfd 100644 > --- a/Documentation/devicetree/bindings/mtd/ingenic,nand.yaml > +++ b/Documentation/devicetree/bindings/mtd/ingenic,nand.yaml > @@ -11,6 +11,7 @@ maintainers: > = > allOf: > - $ref: nand-controller.yaml# > + - $ref: /schemas/memory-controllers/ingenic,nemc-peripherals.yaml# > = > properties: > compatible: > -- > 2.34.1 > = ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/