[PATCH v2 1/2] driver: clk: tegra: partially support PLL clocks

[Svyatoslav Ryhel]

Return PLL id into struct clk if PLL is parsed from device
tree instead of throwing an error. Allow requesting PLL
clock rate via get_rate op.

Signed-off-by: Svyatoslav Ryhel <clamo...@gmail.com>
---
 drivers/clk/tegra/tegra-car-clk.c | 49 ++++++++++++++++++++++++-------
 1 file changed, 39 insertions(+), 10 deletions(-)

diff --git a/drivers/clk/tegra/tegra-car-clk.c 
b/drivers/clk/tegra/tegra-car-clk.c
index 1d61f8dc378..dd4d7791120 100644
--- a/drivers/clk/tegra/tegra-car-clk.c
+++ b/drivers/clk/tegra/tegra-car-clk.c
@@ -10,6 +10,9 @@
 #include <asm/arch/clock.h>
 #include <asm/arch-tegra/clk_rst.h>
 
+#define TEGRA_CAR_CLK_PLL      BIT(0)
+#define TEGRA_CAR_CLK_PERIPH   BIT(1)
+
 static int tegra_car_clk_request(struct clk *clk)
 {
        debug("%s(clk=%p) (dev=%p, id=%lu)\n", __func__, clk, clk->dev,
@@ -20,30 +23,50 @@ static int tegra_car_clk_request(struct clk *clk)
         * varies per SoC) are the peripheral clocks, which use a numbering
         * scheme that matches HW registers 1:1. There are other clock IDs
         * beyond this that are assigned arbitrarily by the Tegra CAR DT
-        * binding. Due to the implementation of this driver, it currently
-        * only supports the peripheral IDs.
+        * binding.
         */
-       if (clk->id >= PERIPH_ID_COUNT)
-               return -EINVAL;
+       if (!(clk->id >= PERIPH_ID_COUNT)) {
+               clk->data |= TEGRA_CAR_CLK_PERIPH;
+               return 0;
+       }
 
-       return 0;
+       /* If check for periph failed, then check for PLL clock id */
+       int id = clk_id_to_pll_id(clk->id);
+
+       if (clock_id_is_pll(id)) {
+               clk->id = id;
+               clk->data |= TEGRA_CAR_CLK_PLL;
+               return 0;
+       }
+
+       return -EINVAL;
 }
 
 static ulong tegra_car_clk_get_rate(struct clk *clk)
 {
-       enum clock_id parent;
+       if (clk->data & TEGRA_CAR_CLK_PLL)
+               return clock_get_rate(clk->id);
 
-       debug("%s(clk=%p) (dev=%p, id=%lu)\n", __func__, clk, clk->dev,
-             clk->id);
+       if (clk->data & TEGRA_CAR_CLK_PERIPH) {
+               enum clock_id parent;
 
-       parent = clock_get_periph_parent(clk->id);
-       return clock_get_periph_rate(clk->id, parent);
+               debug("%s(clk=%p) (dev=%p, id=%lu)\n", __func__,
+                     clk, clk->dev, clk->id);
+
+               parent = clock_get_periph_parent(clk->id);
+               return clock_get_periph_rate(clk->id, parent);
+       }
+
+       return -1U;
 }
 
 static ulong tegra_car_clk_set_rate(struct clk *clk, ulong rate)
 {
        enum clock_id parent;
 
+       if (clk->data & TEGRA_CAR_CLK_PLL)
+               return 0;
+
        debug("%s(clk=%p, rate=%lu) (dev=%p, id=%lu)\n", __func__, clk, rate,
              clk->dev, clk->id);
 
@@ -53,6 +76,9 @@ static ulong tegra_car_clk_set_rate(struct clk *clk, ulong 
rate)
 
 static int tegra_car_clk_enable(struct clk *clk)
 {
+       if (clk->data & TEGRA_CAR_CLK_PLL)
+               return 0;
+
        debug("%s(clk=%p) (dev=%p, id=%lu)\n", __func__, clk, clk->dev,
              clk->id);
 
@@ -63,6 +89,9 @@ static int tegra_car_clk_enable(struct clk *clk)
 
 static int tegra_car_clk_disable(struct clk *clk)
 {
+       if (clk->data & TEGRA_CAR_CLK_PLL)
+               return 0;
+
        debug("%s(clk=%p) (dev=%p, id=%lu)\n", __func__, clk, clk->dev,
              clk->id);
 
-- 
2.43.0