2019-02-13 13:44:19

by Jerome Brunet

Subject: [PATCH] dt-bindings: clock: axg-audio: unexpose controller inputs

Remove the bindings ID of the clock input of the controller. These
clocks are purely internal to the controller, exposing them was a
mistake. Actually, these should not even be in the provider and have
IDs to begin with.

Unexpose these IDs before:
* someone starts using them (even if there no valid reason to do so)
* the actual clocks are removed. The fact that they exist is just the
result of an ugly hack. This will be resolved in CCF when we can
reference DT directly in parent table.

Signed-off-by: Jerome Brunet <[email protected]>
---
drivers/clk/meson/axg-audio.h | 20 ++++++++++++++++++++
include/dt-bindings/clock/axg-audio-clkc.h | 20 --------------------
2 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/drivers/clk/meson/axg-audio.h b/drivers/clk/meson/axg-audio.h
index 7191b39c9d65..644f0b0fddf2 100644
--- a/drivers/clk/meson/axg-audio.h
+++ b/drivers/clk/meson/axg-audio.h
@@ -60,6 +60,26 @@
#define AUD_CLKID_MST5 6
#define AUD_CLKID_MST6 7
#define AUD_CLKID_MST7 8
+#define AUD_CLKID_SLV_SCLK0 9
+#define AUD_CLKID_SLV_SCLK1 10
+#define AUD_CLKID_SLV_SCLK2 11
+#define AUD_CLKID_SLV_SCLK3 12
+#define AUD_CLKID_SLV_SCLK4 13
+#define AUD_CLKID_SLV_SCLK5 14
+#define AUD_CLKID_SLV_SCLK6 15
+#define AUD_CLKID_SLV_SCLK7 16
+#define AUD_CLKID_SLV_SCLK8 17
+#define AUD_CLKID_SLV_SCLK9 18
+#define AUD_CLKID_SLV_LRCLK0 19
+#define AUD_CLKID_SLV_LRCLK1 20
+#define AUD_CLKID_SLV_LRCLK2 21
+#define AUD_CLKID_SLV_LRCLK3 22
+#define AUD_CLKID_SLV_LRCLK4 23
+#define AUD_CLKID_SLV_LRCLK5 24
+#define AUD_CLKID_SLV_LRCLK6 25
+#define AUD_CLKID_SLV_LRCLK7 26
+#define AUD_CLKID_SLV_LRCLK8 27
+#define AUD_CLKID_SLV_LRCLK9 28
#define AUD_CLKID_MST_A_MCLK_SEL 59
#define AUD_CLKID_MST_B_MCLK_SEL 60
#define AUD_CLKID_MST_C_MCLK_SEL 61
diff --git a/include/dt-bindings/clock/axg-audio-clkc.h b/include/dt-bindings/clock/axg-audio-clkc.h
index fd9c362099d9..eafb0de8466b 100644
--- a/include/dt-bindings/clock/axg-audio-clkc.h
+++ b/include/dt-bindings/clock/axg-audio-clkc.h
@@ -7,26 +7,6 @@
#ifndef __AXG_AUDIO_CLKC_BINDINGS_H
#define __AXG_AUDIO_CLKC_BINDINGS_H

-#define AUD_CLKID_SLV_SCLK0 9
-#define AUD_CLKID_SLV_SCLK1 10
-#define AUD_CLKID_SLV_SCLK2 11
-#define AUD_CLKID_SLV_SCLK3 12
-#define AUD_CLKID_SLV_SCLK4 13
-#define AUD_CLKID_SLV_SCLK5 14
-#define AUD_CLKID_SLV_SCLK6 15
-#define AUD_CLKID_SLV_SCLK7 16
-#define AUD_CLKID_SLV_SCLK8 17
-#define AUD_CLKID_SLV_SCLK9 18
-#define AUD_CLKID_SLV_LRCLK0 19
-#define AUD_CLKID_SLV_LRCLK1 20
-#define AUD_CLKID_SLV_LRCLK2 21
-#define AUD_CLKID_SLV_LRCLK3 22
-#define AUD_CLKID_SLV_LRCLK4 23
-#define AUD_CLKID_SLV_LRCLK5 24
-#define AUD_CLKID_SLV_LRCLK6 25
-#define AUD_CLKID_SLV_LRCLK7 26
-#define AUD_CLKID_SLV_LRCLK8 27
-#define AUD_CLKID_SLV_LRCLK9 28
#define AUD_CLKID_DDR_ARB 29
#define AUD_CLKID_PDM 30
#define AUD_CLKID_TDMIN_A 31
--
2.20.1

2019-02-13 13:55:00

by Maxime Jourdan

Subject: Re: [PATCH] dt-bindings: clock: axg-audio: unexpose controller inputs

On Wed, Feb 13, 2019 at 10:59 AM Jerome Brunet <[email protected]> wrote:
>
> Remove the bindings ID of the clock input of the controller. These
> clocks are purely internal to the controller, exposing them was a
> mistake. Actually, these should not even be in the provider and have
> IDs to begin with.
>
> Unexpose these IDs before:
> * someone starts using them (even if there no valid reason to do so)
> * the actual clocks are removed. The fact that they exist is just the
> result of an ugly hack. This will be resolved in CCF when we can
> reference DT directly in parent table.
>
> Signed-off-by: Jerome Brunet <[email protected]>
> ---
> drivers/clk/meson/axg-audio.h | 20 ++++++++++++++++++++
> include/dt-bindings/clock/axg-audio-clkc.h | 20 --------------------
> 2 files changed, 20 insertions(+), 20 deletions(-)
>
> diff --git a/drivers/clk/meson/axg-audio.h b/drivers/clk/meson/axg-audio.h
> index 7191b39c9d65..644f0b0fddf2 100644
> --- a/drivers/clk/meson/axg-audio.h
> +++ b/drivers/clk/meson/axg-audio.h
> @@ -60,6 +60,26 @@
> #define AUD_CLKID_MST5 6
> #define AUD_CLKID_MST6 7
> #define AUD_CLKID_MST7 8
> +#define AUD_CLKID_SLV_SCLK0 9
> +#define AUD_CLKID_SLV_SCLK1 10
> +#define AUD_CLKID_SLV_SCLK2 11
> +#define AUD_CLKID_SLV_SCLK3 12
> +#define AUD_CLKID_SLV_SCLK4 13
> +#define AUD_CLKID_SLV_SCLK5 14
> +#define AUD_CLKID_SLV_SCLK6 15
> +#define AUD_CLKID_SLV_SCLK7 16
> +#define AUD_CLKID_SLV_SCLK8 17
> +#define AUD_CLKID_SLV_SCLK9 18
> +#define AUD_CLKID_SLV_LRCLK0 19
> +#define AUD_CLKID_SLV_LRCLK1 20
> +#define AUD_CLKID_SLV_LRCLK2 21
> +#define AUD_CLKID_SLV_LRCLK3 22
> +#define AUD_CLKID_SLV_LRCLK4 23
> +#define AUD_CLKID_SLV_LRCLK5 24
> +#define AUD_CLKID_SLV_LRCLK6 25
> +#define AUD_CLKID_SLV_LRCLK7 26
> +#define AUD_CLKID_SLV_LRCLK8 27
> +#define AUD_CLKID_SLV_LRCLK9 28
> #define AUD_CLKID_MST_A_MCLK_SEL 59
> #define AUD_CLKID_MST_B_MCLK_SEL 60
> #define AUD_CLKID_MST_C_MCLK_SEL 61
> diff --git a/include/dt-bindings/clock/axg-audio-clkc.h b/include/dt-bindings/clock/axg-audio-clkc.h
> index fd9c362099d9..eafb0de8466b 100644
> --- a/include/dt-bindings/clock/axg-audio-clkc.h
> +++ b/include/dt-bindings/clock/axg-audio-clkc.h
> @@ -7,26 +7,6 @@
> #ifndef __AXG_AUDIO_CLKC_BINDINGS_H
> #define __AXG_AUDIO_CLKC_BINDINGS_H
>
> -#define AUD_CLKID_SLV_SCLK0 9
> -#define AUD_CLKID_SLV_SCLK1 10
> -#define AUD_CLKID_SLV_SCLK2 11
> -#define AUD_CLKID_SLV_SCLK3 12
> -#define AUD_CLKID_SLV_SCLK4 13
> -#define AUD_CLKID_SLV_SCLK5 14
> -#define AUD_CLKID_SLV_SCLK6 15
> -#define AUD_CLKID_SLV_SCLK7 16
> -#define AUD_CLKID_SLV_SCLK8 17
> -#define AUD_CLKID_SLV_SCLK9 18
> -#define AUD_CLKID_SLV_LRCLK0 19
> -#define AUD_CLKID_SLV_LRCLK1 20
> -#define AUD_CLKID_SLV_LRCLK2 21
> -#define AUD_CLKID_SLV_LRCLK3 22
> -#define AUD_CLKID_SLV_LRCLK4 23
> -#define AUD_CLKID_SLV_LRCLK5 24
> -#define AUD_CLKID_SLV_LRCLK6 25
> -#define AUD_CLKID_SLV_LRCLK7 26
> -#define AUD_CLKID_SLV_LRCLK8 27
> -#define AUD_CLKID_SLV_LRCLK9 28
> #define AUD_CLKID_DDR_ARB 29
> #define AUD_CLKID_PDM 30
> #define AUD_CLKID_TDMIN_A 31
> --
> 2.20.1
>

