Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp3878544pxb; Wed, 13 Oct 2021 15:09:45 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx+QzHGIZiDbGPKIm+2oK3xAEUvBsndIEz3pH5y/mcPXqIlesO0yMomTcfuIiKqaHQbklaj X-Received: by 2002:a63:1262:: with SMTP id 34mr1406227pgs.356.1634162985078; Wed, 13 Oct 2021 15:09:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1634162985; cv=none; d=google.com; s=arc-20160816; b=QwNqn3F/0eD+X82woHHeigcPBTMXhQMSQm9oAlP1/VNjGajgv4W1FbDketNd9Uqn+n adam25zV/y9N+2XSdh+M4wUhDm7mOEO9x+msEnlWhRRUdOGfTaIJGcnFaGuLQGiwTcW0 UvkRnYiBhxMg0V70F2Viw9tsQFbp8MBK4poCS6/3r6fOdWvOghYW5apKWK2Ljc7fP0Ln qKdBD+x87xLLawE8sO1szex2lUgpjKOZBEb+FrXuarH0IcA015Q8dz1GCfSqmYQL44U8 Auzd1K9wXRRd6jGbnRs28pIUBE56Kw7gcHODYIhGBXRILsEDkHKs820w3tSul8h7cvuq AmEg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:message-id:date:to:cc:from:subject :references:in-reply-to:content-transfer-encoding:mime-version :dkim-signature; bh=CflRmed1pi8XTq2KCqLAup2YfDqMcp5JmtgvVOGloBU=; b=QNPiHkqIfGN6w6GTon1qRvRuz4g8ZsNv3PrMxqrB5jHkKxkrcVWw3klCyUu2hHZ7lL 4dTbk1bWzvJDm5dVuBcHc1S4Is3yzDhdVhakmkU2B2kZo2NIYw2106E9UnRIbqICKuhW 7gJP8eEgJ4KeYcY1s0LU09beXZbh2k/5oP4k3VpReR+vHBVcnHhv5mlf3HZakJYiYhH7 DAUodF/0winWLHs8LlTLu5GXHaa/qTVLC/k0OExrF0ynn5p3BMXYtNUFLuW1oR+tOpJy 90W32Ntd84GZvMy4i1V0YJHDA+2SVs+xfeMP4rjgW0M1WEoT03472MOBJ1SW0vBxg0uy +8pg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="X9z6/fQt"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w11si1145587plq.378.2021.10.13.15.09.32; Wed, 13 Oct 2021 15:09:45 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="X9z6/fQt"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230289AbhJMWGi (ORCPT + 99 others); Wed, 13 Oct 2021 18:06:38 -0400 Received: from mail.kernel.org ([198.145.29.99]:39436 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229754AbhJMWGh (ORCPT ); Wed, 13 Oct 2021 18:06:37 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 12E24610CF; Wed, 13 Oct 2021 22:04:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1634162674; bh=Pz99xJwgUWewDS8SG1vh0092WbkfB7AaUfBLNp1rW8w=; h=In-Reply-To:References:Subject:From:Cc:To:Date:From; b=X9z6/fQttyLF/h9MxZe8D69kQI/BbTueoj86dTFnYOyjq7ppzSyp9CtZaCfZbwQAx 0Ad8PPb6vN/JujhvZviDsuUe+O1QExqyohHomM34w7HNoXuwtddCO23Z1BhPGo3AWx Jb+qMQLT1kAuPwTTMz/YHmnhmGUODFsh66LoXzLCCskWFJmFfxve5DlB8EnQu3Z8iV xqzgpv9p1hZ8Pf+RRLndcOP/wQEF38Udsden9pshcE8RTKWI27N/EC8Iuxj3X003Fq GvMNko80oPQuOXNvmPhIP0XeB9ww4F36FDvoW/nitoqbIJzRf6xTPo7hOzJrVKHysP b/LRlYY7v15Dg== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable In-Reply-To: <20210923064137.60722-3-zhang.lyra@gmail.com> References: <20210923064137.60722-1-zhang.lyra@gmail.com> <20210923064137.60722-3-zhang.lyra@gmail.com> Subject: Re: [PATCH v4 2/4] dt-bindings: mfd: sprd: Add bindings for ums512 global registers From: Stephen Boyd Cc: linux-clk@vger.kernel.org, devicetree@vger.kernel.org, Baolin Wang , Orson Zhai , Chunyan Zhang , Chunyan Zhang , LKML To: Chunyan Zhang , Rob Herring Date: Wed, 13 Oct 2021 15:04:32 -0700 Message-ID: <163416267274.936110.2784588823311275089@swboyd.mtv.corp.google.com> User-Agent: alot/0.9.1 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Chunyan Zhang (2021-09-22 23:41:35) > diff --git a/Documentation/devicetree/bindings/mfd/sprd,ums512-glbreg.yam= l b/Documentation/devicetree/bindings/mfd/sprd,ums512-glbreg.yaml > new file mode 100644 > index 000000000000..3522f3d2d8de > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/sprd,ums512-glbreg.yaml > @@ -0,0 +1,68 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/mfd/sprd,ums512-glbreg.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Unisoc System Global Register Device Tree Bindings > + > +maintainers: > + - Orson Zhai > + - Baolin Wang > + - Chunyan Zhang > + > +description: > + Unisoc system global registers provide register map > + for clocks and some multimedia modules of the SoC. > + > +properties: > + "#address-cells": true > + "#size-cells": true > + > + compatible: > + items: > + - const: sprd,ums512-glbregs > + - const: syscon > + - const: simple-mfd > + > + ranges: > + maxItems: 1 > + > + reg: > + maxItems: 1 > + > +patternProperties: > + "^.*@[0-9a-f]+$": > + # Child node > + type: object > + $ref: "../clock/sprd,ums512-clk.yaml" > + description: > + Clock controller for the SoC clocks. > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + ap_apb_regs: syscon@71000000 { > + compatible =3D "sprd,ums512-glbregs", "syscon", "simple-mfd"; > + reg =3D <0x71000000 0x3000>; > + #address-cells =3D <1>; > + #size-cells =3D <1>; > + ranges =3D <0 0x71000000 0x3000>; > + > + clock-controller@0 { > + compatible =3D "sprd,ums512-apahb-gate"; Why is this a subnode of a syscon and simple-mfd? Why not put the clock-controller@71000000 directly onto the bus? Does making it a child node help somehow? > + reg =3D <0x0 0x2000>; > + #clock-cells =3D <1>; > + }; > + };