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 X-Spam-Level: X-Spam-Status: No, score=-14.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1FF5BC433C1 for ; Sat, 27 Mar 2021 16:32:10 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 9DCE8617ED for ; Sat, 27 Mar 2021 16:32:09 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9DCE8617ED Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; 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=90G20stTdeQIX7vhzAJikJrVRqPoJloDCkaGspXKXsU=; b=BPIW7XMCgBVXJoFC1s66M/NBK Hsv/W0Pn6kbcej102BnvfcaGA5gc99mZYjFskwUrwbB+fYVTbGX6AoLvlip8cVF6CEeJnOgzsyzRT bTw066ExKakERD7kH14gB3lp23CLD2IEjUBMQVyF4G7fSxmg8ABPil6Sne8+mIEJRG11vIN/dnh6P CWsLtk9fUy8m1E/93jj+YCzONt5PfsYvkjada4TdnakBkeSQY1fBuneC2NVAtaIgaXq+NBrgfiU6d +pn3vo7AuQ+xxHgWWGsv89i2gzJCkBxN9UT2qqQLxoy5pVSLKwxi2tqbqFs/oeKZ+5gXDe4YEwWlG qfsWYgAzg==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lQBpN-005GdL-Ll; Sat, 27 Mar 2021 16:30:21 +0000 Received: from mail-ot1-f53.google.com ([209.85.210.53]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lQBpJ-005Gco-UQ; Sat, 27 Mar 2021 16:30:20 +0000 Received: by mail-ot1-f53.google.com with SMTP id w31-20020a9d36220000b02901f2cbfc9743so8173457otb.7; Sat, 27 Mar 2021 09:30:12 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=8Rp5YRMAuX+nFksmAgBJTg7ZHDI2BYdNvFQYEt8ALSo=; b=lU9JHzzV449sqY2Jm4S0mYqOLPC35Os3diWjXtedNFRlhjtT7SRXs6Z1HA20iLtRfx 1hJuyWqJd8GUTE4dQcRisXGWcydzPHlSN40qwSJ92jvRiMF4otp7jSl5Kolgb+RVAYzi PWU2KqxDzWvd9cZkLzvy0fyR6ncMZxMRcH2hHtxBVm52s4r7pNjr0KQpzVgZg9xqTjUB 0odCe9a3QOimMdqz0FyH0cW6awAsRtebL1kL6/6iBNUzztWlSKst6wZ2wEXHohHoJOxb J0qDYrydrNdAK5PMRQalm0V6rRqk0E9ub2B+0iaknK/UjPdZXWdrOkvgOKNS6Z2sPBQI ElNw== X-Gm-Message-State: AOAM5311oVHOIrvR3bzuZaCtoDTDRCRMqJx1CFfaeW0Xe0BmDugZSwGv yRwMi8r5BhqM3vycgp5+og== X-Google-Smtp-Source: ABdhPJxUWXW6EjXIzgfB/hlFp/ntAvacXisLN585V8F7oaTeQISeOrUptrzfqbveJrvCbvFRXL2CKA== X-Received: by 2002:a05:6830:1e3b:: with SMTP id t27mr9761664otr.209.1616862611114; Sat, 27 Mar 2021 09:30:11 -0700 (PDT) Received: from robh.at.kernel.org ([172.58.99.140]) by smtp.gmail.com with ESMTPSA id w199sm2474969oif.41.2021.03.27.09.30.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 27 Mar 2021 09:30:10 -0700 (PDT) Received: (nullmailer pid 202941 invoked by uid 1000); Sat, 27 Mar 2021 16:30:06 -0000 Date: Sat, 27 Mar 2021 10:30:06 -0600 From: Rob Herring To: Cristian Ciocaltea Cc: Andreas =?iso-8859-1?Q?F=E4rber?= , Manivannan Sadhasivam , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-actions@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/4] dt-bindings: soc: actions: Add Actions Semi Owl socinfo binding Message-ID: <20210327163006.GA185779@robh.at.kernel.org> References: <95105518f61408743d17783099ed9c373a3dfe18.1616178258.git.cristian.ciocaltea@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <95105518f61408743d17783099ed9c373a3dfe18.1616178258.git.cristian.ciocaltea@gmail.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210327_163018_214923_AF9E83D8 X-CRM114-Status: GOOD ( 19.73 ) 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, Mar 19, 2021 at 08:27:59PM +0200, Cristian Ciocaltea wrote: > Add devicetree binding for the Actions Semi Owl SoCs info module. > > Signed-off-by: Cristian Ciocaltea > --- > .../bindings/soc/actions/owl-socinfo.yaml | 71 +++++++++++++++++++ > 1 file changed, 71 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/actions/owl-socinfo.yaml > > diff --git a/Documentation/devicetree/bindings/soc/actions/owl-socinfo.yaml b/Documentation/devicetree/bindings/soc/actions/owl-socinfo.yaml > new file mode 100644 > index 000000000000..3fcb1f584fdf > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/actions/owl-socinfo.yaml > @@ -0,0 +1,71 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/soc/actions/owl-socinfo.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Actions Semi Owl SoC info module > + > +maintainers: > + - Cristian Ciocaltea > + > +description: | > + Actions Semi Owl SoC info module provides access to various information > + about the S500, S700 and S900 SoC variants, such as serial number or id. > + > +patternProperties: > + "^soc(@[0-9a-f]+)?$": Make this a $nodename property. > + type: object > + properties: And move this up to top-level. You need a custom 'select' entry to exclude 'simple-bus'. > + compatible: > + items: > + - enum: > + - actions,s500-soc > + - actions,s700-soc > + - actions,s900-soc > + - const: simple-bus > + > + "#address-cells": > + enum: [1, 2] > + > + "#size-cells": > + enum: [1, 2] > + > + ranges: true > + > + actions,serial-number-addrs: > + description: | > + Contains the physical addresses in DDR memory where the two parts > + of the serial number (low & high) can be read from. > + This is currently supported only on the S500 SoC variant. > + $ref: /schemas/types.yaml#/definitions/uint32-array > + minItems: 2 > + maxItems: 2 Humm, it doesn't really seem you have an actual device or bus here, but are abusing DT to create your socinfo device. As the only property is data in main memory, you should do a compatible for that memory region and put it under reserved-memory. You need that anyway to prevent the kernel from using the memory, right? Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel