linux-kernel - Re: [PATCH 3/3] clk: qcom: gcc-msm8939: Make silicon specific updates for msm8939

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <158752417863.132238.13958544237045504884@swboyd.mtv.corp.google.com>
Date:   Tue, 21 Apr 2020 19:56:18 -0700
From:   Stephen Boyd <sboyd@...nel.org>
To:     Bryan O'Donoghue <bryan.odonoghue@...aro.org>, agross@...nel.org,
        bjorn.andersson@...aro.org, mturquette@...libre.com,
        robh+dt@...nel.org
Cc:     linux-arm-msm@...r.kernel.org, linux-clk@...r.kernel.org,
        linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
        shawn.guo@...aro.org, p.zabel@...gutronix.de,
        Bryan O'Donoghue <bryan.odonoghue@...aro.org>
Subject: Re: [PATCH 3/3] clk: qcom: gcc-msm8939: Make silicon specific updates for msm8939

Quoting Bryan O'Donoghue (2020-04-19 05:18:08)
> The msm8939 is based on the msm8916. It is compatible in several ways but,
> has additional functional blocks added which require additional PLL
> sources. In some cases functional blocks from the msm8916 have different
> clock sources or different supported frequencies.
> 
> This patch encapsulates the conversion from msm8916 to msm8939.
> 
> Cc: Andy Gross <agross@...nel.org>
> Cc: Bjorn Andersson <bjorn.andersson@...aro.org>
> Cc: Michael Turquette <mturquette@...libre.com>
> Cc: Stephen Boyd <sboyd@...nel.org>
> Cc: Rob Herring <robh+dt@...nel.org>
> Cc: Philipp Zabel <p.zabel@...gutronix.de>
> Cc: linux-arm-msm@...r.kernel.org
> Cc: linux-clk@...r.kernel.org
> Cc: linux-kernel@...r.kernel.org
> Cc: devicetree@...r.kernel.org
> Signed-off-by: Shawn Guo <shawn.guo@...aro.org>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
> ---
>  drivers/clk/qcom/gcc-msm8939.c | 832 ++++++++++++++++++++++++++++-----
>  1 file changed, 709 insertions(+), 123 deletions(-)
> 

Please use the new way of specifying clk parents with the clk_hw and DT
based way. Some qcom clk drivers have already migrated. If this is a
copy of the other driver then perhaps also modify that one in the series
so that git can find the copy still. Sigh I still haven't written the
documentation about it!

Look for .parent_data in drivers/clk/qcom/gcc-sc7180.c for the best
guidance.

> @@ -1695,6 +1997,7 @@ static struct clk_branch gcc_blsp1_qup4_spi_apps_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_blsp1_qup5_i2c_apps_clk = {
>         .halt_reg = 0x06020,
>         .clkr = {
> @@ -1712,6 +2015,7 @@ static struct clk_branch gcc_blsp1_qup5_i2c_apps_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_blsp1_qup5_spi_apps_clk = {
>         .halt_reg = 0x0601c,
>         .clkr = {
> @@ -1729,6 +2033,7 @@ static struct clk_branch gcc_blsp1_qup5_spi_apps_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_blsp1_qup6_i2c_apps_clk = {
>         .halt_reg = 0x07020,
>         .clkr = {
> @@ -1746,6 +2051,7 @@ static struct clk_branch gcc_blsp1_qup6_i2c_apps_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_blsp1_qup6_spi_apps_clk = {
>         .halt_reg = 0x0701c,
>         .clkr = {
> @@ -1797,6 +2103,7 @@ static struct clk_branch gcc_blsp1_uart2_apps_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_boot_rom_ahb_clk = {
>         .halt_reg = 0x1300c,
>         .halt_check = BRANCH_HALT_VOTED,
> @@ -2341,6 +2648,7 @@ static struct clk_branch gcc_camss_vfe_axi_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_crypto_ahb_clk = {
>         .halt_reg = 0x16024,
>         .halt_check = BRANCH_HALT_VOTED,
> @@ -2359,6 +2667,7 @@ static struct clk_branch gcc_crypto_ahb_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_crypto_axi_clk = {
>         .halt_reg = 0x16020,
>         .halt_check = BRANCH_HALT_VOTED,
> @@ -2377,6 +2686,7 @@ static struct clk_branch gcc_crypto_axi_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_crypto_clk = {
>         .halt_reg = 0x1601c,
>         .halt_check = BRANCH_HALT_VOTED,
> @@ -2650,6 +3011,7 @@ static struct clk_branch gcc_oxili_gfx3d_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_pdm2_clk = {
>         .halt_reg = 0x4400c,
>         .clkr = {
> @@ -2667,6 +3029,7 @@ static struct clk_branch gcc_pdm2_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */
>  static struct clk_branch gcc_pdm_ahb_clk = {
>         .halt_reg = 0x44004,
>         .clkr = {
> @@ -2684,6 +3047,7 @@ static struct clk_branch gcc_pdm_ahb_clk = {
>         },
>  };
>  
> +/* CBCR/halt_reg value derived from code only */

Drop these useless comments.