[PATCH v2 0/3] GS101: rename cmu_misc clock-names

[PATCH v2 0/3] GS101: rename cmu_misc clock-names

[Tudor Ambarus]

Rename the cmu_misc clock-names to just "bus" and "sss" because
naming is local to the module, so cmu_misc is implied. As the bindings
and the device tree have not made a release yet (v6.8), comply with the
renamed clocks in the device tree and the clock driver.

The patch set is expected to be queued through Krzysztof's tree as he
was the one that queued all the previous gs101 patches.

Changes in v2:
- comply with the new cmu_misc clock names in the clock driver

v1:
- https://lore.kernel.org/all/c6cc6e74-6c3d-439b-8dc1-bc50a88a3d8f@linaro.org/

Tudor Ambarus (3):
  dt-bindings: clock: gs101: rename cmu_misc clock-names
  arm64: dts: exynos: gs101: comply with the new cmu_misc clock names
  clk: samsung: clk-gs101: comply with the new dt cmu_misc clock names

 .../devicetree/bindings/clock/google,gs101-clock.yaml         | 4 ++--
 arch/arm64/boot/dts/exynos/google/gs101.dtsi                  | 2 +-
 drivers/clk/samsung/clk-gs101.c                               | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

-- 
2.43.0.472.g3155946c3a-goog

[PATCH v2 1/3] dt-bindings: clock: gs101: rename cmu_misc clock-names

[Tudor Ambarus]

'bus' and 'ip' are sufficient because naming is local to the module.
As the bindings have not made a release yet, rename the cmu_misc
clock-names.

Fixes: 0a910f160638 ("dt-bindings: clock: Add Google gs101 clock management unit bindings")
Suggested-by: Rob Herring <robh@kernel.org>
Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
---
 .../devicetree/bindings/clock/google,gs101-clock.yaml         | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/clock/google,gs101-clock.yaml b/Documentation/devicetree/bindings/clock/google,gs101-clock.yaml
index 3eebc03a309b..ca7fdada3ff2 100644
--- a/Documentation/devicetree/bindings/clock/google,gs101-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/google,gs101-clock.yaml
@@ -85,8 +85,8 @@ allOf:
 
         clock-names:
           items:
-            - const: dout_cmu_misc_bus
-            - const: dout_cmu_misc_sss
+            - const: bus
+            - const: sss
 
 additionalProperties: false
 
-- 
2.43.0.472.g3155946c3a-goog

Re: [PATCH v2 1/3] dt-bindings: clock: gs101: rename cmu_misc clock-names

[Rob Herring]

On Tue, 09 Jan 2024 11:49:06 +0000, Tudor Ambarus wrote:
> 'bus' and 'ip' are sufficient because naming is local to the module.
> As the bindings have not made a release yet, rename the cmu_misc
> clock-names.
> 
> Fixes: 0a910f160638 ("dt-bindings: clock: Add Google gs101 clock management unit bindings")
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
> ---
>  .../devicetree/bindings/clock/google,gs101-clock.yaml         | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 

Acked-by: Rob Herring <robh@kernel.org>

Re: [PATCH v2 1/3] dt-bindings: clock: gs101: rename cmu_misc clock-names

[Sam Protsenko]

On Tue, Jan 9, 2024 at 5:49 AM Tudor Ambarus <tudor.ambarus@linaro.org> wrote:
>
> 'bus' and 'ip' are sufficient because naming is local to the module.
> As the bindings have not made a release yet, rename the cmu_misc
> clock-names.
>
> Fixes: 0a910f160638 ("dt-bindings: clock: Add Google gs101 clock management unit bindings")
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
> ---

Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org>

>  .../devicetree/bindings/clock/google,gs101-clock.yaml         | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/google,gs101-clock.yaml b/Documentation/devicetree/bindings/clock/google,gs101-clock.yaml
> index 3eebc03a309b..ca7fdada3ff2 100644
> --- a/Documentation/devicetree/bindings/clock/google,gs101-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/google,gs101-clock.yaml
> @@ -85,8 +85,8 @@ allOf:
>
>          clock-names:
>            items:
> -            - const: dout_cmu_misc_bus
> -            - const: dout_cmu_misc_sss
> +            - const: bus
> +            - const: sss
>
>  additionalProperties: false
>
> --
> 2.43.0.472.g3155946c3a-goog
>

