]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
drm: sun4i: rename sun4i_dotclock to sun4i_tcon_dclk
authorRoman Beranek <me@crly.cz>
Fri, 5 May 2023 05:21:09 +0000 (07:21 +0200)
committerMaxime Ripard <maxime@cerno.tech>
Wed, 10 May 2023 14:03:10 +0000 (16:03 +0200)
While the rate of TCON0's DCLK matches dotclock for parallel and LVDS
outputs, this doesn't hold for DSI. The 'D' in DCLK actually stands for
'Data' according to Allwinner's manuals. The clock is mostly referred to
as dclk throughout this driver already anyway, so stick with that.

Signed-off-by: Roman Beranek <me@crly.cz>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Link: https://patchwork.freedesktop.org/patch/msgid/20230505052110.67514-4-me@crly.cz
drivers/gpu/drm/sun4i/Makefile
drivers/gpu/drm/sun4i/sun4i_dotclock.c [deleted file]
drivers/gpu/drm/sun4i/sun4i_dotclock.h [deleted file]
drivers/gpu/drm/sun4i/sun4i_tcon.c
drivers/gpu/drm/sun4i/sun4i_tcon_dclk.c [new file with mode: 0644]
drivers/gpu/drm/sun4i/sun4i_tcon_dclk.h [new file with mode: 0644]

index 0d04f2447b01caf08de771643d2536239c62654c..bad7497a0d11eda452cd27183ea2a8a9b719b999 100644 (file)
@@ -19,7 +19,7 @@ sun8i-mixer-y                 += sun8i_mixer.o sun8i_ui_layer.o \
                                   sun8i_vi_scaler.o sun8i_csc.o
 
 sun4i-tcon-y                   += sun4i_crtc.o
-sun4i-tcon-y                   += sun4i_dotclock.o
+sun4i-tcon-y                   += sun4i_tcon_dclk.o
 sun4i-tcon-y                   += sun4i_lvds.o
 sun4i-tcon-y                   += sun4i_tcon.o
 sun4i-tcon-y                   += sun4i_rgb.o
