From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-io1-f48.google.com (mail-io1-f48.google.com [209.85.166.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7D4EF22424D for ; Sat, 22 Mar 2025 14:27:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.166.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742653657; cv=none; b=MVn2OjoxTfVPZkIIbAPuRDdeThWBsIpTzDTt0MqVxIoOzRHcXMKfWNRJr6W8LP33RnshQ+ZEsYTDU+YnG125IJbPZL2j0Gx8nNXHvXKMDa1wghRvc5taH1v5cqM9bNRvybSf+Odwxjv+GX8XQ8ByV+VR5GXOUvNVv+h/uZcJCz8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742653657; c=relaxed/simple; bh=wpCm+hqOG2wsbtPLPSsueiJdiduXYzbm2JN6/V7qei4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=KA6+uf7P3MMc12sKihG6XLVmdMKY4KSdHglt5ky/Y1DODeaN2R81I/W0UjK0bOOj8PBsyWe46CAFfR1s1l7liVG/IBV/dDaR4Nv41VB8Eqsc8kdFk1VE9s75Qxq6VjB5vtMA7M03XgIR5Zk7pxuxH6wodY8g3np7nN0uRVyrnV4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=riscstar.com; spf=pass smtp.mailfrom=riscstar.com; dkim=pass (2048-bit key) header.d=riscstar-com.20230601.gappssmtp.com header.i=@riscstar-com.20230601.gappssmtp.com header.b=kEHuW3fP; arc=none smtp.client-ip=209.85.166.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=riscstar.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=riscstar.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=riscstar-com.20230601.gappssmtp.com header.i=@riscstar-com.20230601.gappssmtp.com header.b="kEHuW3fP" Received: by mail-io1-f48.google.com with SMTP id ca18e2360f4ac-85e1b1f08a5so84506139f.2 for ; Sat, 22 Mar 2025 07:27:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=riscstar-com.20230601.gappssmtp.com; s=20230601; t=1742653654; x=1743258454; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=ZxLNYJpvB/TXLfXAqOdKhmy/q9FPbJFzYq67r8P8xbE=; b=kEHuW3fPfu726en3O8HTTUvD7b2bnYxF/E6T2FNNhZEAQqq986sLpeCS8/waNy+3zG i8iYZ2PYNQ9Hjh2I+ELvrGFjPj0IFPeajwjcn0HOCZKXnztsMyNGuUfW8ehxjjD9f4aT Cs1AlxPFtF/C4kOhyNl6zDvjeLKqmw+AjcybgoAJrXSM6vbNh68oTvND+XAjI4OuKnHQ 6VIFiVJ/z0Hrq75nE72h7c/ePE/2+4XiSSfjanKyT1b7uV8QcHX7E5r27eXCXGEESv1/ sKEuctf/5Dqj0Q7/lYfUTH7FKD1nqhQTBwaAN4ZUygR+BTE3LzJxrkdZPt4SF37nfGfz zwCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1742653654; x=1743258454; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ZxLNYJpvB/TXLfXAqOdKhmy/q9FPbJFzYq67r8P8xbE=; b=BJDtmaorQokO3KJ+7swfjyxEysPSVxiraGH9yEZegQ3/HBGg/5IJWYYw8D93ffosAS xlsh1tCYBPmi5CbxO9x8GgJORFwwg8oMzSbosEHjiu1sa266z5IxcSf+OSpITYNvn0K2 6Ai+pbqEUbQMCNJIOQZAFyjeROM9HS1N4Urjz2A8/niN2sNX/Eai7jNmoOEIOYNhDSOh 24MHDA6ny0r19IrNMi3UmVTLcUfmEbYUokRYJj4U0VrrUn0QV/6/wORu/wUH8HNF91f+ sQtIEq9zK7acQuMqUsKJuoN0gycaOzzfI3x5ABBvhv8LpY5yFAa9tR5I3glBl3CAay5j BKlQ== X-Forwarded-Encrypted: i=1; AJvYcCWnyY+M1oFWhZR2x+92JnmHgofNE9Xl/b7GzvgeEwGmdomWvqSbQRBWX/6iBkO31PtSRbba8gg1ySOi@vger.kernel.org X-Gm-Message-State: AOJu0YyVt/u4i4QXq2tgTvd2m9VANpt7IU7zwIk6Pjqdso7rN9/xNl9L nzu0oVwTHZbJxkFn/Lfy7/thJyuitqDk60BHEWu6IirrLAnt06vt2iHQIL81unA= X-Gm-Gg: ASbGncsksabFwy7WbRzVNRzeWKIMhk18Ok5i82pnqGMWvRAKfJC1lZc3gGqt+c0g0iV NxlZJWXR90SrVI/fpYD0UHRlVwKqE+jRv+/83AI5a0qAuMm4cP0uZ/YUq2sdBGJSzdOznSAAQRk v5Uc6qUoMMnfoEXXi5WwmxomOw50f77cw3At8Rv6O/IeYgM52Ie8DBOq34V4P/wVeFGsHbje4VJ eMnLjNZ88iEPI85QN/5Cvrommatl8JzxdaFZrWiMBluJLYMUuS8puRRtof2w2Rx67upKYRqWviF 6dEBMuZSpqfXDVXj/p1HdiSu/3wPPxnwn0F/8Nt3/n8KIE/9vb97OjoIzInWP8bvlO1w5GhcLvs 7IbKyObed3EKRSjFirQ== X-Google-Smtp-Source: AGHT+IHxH2JRqswkpmkABiwFlVLWGLbm4QGv8PyCPkc/LYC5FudrlksXf1TKCySAHhTc+btPlG9CLg== X-Received: by 2002:a05:6e02:2585:b0:3d0:235b:4810 with SMTP id e9e14a558f8ab-3d5960bf9b9mr82023985ab.2.1742653654335; Sat, 22 Mar 2025 07:27:34 -0700 (PDT) Received: from [10.211.55.5] (c-73-228-159-35.hsd1.mn.comcast.net. [73.228.159.35]) by smtp.gmail.com with ESMTPSA id 8926c6da1cb9f-4f2cbecb453sm908223173.142.2025.03.22.07.27.32 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 22 Mar 2025 07:27:33 -0700 (PDT) Message-ID: <1d79fb7e-4501-4c62-8379-f00515dec3e4@riscstar.com> Date: Sat, 22 Mar 2025 09:27:32 -0500 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RESEND 1/7] dt-bindings: soc: spacemit: define spacemit,k1-ccu resets To: Yixun Lan Cc: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, p.zabel@pengutronix.de, mturquette@baylibre.com, sboyd@kernel.org, heylenay@4d2.org, guodong@riscstar.com, paul.walmsley@sifive.com, palmer@dabbelt.com, aou@eecs.berkeley.edu, spacemit@lists.linux.dev, devicetree@vger.kernel.org, linux-clk@vger.kernel.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org References: <20250321151831.623575-1-elder@riscstar.com> <20250321151831.623575-2-elder@riscstar.com> <20250321222546-GYA11633@gentoo> Content-Language: en-US From: Alex Elder In-Reply-To: <20250321222546-GYA11633@gentoo> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 3/21/25 5:25 PM, Yixun Lan wrote: > hi Alex: > > On 10:18 Fri 21 Mar , Alex Elder wrote: >> There are additional SpacemiT syscon CCUs whose registers control both >> clocks and resets: RCPU, RCPU2, and APBC2. Unlike those defined >> previously, these will initially support only resets. They do not >> incorporate power domain functionality. >> >> Define the index values for resets associated with all SpacemiT K1 >> syscon nodes, including those with clocks already defined, as well as >> the new ones (without clocks). >> >> Signed-off-by: Alex Elder >> --- >> .../soc/spacemit/spacemit,k1-syscon.yaml | 13 +- >> include/dt-bindings/clock/spacemit,k1-ccu.h | 134 ++++++++++++++++++ >> 2 files changed, 143 insertions(+), 4 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/soc/spacemit/spacemit,k1-syscon.yaml b/Documentation/devicetree/bindings/soc/spacemit/spacemit,k1-syscon.yaml >> index 07a6728e6f864..333c28e075b6c 100644 >> --- a/Documentation/devicetree/bindings/soc/spacemit/spacemit,k1-syscon.yaml >> +++ b/Documentation/devicetree/bindings/soc/spacemit/spacemit,k1-syscon.yaml >> @@ -19,6 +19,9 @@ properties: >> - spacemit,k1-syscon-apbc >> - spacemit,k1-syscon-apmu >> - spacemit,k1-syscon-mpmu >> + - spacemit,k1-syscon-rcpu >> + - spacemit,k1-syscon-rcpu2 >> + - spacemit,k1-syscon-apbc2 >> >> reg: >> maxItems: 1 . . . 32 >> @@ -180,6 +184,60 @@ >> #define CLK_TSEN_BUS 98 >> #define CLK_IPC_AP2AUD_BUS 99 >> >> +/* APBC resets */ >> + > I'd also suggest to drop above blank line, keep style consistent > with others in this file, some same below that I won't comment OK, I'll fix the weird extra line and will drop these blank lines as you suggest in v2. I'll post another version after Sunday. I recognize the merge window means I can't expect reviews during that time, but this code is waiting for the clock code to get accepted anyway. Thanks a lot. -Alex >> +#define RST_UART0 0 >> +#define RST_UART2 1 >> +#define RST_UART3 2 >> +#define RST_UART4 3 >> +#define RST_UART5 4 . . .