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,URIBL_BLOCKED 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 27A92C433E0 for ; Mon, 4 Jan 2021 08:33:27 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 DB5AD20784 for ; Mon, 4 Jan 2021 08:33:26 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org DB5AD20784 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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject: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=TeDAupE6F0XklqfxglvVVSfMFDxLE5CI+Vg9FL14oSQ=; b=NBSLyqGpgkCkLWDMRHU9Vtjdi cEIreB5AF7NmSwNhJNWua+xKB/t73V7yJA/0+k66zeOzbjVMwC/nSk3K/CoV07/S8Vd/qGSenmDVY sk909THvC729rLTTkMb69MX9fJhvSt9wKPSRfM/gCpnzDzxQuUAHbWVXkHxu9e6DHbk1nlc0m3RCK c9yZyJlzGY43dxoZ3XVLSnuWZBI2CgHnBhQSnzkD9KH+x/xfNbBpXjXGB2PSS0KGUDkG8blklCiGM wikfpNuiQqGCfUCB436K1e5UxkiSer2nRXo9+qC/IS9sUKRxFy2PUZ3mXdfhfH5GNkors+zVGb1Lv EQciGwttw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kwLHQ-0002et-UO; Mon, 04 Jan 2021 08:31:56 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kwLHN-0002dT-7V for linux-arm-kernel@lists.infradead.org; Mon, 04 Jan 2021 08:31:54 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id C3C9A20798; Mon, 4 Jan 2021 08:31:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1609749110; bh=/l4LNgcZQuQKp8LjOcoiZ+3D5i2u+KxinFDst/ZT+Xw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=bH9nLgUy6rQaKDLX1sw1beUf9Aq83lpTkkF5ADT9W2qKlL12x7qcISW2nja3aqI1Y WzpG1HIteKGSrWtfL4/MJWWFZPBQZigPtLn7owPGJgh1SKwM/Z0NWd4326bSyG2UVp cbtkAJ/BJ+ksAQAWS715qAMvFPntnXind4CHexEHnKoDA38+VGZGUq5N3KFvPcQKRJ umggJKKxXJrxQebYyCNBK4D1yb/2Cva4LLilwnlwEgsxwCT0/1TdV5UeDY3sdvXAvG TRTNprpXnycMUQpKcKL+L1jkCD7tuLfQvRYX6/z0/rd93m1CzYF32NtvmESwNcScE6 VCK/MmB55ljqQ== Received: by wens.tw (Postfix, from userid 1000) id A190F5F823; Mon, 4 Jan 2021 16:31:48 +0800 (CST) Date: Mon, 4 Jan 2021 16:31:48 +0800 From: Chen-Yu Tsai To: Samuel Holland Subject: Re: [linux-sunxi] [PATCH v2 0/4] Allwinner H6 RSB support Message-ID: References: <20210103100007.32867-1-samuel@sholland.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210103100007.32867-1-samuel@sholland.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210104_033153_504266_ADCE2F5B X-CRM114-Status: GOOD ( 26.40 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Jernej Skrabec , Stephen Boyd , Andre Przywara , Michael Turquette , linux-sunxi@googlegroups.com, linux-kernel@vger.kernel.org, Maxime Ripard , linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, Chen-Yu Tsai , Rob Herring , Philipp Zabel , Linus Walleij , linux-arm-kernel@lists.infradead.org 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 Hi, On Sun, Jan 03, 2021 at 04:00:03AM -0600, Samuel Holland wrote: > The Allwinner H6 SoC contains an RSB controller. It is almost completely > undocumented, so it was missed when doing the initial SoC bringup. > > This series adds the clock/reset, pin configuration, and device tree > node needed to use the RSB controller. Since RSB is faster, simpler, and > generally more reliable than the I2C controller IP in the SoC, switch to > using it where possible. > > This was tested on an Orange Pi 3 and a Pine H64 model B. This series > does not switch the Pine H64 to use RSB, as doing so would prevent > accessing the external RTC that shares the I2C bus. > > Changes v1->v2: > - Put the new values at the end of the DT binding headers > > Samuel Holland (4): > clk: sunxi-ng: h6-r: Add R_APB2_RSB clock and reset > pinctrl: sunxi: h6-r: Add s_rsb pin functions > arm64: dts: allwinner: h6: Add RSB controller node > arm64: dts: allwinner: h6: Use RSB for AXP805 PMIC connection I queued up patches 1, 3, and 4 locally for v5.12. Obviously this won't work unless the pinctrl patch is also queued up, so they won't be pushed out until that happens. Regarding patch 3, I replaced the clock and reset macros with raw numbers to get rid of cross-tree dependencies. The following fix will be posted for v5.12 later on during its RC cycle. ------------------------ >8 ------------------------ commit 0b4781666adc5e19c4d4fb4a2bff33883181cc39 Author: Chen-Yu Tsai Date: Mon Jan 4 16:19:17 2021 +0800 arm64: dts: allwinner: h6: Switch to macros for RSB clock/reset indices The macros for the clock and reset indices for the RSB hardware block were replaced with raw numbers when the RSB controller node was added. This was done to avoid cross-tree dependencies. Now that both the clk and DT changes have been merged, we can switch back to using the macros. Signed-off-by: Chen-Yu Tsai diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi index d897697849d6..b043beea8e6e 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi +++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi @@ -988,9 +988,9 @@ r_rsb: rsb@7083000 { compatible = "allwinner,sun8i-a23-rsb"; reg = <0x07083000 0x400>; interrupts = ; - clocks = <&r_ccu 13>; + clocks = <&r_ccu CLK_R_APB2_RSB>; clock-frequency = <3000000>; - resets = <&r_ccu 7>; + resets = <&r_ccu RST_R_APB2_RSB>; pinctrl-names = "default"; pinctrl-0 = <&r_rsb_pins>; status = "disabled"; ------------------------ >8 ------------------------ > > .../dts/allwinner/sun50i-h6-beelink-gs1.dts | 38 +++++++++---------- > .../dts/allwinner/sun50i-h6-orangepi-3.dts | 14 +++---- > .../dts/allwinner/sun50i-h6-orangepi.dtsi | 22 +++++------ > arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 19 ++++++++++ > drivers/clk/sunxi-ng/ccu-sun50i-h6-r.c | 5 +++ > drivers/clk/sunxi-ng/ccu-sun50i-h6-r.h | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun50i-h6-r.c | 2 + > include/dt-bindings/clock/sun50i-h6-r-ccu.h | 2 + > include/dt-bindings/reset/sun50i-h6-r-ccu.h | 1 + > 9 files changed, 67 insertions(+), 38 deletions(-) > > -- > 2.26.2 > > -- > You received this message because you are subscribed to the Google Groups "linux-sunxi" group. > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com. > To view this discussion on the web, visit https://groups.google.com/d/msgid/linux-sunxi/20210103100007.32867-1-samuel%40sholland.org. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel