From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 16878C5321D for ; Mon, 26 Aug 2024 12:40:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:From: To:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=HpL57b35Xgk1jI0euyh2wxEnpr5lIQdBh0j+DUbMFwU=; b=b1ySvA4jclzS6cf87oDygEWZbV JGoojVmKLgNQwxDxog7VsEU89eeQ+IkGlc647B+oIbe5KvMb58HIkB8FV6CVdxT5k01ByD6umo38m 3dC4y+hsyRWtdQ7NE0CBc4CNKEpDEENWYvozWGb70wHDKaEdpFMY+JhIFJFdg19IldnYtoAnggOzu yL40LTkcQf3Zgl3N1ggow6X8J/iqeHIHGSGXx68eNH3pVNb9P3rsyk7QnBYQg/nL06gzRvdiHm/OX dgOKWwsBiTNlH2uOLLZEsVok2BS3rwOxJS3/64HRJvEj/5PfcyLXysQJ6kX2V9GSvQen5E2csBi6d aEbl17uA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1siZ1E-00000007LJv-2Hw6; Mon, 26 Aug 2024 12:40:24 +0000 Received: from mail-4316.protonmail.ch ([185.70.43.16]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1siYzE-00000007Km5-3q78 for linux-arm-kernel@lists.infradead.org; Mon, 26 Aug 2024 12:38:22 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=protonmail.com; s=protonmail3; t=1724675899; x=1724935099; bh=HpL57b35Xgk1jI0euyh2wxEnpr5lIQdBh0j+DUbMFwU=; h=Date:To:From:Cc:Subject:Message-ID:In-Reply-To:References: Feedback-ID:From:To:Cc:Date:Subject:Reply-To:Feedback-ID: Message-ID:BIMI-Selector; b=rnNGkq2ZE8jLrADIbN27FF51VfSijQ8xq1Cmv7Os9Fln2bc0c/40TAILXm7/xX+DO ZIsIwqMA90v3vqe7RUt0KjkbJqo0cQzU1+FYOzukNN6PNZdXtrZZ98mpaK7qKvjDBG p6Gbi6kE3uK0Ak/H2Dv1OleHZ021rboi/cASOJ2TliaVtWuSpedrwQBp7K0McCS+/f JnXaCaCCfCMUUUnCajfAP0xJv0jTOnvbffGWcBKkGES2yYlZEokld9xdopQYcbx4LQ DHJGhq3emP28pPdBGPJFV+frQS0zlxSzaBfF/e/XRiOg793KaewaoHYA5MpjwWyAS4 GFMFfNQ34hjng== Date: Mon, 26 Aug 2024 12:38:14 +0000 To: Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Michal Simek , Greg Kroah-Hartman From: Harry Austen Cc: Shubhrajyoti Datta , Dave Ertman , Ira Weiny , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Harry Austen Subject: [PATCH v3 2/9] clk: clocking-wizard: use newer clk_hw API Message-ID: <20240826123602.1872-3-hpausten@protonmail.com> In-Reply-To: <20240826123602.1872-1-hpausten@protonmail.com> References: <20240826123602.1872-1-hpausten@protonmail.com> Feedback-ID: 53116287:user:proton X-Pm-Message-ID: 540320f489ab97a665acd4ab2d014861d1b470f7 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240826_053821_141328_D1CECCF6 X-CRM114-Status: GOOD ( 12.92 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Utilise clock provider API with struct clk_hw instances instead of the consumer-side struct clk. Signed-off-by: Harry Austen --- drivers/clk/xilinx/clk-xlnx-clock-wizard.c | 77 +++++++++++----------- 1 file changed, 40 insertions(+), 37 deletions(-) diff --git a/drivers/clk/xilinx/clk-xlnx-clock-wizard.c b/drivers/clk/xilin= x/clk-xlnx-clock-wizard.c index 0ca045849ea3e..cd795a4952099 100644 --- a/drivers/clk/xilinx/clk-xlnx-clock-wizard.c +++ b/drivers/clk/xilinx/clk-xlnx-clock-wizard.c @@ -17,6 +17,7 @@ #include #include #include +#include #include #include =20 @@ -121,26 +122,24 @@ enum clk_wzrd_int_clks { /** * struct clk_wzrd - Clock wizard private data structure * - * @clk_data:=09=09Clock data * @nb:=09=09=09Notifier block * @base:=09=09Memory base * @clk_in1:=09=09Handle to input clock 'clk_in1' * @axi_clk:=09=09Handle to input clock 's_axi_aclk' * @clks_internal:=09Internal clocks - * @clkout:=09=09Output clocks * @speed_grade:=09Speed grade of the device * @suspended:=09=09Flag indicating power state of the device + * @clk_data:=09=09Output clock data */ struct clk_wzrd { -=09struct clk_onecell_data clk_data; =09struct notifier_block nb; =09void __iomem *base; =09struct clk *clk_in1; =09struct clk *axi_clk; -=09struct clk *clks_internal[wzrd_clk_int_max]; -=09struct clk *clkout[WZRD_NUM_OUTPUTS]; +=09struct clk_hw *clks_internal[wzrd_clk_int_max]; =09unsigned int speed_grade; =09bool suspended; +=09struct clk_hw_onecell_data clk_data; }; =20 /** @@ -765,7 +764,7 @@ static const struct clk_ops clk_wzrd_clk_divider_ops_f = =3D { =09.recalc_rate =3D clk_wzrd_recalc_ratef, }; =20 -static struct clk *clk_wzrd_register_divf(struct device *dev, +static struct clk_hw *clk_wzrd_register_divf(struct device *dev, =09=09=09=09=09 const char *name, =09=09=09=09=09 const char *parent_name, =09=09=09=09=09 unsigned long flags, @@ -805,10 +804,10 @@ static struct clk *clk_wzrd_register_divf(struct devi= ce *dev, =09if (ret) =09=09return ERR_PTR(ret); =20 -=09return hw->clk; +=09return hw; } =20 -static struct clk *clk_wzrd_ver_register_divider(struct device *dev, +static struct clk_hw *clk_wzrd_ver_register_divider(struct device *dev, =09=09=09=09=09=09 const char *name, =09=09=09=09=09=09 const char *parent_name, =09=09=09=09=09=09 unsigned long flags, @@ -852,10 +851,10 @@ static struct clk *clk_wzrd_ver_register_divider(stru= ct device *dev, =09if (ret) =09=09return ERR_PTR(ret); =20 -=09return hw->clk; +=09return hw; } =20 -static struct clk *clk_wzrd_register_divider(struct device *dev, +static struct clk_hw *clk_wzrd_register_divider(struct device *dev, =09=09=09=09=09 const char *name, =09=09=09=09=09 const char *parent_name, =09=09=09=09=09 unsigned long flags, @@ -898,7 +897,7 @@ static struct clk *clk_wzrd_register_divider(struct dev= ice *dev, =09if (ret) =09=09return ERR_PTR(ret); =20 -=09return hw->clk; +=09return hw; } =20 static int clk_wzrd_clk_notifier(struct notifier_block *nb, unsigned long = event, @@ -978,7 +977,12 @@ static int clk_wzrd_probe(struct platform_device *pdev= ) =09int nr_outputs; =09int i, ret; =20 -=09clk_wzrd =3D devm_kzalloc(&pdev->dev, sizeof(*clk_wzrd), GFP_KERNEL); +=09ret =3D of_property_read_u32(np, "xlnx,nr-outputs", &nr_outputs); +=09if (ret || nr_outputs > WZRD_NUM_OUTPUTS) +=09=09return -EINVAL; + +=09clk_wzrd =3D devm_kzalloc(&pdev->dev, struct_size(clk_wzrd, clk_data.hw= s, nr_outputs), +=09=09=09=09GFP_KERNEL); =09if (!clk_wzrd) =09=09return -ENOMEM; =09platform_set_drvdata(pdev, clk_wzrd); @@ -1016,17 +1020,13 @@ static int clk_wzrd_probe(struct platform_device *p= dev) =09if (data) =09=09is_versal =3D data->is_versal; =20 -=09ret =3D of_property_read_u32(np, "xlnx,nr-outputs", &nr_outputs); -=09if (ret || nr_outputs > WZRD_NUM_OUTPUTS) -=09=09return -EINVAL; - =09clkout_name =3D devm_kasprintf(&pdev->dev, GFP_KERNEL, "%s_out0", dev_n= ame(&pdev->dev)); =09if (!clkout_name) =09=09return -ENOMEM; =20 =09if (is_versal) { =09=09if (nr_outputs =3D=3D 1) { -=09=09=09clk_wzrd->clkout[0] =3D clk_wzrd_ver_register_divider +=09=09=09clk_wzrd->clk_data.hws[0] =3D clk_wzrd_ver_register_divider =09=09=09=09(&pdev->dev, clkout_name, =09=09=09=09__clk_get_name(clk_wzrd->clk_in1), 0, =09=09=09=09clk_wzrd->base, WZRD_CLK_CFG_REG(is_versal, 3), @@ -1059,7 +1059,7 @@ static int clk_wzrd_probe(struct platform_device *pde= v) =09=09div =3D 64; =09} else { =09=09if (nr_outputs =3D=3D 1) { -=09=09=09clk_wzrd->clkout[0] =3D clk_wzrd_register_divider +=09=09=09clk_wzrd->clk_data.hws[0] =3D clk_wzrd_register_divider =09=09=09=09(&pdev->dev, clkout_name, =09=09=09=09__clk_get_name(clk_wzrd->clk_in1), 0, =09=09=09=09clk_wzrd->base, WZRD_CLK_CFG_REG(is_versal, 3), @@ -1082,7 +1082,7 @@ static int clk_wzrd_probe(struct platform_device *pde= v) =09clk_name =3D devm_kasprintf(&pdev->dev, GFP_KERNEL, "%s_mul", dev_name(= &pdev->dev)); =09if (!clk_name) =09=09return -ENOMEM; -=09clk_wzrd->clks_internal[wzrd_clk_mul] =3D clk_register_fixed_factor +=09clk_wzrd->clks_internal[wzrd_clk_mul] =3D clk_hw_register_fixed_factor =09=09=09(&pdev->dev, clk_name, =09=09=09 __clk_get_name(clk_wzrd->clk_in1), =09=09=090, mult, div); @@ -1108,15 +1108,15 @@ static int clk_wzrd_probe(struct platform_device *p= dev) =09=09if (!div) =09=09=09div =3D 1; =20 -=09=09clk_mul_name =3D __clk_get_name(clk_wzrd->clks_internal[wzrd_clk_mul= ]); +=09=09clk_mul_name =3D clk_hw_get_name(clk_wzrd->clks_internal[wzrd_clk_mu= l]); =09=09clk_wzrd->clks_internal[wzrd_clk_mul_div] =3D -=09=09=09clk_register_fixed_factor(&pdev->dev, clk_name, -=09=09=09=09=09=09 clk_mul_name, 0, 1, div); +=09=09=09clk_hw_register_fixed_factor(&pdev->dev, clk_name, +=09=09=09=09=09=09 clk_mul_name, 0, 1, div); =09} else { =09=09ctrl_reg =3D clk_wzrd->base + WZRD_CLK_CFG_REG(is_versal, 0); -=09=09clk_wzrd->clks_internal[wzrd_clk_mul_div] =3D clk_register_divider +=09=09clk_wzrd->clks_internal[wzrd_clk_mul_div] =3D clk_hw_register_divide= r =09=09=09(&pdev->dev, clk_name, -=09=09=09 __clk_get_name(clk_wzrd->clks_internal[wzrd_clk_mul]), +=09=09=09 clk_hw_get_name(clk_wzrd->clks_internal[wzrd_clk_mul]), =09=09=09flags, ctrl_reg, 0, 8, CLK_DIVIDER_ONE_BASED | =09=09=09CLK_DIVIDER_ALLOW_ZERO, &clkwzrd_lock); =09} @@ -1136,7 +1136,7 @@ static int clk_wzrd_probe(struct platform_device *pde= v) =09=09} =20 =09=09if (is_versal) { -=09=09=09clk_wzrd->clkout[i] =3D clk_wzrd_ver_register_divider +=09=09=09clk_wzrd->clk_data.hws[i] =3D clk_wzrd_ver_register_divider =09=09=09=09=09=09(&pdev->dev, =09=09=09=09=09=09 clkout_name, clk_name, 0, =09=09=09=09=09=09 clk_wzrd->base, @@ -1148,7 +1148,7 @@ static int clk_wzrd_probe(struct platform_device *pde= v) =09=09=09=09=09=09 DIV_O, &clkwzrd_lock); =09=09} else { =09=09=09if (!i) -=09=09=09=09clk_wzrd->clkout[i] =3D clk_wzrd_register_divf +=09=09=09=09clk_wzrd->clk_data.hws[i] =3D clk_wzrd_register_divf =09=09=09=09=09(&pdev->dev, clkout_name, clk_name, flags, clk_wzrd->base, =09=09=09=09=09(WZRD_CLK_CFG_REG(is_versal, 2) + i * 12), =09=09=09=09=09WZRD_CLKOUT_DIVIDE_SHIFT, @@ -1156,7 +1156,7 @@ static int clk_wzrd_probe(struct platform_device *pde= v) =09=09=09=09=09CLK_DIVIDER_ONE_BASED | CLK_DIVIDER_ALLOW_ZERO, =09=09=09=09=09DIV_O, &clkwzrd_lock); =09=09=09else -=09=09=09=09clk_wzrd->clkout[i] =3D clk_wzrd_register_divider +=09=09=09=09clk_wzrd->clk_data.hws[i] =3D clk_wzrd_register_divider =09=09=09=09=09(&pdev->dev, clkout_name, clk_name, 0, clk_wzrd->base, =09=09=09=09=09(WZRD_CLK_CFG_REG(is_versal, 2) + i * 12), =09=09=09=09=09WZRD_CLKOUT_DIVIDE_SHIFT, @@ -1164,22 +1164,25 @@ static int clk_wzrd_probe(struct platform_device *p= dev) =09=09=09=09=09CLK_DIVIDER_ONE_BASED | CLK_DIVIDER_ALLOW_ZERO, =09=09=09=09=09DIV_O, &clkwzrd_lock); =09=09} -=09=09if (IS_ERR(clk_wzrd->clkout[i])) { +=09=09if (IS_ERR(clk_wzrd->clk_data.hws[i])) { =09=09=09int j; =20 =09=09=09for (j =3D i + 1; j < nr_outputs; j++) -=09=09=09=09clk_unregister(clk_wzrd->clkout[j]); +=09=09=09=09clk_hw_unregister(clk_wzrd->clk_data.hws[j]); =09=09=09dev_err(&pdev->dev, =09=09=09=09"unable to register divider clock\n"); -=09=09=09ret =3D PTR_ERR(clk_wzrd->clkout[i]); +=09=09=09ret =3D PTR_ERR(clk_wzrd->clk_data.hws[i]); =09=09=09goto err_rm_int_clks; =09=09} =09} =20 out: -=09clk_wzrd->clk_data.clks =3D clk_wzrd->clkout; -=09clk_wzrd->clk_data.clk_num =3D ARRAY_SIZE(clk_wzrd->clkout); -=09of_clk_add_provider(np, of_clk_src_onecell_get, &clk_wzrd->clk_data); +=09clk_wzrd->clk_data.num =3D nr_outputs; +=09ret =3D of_clk_add_hw_provider(pdev->dev.of_node, of_clk_hw_onecell_get= , &clk_wzrd->clk_data); +=09if (ret) { +=09=09dev_err(&pdev->dev, "unable to register clock provider\n"); +=09=09return ret; +=09} =20 =09if (clk_wzrd->speed_grade) { =09=09clk_wzrd->nb.notifier_call =3D clk_wzrd_clk_notifier; @@ -1200,9 +1203,9 @@ static int clk_wzrd_probe(struct platform_device *pde= v) =09return 0; =20 err_rm_int_clks: -=09clk_unregister(clk_wzrd->clks_internal[1]); +=09clk_hw_unregister(clk_wzrd->clks_internal[1]); err_rm_int_clk: -=09clk_unregister(clk_wzrd->clks_internal[0]); +=09clk_hw_unregister(clk_wzrd->clks_internal[0]); =09return ret; } =20 @@ -1214,9 +1217,9 @@ static void clk_wzrd_remove(struct platform_device *p= dev) =09of_clk_del_provider(pdev->dev.of_node); =20 =09for (i =3D 0; i < WZRD_NUM_OUTPUTS; i++) -=09=09clk_unregister(clk_wzrd->clkout[i]); +=09=09clk_hw_unregister(clk_wzrd->clk_data.hws[i]); =09for (i =3D 0; i < wzrd_clk_int_max; i++) -=09=09clk_unregister(clk_wzrd->clks_internal[i]); +=09=09clk_hw_unregister(clk_wzrd->clks_internal[i]); } =20 static const struct of_device_id clk_wzrd_ids[] =3D { --=20 2.46.0