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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Wed, 20 Mar 2019 09:49:31 +0100
From:   Neil Armstrong <narmstrong@...libre.com>
To:     Jerome Brunet <jbrunet@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>,
        Michael Turquette <mturquette@...libre.com>
Cc:     Kevin Hilman <khilman@...libre.com>, linux-kernel@...r.kernel.org,
        linux-clk@...r.kernel.org, linux-amlogic@...ts.infradead.org,
        linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
        Maxime Jourdan <mjourdan@...libre.com>
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 <jbrunet@...libre.com>
> ---
>  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