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 35075C4332F for ; Thu, 10 Nov 2022 13:50:37 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=/DJiq//graxnYfIElSYMk8eJYGfKibBnVx7QXkGx4cg=; b=bFXeaOAfjaPPc0 y2S6D1L5e9+uhdna6ByVTFkdL9HEn2TnsAaqk3aFfRarrCQ9pCmAAAqS3fXXqx+K2QSvaj+CFYO9D kAjqMAMp3T0DPm340s8DldTeYpeSiR6UBGn0Sq8pM8FxV3HcoynVF7ncV6b7Em3PK9jV2h5tVl0iv hPVJnfLY2yNw2vU/fTHpi50RntMzAe4hrmGkTDkQocsSWJ5aHx24oM3FaHWnV7BE4NLY00xF6dERu UfCWr6DnaqWeFqiTd9vZ/1cJ6x1c4hMycC2zNgHbbabu24RL4QwXZ148+nbV4s+IMJB1RJtQZQFop 7jPbHgaQBZ30jgQSdnGA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ot7vq-0069R5-Of; Thu, 10 Nov 2022 13:49:26 +0000 Received: from mail-oa1-f53.google.com ([209.85.160.53]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ot7vn-0069N4-Cl for linux-arm-kernel@lists.infradead.org; Thu, 10 Nov 2022 13:49:25 +0000 Received: by mail-oa1-f53.google.com with SMTP id 586e51a60fabf-13bd2aea61bso2255336fac.0 for ; Thu, 10 Nov 2022 05:49:18 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=sNqJFWW1l/RwQMBOBINJbE+k5PGilLqiAp/MdKz97eQ=; b=xh77OkJnmCQU2g452VV97+ZQ0PGsyFevA65LU98g907IR6hlt5h2QbbiTFXMfKyeVy yQOqWvKrXhZFmuCMx1LEdtxQPQSCjRWIAfnSpcy37kuAOY/u+FI2oDFuJZTSlIp92Ghe wJKqVEoIGf7QhjmhfcDAeYbQRr+MNOY05hunjPr2VqpKBHqtR+WDVK6KVpdcGU3i27mq 09l4yNsND0pTTJDnCWQ68I57u9rfucZahbT1QUbhNGvNxlZKQbs8/UH3NuOs7n6Lz7ju W6F/IyqSu0uGC7KGd6JrzoibrwCFAtoM0GVC+j+3X0UXUBz2iDY9bIjo/Jd0jLpIEmty 7qFg== X-Gm-Message-State: ACrzQf3GlvrRH/4rtsFIrI2wX0msTpXo1/K5zVS/foZvanV/V83mua1b nWuCLjn5ZAeTWmukcZkD5A== X-Google-Smtp-Source: AMsMyM5uF7ern7BL/3lKtN7O7XEt5ACzXfcdLIxfjhpgOjP+LI8pn3YJSIWJ48iw4kzzXWrQMBh+3Q== X-Received: by 2002:a05:6870:3488:b0:12c:19b0:f4bc with SMTP id n8-20020a056870348800b0012c19b0f4bcmr46679794oah.238.1668088157470; Thu, 10 Nov 2022 05:49:17 -0800 (PST) Received: from robh_at_kernel.org (66-90-144-107.dyn.grandenetworks.net. [66.90.144.107]) by smtp.gmail.com with ESMTPSA id x51-20020a9d37b6000000b00666a5b5d20fsm6348482otb.32.2022.11.10.05.49.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 10 Nov 2022 05:49:16 -0800 (PST) Received: (nullmailer pid 220983 invoked by uid 1000); Thu, 10 Nov 2022 13:49:18 -0000 Date: Thu, 10 Nov 2022 07:49:18 -0600 From: Rob Herring To: Miquel Raynal Cc: Krzysztof Kozlowski , devicetree@vger.kernel.org, Srinivas Kandagatla , linux-kernel@vger.kernel.org, Robert Marko , Luka Perkov , Thomas Petazzoni , Michael Walle , linux-arm-kernel@lists.infradead.org, Bartosz Golaszewski , Christian Eggers , Cory Tusar Subject: Re: [PATCH v3 4/6] dt-bindings: nvmem: add YAML schema for the sl28 vpd layout Message-ID: <20221110134918.GB3436769-robh@kernel.org> References: <20221104163833.1289857-1-miquel.raynal@bootlin.com> <20221104163833.1289857-5-miquel.raynal@bootlin.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20221104163833.1289857-5-miquel.raynal@bootlin.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221110_054923_453447_03C9C189 X-CRM114-Status: GOOD ( 22.99 ) 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 Fri, Nov 04, 2022 at 05:38:31PM +0100, Miquel Raynal wrote: > From: Michael Walle > > Add a schema for the NVMEM layout on Kontron's sl28 boards. > > Signed-off-by: Michael Walle > Signed-off-by: Miquel Raynal > --- > .../nvmem/layouts/kontron,sl28-vpd.yaml | 60 +++++++++++++++++++ > .../bindings/nvmem/layouts/nvmem-layout.yaml | 3 + > 2 files changed, 63 insertions(+) > create mode 100644 Documentation/devicetree/bindings/nvmem/layouts/kontron,sl28-vpd.yaml > > diff --git a/Documentation/devicetree/bindings/nvmem/layouts/kontron,sl28-vpd.yaml b/Documentation/devicetree/bindings/nvmem/layouts/kontron,sl28-vpd.yaml > new file mode 100644 > index 000000000000..44088c8b4153 > --- /dev/null > +++ b/Documentation/devicetree/bindings/nvmem/layouts/kontron,sl28-vpd.yaml > @@ -0,0 +1,60 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/nvmem/layouts/kontron,sl28-vpd.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: NVMEM layout of the Kontron SMARC-sAL28 vital product data > + > +maintainers: > + - Michael Walle > + > +description: > + The vital product data (VPD) of the sl28 boards contains a serial > + number and a base MAC address. The actual MAC addresses for the > + on-board ethernet devices are derived from this base MAC address by > + adding an offset. > + > +properties: > + compatible: > + const: kontron,sl28-vpd > + > + serial-number: > + type: object > + description: The board's serial number > + > + base-mac-address: > + type: object > + description: > + Base MAC address for all on-module network interfaces. The first > + argument of the phandle will be treated as an offset. > + > + properties: > + "#nvmem-cell-cells": > + const: 1 > + > + additionalProperties: false > + > +required: > + - compatible > + > +additionalProperties: false > + > +examples: > + - | > + otp-1 { > + compatible = "user-otp"; > + > + nvmem-layout { > + compatible = "kontron,sl28-vpd"; > + > + serial_number: serial-number { > + }; > + > + base_mac_address: base-mac-address { > + #nvmem-cell-cells = <1>; > + }; > + }; > + }; > + > +... > diff --git a/Documentation/devicetree/bindings/nvmem/layouts/nvmem-layout.yaml b/Documentation/devicetree/bindings/nvmem/layouts/nvmem-layout.yaml > index ecc7c37cbc1f..f64ea2fa362d 100644 > --- a/Documentation/devicetree/bindings/nvmem/layouts/nvmem-layout.yaml > +++ b/Documentation/devicetree/bindings/nvmem/layouts/nvmem-layout.yaml > @@ -17,6 +17,9 @@ description: | > define and might require dynamic reading of the NVMEM device in order to > perform their parsing. The nvmem-layout container is here to describe these. > > +oneOf: > + - $ref: kontron,sl28-vpd.yaml This is the other way around from how we normally structure things. Normally, the specific schema would reference the common/base schema. This works, though you will be applying the schema twice. Once here and then by matching on compatible string. Not a big deal as that happens fairly often, but a 'select: false' in kontron,sl28-vpd.yaml would prevent that. This way does more to enforce the overall structure of nodes. The one downside I see with it this way is nvmem-layout can't ever have common properties defined without listing them in each layout schema. In the end, I'm okay with either way. Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel