Hi Lukasz,

On 1/28/20 9:20 AM, Lukasz Majewski wrote:
Hi Giulio,

Add support for PLLV3 AV type.

If this code has been ported from Linux kernel, then provide SHA1,
branch, and commit message.

Oh sorry,

Code taken from Linux with below commit:
"ARM: imx: add common clock support for pllv3"
(sha1: a3f6b9dbf2a964b85f0523e577807d6f63d6b29d)

Hope this way is ok, I imitated other commit logs in u-boot.

Best regards
--
Giulio Benetti
Benetti Engineering sas

Acked-by: Lukasz Majewski <lu...@denx.de>


Signed-off-by: Giulio Benetti <giulio.bene...@benettiengineering.com>
---
  drivers/clk/imx/clk-pllv3.c | 76
+++++++++++++++++++++++++++++++++++++ 1 file changed, 76 insertions(+)

diff --git a/drivers/clk/imx/clk-pllv3.c b/drivers/clk/imx/clk-pllv3.c
index d5087a104e..fc16416d5f 100644
--- a/drivers/clk/imx/clk-pllv3.c
+++ b/drivers/clk/imx/clk-pllv3.c
@@ -6,6 +6,7 @@
#include <common.h>
  #include <asm/io.h>
+#include <div64.h>
  #include <malloc.h>
  #include <clk-uclass.h>
  #include <dm/device.h>
@@ -16,6 +17,10 @@
  #define UBOOT_DM_CLK_IMX_PLLV3_GENERIC        "imx_clk_pllv3_generic"
  #define UBOOT_DM_CLK_IMX_PLLV3_SYS    "imx_clk_pllv3_sys"
  #define UBOOT_DM_CLK_IMX_PLLV3_USB    "imx_clk_pllv3_usb"
+#define UBOOT_DM_CLK_IMX_PLLV3_AV      "imx_clk_pllv3_av"
+
+#define PLL_NUM_OFFSET         0x10
+#define PLL_DENOM_OFFSET       0x20
#define BM_PLL_POWER (0x1 << 12)
  #define BM_PLL_LOCK           (0x1 << 31)
@@ -143,6 +148,65 @@ static const struct clk_ops clk_pllv3_sys_ops = {
        .set_rate       = clk_pllv3_sys_set_rate,
  };
+static ulong clk_pllv3_av_get_rate(struct clk *clk)
+{
+       struct clk_pllv3 *pll = to_clk_pllv3(clk);
+       unsigned long parent_rate = clk_get_parent_rate(clk);
+       u32 mfn = readl(pll->base + PLL_NUM_OFFSET);
+       u32 mfd = readl(pll->base + PLL_DENOM_OFFSET);
+       u32 div = readl(pll->base) & pll->div_mask;
+       u64 temp64 = (u64)parent_rate;
+
+       temp64 *= mfn;
+       do_div(temp64, mfd);
+
+       return parent_rate * div + (unsigned long)temp64;
+}
+
+static ulong clk_pllv3_av_set_rate(struct clk *clk, ulong rate)
+{
+       struct clk_pllv3 *pll = to_clk_pllv3(clk);
+       unsigned long parent_rate = clk_get_parent_rate(clk);
+       unsigned long min_rate = parent_rate * 27;
+       unsigned long max_rate = parent_rate * 54;
+       u32 val, div;
+       u32 mfn, mfd = 1000000;
+       u32 max_mfd = 0x3FFFFFFF;
+       u64 temp64;
+
+       if (rate < min_rate || rate > max_rate)
+               return -EINVAL;
+
+       if (parent_rate <= max_mfd)
+               mfd = parent_rate;
+
+       div = rate / parent_rate;
+       temp64 = (u64)(rate - div * parent_rate);
+       temp64 *= mfd;
+       do_div(temp64, parent_rate);
+       mfn = temp64;
+
+       val = readl(pll->base);
+       val &= ~pll->div_mask;
+       val |= div;
+       writel(val, pll->base);
+       writel(mfn, pll->base + PLL_NUM_OFFSET);
+       writel(mfd, pll->base + PLL_DENOM_OFFSET);
+
+       /* Wait for PLL to lock */
+       while (!(readl(pll->base) & BM_PLL_LOCK))
+               ;
+
+       return 0;
+}
+
+static const struct clk_ops clk_pllv3_av_ops = {
+       .enable         = clk_pllv3_generic_enable,
+       .disable        = clk_pllv3_generic_disable,
+       .get_rate       = clk_pllv3_av_get_rate,
+       .set_rate       = clk_pllv3_av_set_rate,
+};
+
  struct clk *imx_clk_pllv3(enum imx_pllv3_type type, const char *name,
                          const char *parent_name, void __iomem
*base, u32 div_mask)
@@ -174,6 +238,11 @@ struct clk *imx_clk_pllv3(enum imx_pllv3_type
type, const char *name, pll->div_shift = 1;
                pll->powerup_set = true;
                break;
+       case IMX_PLLV3_AV:
+               drv_name = UBOOT_DM_CLK_IMX_PLLV3_AV;
+               pll->div_shift = 0;
+               pll->powerup_set = false;
+               break;
        default:
                kfree(pll);
                return ERR_PTR(-ENOTSUPP);
@@ -212,3 +281,10 @@ U_BOOT_DRIVER(clk_pllv3_usb) = {
        .ops    = &clk_pllv3_generic_ops,
        .flags = DM_FLAG_PRE_RELOC,
  };
+
+U_BOOT_DRIVER(clk_pllv3_av) = {
+       .name   = UBOOT_DM_CLK_IMX_PLLV3_AV,
+       .id     = UCLASS_CLK,
+       .ops    = &clk_pllv3_av_ops,
+       .flags = DM_FLAG_PRE_RELOC,
+};




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de


Reply via email to