From: Stephen Boyd <stephen.boyd@linaro.org> To: Michael Turquette <mturquette@baylibre.com>, Stephen Boyd <sboyd@codeaurora.org> Cc: linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, "Guenter Roeck" <linux@roeck-us.net>, "Sören Brinkmann" <soren.brinkmann@xilinx.com> Subject: [PATCH 29/34] clk: si570: Migrate to clk_hw based OF and registration APIs Date: Wed, 1 Jun 2016 16:15:28 -0700 [thread overview] Message-ID: <20160601231533.9354-30-stephen.boyd@linaro.org> (raw) In-Reply-To: <20160601231533.9354-1-stephen.boyd@linaro.org> Now that we have clk_hw based provider APIs to register clks, we can get rid of struct clk pointers while registering clks in these drivers, allowing us to move closer to a clear split of consumer and provider clk APIs. Cc: Guenter Roeck <linux@roeck-us.net> Cc: Sören Brinkmann <soren.brinkmann@xilinx.com> Signed-off-by: Stephen Boyd <stephen.boyd@linaro.org> --- See commit 58657d189a2f and it's children for details on this new registration API. drivers/clk/clk-si570.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/clk/clk-si570.c b/drivers/clk/clk-si570.c index d56648521a95..646af1d1898d 100644 --- a/drivers/clk/clk-si570.c +++ b/drivers/clk/clk-si570.c @@ -408,7 +408,6 @@ static int si570_probe(struct i2c_client *client, { struct clk_si570 *data; struct clk_init_data init; - struct clk *clk; u32 initial_fout, factory_fout, stability; int err; enum clk_si570_variant variant = id->driver_data; @@ -462,13 +461,13 @@ static int si570_probe(struct i2c_client *client, if (err) return err; - clk = devm_clk_register(&client->dev, &data->hw); - if (IS_ERR(clk)) { + err = devm_clk_hw_register(&client->dev, &data->hw); + if (err) { dev_err(&client->dev, "clock registration failed\n"); - return PTR_ERR(clk); + return err; } - err = of_clk_add_provider(client->dev.of_node, of_clk_src_simple_get, - clk); + err = of_clk_add_hw_provider(client->dev.of_node, of_clk_hw_simple_get, + &data->hw); if (err) { dev_err(&client->dev, "unable to add clk provider\n"); return err; @@ -477,7 +476,7 @@ static int si570_probe(struct i2c_client *client, /* Read the requested initial output frequency from device tree */ if (!of_property_read_u32(client->dev.of_node, "clock-frequency", &initial_fout)) { - err = clk_set_rate(clk, initial_fout); + err = clk_set_rate(data->hw.clk, initial_fout); if (err) { of_clk_del_provider(client->dev.of_node); return err; -- 2.7.4
WARNING: multiple messages have this Message-ID (diff)
From: stephen.boyd@linaro.org (Stephen Boyd) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 29/34] clk: si570: Migrate to clk_hw based OF and registration APIs Date: Wed, 1 Jun 2016 16:15:28 -0700 [thread overview] Message-ID: <20160601231533.9354-30-stephen.boyd@linaro.org> (raw) In-Reply-To: <20160601231533.9354-1-stephen.boyd@linaro.org> Now that we have clk_hw based provider APIs to register clks, we can get rid of struct clk pointers while registering clks in these drivers, allowing us to move closer to a clear split of consumer and provider clk APIs. Cc: Guenter Roeck <linux@roeck-us.net> Cc: S?ren Brinkmann <soren.brinkmann@xilinx.com> Signed-off-by: Stephen Boyd <stephen.boyd@linaro.org> --- See commit 58657d189a2f and it's children for details on this new registration API. drivers/clk/clk-si570.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/clk/clk-si570.c b/drivers/clk/clk-si570.c index d56648521a95..646af1d1898d 100644 --- a/drivers/clk/clk-si570.c +++ b/drivers/clk/clk-si570.c @@ -408,7 +408,6 @@ static int si570_probe(struct i2c_client *client, { struct clk_si570 *data; struct clk_init_data init; - struct clk *clk; u32 initial_fout, factory_fout, stability; int err; enum clk_si570_variant variant = id->driver_data; @@ -462,13 +461,13 @@ static int si570_probe(struct i2c_client *client, if (err) return err; - clk = devm_clk_register(&client->dev, &data->hw); - if (IS_ERR(clk)) { + err = devm_clk_hw_register(&client->dev, &data->hw); + if (err) { dev_err(&client->dev, "clock registration failed\n"); - return PTR_ERR(clk); + return err; } - err = of_clk_add_provider(client->dev.of_node, of_clk_src_simple_get, - clk); + err = of_clk_add_hw_provider(client->dev.of_node, of_clk_hw_simple_get, + &data->hw); if (err) { dev_err(&client->dev, "unable to add clk provider\n"); return err; @@ -477,7 +476,7 @@ static int si570_probe(struct i2c_client *client, /* Read the requested initial output frequency from device tree */ if (!of_property_read_u32(client->dev.of_node, "clock-frequency", &initial_fout)) { - err = clk_set_rate(clk, initial_fout); + err = clk_set_rate(data->hw.clk, initial_fout); if (err) { of_clk_del_provider(client->dev.of_node); return err; -- 2.7.4
next prev parent reply other threads:[~2016-06-01 23:18 UTC|newest] Thread overview: 179+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-06-01 23:14 [PATCH 00/34] Convert clk providers to clk_hw based APIs (part 1) Stephen Boyd 2016-06-01 23:14 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 01/34] clk: qcom: Migrate to clk_hw based registration and OF APIs Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 02/34] clk: at91: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-07 16:36 ` Alexandre Belloni 2016-06-07 16:36 ` Alexandre Belloni 2016-06-07 16:40 ` Boris Brezillon 2016-06-07 16:40 ` Boris Brezillon 2016-09-15 0:39 ` Stephen Boyd 2016-09-15 0:39 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 03/34] clk: highbank: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-02 1:02 ` Rob Herring 2016-06-02 1:02 ` Rob Herring 2016-06-02 1:02 ` Rob Herring 2016-06-30 19:25 ` Stephen Boyd 2016-06-30 19:25 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 04/34] clk: bcm2835: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-02 18:25 ` Eric Anholt 2016-06-02 18:25 ` Eric Anholt 2016-09-15 0:36 ` Stephen Boyd 2016-09-15 0:36 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 05/34] clk: bcm: iproc: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-02 17:22 ` Ray Jui 2016-06-02 17:22 ` Ray Jui 2016-06-30 19:27 ` Stephen Boyd 2016-06-30 19:27 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 06/34] clk: bcm: kona: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-24 23:08 ` Stephen Boyd 2016-08-24 23:08 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 07/34] clk: berlin: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-05 17:41 ` Alexandre Belloni 2016-06-05 17:41 ` Alexandre Belloni 2016-06-07 8:40 ` Stephen Boyd 2016-06-07 8:40 ` Stephen Boyd 2016-06-07 16:33 ` Alexandre Belloni 2016-06-07 16:33 ` Alexandre Belloni 2016-06-01 23:15 ` [PATCH 08/34] clk: asm9260: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-24 23:09 ` Stephen Boyd 2016-08-24 23:09 ` Stephen Boyd 2016-08-24 23:10 ` Stephen Boyd 2016-08-24 23:10 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 09/34] clk: axi-clkgen: Migrate to clk_hw based OF and registration APIs Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-24 23:11 ` Stephen Boyd 2016-08-24 23:11 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 10/34] clk: axm5516: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-24 23:12 ` Stephen Boyd 2016-08-24 23:12 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 11/34] clk: cdce: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-24 23:13 ` Stephen Boyd 2016-08-24 23:13 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 12/34] clk: cdce925: Migrate to clk_hw based OF and provider APIs Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:19 ` Stephen Boyd 2016-08-25 0:19 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 13/34] clk: clps711x: Migrate to clk_hw based OF and registration APIs Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:20 ` Stephen Boyd 2016-08-25 0:20 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 14/34] clk: cs2000: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:21 ` Stephen Boyd 2016-08-25 0:21 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 15/34] clk: efm32gg: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:22 ` Stephen Boyd 2016-08-25 0:22 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 16/34] clk: ls1x: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-19 0:09 ` Stephen Boyd 2016-08-19 0:09 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 17/34] clk: maxgen: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-07 18:50 ` Javier Martinez Canillas 2016-06-07 18:50 ` Javier Martinez Canillas 2016-06-07 18:50 ` Javier Martinez Canillas 2016-06-07 18:55 ` Javier Martinez Canillas 2016-06-07 18:55 ` Javier Martinez Canillas 2016-06-07 18:55 ` Javier Martinez Canillas 2016-08-16 20:06 ` Stephen Boyd 2016-08-16 20:06 ` Stephen Boyd 2016-08-16 20:06 ` Stephen Boyd 2016-08-17 3:00 ` Javier Martinez Canillas 2016-08-17 3:00 ` Javier Martinez Canillas 2016-08-17 3:00 ` Javier Martinez Canillas 2016-06-01 23:15 ` [PATCH 18/34] clk: mb86s7x: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:27 ` Stephen Boyd 2016-08-25 0:27 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 19/34] clk: moxart: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:25 ` Stephen Boyd 2016-08-25 0:25 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 20/34] clk: nomadik: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-02 12:06 ` Linus Walleij 2016-06-02 12:06 ` Linus Walleij 2016-06-02 12:06 ` Linus Walleij 2016-06-30 19:26 ` Stephen Boyd 2016-06-30 19:26 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 21/34] clk: nspire: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:25 ` Stephen Boyd 2016-08-25 0:25 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 22/34] clk: palmas: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:25 ` Stephen Boyd 2016-08-25 0:25 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 23/34] clk: pwm: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:25 ` Stephen Boyd 2016-08-25 0:25 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 24/34] clk: rk808: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 25/34] clk: s2mps11: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-03 7:18 ` Krzysztof Kozlowski 2016-06-03 7:18 ` Krzysztof Kozlowski 2016-06-08 7:22 ` Andi Shyti 2016-06-08 7:22 ` Andi Shyti 2016-06-30 19:29 ` Stephen Boyd 2016-06-30 19:29 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 26/34] clk: scpi: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 27/34] clk: si514: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 28/34] clk: si5351: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd [this message] 2016-06-01 23:15 ` [PATCH 29/34] clk: si570: " Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-08-25 0:30 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 30/34] clk: stm32f3: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-06 13:48 ` Daniel Thompson 2016-06-06 13:48 ` Daniel Thompson 2016-06-07 8:37 ` Stephen Boyd 2016-06-07 8:37 ` Stephen Boyd 2016-06-07 8:37 ` Stephen Boyd 2016-06-30 19:28 ` Stephen Boyd 2016-06-30 19:28 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 31/34] clk: twl6040: Migrate to clk_hw based " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:34 ` Stephen Boyd 2016-08-25 0:34 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 32/34] clk: u300: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-06-02 12:06 ` Linus Walleij 2016-06-02 12:06 ` Linus Walleij 2016-06-02 12:06 ` Linus Walleij 2016-06-30 19:26 ` Stephen Boyd 2016-06-30 19:26 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 33/34] clk: vt8500: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:36 ` Stephen Boyd 2016-08-25 0:36 ` Stephen Boyd 2016-06-01 23:15 ` [PATCH 34/34] clk: wm831x: " Stephen Boyd 2016-06-01 23:15 ` Stephen Boyd 2016-08-25 0:36 ` Stephen Boyd 2016-08-25 0:36 ` Stephen Boyd
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20160601231533.9354-30-stephen.boyd@linaro.org \ --to=stephen.boyd@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@roeck-us.net \ --cc=mturquette@baylibre.com \ --cc=sboyd@codeaurora.org \ --cc=soren.brinkmann@xilinx.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.