diff --git a/drivers/gpu/drm/sun4i/sun4i_dotclock.c b/drivers/gpu/drm/sun4i/sun4i_dotclock.c
deleted file mode 100644 (file)
index 417ade3..0000000
+++ /dev/null
@@ -1,206 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0-or-later
-/*
- * Copyright (C) 2016 Free Electrons
- * Copyright (C) 2016 NextThing Co
- *
- * Maxime Ripard <maxime.ripard@free-electrons.com>
- */
-
-#include <linux/clk-provider.h>
-#include <linux/regmap.h>
-
-#include "sun4i_tcon.h"
-#include "sun4i_dotclock.h"
-
-struct sun4i_dclk {
-       struct clk_hw           hw;
-       struct regmap           *regmap;
-       struct sun4i_tcon       *tcon;
-};
-
-static inline struct sun4i_dclk *hw_to_dclk(struct clk_hw *hw)
-{
-       return container_of(hw, struct sun4i_dclk, hw);
-}
-
-static void sun4i_dclk_disable(struct clk_hw *hw)
-{
-       struct sun4i_dclk *dclk = hw_to_dclk(hw);
-
-       regmap_update_bits(dclk->regmap, SUN4I_TCON0_DCLK_REG,
-                          BIT(SUN4I_TCON0_DCLK_GATE_BIT), 0);
-}
-
-static int sun4i_dclk_enable(struct clk_hw *hw)
-{
-       struct sun4i_dclk *dclk = hw_to_dclk(hw);
-
-       return regmap_update_bits(dclk->regmap, SUN4I_TCON0_DCLK_REG,
-                                 BIT(SUN4I_TCON0_DCLK_GATE_BIT),
-                                 BIT(SUN4I_TCON0_DCLK_GATE_BIT));
-}
-
-static int sun4i_dclk_is_enabled(struct clk_hw *hw)
-{
-       struct sun4i_dclk *dclk = hw_to_dclk(hw);
-       u32 val;
-
-       regmap_read(dclk->regmap, SUN4I_TCON0_DCLK_REG, &val);
-
-       return val & BIT(SUN4I_TCON0_DCLK_GATE_BIT);
-}
-
-static unsigned long sun4i_dclk_recalc_rate(struct clk_hw *hw,
-                                           unsigned long parent_rate)
-{
-       struct sun4i_dclk *dclk = hw_to_dclk(hw);
-       u32 val;
-
-       regmap_read(dclk->regmap, SUN4I_TCON0_DCLK_REG, &val);
-
-       val >>= SUN4I_TCON0_DCLK_DIV_SHIFT;
-       val &= (1 << SUN4I_TCON0_DCLK_DIV_WIDTH) - 1;
-
-       if (!val)
-               val = 1;
-
-       return parent_rate / val;
-}
-
-static long sun4i_dclk_round_rate(struct clk_hw *hw, unsigned long rate,
-                                 unsigned long *parent_rate)
-{
-       struct sun4i_dclk *dclk = hw_to_dclk(hw);
-       struct sun4i_tcon *tcon = dclk->tcon;
-       unsigned long best_parent = 0;
-       u8 best_div = 1;
-       int i;
-
-       for (i = tcon->dclk_min_div; i <= tcon->dclk_max_div; i++) {
-               u64 ideal = (u64)rate * i;
-               unsigned long rounded;
-
-               /*
-                * ideal has overflowed the max value that can be stored in an
-                * unsigned long, and every clk operation we might do on a
-                * truncated u64 value will give us incorrect results.
-                * Let's just stop there since bigger dividers will result in
-                * the same overflow issue.
-                */
-               if (ideal > ULONG_MAX)
-                       goto out;
-
-               rounded = clk_hw_round_rate(clk_hw_get_parent(hw),
-                                           ideal);
-
-               if (rounded == ideal) {
-                       best_parent = rounded;
-                       best_div = i;
-                       goto out;
-               }
-
-               if (abs(rate - rounded / i) <
-                   abs(rate - best_parent / best_div)) {
-                       best_parent = rounded;
-                       best_div = i;
-               }
-       }
-
-out:
-       *parent_rate = best_parent;
-
-       return best_parent / best_div;
-}
-
-static int sun4i_dclk_set_rate(struct clk_hw *hw, unsigned long rate,
-                              unsigned long parent_rate)
-{
-       struct sun4i_dclk *dclk = hw_to_dclk(hw);
-       u8 div = parent_rate / rate;
-
-       return regmap_update_bits(dclk->regmap, SUN4I_TCON0_DCLK_REG,
-                                 GENMASK(6, 0), div);
-}
-
-static int sun4i_dclk_get_phase(struct clk_hw *hw)
-{
-       struct sun4i_dclk *dclk = hw_to_dclk(hw);
-       u32 val;
-
-       regmap_read(dclk->regmap, SUN4I_TCON0_IO_POL_REG, &val);
-
-       val >>= 28;
-       val &= 3;
-
-       return val * 120;
-}
-
-static int sun4i_dclk_set_phase(struct clk_hw *hw, int degrees)
-{
-       struct sun4i_dclk *dclk = hw_to_dclk(hw);
-       u32 val = degrees / 120;
-
-       val <<= 28;
-
-       regmap_update_bits(dclk->regmap, SUN4I_TCON0_IO_POL_REG,
-                          GENMASK(29, 28),
-                          val);
-
-       return 0;
-}
-
-static const struct clk_ops sun4i_dclk_ops = {
-       .disable        = sun4i_dclk_disable,
-       .enable         = sun4i_dclk_enable,
-       .is_enabled     = sun4i_dclk_is_enabled,
-
-       .recalc_rate    = sun4i_dclk_recalc_rate,
-       .round_rate     = sun4i_dclk_round_rate,
-       .set_rate       = sun4i_dclk_set_rate,
-
-       .get_phase      = sun4i_dclk_get_phase,
-       .set_phase      = sun4i_dclk_set_phase,
-};
-
-int sun4i_dclk_create(struct device *dev, struct sun4i_tcon *tcon)
-{
-       const char *clk_name, *parent_name;
-       struct clk_init_data init;
-       struct sun4i_dclk *dclk;
-       int ret;
-
-       parent_name = __clk_get_name(tcon->sclk0);
-       ret = of_property_read_string_index(dev->of_node,
-                                           "clock-output-names", 0,
-                                           &clk_name);
-       if (ret)
-               return ret;
-
-       dclk = devm_kzalloc(dev, sizeof(*dclk), GFP_KERNEL);
-       if (!dclk)
-               return -ENOMEM;
-       dclk->tcon = tcon;
-
-       init.name = clk_name;
-       init.ops = &sun4i_dclk_ops;
-       init.parent_names = &parent_name;
-       init.num_parents = 1;
-       init.flags = CLK_SET_RATE_PARENT;
-
-       dclk->regmap = tcon->regs;
-       dclk->hw.init = &init;
-
-       tcon->dclk = clk_register(dev, &dclk->hw);
-       if (IS_ERR(tcon->dclk))
-               return PTR_ERR(tcon->dclk);
-
-       return 0;
-}
-EXPORT_SYMBOL(sun4i_dclk_create);
-
-int sun4i_dclk_free(struct sun4i_tcon *tcon)
-{
-       clk_unregister(tcon->dclk);
-       return 0;
-}
-EXPORT_SYMBOL(sun4i_dclk_free);
diff --git a/drivers/gpu/drm/sun4i/sun4i_dotclock.h b/drivers/gpu/drm/sun4i/sun4i_dotclock.h
deleted file mode 100644 (file)
index ac60da2..0000000
+++ /dev/null
@@ -1,17 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-or-later */
-/*
- * Copyright (C) 2015 Free Electrons
- * Copyright (C) 2015 NextThing Co
- *
- * Maxime Ripard <maxime.ripard@free-electrons.com>
- */
-
-#ifndef _SUN4I_DOTCLOCK_H_
-#define _SUN4I_DOTCLOCK_H_
-
-struct sun4i_tcon;
-
-int sun4i_dclk_create(struct device *dev, struct sun4i_tcon *tcon);
-int sun4i_dclk_free(struct sun4i_tcon *tcon);
-
-#endif /* _SUN4I_DOTCLOCK_H_ */
index 523a6d7879210b08960032a72b894d7da8d9b65a..eec26b1faa4ba3fc425b4ab64bbd5bfcae659581 100644 (file)
 #include <uapi/drm/drm_mode.h>
 
 #include "sun4i_crtc.h"