[PATCH v2 2/3] arm64: dts: exynos: gs101: comply with the new cmu_misc clock names

[Tudor Ambarus]

The cmu_misc clock-names were renamed to just "bus" and "sss" because
naming is local to the module, so cmu_misc is implied. As the bindings
and the device tree have not made a release yet, comply with the
renamed clocks.

Suggested-by: Rob Herring <robh@kernel.org>
Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
---
 arch/arm64/boot/dts/exynos/google/gs101.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/exynos/google/gs101.dtsi b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
index 9747cb3fa03a..d838e3a7af6e 100644
--- a/arch/arm64/boot/dts/exynos/google/gs101.dtsi
+++ b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
@@ -289,7 +289,7 @@ cmu_misc: clock-controller@10010000 {
 			#clock-cells = <1>;
 			clocks = <&cmu_top CLK_DOUT_CMU_MISC_BUS>,
 				 <&cmu_top CLK_DOUT_CMU_MISC_SSS>;
-			clock-names = "dout_cmu_misc_bus", "dout_cmu_misc_sss";
+			clock-names = "bus", "sss";
 		};
 
 		watchdog_cl0: watchdog@10060000 {
-- 
2.43.0.472.g3155946c3a-goog

[PATCH v2 3/3] clk: samsung: clk-gs101: comply with the new dt cmu_misc clock names

[Tudor Ambarus]

The cmu_misc clock-names were renamed to just "bus" and "sss" because
naming is local to the module, so cmu_misc is implied. As the bindings
and the device tree have not made a release yet, comply with the
renamed clocks.

Suggested-by: Rob Herring <robh@kernel.org>
Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
---
 drivers/clk/samsung/clk-gs101.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/samsung/clk-gs101.c b/drivers/clk/samsung/clk-gs101.c
index 0964bb11657f..782993951fff 100644
--- a/drivers/clk/samsung/clk-gs101.c
+++ b/drivers/clk/samsung/clk-gs101.c
@@ -2475,7 +2475,7 @@ static const struct samsung_cmu_info misc_cmu_info __initconst = {
 	.nr_clk_ids		= CLKS_NR_MISC,
 	.clk_regs		= misc_clk_regs,
 	.nr_clk_regs		= ARRAY_SIZE(misc_clk_regs),
-	.clk_name		= "dout_cmu_misc_bus",
+	.clk_name		= "bus",
 };
 
 /* ---- platform_driver ----------------------------------------------------- */
-- 
2.43.0.472.g3155946c3a-goog

Re: [PATCH v2 0/3] GS101: rename cmu_misc clock-names

[Krzysztof Kozlowski]

On Tue, 09 Jan 2024 11:49:05 +0000, Tudor Ambarus wrote:
> Rename the cmu_misc clock-names to just "bus" and "sss" because
> naming is local to the module, so cmu_misc is implied. As the bindings
> and the device tree have not made a release yet (v6.8), comply with the
> renamed clocks in the device tree and the clock driver.
> 
> The patch set is expected to be queued through Krzysztof's tree as he
> was the one that queued all the previous gs101 patches.
> 
> [...]

Applied, thanks!

[1/3] dt-bindings: clock: gs101: rename cmu_misc clock-names
      https://git.kernel.org/krzk/linux/c/1755c4b0372a2cf1e7124956b8cfebcb51083208
[2/3] arm64: dts: exynos: gs101: comply with the new cmu_misc clock names
      https://git.kernel.org/krzk/linux/c/80c86ff6800b857c8008cebe7b8d22a6e574e68d
[3/3] clk: samsung: clk-gs101: comply with the new dt cmu_misc clock names
      https://git.kernel.org/krzk/linux/c/d76c762e7ee04af79e1c127422e0bbcb5f123018

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>