Acked-by: Maxime Jourdan <[email protected]>

2019-02-28 20:23:03

by Rob Herring

Subject: Re: [PATCH] dt-bindings: clock: axg-audio: unexpose controller inputs

On Wed, 13 Feb 2019 10:58:35 +0100, Jerome Brunet wrote:
> Remove the bindings ID of the clock input of the controller. These
> clocks are purely internal to the controller, exposing them was a
> mistake. Actually, these should not even be in the provider and have
> IDs to begin with.
>
> Unexpose these IDs before:
> * someone starts using them (even if there no valid reason to do so)
> * the actual clocks are removed. The fact that they exist is just the
> result of an ugly hack. This will be resolved in CCF when we can
> reference DT directly in parent table.
>
> Signed-off-by: Jerome Brunet <[email protected]>
> ---
> drivers/clk/meson/axg-audio.h | 20 ++++++++++++++++++++
> include/dt-bindings/clock/axg-audio-clkc.h | 20 --------------------
> 2 files changed, 20 insertions(+), 20 deletions(-)
>

Reviewed-by: Rob Herring <[email protected]>

2019-03-20 08:50:54

by Neil Armstrong

Subject: Re: [PATCH] dt-bindings: clock: axg-audio: unexpose controller inputs

On 13/02/2019 10:58, Jerome Brunet wrote:
> Remove the bindings ID of the clock input of the controller. These
> clocks are purely internal to the controller, exposing them was a
> mistake. Actually, these should not even be in the provider and have
> IDs to begin with.
>
> Unexpose these IDs before:
> * someone starts using them (even if there no valid reason to do so)
> * the actual clocks are removed. The fact that they exist is just the
> result of an ugly hack. This will be resolved in CCF when we can
> reference DT directly in parent table.
>
> Signed-off-by: Jerome Brunet <[email protected]>
> ---
> drivers/clk/meson/axg-audio.h | 20 ++++++++++++++++++++
> include/dt-bindings/clock/axg-audio-clkc.h | 20 --------------------
> 2 files changed, 20 insertions(+), 20 deletions(-)
>
> diff --git a/drivers/clk/meson/axg-audio.h b/drivers/clk/meson/axg-audio.h
> index 7191b39c9d65..644f0b0fddf2 100644
> --- a/drivers/clk/meson/axg-audio.h
> +++ b/drivers/clk/meson/axg-audio.h
> @@ -60,6 +60,26 @@
> #define AUD_CLKID_MST5 6
> #define AUD_CLKID_MST6 7
> #define AUD_CLKID_MST7 8
> +#define AUD_CLKID_SLV_SCLK0 9
> +#define AUD_CLKID_SLV_SCLK1 10
> +#define AUD_CLKID_SLV_SCLK2 11
> +#define AUD_CLKID_SLV_SCLK3 12
> +#define AUD_CLKID_SLV_SCLK4 13
> +#define AUD_CLKID_SLV_SCLK5 14
> +#define AUD_CLKID_SLV_SCLK6 15
> +#define AUD_CLKID_SLV_SCLK7 16
> +#define AUD_CLKID_SLV_SCLK8 17
> +#define AUD_CLKID_SLV_SCLK9 18
> +#define AUD_CLKID_SLV_LRCLK0 19
> +#define AUD_CLKID_SLV_LRCLK1 20
> +#define AUD_CLKID_SLV_LRCLK2 21
> +#define AUD_CLKID_SLV_LRCLK3 22
> +#define AUD_CLKID_SLV_LRCLK4 23
> +#define AUD_CLKID_SLV_LRCLK5 24
> +#define AUD_CLKID_SLV_LRCLK6 25
> +#define AUD_CLKID_SLV_LRCLK7 26
> +#define AUD_CLKID_SLV_LRCLK8 27
> +#define AUD_CLKID_SLV_LRCLK9 28
> #define AUD_CLKID_MST_A_MCLK_SEL 59
> #define AUD_CLKID_MST_B_MCLK_SEL 60
> #define AUD_CLKID_MST_C_MCLK_SEL 61
> diff --git a/include/dt-bindings/clock/axg-audio-clkc.h b/include/dt-bindings/clock/axg-audio-clkc.h
> index fd9c362099d9..eafb0de8466b 100644
> --- a/include/dt-bindings/clock/axg-audio-clkc.h
> +++ b/include/dt-bindings/clock/axg-audio-clkc.h
> @@ -7,26 +7,6 @@
> #ifndef __AXG_AUDIO_CLKC_BINDINGS_H
> #define __AXG_AUDIO_CLKC_BINDINGS_H
>
> -#define AUD_CLKID_SLV_SCLK0 9
> -#define AUD_CLKID_SLV_SCLK1 10
> -#define AUD_CLKID_SLV_SCLK2 11
> -#define AUD_CLKID_SLV_SCLK3 12
> -#define AUD_CLKID_SLV_SCLK4 13
> -#define AUD_CLKID_SLV_SCLK5 14
> -#define AUD_CLKID_SLV_SCLK6 15
> -#define AUD_CLKID_SLV_SCLK7 16
> -#define AUD_CLKID_SLV_SCLK8 17
> -#define AUD_CLKID_SLV_SCLK9 18
> -#define AUD_CLKID_SLV_LRCLK0 19
> -#define AUD_CLKID_SLV_LRCLK1 20
> -#define AUD_CLKID_SLV_LRCLK2 21
> -#define AUD_CLKID_SLV_LRCLK3 22
> -#define AUD_CLKID_SLV_LRCLK4 23
> -#define AUD_CLKID_SLV_LRCLK5 24
> -#define AUD_CLKID_SLV_LRCLK6 25
> -#define AUD_CLKID_SLV_LRCLK7 26
> -#define AUD_CLKID_SLV_LRCLK8 27
> -#define AUD_CLKID_SLV_LRCLK9 28
> #define AUD_CLKID_DDR_ARB 29
> #define AUD_CLKID_PDM 30
> #define AUD_CLKID_TDMIN_A 31
>


Applied to next/drivers