Message ID | 20250509132414.3752159-1-raghav.s@samsung.com |
---|---|
State | Superseded |
Headers | show |
Series | [v1] dt-bindings: clock: exynosautov920: add hsi2 clock definitions | expand |
> -----Original Message----- > From: Raghav Sharma <raghav.s@samsung.com> > Sent: Friday, May 9, 2025 6:54 PM > To: krzk@kernel.org; s.nawrocki@samsung.com; cw00.choi@samsung.com; > alim.akhtar@samsung.com; mturquette@baylibre.com; sboyd@kernel.org; > robh@kernel.org; conor+dt@kernel.org; sunyeal.hong@samsung.com; > shin.son@samsung.com > Cc: linux-samsung-soc@vger.kernel.org; linux-clk@vger.kernel.org; > devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux- > kernel@vger.kernel.org; Raghav Sharma <raghav.s@samsung.com> > Subject: [PATCH v1] dt-bindings: clock: exynosautov920: add hsi2 clock > definitions > > Add device tree clock binding definitions for CMU_HSI2 > > Signed-off-by: Raghav Sharma <raghav.s@samsung.com> > --- > .../clock/samsung,exynosautov920-clock.yaml | 29 +++++++++++++++++-- > .../clock/samsung,exynosautov920.h | 9 ++++++ > 2 files changed, 36 insertions(+), 2 deletions(-) > Hmm, you sent three separate patches that are interdependent but did not mention their dependencies. e.g. The patch that adds the clock driver cannot be compiled without the one that adds the header file, which is part of this binding patch Go through the below section of submitting-patches documentation https://www.kernel.org/doc/html/latest/process/submitting-patches.html#separate-your-changes Make sure that each patch is individually compilable (or dependencies are mentioned clearly) and bisectable ask someone if you are unsure of what you are submitting.
diff --git a/Documentation/devicetree/bindings/clock/samsung,exynosautov920-clock.yaml b/Documentation/devicetree/bindings/clock/samsung,exynosautov920-clock.yaml index 6961a68098f4..3cbb1dc8d828 100644 --- a/Documentation/devicetree/bindings/clock/samsung,exynosautov920-clock.yaml +++ b/Documentation/devicetree/bindings/clock/samsung,exynosautov920-clock.yaml @@ -41,14 +41,15 @@ properties: - samsung,exynosautov920-cmu-misc - samsung,exynosautov920-cmu-hsi0 - samsung,exynosautov920-cmu-hsi1 + - samsung,exynosautov920-cmu-hsi2 clocks: minItems: 1 - maxItems: 4 + maxItems: 5 clock-names: minItems: 1 - maxItems: 4 + maxItems: 5 "#clock-cells": const: 1 @@ -201,6 +202,30 @@ allOf: - const: usbdrd - const: mmc_card + - if: + properties: + compatible: + contains: + const: samsung,exynosautov920-cmu-hsi2 + + then: + properties: + clocks: + items: + - description: External reference clock (38.4 MHz) + - description: CMU_HSI2 NOC clock (from CMU_TOP) + - description: CMU_HSI2 NOC UFS clock (from CMU_TOP) + - description: CMU_HSI2 UFS EMBD clock (from CMU_TOP) + - description: CMU_HSI2 ETHERNET clock (from CMU_TOP) + + clock-names: + items: + - const: oscclk + - const: noc + - const: ufs + - const: embd + - const: ethernet + required: - compatible - "#clock-cells" diff --git a/include/dt-bindings/clock/samsung,exynosautov920.h b/include/dt-bindings/clock/samsung,exynosautov920.h index 5e6896e9627f..93e6233d1358 100644 --- a/include/dt-bindings/clock/samsung,exynosautov920.h +++ b/include/dt-bindings/clock/samsung,exynosautov920.h @@ -286,4 +286,13 @@ #define CLK_MOUT_HSI1_USBDRD_USER 3 #define CLK_MOUT_HSI1_USBDRD 4 +/* CMU_HSI2 */ +#define FOUT_PLL_ETH 1 +#define CLK_MOUT_HSI2_NOC_UFS_USER 2 +#define CLK_MOUT_HSI2_UFS_EMBD_USER 3 +#define CLK_MOUT_HSI2_ETHERNET 4 +#define CLK_MOUT_HSI2_ETHERNET_USER 5 +#define CLK_DOUT_HSI2_ETHERNET 6 +#define CLK_DOUT_HSI2_ETHERNET_PTP 7 + #endif /* _DT_BINDINGS_CLOCK_EXYNOSAUTOV920_H */
Add device tree clock binding definitions for CMU_HSI2 Signed-off-by: Raghav Sharma <raghav.s@samsung.com> --- .../clock/samsung,exynosautov920-clock.yaml | 29 +++++++++++++++++-- .../clock/samsung,exynosautov920.h | 9 ++++++ 2 files changed, 36 insertions(+), 2 deletions(-)