From: Rob Herring <robh@kernel.org>
To: Luo Jie <quic_luoj@quicinc.com>
Cc: Georgi Djakov <djakov@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Anusha Rao <quic_anusha@quicinc.com>,
Konrad Dybcio <konradybcio@kernel.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Richard Cochran <richardcochran@gmail.com>,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <will@kernel.org>,
linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-clk@vger.kernel.org, netdev@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, quic_kkumarcs@quicinc.com,
quic_linchen@quicinc.com, quic_leiwei@quicinc.com,
quic_pavir@quicinc.com, quic_suruchia@quicinc.com
Subject: Re: [PATCH v3 07/10] dt-bindings: clock: qcom: Add NSS clock controller for IPQ5424 SoC
Date: Thu, 10 Jul 2025 17:55:39 -0500 [thread overview]
Message-ID: <20250710225539.GA29510-robh@kernel.org> (raw)
In-Reply-To: <20250710-qcom_ipq5424_nsscc-v3-7-f149dc461212@quicinc.com>
On Thu, Jul 10, 2025 at 08:28:15PM +0800, Luo Jie wrote:
> NSS clock controller provides the clocks and resets to the networking
> blocks such as PPE (Packet Process Engine) and UNIPHY (PCS) on IPQ5424
> devices.
>
> Add the compatible "qcom,ipq5424-nsscc" support based on the current
> IPQ9574 NSS clock controller DT binding file. ICC clocks are always
> provided by the NSS clock controller of IPQ9574 and IPQ5424, so add
> interconnect-cells as required DT property.
>
> Also add master/slave ids for IPQ5424 networking interfaces, which is
> used by nss-ipq5424 driver for providing interconnect services using
> icc-clk framework.
>
> Signed-off-by: Luo Jie <quic_luoj@quicinc.com>
> ---
> .../bindings/clock/qcom,ipq9574-nsscc.yaml | 14 +++--
> include/dt-bindings/clock/qcom,ipq5424-nsscc.h | 65 ++++++++++++++++++++++
> include/dt-bindings/interconnect/qcom,ipq5424.h | 13 +++++
> include/dt-bindings/reset/qcom,ipq5424-nsscc.h | 46 +++++++++++++++
> 4 files changed, 134 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/qcom,ipq9574-nsscc.yaml b/Documentation/devicetree/bindings/clock/qcom,ipq9574-nsscc.yaml
> index b9ca69172adc..86ee9ffb2eda 100644
> --- a/Documentation/devicetree/bindings/clock/qcom,ipq9574-nsscc.yaml
> +++ b/Documentation/devicetree/bindings/clock/qcom,ipq9574-nsscc.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/clock/qcom,ipq9574-nsscc.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Qualcomm Networking Sub System Clock & Reset Controller on IPQ9574
> +title: Qualcomm Networking Sub System Clock & Reset Controller on IPQ9574 and IPQ5424
>
> maintainers:
> - Bjorn Andersson <andersson@kernel.org>
> @@ -12,15 +12,19 @@ maintainers:
>
> description: |
> Qualcomm networking sub system clock control module provides the clocks,
> - resets on IPQ9574
> + resets on IPQ9574 and IPQ5424
>
> - See also::
> + See also:
> + include/dt-bindings/clock/qcom,ipq5424-nsscc.h
> include/dt-bindings/clock/qcom,ipq9574-nsscc.h
> + include/dt-bindings/reset/qcom,ipq5424-nsscc.h
> include/dt-bindings/reset/qcom,ipq9574-nsscc.h
>
> properties:
> compatible:
> - const: qcom,ipq9574-nsscc
> + enum:
> + - qcom,ipq5424-nsscc
> + - qcom,ipq9574-nsscc
>
> clocks:
> items:
> @@ -57,6 +61,7 @@ required:
> - compatible
> - clocks
> - clock-names
> + - '#interconnect-cells'
You just made this required for everyone. Again, that's an ABI change.
>
> allOf:
> - $ref: qcom,gcc.yaml#
> @@ -94,5 +99,6 @@ examples:
> "bus";
> #clock-cells = <1>;
> #reset-cells = <1>;
> + #interconnect-cells = <1>;
> };
> ...
> diff --git a/include/dt-bindings/clock/qcom,ipq5424-nsscc.h b/include/dt-bindings/clock/qcom,ipq5424-nsscc.h
> new file mode 100644
> index 000000000000..59ce056ead93
> --- /dev/null
> +++ b/include/dt-bindings/clock/qcom,ipq5424-nsscc.h
> @@ -0,0 +1,65 @@
> +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
> +/*
> + * Copyright (c) 2025, Qualcomm Innovation Center, Inc. All rights reserved.
> + */
> +
> +#ifndef _DT_BINDINGS_CLOCK_QCOM_IPQ5424_NSSCC_H
> +#define _DT_BINDINGS_CLOCK_QCOM_IPQ5424_NSSCC_H
> +
> +/* NSS_CC clocks */
> +#define NSS_CC_CE_APB_CLK 0
> +#define NSS_CC_CE_AXI_CLK 1
> +#define NSS_CC_CE_CLK_SRC 2
> +#define NSS_CC_CFG_CLK_SRC 3
> +#define NSS_CC_DEBUG_CLK 4
> +#define NSS_CC_EIP_BFDCD_CLK_SRC 5
> +#define NSS_CC_EIP_CLK 6
> +#define NSS_CC_NSS_CSR_CLK 7
> +#define NSS_CC_NSSNOC_CE_APB_CLK 8
> +#define NSS_CC_NSSNOC_CE_AXI_CLK 9
> +#define NSS_CC_NSSNOC_EIP_CLK 10
> +#define NSS_CC_NSSNOC_NSS_CSR_CLK 11
> +#define NSS_CC_NSSNOC_PPE_CFG_CLK 12
> +#define NSS_CC_NSSNOC_PPE_CLK 13
> +#define NSS_CC_PORT1_MAC_CLK 14
> +#define NSS_CC_PORT1_RX_CLK 15
> +#define NSS_CC_PORT1_RX_CLK_SRC 16
> +#define NSS_CC_PORT1_RX_DIV_CLK_SRC 17
> +#define NSS_CC_PORT1_TX_CLK 18
> +#define NSS_CC_PORT1_TX_CLK_SRC 19
> +#define NSS_CC_PORT1_TX_DIV_CLK_SRC 20
> +#define NSS_CC_PORT2_MAC_CLK 21
> +#define NSS_CC_PORT2_RX_CLK 22
> +#define NSS_CC_PORT2_RX_CLK_SRC 23
> +#define NSS_CC_PORT2_RX_DIV_CLK_SRC 24
> +#define NSS_CC_PORT2_TX_CLK 25
> +#define NSS_CC_PORT2_TX_CLK_SRC 26
> +#define NSS_CC_PORT2_TX_DIV_CLK_SRC 27
> +#define NSS_CC_PORT3_MAC_CLK 28
> +#define NSS_CC_PORT3_RX_CLK 29
> +#define NSS_CC_PORT3_RX_CLK_SRC 30
> +#define NSS_CC_PORT3_RX_DIV_CLK_SRC 31
> +#define NSS_CC_PORT3_TX_CLK 32
> +#define NSS_CC_PORT3_TX_CLK_SRC 33
> +#define NSS_CC_PORT3_TX_DIV_CLK_SRC 34
> +#define NSS_CC_PPE_CLK_SRC 35
> +#define NSS_CC_PPE_EDMA_CFG_CLK 36
> +#define NSS_CC_PPE_EDMA_CLK 37
> +#define NSS_CC_PPE_SWITCH_BTQ_CLK 38
> +#define NSS_CC_PPE_SWITCH_CFG_CLK 39
> +#define NSS_CC_PPE_SWITCH_CLK 40
> +#define NSS_CC_PPE_SWITCH_IPE_CLK 41
> +#define NSS_CC_UNIPHY_PORT1_RX_CLK 42
> +#define NSS_CC_UNIPHY_PORT1_TX_CLK 43
> +#define NSS_CC_UNIPHY_PORT2_RX_CLK 44
> +#define NSS_CC_UNIPHY_PORT2_TX_CLK 45
> +#define NSS_CC_UNIPHY_PORT3_RX_CLK 46
> +#define NSS_CC_UNIPHY_PORT3_TX_CLK 47
> +#define NSS_CC_XGMAC0_PTP_REF_CLK 48
> +#define NSS_CC_XGMAC0_PTP_REF_DIV_CLK_SRC 49
> +#define NSS_CC_XGMAC1_PTP_REF_CLK 50
> +#define NSS_CC_XGMAC1_PTP_REF_DIV_CLK_SRC 51
> +#define NSS_CC_XGMAC2_PTP_REF_CLK 52
> +#define NSS_CC_XGMAC2_PTP_REF_DIV_CLK_SRC 53
> +
> +#endif
> diff --git a/include/dt-bindings/interconnect/qcom,ipq5424.h b/include/dt-bindings/interconnect/qcom,ipq5424.h
> index 66cd9a9ece03..a78604beff99 100644
> --- a/include/dt-bindings/interconnect/qcom,ipq5424.h
> +++ b/include/dt-bindings/interconnect/qcom,ipq5424.h
> @@ -27,4 +27,17 @@
> #define MASTER_NSSNOC_SNOC_1 22
> #define SLAVE_NSSNOC_SNOC_1 23
>
> +#define MASTER_NSSNOC_PPE 0
> +#define SLAVE_NSSNOC_PPE 1
> +#define MASTER_NSSNOC_PPE_CFG 2
> +#define SLAVE_NSSNOC_PPE_CFG 3
> +#define MASTER_NSSNOC_NSS_CSR 4
> +#define SLAVE_NSSNOC_NSS_CSR 5
> +#define MASTER_NSSNOC_CE_AXI 6
> +#define SLAVE_NSSNOC_CE_AXI 7
> +#define MASTER_NSSNOC_CE_APB 8
> +#define SLAVE_NSSNOC_CE_APB 9
> +#define MASTER_NSSNOC_EIP 10
> +#define SLAVE_NSSNOC_EIP 11
> +
> #endif /* INTERCONNECT_QCOM_IPQ5424_H */
> diff --git a/include/dt-bindings/reset/qcom,ipq5424-nsscc.h b/include/dt-bindings/reset/qcom,ipq5424-nsscc.h
> new file mode 100644
> index 000000000000..f2f7eaa28b21
> --- /dev/null
> +++ b/include/dt-bindings/reset/qcom,ipq5424-nsscc.h
> @@ -0,0 +1,46 @@
> +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
> +/*
> + * Copyright (c) 2025, Qualcomm Innovation Center, Inc. All rights reserved.
> + */
> +
> +#ifndef _DT_BINDINGS_RESET_QCOM_IPQ5424_NSSCC_H
> +#define _DT_BINDINGS_RESET_QCOM_IPQ5424_NSSCC_H
> +
> +#define NSS_CC_CE_APB_CLK_ARES 0
> +#define NSS_CC_CE_AXI_CLK_ARES 1
> +#define NSS_CC_DEBUG_CLK_ARES 2
> +#define NSS_CC_EIP_CLK_ARES 3
> +#define NSS_CC_NSS_CSR_CLK_ARES 4
> +#define NSS_CC_NSSNOC_CE_APB_CLK_ARES 5
> +#define NSS_CC_NSSNOC_CE_AXI_CLK_ARES 6
> +#define NSS_CC_NSSNOC_EIP_CLK_ARES 7
> +#define NSS_CC_NSSNOC_NSS_CSR_CLK_ARES 8
> +#define NSS_CC_NSSNOC_PPE_CLK_ARES 9
> +#define NSS_CC_NSSNOC_PPE_CFG_CLK_ARES 10
> +#define NSS_CC_PORT1_MAC_CLK_ARES 11
> +#define NSS_CC_PORT1_RX_CLK_ARES 12
> +#define NSS_CC_PORT1_TX_CLK_ARES 13
> +#define NSS_CC_PORT2_MAC_CLK_ARES 14
> +#define NSS_CC_PORT2_RX_CLK_ARES 15
> +#define NSS_CC_PORT2_TX_CLK_ARES 16
> +#define NSS_CC_PORT3_MAC_CLK_ARES 17
> +#define NSS_CC_PORT3_RX_CLK_ARES 18
> +#define NSS_CC_PORT3_TX_CLK_ARES 19
> +#define NSS_CC_PPE_BCR 20
> +#define NSS_CC_PPE_EDMA_CLK_ARES 21
> +#define NSS_CC_PPE_EDMA_CFG_CLK_ARES 22
> +#define NSS_CC_PPE_SWITCH_BTQ_CLK_ARES 23
> +#define NSS_CC_PPE_SWITCH_CLK_ARES 24
> +#define NSS_CC_PPE_SWITCH_CFG_CLK_ARES 25
> +#define NSS_CC_PPE_SWITCH_IPE_CLK_ARES 26
> +#define NSS_CC_UNIPHY_PORT1_RX_CLK_ARES 27
> +#define NSS_CC_UNIPHY_PORT1_TX_CLK_ARES 28
> +#define NSS_CC_UNIPHY_PORT2_RX_CLK_ARES 29
> +#define NSS_CC_UNIPHY_PORT2_TX_CLK_ARES 30
> +#define NSS_CC_UNIPHY_PORT3_RX_CLK_ARES 31
> +#define NSS_CC_UNIPHY_PORT3_TX_CLK_ARES 32
> +#define NSS_CC_XGMAC0_PTP_REF_CLK_ARES 33
> +#define NSS_CC_XGMAC1_PTP_REF_CLK_ARES 34
> +#define NSS_CC_XGMAC2_PTP_REF_CLK_ARES 35
> +
> +#endif
>
> --
> 2.34.1
>
next prev parent reply other threads:[~2025-07-11 1:20 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-10 12:28 [PATCH v3 00/10] Add Network Subsystem (NSS) clock controller support for IPQ5424 SoC Luo Jie
2025-07-10 12:28 ` [PATCH v3 01/10] dt-bindings: interconnect: Add Qualcomm IPQ5424 NSSNOC IDs Luo Jie
2025-07-18 13:41 ` Georgi Djakov
2025-07-10 12:28 ` [PATCH v3 02/10] clk: qcom: ipq5424: Enable NSS NoC clocks to use icc-clk Luo Jie
2025-07-17 20:40 ` Konrad Dybcio
2025-07-18 9:21 ` Luo Jie
2025-07-10 12:28 ` [PATCH v3 03/10] dt-bindings: clock: gcc-ipq5424: Add definition for GPLL0_OUT_AUX Luo Jie
2025-07-10 12:28 ` [PATCH v3 04/10] clock: qcom: gcc-ipq5424: Add gpll0_out_aux clock Luo Jie
2025-07-17 20:41 ` Konrad Dybcio
2025-07-18 9:12 ` Luo Jie
2025-07-10 12:28 ` [PATCH v3 05/10] dt-bindings: clock: ipq9574: Rename NSS CC source clocks to drop rate Luo Jie
2025-07-10 22:54 ` Rob Herring
2025-07-11 12:15 ` Konrad Dybcio
2025-07-18 9:12 ` Luo Jie
2025-07-18 9:28 ` Konrad Dybcio
2025-07-18 15:51 ` Luo Jie
2025-07-29 13:53 ` Konrad Dybcio
2025-07-29 13:57 ` Krzysztof Kozlowski
2025-07-30 11:50 ` Luo Jie
2025-07-10 12:28 ` [PATCH v3 06/10] arm64: dts: qcom: ipq9574: Rename NSSCC source clock names " Luo Jie
2025-07-10 12:28 ` [PATCH v3 07/10] dt-bindings: clock: qcom: Add NSS clock controller for IPQ5424 SoC Luo Jie
2025-07-10 22:55 ` Rob Herring [this message]
2025-07-11 12:16 ` Konrad Dybcio
2025-07-18 9:05 ` Luo Jie
2025-07-10 12:28 ` [PATCH v3 08/10] clk: qcom: Add NSS clock controller driver for IPQ5424 Luo Jie
2025-07-10 12:28 ` [PATCH v3 09/10] arm64: dts: qcom: ipq5424: Add NSS clock controller node Luo Jie
2025-07-10 12:28 ` [PATCH v3 10/10] arm64: defconfig: Build NSS clock controller driver for IPQ5424 Luo Jie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250710225539.GA29510-robh@kernel.org \
--to=robh@kernel.org \
--cc=andersson@kernel.org \
--cc=catalin.marinas@arm.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=djakov@kernel.org \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=mturquette@baylibre.com \
--cc=netdev@vger.kernel.org \
--cc=p.zabel@pengutronix.de \
--cc=quic_anusha@quicinc.com \
--cc=quic_kkumarcs@quicinc.com \
--cc=quic_leiwei@quicinc.com \
--cc=quic_linchen@quicinc.com \
--cc=quic_luoj@quicinc.com \
--cc=quic_pavir@quicinc.com \
--cc=quic_suruchia@quicinc.com \
--cc=richardcochran@gmail.com \
--cc=sboyd@kernel.org \
--cc=will@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.