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 41C5FC3DA4A for ; Mon, 19 Aug 2024 14:07:30 +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:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=HE6y6oinzndXIJ/oZ2w4794yHzPY6L4u8kCKskeJp5c=; b=bk4ozSyUeEncy5 2QlVPrIwJsBhcMfsKuI1yqdbpaRt52yZ508cUIZhT6SuxPE4985ifwgeW8XztsQx9pOwMow66QFY7 DRlee2P7RAIP90V54h4MLA9HiuuCMZv7PtDJWMq5W5mYQPRjnYIxvf4dq5nyPHgrM5tMBaPHoKaAx yv5J4Bkvo/PIT2Fb5nWvXitxmz8BSczF5725zCKJ014nwtoFm5bvUcMptYVeSPi8IT8OSiqLEehhr 68yc1rbEMmjXItNWI9h3m1yRl7Fxuixzg0zlcR1ovuwXF69HD6GGhXKzqDVmftJrIbxyxgsRiKXBW FJha7TVGcEbazrsA0gRQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sg32b-00000001h7r-1Pct; Mon, 19 Aug 2024 14:07:25 +0000 Received: from sender4-op-o12.zoho.com ([136.143.188.12]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sg32N-00000001h5d-0Sfi; Mon, 19 Aug 2024 14:07:22 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1724076419; cv=none; d=zohomail.com; s=zohoarc; b=KFcBHaTHpEoAOvPDekVVbOVDYrcbpTePmOuzBmIUQMVGE99DllJ134ePJ4eaFXeSpQ7FfmvGBHVj2TiwxkUdoPN2xwkewqIZxdQTJ5WNBSxBlTwD1x9iNNyENagASrY+kX1zphmOQw1CxLq4c54ihi+EzFqh+x4JYcMTeGvhzCU= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1724076419; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=ZkV3lCqrhCL/CEHyK/B3/ruXUMXYJqM7j2j33wHPH7Y=; b=OB8YJp7mDp1naTGor1Y9XuwAWslUbAoC5jSC0YopnC9fY+aasCJwfneU6++FOBwYxd/SMffAdL2A7K0OmAhbiTDu9GmzQjFC8yK8Eg6vLkIUzH45fPiLKk1YSGk6qTY1C4uqXyz+9CQy/0qgVwJ906zITTUG/brnUctHylGy6tM= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=detlev.casanova@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1724076419; s=zohomail; d=collabora.com; i=detlev.casanova@collabora.com; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Transfer-Encoding:Content-Type:Message-Id:Reply-To; bh=ZkV3lCqrhCL/CEHyK/B3/ruXUMXYJqM7j2j33wHPH7Y=; b=XoECn+5ajwSz41GDT1zGVJA0lMOdtpxgBxr36oMfTdnkREm/BbUkzdYHXd4DvIy1 trOao3WT3jh8czfmuvkE+vpgSZKhJ+rRx5SVave+P5Jn+iaEb7pHsTfN7q9kLFLE5wn jYd7koXzjlU5EP8ZN8IpSehrNtugsFCERWO1X3wk= Received: by mx.zohomail.com with SMTPS id 1724076416454854.9649861700929; Mon, 19 Aug 2024 07:06:56 -0700 (PDT) From: Detlev Casanova To: Conor Dooley Cc: linux-kernel@vger.kernel.org, Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Philipp Zabel , Elaine Zhang , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, kernel@collabora.com, Sugar Zhang Subject: Re: [PATCH v5 1/2] dt-bindings: clock, reset: Add support for rk3576 Date: Mon, 19 Aug 2024 10:08:31 -0400 Message-ID: <5018731.31r3eYUQgx@trenzalore> In-Reply-To: <20240815-tassel-whacking-a460ee2ebd41@spud> References: <20240814222159.2598-1-detlev.casanova@collabora.com> <20240814222159.2598-2-detlev.casanova@collabora.com> <20240815-tassel-whacking-a460ee2ebd41@spud> MIME-Version: 1.0 X-ZohoMailClient: External X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240819_070711_412822_3DA0A156 X-CRM114-Status: GOOD ( 26.87 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org Hi Conor, On Thursday, 15 August 2024 11:07:46 EDT Conor Dooley wrote: > On Wed, Aug 14, 2024 at 06:19:22PM -0400, Detlev Casanova wrote: > > Add clock and reset ID defines for rk3576. > > > > Compared to the downstream bindings written by Elaine, this uses > > continous gapless IDs starting at 0. Thus all numbers are > > different between downstream and upstream, but names are kept > > exactly the same. > > > > Also add documentation for the rk3576 CRU core. > > > > Signed-off-by: Elaine Zhang > > Signed-off-by: Sugar Zhang > > Signed-off-by: Detlev Casanova > > --- > > > > .../bindings/clock/rockchip,rk3576-cru.yaml | 64 ++ > > .../dt-bindings/clock/rockchip,rk3576-cru.h | 592 ++++++++++++++++++ > > .../dt-bindings/reset/rockchip,rk3576-cru.h | 564 +++++++++++++++++ > > 3 files changed, 1220 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/clock/rockchip,rk3576-cru.yaml create > > mode 100644 include/dt-bindings/clock/rockchip,rk3576-cru.h create mode > > 100644 include/dt-bindings/reset/rockchip,rk3576-cru.h> > > diff --git > > a/Documentation/devicetree/bindings/clock/rockchip,rk3576-cru.yaml > > b/Documentation/devicetree/bindings/clock/rockchip,rk3576-cru.yaml new > > file mode 100644 > > index 0000000000000..d69985e6fa0ce > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/clock/rockchip,rk3576-cru.yaml > > @@ -0,0 +1,64 @@ > > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/clock/rockchip,rk3576-cru.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Rockchip rk3576 Family Clock and Reset Control Module > > + > > +maintainers: > > + - Elaine Zhang > > + - Heiko Stuebner > > + - Detlev Casanova > > + > > +description: > > + The RK3576 clock controller generates the clock and also implements a > > reset + controller for SoC peripherals. For example it provides > > SCLK_UART2 and + PCLK_UART2, as well as SRST_P_UART2 and SRST_S_UART2 > > for the second UART + module. > > + > > +properties: > > + compatible: > > + const: rockchip,rk3576-cru > > + > > + reg: > > + maxItems: 1 > > + > > + "#clock-cells": > > + const: 1 > > + > > + "#reset-cells": > > + const: 1 > > + > > + clocks: > > + maxItems: 2 > > + > > + clock-names: > > + items: > > + - const: xin24m > > + - const: xin32k > > + > > + rockchip,grf: > > + $ref: /schemas/types.yaml#/definitions/phandle > > + description: > > > + phandle to the syscon managing the "general register files". It is > > used + for GRF muxes, if missing any muxes present in the GRF will > > not be + available. > > Two questions on this property: > - you only support one soc, why is this optional? It is optional because only used for some specific clocks. The SoC can still be used without this, but some devices might not work (Not tested but USB PHYs might not be working without the GRF) This is also set as optional in similar rockchip CRU bindings (rk3588). > - why can't you look it up by compatible? These bindings are specific to one compatible only. It is very similar to rk3588 but it looks like all rockchip CRU driver has its own yaml file, so I followed that trend instead of merging with the rk3588 CRU bindings. Regards, Detlev. _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip