Re: [PATCH 02/11] clk: qcom: gpucc-sdm660: use parent_hws instead of parent_data

Marijn Suijten
2021-12-08T22:45:03Z

On 2021-12-08 20:54:21, Dmitry Baryshkov wrote:
> If all parents are specified as clk_hw, we can use parent_hws instead of
> parent_data.
>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org>

> ---
> drivers/clk/qcom/gpucc-sdm660.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/clk/qcom/gpucc-sdm660.c b/drivers/clk/qcom/gpucc-sdm660.c
> index 26e17f349a77..27a506a78a25 100644
> --- a/drivers/clk/qcom/gpucc-sdm660.c
> +++ b/drivers/clk/qcom/gpucc-sdm660.c
> @@ -65,8 +65,8 @@ static struct clk_alpha_pll gpu_pll0_pll_out_main = {
> .num_vco = ARRAY_SIZE(gpu_vco),
> .clkr.hw.init = &(struct clk_init_data){
> .name = "gpu_pll0_pll_out_main",
> - .parent_data = &(const struct clk_parent_data){
> - .hw = &gpucc_cxo_clk.clkr.hw,
> + .parent_hws = (const struct clk_hw*[]){
> + &gpucc_cxo_clk.clkr.hw,
> },
> .num_parents = 1,
> .ops = &clk_alpha_pll_ops,
> @@ -80,8 +80,8 @@ static struct clk_alpha_pll gpu_pll1_pll_out_main = {
> .num_vco = ARRAY_SIZE(gpu_vco),
> .clkr.hw.init = &(struct clk_init_data){
> .name = "gpu_pll1_pll_out_main",
> - .parent_data = &(const struct clk_parent_data){
> - .hw = &gpucc_cxo_clk.clkr.hw,
> + .parent_hws = (const struct clk_hw*[]){
> + &gpucc_cxo_clk.clkr.hw,
> },
> .num_parents = 1,
> .ops = &clk_alpha_pll_ops,
> @@ -134,8 +134,8 @@ static struct clk_branch gpucc_gfx3d_clk = {
> .enable_mask = BIT(0),
> .hw.init = &(struct clk_init_data){
> .name = "gpucc_gfx3d_clk",
> - .parent_data = &(const struct clk_parent_data){
> - .hw = &gfx3d_clk_src.rcg.clkr.hw,
> + .parent_hws = (const struct clk_hw*[]){
> + &gfx3d_clk_src.rcg.clkr.hw,
> },
> .num_parents = 1,
> .ops = &clk_branch2_ops,
> --
> 2.33.0
>