-#include "sun4i_dotclock.h"
 #include "sun4i_drv.h"
 #include "sun4i_lvds.h"
 #include "sun4i_rgb.h"
 #include "sun4i_tcon.h"
 #include "sun6i_mipi_dsi.h"
+#include "sun4i_tcon_dclk.h"
 #include "sun8i_tcon_top.h"
 #include "sunxi_engine.h"
 
@@ -1237,14 +1237,14 @@ static int sun4i_tcon_bind(struct device *dev, struct device *master,
        ret = sun4i_tcon_init_irq(dev, tcon);
        if (ret) {
                dev_err(dev, "Couldn't init our TCON interrupts\n");
-               goto err_free_dotclock;
+               goto err_free_dclk;
        }
 
        tcon->crtc = sun4i_crtc_init(drm, engine, tcon);
        if (IS_ERR(tcon->crtc)) {
                dev_err(dev, "Couldn't create our CRTC\n");
                ret = PTR_ERR(tcon->crtc);
-               goto err_free_dotclock;
+               goto err_free_dclk;
        }
 
        if (tcon->quirks->has_channel_0) {
@@ -1264,7 +1264,7 @@ static int sun4i_tcon_bind(struct device *dev, struct device *master,
                of_node_put(remote);
 
                if (ret < 0)
-                       goto err_free_dotclock;
+                       goto err_free_dclk;
        }
 
        if (tcon->quirks->needs_de_be_mux) {
@@ -1290,7 +1290,7 @@ static int sun4i_tcon_bind(struct device *dev, struct device *master,
 
        return 0;
 
-err_free_dotclock:
+err_free_dclk:
        if (tcon->quirks->has_channel_0)
                sun4i_dclk_free(tcon);
 err_free_clocks:
diff --git a/drivers/gpu/drm/sun4i/sun4i_tcon_dclk.c b/drivers/gpu/drm/sun4i/sun4i_tcon_dclk.c
new file mode 100644 (file)
index 0000000..03d7de1
--- /dev/null
@@ -0,0 +1,206 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Copyright (C) 2016 Free Electrons
+ * Copyright (C) 2016 NextThing Co
+ *
+ * Maxime Ripard <maxime.ripard@free-electrons.com>
+ */
+
+#include <linux/clk-provider.h>
+#include <linux/regmap.h>
+
+#include "sun4i_tcon.h"
+#include "sun4i_tcon_dclk.h"
+
+struct sun4i_dclk {
+       struct clk_hw           hw;
+       struct regmap           *regmap;
+       struct sun4i_tcon       *tcon;
+};
+
+static inline struct sun4i_dclk *hw_to_dclk(struct clk_hw *hw)
+{
+       return container_of(hw, struct sun4i_dclk, hw);
+}
+
+static void sun4i_dclk_disable(struct clk_hw *hw)
+{
+       struct sun4i_dclk *dclk = hw_to_dclk(hw);
+
+       regmap_update_bits(dclk->regmap, SUN4I_TCON0_DCLK_REG,
+                          BIT(SUN4I_TCON0_DCLK_GATE_BIT), 0);
+}
+
+static int sun4i_dclk_enable(struct clk_hw *hw)
+{
+       struct sun4i_dclk *dclk = hw_to_dclk(hw);
+
+       return regmap_update_bits(dclk->regmap, SUN4I_TCON0_DCLK_REG,
+                                 BIT(SUN4I_TCON0_DCLK_GATE_BIT),
+                                 BIT(SUN4I_TCON0_DCLK_GATE_BIT));
+}
+
+static int sun4i_dclk_is_enabled(struct clk_hw *hw)
+{
+       struct sun4i_dclk *dclk = hw_to_dclk(hw);
+       u32 val;
+
+       regmap_read(dclk->regmap, SUN4I_TCON0_DCLK_REG, &val);
+
+       return val & BIT(SUN4I_TCON0_DCLK_GATE_BIT);
+}
+
+static unsigned long sun4i_dclk_recalc_rate(struct clk_hw *hw,
+                                           unsigned long parent_rate)
+{
+       struct sun4i_dclk *dclk = hw_to_dclk(hw);
+       u32 val;
+
+       regmap_read(dclk->regmap, SUN4I_TCON0_DCLK_REG, &val);
+
+       val >>= SUN4I_TCON0_DCLK_DIV_SHIFT;
+       val &= (1 << SUN4I_TCON0_DCLK_DIV_WIDTH) - 1;
+
+       if (!val)
+               val = 1;
+
+       return parent_rate / val;
+}
+
+static long sun4i_dclk_round_rate(struct clk_hw *hw, unsigned long rate,
+                                 unsigned long *parent_rate)
+{
+       struct sun4i_dclk *dclk = hw_to_dclk(hw);
+       struct sun4i_tcon *tcon = dclk->tcon;
+       unsigned long best_parent = 0;
+       u8 best_div = 1;
+       int i;
+
+       for (i = tcon->dclk_min_div; i <= tcon->dclk_max_div; i++) {
+               u64 ideal = (u64)rate * i;
+               unsigned long rounded;
+
+               /*
+                * ideal has overflowed the max value that can be stored in an
+                * unsigned long, and every clk operation we might do on a
+                * truncated u64 value will give us incorrect results.
+                * Let's just stop there since bigger dividers will result in
+                * the same overflow issue.
+                */
+               if (ideal > ULONG_MAX)
+                       goto out;
+
+               rounded = clk_hw_round_rate(clk_hw_get_parent(hw),
+                                           ideal);
+
+               if (rounded == ideal) {
+                       best_parent = rounded;
+                       best_div = i;
+                       goto out;
+               }
+
+               if (abs(rate - rounded / i) <
+                   abs(rate - best_parent / best_div)) {
+                       best_parent = rounded;
+                       best_div = i;
+               }
+       }
+
+out:
+       *parent_rate = best_parent;
+
+       return best_parent / best_div;
+}
+
+static int sun4i_dclk_set_rate(struct clk_hw *hw, unsigned long rate,
+                              unsigned long parent_rate)
+{
+       struct sun4i_dclk *dclk = hw_to_dclk(hw);
+       u8 div = parent_rate / rate;
+
+       return regmap_update_bits(dclk->regmap, SUN4I_TCON0_DCLK_REG,
+                                 GENMASK(6, 0), div);
+}
+
+static int sun4i_dclk_get_phase(struct clk_hw *hw)
+{
+       struct sun4i_dclk *dclk = hw_to_dclk(hw);
+       u32 val;
+
+       regmap_read(dclk->regmap, SUN4I_TCON0_IO_POL_REG, &val);
+
+       val >>= 28;
+       val &= 3;
+
+       return val * 120;
+}
+
+static int sun4i_dclk_set_phase(struct clk_hw *hw, int degrees)
+{
+       struct sun4i_dclk *dclk = hw_to_dclk(hw);
+       u32 val = degrees / 120;
+
+       val <<= 28;
+
+       regmap_update_bits(dclk->regmap, SUN4I_TCON0_IO_POL_REG,
+                          GENMASK(29, 28),
+                          val);
+
+       return 0;
+}
+
+static const struct clk_ops sun4i_dclk_ops = {
+       .disable        = sun4i_dclk_disable,
+       .enable         = sun4i_dclk_enable,
+       .is_enabled     = sun4i_dclk_is_enabled,
+
+       .recalc_rate    = sun4i_dclk_recalc_rate,
+       .round_rate     = sun4i_dclk_round_rate,
+       .set_rate       = sun4i_dclk_set_rate,
+
+       .get_phase      = sun4i_dclk_get_phase,
+       .set_phase      = sun4i_dclk_set_phase,
+};
+
+int sun4i_dclk_create(struct device *dev, struct sun4i_tcon *tcon)
+{
+       const char *clk_name, *parent_name;
+       struct clk_init_data init;
+       struct sun4i_dclk *dclk;
+       int ret;
+
+       parent_name = __clk_get_name(tcon->sclk0);
+       ret = of_property_read_string_index(dev->of_node,
+                                           "clock-output-names", 0,
+                                           &clk_name);
+       if (ret)
+               return ret;
+
+       dclk = devm_kzalloc(dev, sizeof(*dclk), GFP_KERNEL);
+       if (!dclk)
+               return -ENOMEM;
+       dclk->tcon = tcon;
+
+       init.name = clk_name;
+       init.ops = &sun4i_dclk_ops;
+       init.parent_names = &parent_name;
+       init.num_parents = 1;
+       init.flags = CLK_SET_RATE_PARENT;
+
+       dclk->regmap = tcon->regs;
+       dclk->hw.init = &init;
+
+       tcon->dclk = clk_register(dev, &dclk->hw);
+       if (IS_ERR(tcon->dclk))
+               return PTR_ERR(tcon->dclk);
+
+       return 0;
+}
+EXPORT_SYMBOL(sun4i_dclk_create);
+
+int sun4i_dclk_free(struct sun4i_tcon *tcon)
+{
+       clk_unregister(tcon->dclk);
+       return 0;
+}
+EXPORT_SYMBOL(sun4i_dclk_free);
diff --git a/drivers/gpu/drm/sun4i/sun4i_tcon_dclk.h b/drivers/gpu/drm/sun4i/sun4i_tcon_dclk.h
new file mode 100644 (file)
index 0000000..ac60da2
--- /dev/null
@@ -0,0 +1,17 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+/*
+ * Copyright (C) 2015 Free Electrons
+ * Copyright (C) 2015 NextThing Co
+ *
+ * Maxime Ripard <maxime.ripard@free-electrons.com>
+ */
+
+#ifndef _SUN4I_DOTCLOCK_H_
+#define _SUN4I_DOTCLOCK_H_
+
+struct sun4i_tcon;
+
+int sun4i_dclk_create(struct device *dev, struct sun4i_tcon *tcon);
+int sun4i_dclk_free(struct sun4i_tcon *tcon);
+
+#endif /* _SUN4I_DOTCLOCK_H_ */