From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id C4E693AAF6A; Sun, 20 Sep 2026 15:13:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789917226; cv=none; b=BGzQ5nVUIGqB+Cbu+nC5LoT40FVZBuEmokg1gpvaDs3sJ2sDR7yNMtCf1HCD/7GY9ufa4MJdo7njtp7KqpbJnmCpqBxvV65VluWT0Hk2ZWMN9NPEyJIDHn+OEshs50g5h8o3/1qF5csqRhAZWlJ+YPW8ZphMTkriwDvglrzSZN4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789917226; c=relaxed/simple; bh=SSUGOkvZlKcwT3YQ7ugwa08tm7XRyK2mRxan7tsBq9A=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=LFTYrmbDtjsNZYaNa7Uj12SCBnTA/nNoAlbI9d7obt2SRmd8UZFtPXNfvFsx7xRaecbAcO3RGRnhIz29l/hcK79u9nSw4K5hCPIBKygrLI1thiFL/i047id+vK5Gg4HKkwEEXk28LtWSEijfNYDrCZHT9DrfZ/ZqzLK5NBXi1+E= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=uBDAHjhB; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="uBDAHjhB" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 22EA61650; Sun, 20 Sep 2026 08:13:38 -0700 (PDT) Received: from ryzen.lan (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 36F293F632; Sun, 20 Sep 2026 08:13:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1789917221; bh=SSUGOkvZlKcwT3YQ7ugwa08tm7XRyK2mRxan7tsBq9A=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=uBDAHjhBHkkPwi628BqqIFlQ+INGYryGrQdUfxjc10DPsc6erfmlXh9zJcVrJ3Li5 YhNz8u7RxpgHCmj7AHjpjl57NcWuDNc0lyXiuRMvqgpYvzjvoVqGlHBEcIljCTNcjO 4XmJx/k8MUIg6BaZ9K3VisgZGammxqfukhfY/ERA= Date: Sun, 20 Sep 2026 17:08:46 +0200 From: Andre Przywara To: Junhui Liu Cc: Stephen Boyd , Brian Masney , Jerome Brunet , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Philipp Zabel , Paul Walmsley , Palmer Dabbelt , Albert Ou , Alexandre Ghiti , Richard Cochran , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, netdev@vger.kernel.org, Jerome Brunet , Enzo Adriano Subject: Re: [PATCH v3 8/8] clk: sunxi-ng: a733: Add reset lines Message-ID: <20260920170846.087bd42d@ryzen.lan> In-Reply-To: <20260905-a733-clk-v3-8-50f238f39c78@pigmoral.tech> References: <20260905-a733-clk-v3-0-50f238f39c78@pigmoral.tech> <20260905-a733-clk-v3-8-50f238f39c78@pigmoral.tech> Organization: Arm Ltd. X-Mailer: Claws Mail 4.4.0 (GTK 3.24.31; x86_64-slackware-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Sat, 05 Sep 2026 00:45:37 +0800 Junhui Liu wrote: Hi, > Add the reset lines for the Allwinner A733 SoC. These reset control bits > are integrated into the Bus Gate Reset (BGR) registers, typically > sharing the same register address with their corresponding bus clock > gates. Integrate them into the main CCU driver using the existing > sunxi-ng ccu_reset framework, allowing the CCU to also function as a > reset controller for the SoC. > > Tested-by: Jerome Brunet > Signed-off-by: Junhui Liu Many thanks, my comments have all been addressed: Reviewed-by: Andre Przywara Cheers, Andre > --- > drivers/clk/sunxi-ng/ccu-sun60i-a733.c | 126 +++++++++++++++++++++++++++++++++ > 1 file changed, 126 insertions(+) > > diff --git a/drivers/clk/sunxi-ng/ccu-sun60i-a733.c b/drivers/clk/sunxi-ng/ccu-sun60i-a733.c > index c124b09855c0..b7fc2cfc626c 100644 > --- a/drivers/clk/sunxi-ng/ccu-sun60i-a733.c > +++ b/drivers/clk/sunxi-ng/ccu-sun60i-a733.c > @@ -19,6 +19,7 @@ > #include > > #include "ccu_common.h" > +#include "ccu_reset.h" > > #include "ccu_div.h" > #include "ccu_gate.h" > @@ -2183,11 +2184,136 @@ static struct clk_hw_onecell_data sun60i_a733_hw_clks = { > .num = CLK_FANOUT3 + 1, > }; > > +static const struct ccu_reset_map sun60i_a733_ccu_resets[] = { > + [RST_BUS_ITS_PCIE0] = { 0x574, BIT(16) }, > + [RST_BUS_IOMMU0_SYS] = { 0x58c, BIT(16) }, > + [RST_BUS_MSI_LITE0_AHB] = { 0x594, BIT(16) }, > + [RST_BUS_MSI_LITE0_MBUS] = { 0x594, BIT(17) }, > + [RST_BUS_MSI_LITE1_AHB] = { 0x59c, BIT(16) }, > + [RST_BUS_MSI_LITE1_MBUS] = { 0x59c, BIT(17) }, > + [RST_BUS_MSI_LITE2_AHB] = { 0x5a4, BIT(16) }, > + [RST_BUS_MSI_LITE2_MBUS] = { 0x5a4, BIT(17) }, > + [RST_BUS_IOMMU1_SYS] = { 0x5b4, BIT(16) }, > + [RST_BUS_DMA0] = { 0x704, BIT(16) }, > + [RST_BUS_DMA1] = { 0x70c, BIT(16) }, > + [RST_BUS_SPINLOCK] = { 0x724, BIT(16) }, > + [RST_BUS_MSGBOX] = { 0x744, BIT(16) }, > + [RST_BUS_PWM0] = { 0x784, BIT(16) }, > + [RST_BUS_PWM1] = { 0x78c, BIT(16) }, > + [RST_BUS_DBG] = { 0x7a4, BIT(16) }, > + [RST_BUS_SYSDAP] = { 0x7ac, BIT(16) }, > + [RST_BUS_TIMER0] = { 0x850, BIT(16) }, > + [RST_BUS_DE0] = { 0xa04, BIT(16) }, > + [RST_BUS_DI] = { 0xa24, BIT(16) }, > + [RST_BUS_G2D] = { 0xa44, BIT(16) }, > + [RST_BUS_EINK] = { 0xa6c, BIT(16) }, > + [RST_BUS_DE_SYS] = { 0xa74, BIT(16) }, > + [RST_BUS_VE_ENC0] = { 0xa8c, BIT(16) }, > + [RST_BUS_VE_DEC0] = { 0xa8c, BIT(18) }, > + [RST_BUS_CE] = { 0xac4, BIT(16) }, > + [RST_BUS_CE_SYS] = { 0xac4, BIT(17) }, > + [RST_BUS_NPU_CORE] = { 0xb04, BIT(16) }, > + [RST_BUS_NPU_AXI] = { 0xb04, BIT(17) }, > + [RST_BUS_NPU_AHB] = { 0xb04, BIT(18) }, > + [RST_BUS_NPU_SRAM] = { 0xb04, BIT(19) }, > + [RST_BUS_GPU0] = { 0xb24, BIT(16) }, > + [RST_BUS_DRAM0] = { 0xc0c, BIT(16) }, > + [RST_BUS_NAND0] = { 0xc8c, BIT(16) }, > + [RST_BUS_MMC0] = { 0xd0c, BIT(16) }, > + [RST_BUS_MMC1] = { 0xd1c, BIT(16) }, > + [RST_BUS_MMC2] = { 0xd2c, BIT(16) }, > + [RST_BUS_MMC3] = { 0xd3c, BIT(16) }, > + [RST_BUS_UFS_AHB] = { 0xd8c, BIT(16) }, > + [RST_BUS_UFS_AXI] = { 0xd8c, BIT(17) }, > + [RST_BUS_UFS_PHY] = { 0xd8c, BIT(18) }, > + [RST_BUS_UFS_CORE] = { 0xd8c, BIT(19) }, > + [RST_BUS_UART0] = { 0xe00, BIT(16) }, > + [RST_BUS_UART1] = { 0xe04, BIT(16) }, > + [RST_BUS_UART2] = { 0xe08, BIT(16) }, > + [RST_BUS_UART3] = { 0xe0c, BIT(16) }, > + [RST_BUS_UART4] = { 0xe10, BIT(16) }, > + [RST_BUS_UART5] = { 0xe14, BIT(16) }, > + [RST_BUS_UART6] = { 0xe18, BIT(16) }, > + [RST_BUS_I2C0] = { 0xe80, BIT(16) }, > + [RST_BUS_I2C1] = { 0xe84, BIT(16) }, > + [RST_BUS_I2C2] = { 0xe88, BIT(16) }, > + [RST_BUS_I2C3] = { 0xe8c, BIT(16) }, > + [RST_BUS_I2C4] = { 0xe90, BIT(16) }, > + [RST_BUS_I2C5] = { 0xe94, BIT(16) }, > + [RST_BUS_I2C6] = { 0xe98, BIT(16) }, > + [RST_BUS_I2C7] = { 0xe9c, BIT(16) }, > + [RST_BUS_I2C8] = { 0xea0, BIT(16) }, > + [RST_BUS_I2C9] = { 0xea4, BIT(16) }, > + [RST_BUS_I2C10] = { 0xea8, BIT(16) }, > + [RST_BUS_I2C11] = { 0xeac, BIT(16) }, > + [RST_BUS_I2C12] = { 0xeb0, BIT(16) }, > + [RST_BUS_SPI0] = { 0xf04, BIT(16) }, > + [RST_BUS_SPI1] = { 0xf0c, BIT(16) }, > + [RST_BUS_SPI2] = { 0xf14, BIT(16) }, > + [RST_BUS_SPIF] = { 0xf1c, BIT(16) }, > + [RST_BUS_SPI3] = { 0xf24, BIT(16) }, > + [RST_BUS_SPI4] = { 0xf2c, BIT(16) }, /* From the vendor kernel. */ > + [RST_BUS_GPADC0] = { 0xfc4, BIT(16) }, > + [RST_BUS_THS0] = { 0xfe4, BIT(16) }, > + [RST_BUS_IRRX] = { 0x1004, BIT(16) }, > + [RST_BUS_IRTX] = { 0x100c, BIT(16) }, > + [RST_BUS_LRADC] = { 0x1024, BIT(16) }, > + [RST_BUS_SGPIO] = { 0x1064, BIT(16) }, /* From the vendor kernel. */ > + [RST_BUS_LPC] = { 0x1084, BIT(16) }, /* From the vendor kernel. */ > + [RST_BUS_I2SPCM0] = { 0x120c, BIT(16) }, > + [RST_BUS_I2SPCM1] = { 0x121c, BIT(16) }, > + [RST_BUS_I2SPCM2] = { 0x122c, BIT(16) }, > + [RST_BUS_I2SPCM3] = { 0x123c, BIT(16) }, > + [RST_BUS_I2SPCM4] = { 0x124c, BIT(16) }, > + [RST_BUS_SPDIF] = { 0x128c, BIT(16) }, > + [RST_BUS_DMIC] = { 0x12cc, BIT(16) }, > + [RST_USB_PHY0] = { 0x1300, BIT(30) }, > + [RST_BUS_OHCI0] = { 0x1304, BIT(16) }, > + [RST_BUS_EHCI0] = { 0x1304, BIT(20) }, > + [RST_BUS_OTG] = { 0x1304, BIT(24) }, > + [RST_USB_PHY1] = { 0x1308, BIT(30) }, > + [RST_BUS_OHCI1] = { 0x130c, BIT(16) }, > + [RST_BUS_EHCI1] = { 0x130c, BIT(20) }, > + [RST_BUS_USB2] = { 0x135c, BIT(16) }, > + [RST_BUS_PCIE_PWRUP] = { 0x138c, BIT(16) }, > + [RST_BUS_PCIE] = { 0x138c, BIT(17) }, > + [RST_BUS_SERDES] = { 0x13c4, BIT(16) }, > + [RST_BUS_GMAC0] = { 0x141c, BIT(16) }, > + [RST_BUS_GMAC0_AXI] = { 0x141c, BIT(17) }, > + [RST_BUS_GMAC1] = { 0x142c, BIT(16) }, /* From the vendor kernel. */ > + [RST_BUS_GMAC1_AXI] = { 0x142c, BIT(17) }, /* From the vendor kernel. */ > + [RST_BUS_TCON_LCD0] = { 0x1504, BIT(16) }, > + [RST_BUS_TCON_LCD1] = { 0x150c, BIT(16) }, > + [RST_BUS_TCON_LCD2] = { 0x1514, BIT(16) }, /* From the vendor kernel. */ > + [RST_BUS_LVDS0] = { 0x1544, BIT(16) }, > + [RST_BUS_LVDS1] = { 0x154c, BIT(16) }, > + [RST_BUS_DSI0] = { 0x1584, BIT(16) }, > + [RST_BUS_DSI1] = { 0x158c, BIT(16) }, > + [RST_BUS_TCON_TV0] = { 0x1604, BIT(16) }, > + [RST_BUS_TCON_TV1] = { 0x160c, BIT(16) }, > + [RST_BUS_EDP] = { 0x164c, BIT(16) }, > + [RST_BUS_HDMI_MAIN] = { 0x168c, BIT(16) }, > + [RST_BUS_HDMI_SUB] = { 0x168c, BIT(17) }, > + [RST_BUS_HDMI_HDCP] = { 0x168c, BIT(18) }, > + [RST_BUS_DPSS_TOP0] = { 0x16c4, BIT(16) }, > + [RST_BUS_DPSS_TOP1] = { 0x16cc, BIT(16) }, > + [RST_BUS_VIDEO_OUT0] = { 0x16e4, BIT(16) }, > + [RST_BUS_VIDEO_OUT1] = { 0x16ec, BIT(16) }, > + [RST_BUS_LEDC] = { 0x1704, BIT(16) }, > + [RST_BUS_DSC] = { 0x1744, BIT(16) }, > + [RST_BUS_CSI] = { 0x1844, BIT(16) }, > + [RST_BUS_VIDEO_IN] = { 0x1884, BIT(16) }, > + [RST_BUS_APB2JTAG] = { 0x1c04, BIT(16) }, > +}; > + > static const struct sunxi_ccu_desc sun60i_a733_ccu_desc = { > .ccu_clks = sun60i_a733_ccu_clks, > .num_ccu_clks = ARRAY_SIZE(sun60i_a733_ccu_clks), > > .hw_clks = &sun60i_a733_hw_clks, > + > + .resets = sun60i_a733_ccu_resets, > + .num_resets = ARRAY_SIZE(sun60i_a733_ccu_resets), > }; > > static const u32 pll_regs[] = { >