002-v6.2-05-clk-qcom-krait-cc-convert-to-devm_clk_hw_register.patch 2.5 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104
  1. From 8ea9fb841a7e528bc8ae79d726ce951dcf7b46e2 Mon Sep 17 00:00:00 2001
  2. From: Christian Marangi <[email protected]>
  3. Date: Wed, 9 Nov 2022 01:56:30 +0100
  4. Subject: [PATCH 5/6] clk: qcom: krait-cc: convert to devm_clk_hw_register
  5. clk_register is now deprecated. Convert the driver to devm_clk_hw_register.
  6. Signed-off-by: Christian Marangi <[email protected]>
  7. Signed-off-by: Bjorn Andersson <[email protected]>
  8. Link: https://lore.kernel.org/r/[email protected]
  9. ---
  10. drivers/clk/qcom/krait-cc.c | 31 +++++++++++++++++++------------
  11. 1 file changed, 19 insertions(+), 12 deletions(-)
  12. --- a/drivers/clk/qcom/krait-cc.c
  13. +++ b/drivers/clk/qcom/krait-cc.c
  14. @@ -79,8 +79,7 @@ krait_add_div(struct device *dev, int id
  15. .flags = CLK_SET_RATE_PARENT,
  16. };
  17. const char *p_names[1];
  18. - struct clk *clk;
  19. - int cpu;
  20. + int cpu, ret;
  21. div = devm_kzalloc(dev, sizeof(*div), GFP_KERNEL);
  22. if (!div)
  23. @@ -103,8 +102,8 @@ krait_add_div(struct device *dev, int id
  24. return -ENOMEM;
  25. }
  26. - clk = devm_clk_register(dev, &div->hw);
  27. - if (IS_ERR(clk))
  28. + ret = devm_clk_hw_register(dev, &div->hw);
  29. + if (ret)
  30. goto err;
  31. /* clk-krait ignore any rate change if mux is not flagged as enabled */
  32. @@ -118,7 +117,7 @@ err:
  33. kfree(p_names[0]);
  34. kfree(init.name);
  35. - return PTR_ERR_OR_ZERO(clk);
  36. + return ret;
  37. }
  38. static int
  39. @@ -137,7 +136,6 @@ krait_add_sec_mux(struct device *dev, in
  40. .ops = &krait_mux_clk_ops,
  41. .flags = CLK_SET_RATE_PARENT,
  42. };
  43. - struct clk *clk;
  44. mux = devm_kzalloc(dev, sizeof(*mux), GFP_KERNEL);
  45. if (!mux)
  46. @@ -166,14 +164,16 @@ krait_add_sec_mux(struct device *dev, in
  47. if (unique_aux) {
  48. sec_mux_list[0] = kasprintf(GFP_KERNEL, "acpu%s_aux", s);
  49. if (!sec_mux_list[0]) {
  50. - clk = ERR_PTR(-ENOMEM);
  51. + ret = -ENOMEM;
  52. goto err_aux;
  53. }
  54. }
  55. - clk = devm_clk_register(dev, &mux->hw);
  56. + ret = devm_clk_hw_register(dev, &mux->hw);
  57. + if (ret)
  58. + goto unique_aux;
  59. - ret = krait_notifier_register(dev, clk, mux);
  60. + ret = krait_notifier_register(dev, mux->hw.clk, mux);
  61. if (ret)
  62. goto unique_aux;
  63. @@ -189,7 +189,7 @@ unique_aux:
  64. kfree(sec_mux_list[0]);
  65. err_aux:
  66. kfree(init.name);
  67. - return PTR_ERR_OR_ZERO(clk);
  68. + return ret;
  69. }
  70. static struct clk *
  71. @@ -241,11 +241,18 @@ krait_add_pri_mux(struct device *dev, in
  72. goto err_p2;
  73. }
  74. - clk = devm_clk_register(dev, &mux->hw);
  75. + ret = devm_clk_hw_register(dev, &mux->hw);
  76. + if (ret) {
  77. + clk = ERR_PTR(ret);
  78. + goto err_p3;
  79. + }
  80. +
  81. + clk = mux->hw.clk;
  82. ret = krait_notifier_register(dev, clk, mux);
  83. if (ret)
  84. - goto err_p3;
  85. + clk = ERR_PTR(ret);
  86. +
  87. err_p3:
  88. kfree(p_names[2]);
  89. err_p2: