lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20150113002318.20842.90316@quantum>
Date:	Mon, 12 Jan 2015 16:23:18 -0800
From:	Mike Turquette <mturquette@...aro.org>
To:	Max Filippov <jcmvbkbc@...il.com>, linux-kernel@...r.kernel.org,
	devicetree@...r.kernel.org
Cc:	"Stephen Boyd" <sboyd@...eaurora.org>,
	"Rob Herring" <robh+dt@...nel.org>,
	"Pawel Moll" <pawel.moll@....com>,
	"Mark Rutland" <mark.rutland@....com>,
	"Ian Campbell" <ijc+devicetree@...lion.org.uk>,
	"Kumar Gala" <galak@...eaurora.org>,
	"Grant Likely" <grant.likely@...aro.org>,
	"Max Filippov" <jcmvbkbc@...il.com>
Subject: Re: [PATCH v2] clk: TI CDCE706 clock synthesizer driver

Quoting Max Filippov (2015-01-11 23:20:46)
> The driver allows using CDCE706 in its default configuration recorded in
> EEPROM and adjusting of synthesized clocks by consumers.
> 
> Signed-off-by: Max Filippov <jcmvbkbc@...il.com>

Applied.

Regards,
Mike

> ---
> Changes v1->v2:
> - add example usage to devicetree binding doc.
> 
>  .../devicetree/bindings/clock/ti,cdce706.txt       |  42 ++
>  MAINTAINERS                                        |   5 +
>  drivers/clk/Kconfig                                |   8 +
>  drivers/clk/Makefile                               |   1 +
>  drivers/clk/clk-cdce706.c                          | 700 +++++++++++++++++++++
>  5 files changed, 756 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/clock/ti,cdce706.txt
>  create mode 100644 drivers/clk/clk-cdce706.c
> 
> diff --git a/Documentation/devicetree/bindings/clock/ti,cdce706.txt b/Documentation/devicetree/bindings/clock/ti,cdce706.txt
> new file mode 100644
> index 0000000..616836e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/clock/ti,cdce706.txt
> @@ -0,0 +1,42 @@
> +Bindings for Texas Instruments CDCE706 programmable 3-PLL clock
> +synthesizer/multiplier/divider.
> +
> +Reference: http://www.ti.com/lit/ds/symlink/cdce706.pdf
> +
> +I2C device node required properties:
> +- compatible: shall be "ti,cdce706".
> +- reg: i2c device address, shall be in range [0x68...0x6b].
> +- #clock-cells: from common clock binding; shall be set to 1.
> +- clocks: from common clock binding; list of parent clock
> +  handles, shall be reference clock(s) connected to CLK_IN0
> +  and CLK_IN1 pins.
> +- clock-names: shall be clk_in0 and/or clk_in1. Use clk_in0
> +  in case of crystal oscillator or differential signal input
> +  configuration. Use clk_in0 and clk_in1 in case of independent
> +  single-ended LVCMOS inputs configuration.
> +
> +Example:
> +
> +       clocks {
> +               clk54: clk54 {
> +                       #clock-cells = <0>;
> +                       compatible = "fixed-clock";
> +                       clock-frequency = <54000000>;
> +               };
> +       };
> +       ...
> +       i2c0: i2c-master@...90000 {
> +               ...
> +               cdce706: clock-synth@69 {
> +                       compatible = "ti,cdce706";
> +                       #clock-cells = <1>;
> +                       reg = <0x69>;
> +                       clocks = <&clk54>;
> +                       clock-names = "clk_in0";
> +               };
> +       };
> +       ...
> +       simple-audio-card,codec {
> +               ...
> +               clocks = <&cdce706 4>;
> +       };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f1eb40f..923d9b3 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9536,6 +9536,11 @@ L:       linux-pm@...r.kernel.org
>  S:     Supported
>  F:     drivers/thermal/ti-soc-thermal/
>  
> +TI CDCE706 CLOCK DRIVER
> +M:     Max Filippov <jcmvbkbc@...il.com>
> +S:     Maintained
> +F:     drivers/clk/clk-cdce706.c
> +
>  TI CLOCK DRIVER
>  M:     Tero Kristo <t-kristo@...com>
>  L:     linux-omap@...r.kernel.org
> diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig
> index 3f44f292..975af6a 100644
> --- a/drivers/clk/Kconfig
> +++ b/drivers/clk/Kconfig
> @@ -134,6 +134,14 @@ config COMMON_CLK_PXA
>         ---help---
>           Sypport for the Marvell PXA SoC.
>  
> +config COMMON_CLK_CDCE706
> +       tristate "Clock driver for TI CDCE706 clock synthesizer"
> +       depends on I2C
> +       select REGMAP_I2C
> +       select RATIONAL
> +       ---help---
> +         This driver supports TI CDCE706 programmable 3-PLL clock synthesizer.
> +
>  source "drivers/clk/qcom/Kconfig"
>  
>  endmenu
> diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile
> index d5fba5b..929e11a 100644
> --- a/drivers/clk/Makefile
> +++ b/drivers/clk/Makefile
> @@ -19,6 +19,7 @@ endif
>  obj-$(CONFIG_COMMON_CLK_AXI_CLKGEN)    += clk-axi-clkgen.o
>  obj-$(CONFIG_ARCH_AXXIA)               += clk-axm5516.o
>  obj-$(CONFIG_ARCH_BCM2835)             += clk-bcm2835.o
> +obj-$(CONFIG_COMMON_CLK_CDCE706)       += clk-cdce706.o
>  obj-$(CONFIG_ARCH_CLPS711X)            += clk-clps711x.o
>  obj-$(CONFIG_ARCH_EFM32)               += clk-efm32gg.o
>  obj-$(CONFIG_ARCH_HIGHBANK)            += clk-highbank.o
> diff --git a/drivers/clk/clk-cdce706.c b/drivers/clk/clk-cdce706.c
> new file mode 100644
> index 0000000..c386ad2
> --- /dev/null
> +++ b/drivers/clk/clk-cdce706.c
> @@ -0,0 +1,700 @@
> +/*
> + * TI CDCE706 programmable 3-PLL clock synthesizer driver
> + *
> + * Copyright (c) 2014 Cadence Design Systems Inc.
> + *
> + * Reference: http://www.ti.com/lit/ds/symlink/cdce706.pdf
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/clk-provider.h>
> +#include <linux/delay.h>
> +#include <linux/i2c.h>
> +#include <linux/interrupt.h>
> +#include <linux/mod_devicetable.h>
> +#include <linux/module.h>
> +#include <linux/of.h>
> +#include <linux/rational.h>
> +#include <linux/regmap.h>
> +#include <linux/slab.h>
> +
> +#define CDCE706_CLKIN_CLOCK            10
> +#define CDCE706_CLKIN_SOURCE           11
> +#define CDCE706_PLL_M_LOW(pll)         (1 + 3 * (pll))
> +#define CDCE706_PLL_N_LOW(pll)         (2 + 3 * (pll))
> +#define CDCE706_PLL_HI(pll)            (3 + 3 * (pll))
> +#define CDCE706_PLL_MUX                        3
> +#define CDCE706_PLL_FVCO               6
> +#define CDCE706_DIVIDER(div)           (13 + (div))
> +#define CDCE706_CLKOUT(out)            (19 + (out))
> +
> +#define CDCE706_CLKIN_CLOCK_MASK       0x10
> +#define CDCE706_CLKIN_SOURCE_SHIFT     6
> +#define CDCE706_CLKIN_SOURCE_MASK      0xc0
> +#define CDCE706_CLKIN_SOURCE_LVCMOS    0x40
> +
> +#define CDCE706_PLL_MUX_MASK(pll)      (0x80 >> (pll))
> +#define CDCE706_PLL_LOW_M_MASK         0xff
> +#define CDCE706_PLL_LOW_N_MASK         0xff
> +#define CDCE706_PLL_HI_M_MASK          0x1
> +#define CDCE706_PLL_HI_N_MASK          0x1e
> +#define CDCE706_PLL_HI_N_SHIFT         1
> +#define CDCE706_PLL_M_MAX              0x1ff
> +#define CDCE706_PLL_N_MAX              0xfff
> +#define CDCE706_PLL_FVCO_MASK(pll)     (0x80 >> (pll))
> +#define CDCE706_PLL_FREQ_MIN            80000000
> +#define CDCE706_PLL_FREQ_MAX           300000000
> +#define CDCE706_PLL_FREQ_HI            180000000
> +
> +#define CDCE706_DIVIDER_PLL(div)       (9 + (div) - ((div) > 2) - ((div) > 4))
> +#define CDCE706_DIVIDER_PLL_SHIFT(div) ((div) < 2 ? 5 : 3 * ((div) & 1))
> +#define CDCE706_DIVIDER_PLL_MASK(div)  (0x7 << CDCE706_DIVIDER_PLL_SHIFT(div))
> +#define CDCE706_DIVIDER_DIVIDER_MASK   0x7f
> +#define CDCE706_DIVIDER_DIVIDER_MAX    0x7f
> +
> +#define CDCE706_CLKOUT_DIVIDER_MASK    0x7
> +#define CDCE706_CLKOUT_ENABLE_MASK     0x8
> +
> +static struct regmap_config cdce706_regmap_config = {
> +       .reg_bits = 8,
> +       .val_bits = 8,
> +       .val_format_endian = REGMAP_ENDIAN_NATIVE,
> +};
> +
> +#define to_hw_data(phw) (container_of((phw), struct cdce706_hw_data, hw))
> +
> +struct cdce706_hw_data {
> +       struct cdce706_dev_data *dev_data;
> +       unsigned idx;
> +       unsigned parent;
> +       struct clk *clk;
> +       struct clk_hw hw;
> +       unsigned div;
> +       unsigned mul;
> +       unsigned mux;
> +};
> +
> +struct cdce706_dev_data {
> +       struct i2c_client *client;
> +       struct regmap *regmap;
> +       struct clk_onecell_data onecell;
> +       struct clk *clks[6];
> +       struct clk *clkin_clk[2];
> +       const char *clkin_name[2];
> +       struct cdce706_hw_data clkin[1];
> +       struct cdce706_hw_data pll[3];
> +       struct cdce706_hw_data divider[6];
> +       struct cdce706_hw_data clkout[6];
> +};
> +
> +static const char * const cdce706_source_name[] = {
> +       "clk_in0", "clk_in1",
> +};
> +
> +static const char *cdce706_clkin_name[] = {
> +       "clk_in",
> +};
> +
> +static const char * const cdce706_pll_name[] = {
> +       "pll1", "pll2", "pll3",
> +};
> +
> +static const char *cdce706_divider_parent_name[] = {
> +       "clk_in", "pll1", "pll2", "pll2", "pll3",
> +};
> +
> +static const char *cdce706_divider_name[] = {
> +       "p0", "p1", "p2", "p3", "p4", "p5",
> +};
> +
> +static const char * const cdce706_clkout_name[] = {
> +       "clk_out0", "clk_out1", "clk_out2", "clk_out3", "clk_out4", "clk_out5",
> +};
> +
> +static int cdce706_reg_read(struct cdce706_dev_data *dev_data, unsigned reg,
> +                           unsigned *val)
> +{
> +       int rc = regmap_read(dev_data->regmap, reg | 0x80, val);
> +
> +       if (rc < 0)
> +               dev_err(&dev_data->client->dev, "error reading reg %u", reg);
> +       return rc;
> +}
> +
> +static int cdce706_reg_write(struct cdce706_dev_data *dev_data, unsigned reg,
> +                            unsigned val)
> +{
> +       int rc = regmap_write(dev_data->regmap, reg | 0x80, val);
> +
> +       if (rc < 0)
> +               dev_err(&dev_data->client->dev, "error writing reg %u", reg);
> +       return rc;
> +}
> +
> +static int cdce706_reg_update(struct cdce706_dev_data *dev_data, unsigned reg,
> +                             unsigned mask, unsigned val)
> +{
> +       int rc = regmap_update_bits(dev_data->regmap, reg | 0x80, mask, val);
> +
> +       if (rc < 0)
> +               dev_err(&dev_data->client->dev, "error updating reg %u", reg);
> +       return rc;
> +}
> +
> +static int cdce706_clkin_set_parent(struct clk_hw *hw, u8 index)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       hwd->parent = index;
> +       return 0;
> +}
> +
> +static u8 cdce706_clkin_get_parent(struct clk_hw *hw)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       return hwd->parent;
> +}
> +
> +static const struct clk_ops cdce706_clkin_ops = {
> +       .set_parent = cdce706_clkin_set_parent,
> +       .get_parent = cdce706_clkin_get_parent,
> +};
> +
> +static unsigned long cdce706_pll_recalc_rate(struct clk_hw *hw,
> +                                            unsigned long parent_rate)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       dev_dbg(&hwd->dev_data->client->dev,
> +               "%s, pll: %d, mux: %d, mul: %u, div: %u\n",
> +               __func__, hwd->idx, hwd->mux, hwd->mul, hwd->div);
> +
> +       if (!hwd->mux) {
> +               if (hwd->div && hwd->mul) {
> +                       u64 res = (u64)parent_rate * hwd->mul;
> +
> +                       do_div(res, hwd->div);
> +                       return res;
> +               }
> +       } else {
> +               if (hwd->div)
> +                       return parent_rate / hwd->div;
> +       }
> +       return 0;
> +}
> +
> +static long cdce706_pll_round_rate(struct clk_hw *hw, unsigned long rate,
> +                                  unsigned long *parent_rate)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +       unsigned long mul, div;
> +       u64 res;
> +
> +       dev_dbg(&hwd->dev_data->client->dev,
> +               "%s, rate: %lu, parent_rate: %lu\n",
> +               __func__, rate, *parent_rate);
> +
> +       rational_best_approximation(rate, *parent_rate,
> +                                   CDCE706_PLL_N_MAX, CDCE706_PLL_M_MAX,
> +                                   &mul, &div);
> +       hwd->mul = mul;
> +       hwd->div = div;
> +
> +       dev_dbg(&hwd->dev_data->client->dev,
> +               "%s, pll: %d, mul: %lu, div: %lu\n",
> +               __func__, hwd->idx, mul, div);
> +
> +       res = (u64)*parent_rate * hwd->mul;
> +       do_div(res, hwd->div);
> +       return res;
> +}
> +
> +static int cdce706_pll_set_rate(struct clk_hw *hw, unsigned long rate,
> +                               unsigned long parent_rate)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +       unsigned long mul = hwd->mul, div = hwd->div;
> +       int err;
> +
> +       dev_dbg(&hwd->dev_data->client->dev,
> +               "%s, pll: %d, mul: %lu, div: %lu\n",
> +               __func__, hwd->idx, mul, div);
> +
> +       err = cdce706_reg_update(hwd->dev_data,
> +                                CDCE706_PLL_HI(hwd->idx),
> +                                CDCE706_PLL_HI_M_MASK | CDCE706_PLL_HI_N_MASK,
> +                                ((div >> 8) & CDCE706_PLL_HI_M_MASK) |
> +                                ((mul >> (8 - CDCE706_PLL_HI_N_SHIFT)) &
> +                                 CDCE706_PLL_HI_N_MASK));
> +       if (err < 0)
> +               return err;
> +
> +       err = cdce706_reg_write(hwd->dev_data,
> +                               CDCE706_PLL_M_LOW(hwd->idx),
> +                               div & CDCE706_PLL_LOW_M_MASK);
> +       if (err < 0)
> +               return err;
> +
> +       err = cdce706_reg_write(hwd->dev_data,
> +                               CDCE706_PLL_N_LOW(hwd->idx),
> +                               mul & CDCE706_PLL_LOW_N_MASK);
> +       if (err < 0)
> +               return err;
> +
> +       err = cdce706_reg_update(hwd->dev_data,
> +                                CDCE706_PLL_FVCO,
> +                                CDCE706_PLL_FVCO_MASK(hwd->idx),
> +                                rate > CDCE706_PLL_FREQ_HI ?
> +                                CDCE706_PLL_FVCO_MASK(hwd->idx) : 0);
> +       return err;
> +}
> +
> +static const struct clk_ops cdce706_pll_ops = {
> +       .recalc_rate = cdce706_pll_recalc_rate,
> +       .round_rate = cdce706_pll_round_rate,
> +       .set_rate = cdce706_pll_set_rate,
> +};
> +
> +static int cdce706_divider_set_parent(struct clk_hw *hw, u8 index)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       if (hwd->parent == index)
> +               return 0;
> +       hwd->parent = index;
> +       return cdce706_reg_update(hwd->dev_data,
> +                                 CDCE706_DIVIDER_PLL(hwd->idx),
> +                                 CDCE706_DIVIDER_PLL_MASK(hwd->idx),
> +                                 index << CDCE706_DIVIDER_PLL_SHIFT(hwd->idx));
> +}
> +
> +static u8 cdce706_divider_get_parent(struct clk_hw *hw)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       return hwd->parent;
> +}
> +
> +static unsigned long cdce706_divider_recalc_rate(struct clk_hw *hw,
> +                                                unsigned long parent_rate)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       dev_dbg(&hwd->dev_data->client->dev,
> +               "%s, divider: %d, div: %u\n",
> +               __func__, hwd->idx, hwd->div);
> +       if (hwd->div)
> +               return parent_rate / hwd->div;
> +       return 0;
> +}
> +
> +static long cdce706_divider_round_rate(struct clk_hw *hw, unsigned long rate,
> +                                      unsigned long *parent_rate)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +       struct cdce706_dev_data *cdce = hwd->dev_data;
> +       unsigned long mul, div;
> +
> +       dev_dbg(&hwd->dev_data->client->dev,
> +               "%s, rate: %lu, parent_rate: %lu\n",
> +               __func__, rate, *parent_rate);
> +
> +       rational_best_approximation(rate, *parent_rate,
> +                                   1, CDCE706_DIVIDER_DIVIDER_MAX,
> +                                   &mul, &div);
> +       if (!mul)
> +               div = CDCE706_DIVIDER_DIVIDER_MAX;
> +
> +       if (__clk_get_flags(hw->clk) & CLK_SET_RATE_PARENT) {
> +               unsigned long best_diff = rate;
> +               unsigned long best_div = 0;
> +               struct clk *gp_clk = cdce->clkin_clk[cdce->clkin[0].parent];
> +               unsigned long gp_rate = gp_clk ? clk_get_rate(gp_clk) : 0;
> +
> +               for (div = CDCE706_PLL_FREQ_MIN / rate; best_diff &&
> +                    div <= CDCE706_PLL_FREQ_MAX / rate; ++div) {
> +                       unsigned long n, m;
> +                       unsigned long diff;
> +                       unsigned long div_rate;
> +                       u64 div_rate64;
> +
> +                       if (rate * div < CDCE706_PLL_FREQ_MIN)
> +                               continue;
> +
> +                       rational_best_approximation(rate * div, gp_rate,
> +                                                   CDCE706_PLL_N_MAX,
> +                                                   CDCE706_PLL_M_MAX,
> +                                                   &n, &m);
> +                       div_rate64 = (u64)gp_rate * n;
> +                       do_div(div_rate64, m);
> +                       do_div(div_rate64, div);
> +                       div_rate = div_rate64;
> +                       diff = max(div_rate, rate) - min(div_rate, rate);
> +
> +                       if (diff < best_diff) {
> +                               best_diff = diff;
> +                               best_div = div;
> +                               dev_dbg(&hwd->dev_data->client->dev,
> +                                       "%s, %lu * %lu / %lu / %lu = %lu\n",
> +                                       __func__, gp_rate, n, m, div, div_rate);
> +                       }
> +               }
> +
> +               div = best_div;
> +
> +               dev_dbg(&hwd->dev_data->client->dev,
> +                       "%s, altering parent rate: %lu -> %lu\n",
> +                       __func__, *parent_rate, rate * div);
> +               *parent_rate = rate * div;
> +       }
> +       hwd->div = div;
> +
> +       dev_dbg(&hwd->dev_data->client->dev,
> +               "%s, divider: %d, div: %lu\n",
> +               __func__, hwd->idx, div);
> +
> +       return *parent_rate / div;
> +}
> +
> +static int cdce706_divider_set_rate(struct clk_hw *hw, unsigned long rate,
> +                                   unsigned long parent_rate)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       dev_dbg(&hwd->dev_data->client->dev,
> +               "%s, divider: %d, div: %u\n",
> +               __func__, hwd->idx, hwd->div);
> +
> +       return cdce706_reg_update(hwd->dev_data,
> +                                 CDCE706_DIVIDER(hwd->idx),
> +                                 CDCE706_DIVIDER_DIVIDER_MASK,
> +                                 hwd->div);
> +}
> +
> +static const struct clk_ops cdce706_divider_ops = {
> +       .set_parent = cdce706_divider_set_parent,
> +       .get_parent = cdce706_divider_get_parent,
> +       .recalc_rate = cdce706_divider_recalc_rate,
> +       .round_rate = cdce706_divider_round_rate,
> +       .set_rate = cdce706_divider_set_rate,
> +};
> +
> +static int cdce706_clkout_prepare(struct clk_hw *hw)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       return cdce706_reg_update(hwd->dev_data, CDCE706_CLKOUT(hwd->idx),
> +                                 CDCE706_CLKOUT_ENABLE_MASK,
> +                                 CDCE706_CLKOUT_ENABLE_MASK);
> +}
> +
> +static void cdce706_clkout_unprepare(struct clk_hw *hw)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       cdce706_reg_update(hwd->dev_data, CDCE706_CLKOUT(hwd->idx),
> +                          CDCE706_CLKOUT_ENABLE_MASK, 0);
> +}
> +
> +static int cdce706_clkout_set_parent(struct clk_hw *hw, u8 index)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       if (hwd->parent == index)
> +               return 0;
> +       hwd->parent = index;
> +       return cdce706_reg_update(hwd->dev_data,
> +                                 CDCE706_CLKOUT(hwd->idx),
> +                                 CDCE706_CLKOUT_ENABLE_MASK, index);
> +}
> +
> +static u8 cdce706_clkout_get_parent(struct clk_hw *hw)
> +{
> +       struct cdce706_hw_data *hwd = to_hw_data(hw);
> +
> +       return hwd->parent;
> +}
> +
> +static unsigned long cdce706_clkout_recalc_rate(struct clk_hw *hw,
> +                                               unsigned long parent_rate)
> +{
> +       return parent_rate;
> +}
> +
> +static long cdce706_clkout_round_rate(struct clk_hw *hw, unsigned long rate,
> +                                     unsigned long *parent_rate)
> +{
> +       *parent_rate = rate;
> +       return rate;
> +}
> +
> +static int cdce706_clkout_set_rate(struct clk_hw *hw, unsigned long rate,
> +                                  unsigned long parent_rate)
> +{
> +       return 0;
> +}
> +
> +static const struct clk_ops cdce706_clkout_ops = {
> +       .prepare = cdce706_clkout_prepare,
> +       .unprepare = cdce706_clkout_unprepare,
> +       .set_parent = cdce706_clkout_set_parent,
> +       .get_parent = cdce706_clkout_get_parent,
> +       .recalc_rate = cdce706_clkout_recalc_rate,
> +       .round_rate = cdce706_clkout_round_rate,
> +       .set_rate = cdce706_clkout_set_rate,
> +};
> +
> +static int cdce706_register_hw(struct cdce706_dev_data *cdce,
> +                              struct cdce706_hw_data *hw, unsigned num_hw,
> +                              const char * const *clk_names,
> +                              struct clk_init_data *init)
> +{
> +       unsigned i;
> +
> +       for (i = 0; i < num_hw; ++i, ++hw) {
> +               init->name = clk_names[i];
> +               hw->dev_data = cdce;
> +               hw->idx = i;
> +               hw->hw.init = init;
> +               hw->clk = devm_clk_register(&cdce->client->dev,
> +                                           &hw->hw);
> +               if (IS_ERR(hw->clk)) {
> +                       dev_err(&cdce->client->dev, "Failed to register %s\n",
> +                               clk_names[i]);
> +                       return PTR_ERR(hw->clk);
> +               }
> +       }
> +       return 0;
> +}
> +
> +static int cdce706_register_clkin(struct cdce706_dev_data *cdce)
> +{
> +       struct clk_init_data init = {
> +               .ops = &cdce706_clkin_ops,
> +               .parent_names = cdce->clkin_name,
> +               .num_parents = ARRAY_SIZE(cdce->clkin_name),
> +       };
> +       unsigned i;
> +       int ret;
> +       unsigned clock, source;
> +
> +       for (i = 0; i < ARRAY_SIZE(cdce->clkin_name); ++i) {
> +               struct clk *parent = devm_clk_get(&cdce->client->dev,
> +                                                 cdce706_source_name[i]);
> +
> +               if (IS_ERR(parent)) {
> +                       cdce->clkin_name[i] = cdce706_source_name[i];
> +               } else {
> +                       cdce->clkin_name[i] = __clk_get_name(parent);
> +                       cdce->clkin_clk[i] = parent;
> +               }
> +       }
> +
> +       ret = cdce706_reg_read(cdce, CDCE706_CLKIN_SOURCE, &source);
> +       if (ret < 0)
> +               return ret;
> +       if ((source & CDCE706_CLKIN_SOURCE_MASK) ==
> +           CDCE706_CLKIN_SOURCE_LVCMOS) {
> +               ret = cdce706_reg_read(cdce, CDCE706_CLKIN_CLOCK, &clock);
> +               if (ret < 0)
> +                       return ret;
> +               cdce->clkin[0].parent = !!(clock & CDCE706_CLKIN_CLOCK_MASK);
> +       }
> +
> +       ret = cdce706_register_hw(cdce, cdce->clkin,
> +                                 ARRAY_SIZE(cdce->clkin),
> +                                 cdce706_clkin_name, &init);
> +       return ret;
> +}
> +
> +static int cdce706_register_plls(struct cdce706_dev_data *cdce)
> +{
> +       struct clk_init_data init = {
> +               .ops = &cdce706_pll_ops,
> +               .parent_names = cdce706_clkin_name,
> +               .num_parents = ARRAY_SIZE(cdce706_clkin_name),
> +       };
> +       unsigned i;
> +       int ret;
> +       unsigned mux;
> +
> +       ret = cdce706_reg_read(cdce, CDCE706_PLL_MUX, &mux);
> +       if (ret < 0)
> +               return ret;
> +
> +       for (i = 0; i < ARRAY_SIZE(cdce->pll); ++i) {
> +               unsigned m, n, v;
> +
> +               ret = cdce706_reg_read(cdce, CDCE706_PLL_M_LOW(i), &m);
> +               if (ret < 0)
> +                       return ret;
> +               ret = cdce706_reg_read(cdce, CDCE706_PLL_N_LOW(i), &n);
> +               if (ret < 0)
> +                       return ret;
> +               ret = cdce706_reg_read(cdce, CDCE706_PLL_HI(i), &v);
> +               if (ret < 0)
> +                       return ret;
> +               cdce->pll[i].div = m | ((v & CDCE706_PLL_HI_M_MASK) << 8);
> +               cdce->pll[i].mul = n | ((v & CDCE706_PLL_HI_N_MASK) <<
> +                                       (8 - CDCE706_PLL_HI_N_SHIFT));
> +               cdce->pll[i].mux = mux & CDCE706_PLL_MUX_MASK(i);
> +               dev_dbg(&cdce->client->dev,
> +                       "%s: i: %u, div: %u, mul: %u, mux: %d\n", __func__, i,
> +                       cdce->pll[i].div, cdce->pll[i].mul, cdce->pll[i].mux);
> +       }
> +
> +       ret = cdce706_register_hw(cdce, cdce->pll,
> +                                 ARRAY_SIZE(cdce->pll),
> +                                 cdce706_pll_name, &init);
> +       return ret;
> +}
> +
> +static int cdce706_register_dividers(struct cdce706_dev_data *cdce)
> +{
> +       struct clk_init_data init = {
> +               .ops = &cdce706_divider_ops,
> +               .parent_names = cdce706_divider_parent_name,
> +               .num_parents = ARRAY_SIZE(cdce706_divider_parent_name),
> +               .flags = CLK_SET_RATE_PARENT,
> +       };
> +       unsigned i;
> +       int ret;
> +
> +       for (i = 0; i < ARRAY_SIZE(cdce->divider); ++i) {
> +               unsigned val;
> +
> +               ret = cdce706_reg_read(cdce, CDCE706_DIVIDER_PLL(i), &val);
> +               if (ret < 0)
> +                       return ret;
> +               cdce->divider[i].parent =
> +                       (val & CDCE706_DIVIDER_PLL_MASK(i)) >>
> +                       CDCE706_DIVIDER_PLL_SHIFT(i);
> +
> +               ret = cdce706_reg_read(cdce, CDCE706_DIVIDER(i), &val);
> +               if (ret < 0)
> +                       return ret;
> +               cdce->divider[i].div = val & CDCE706_DIVIDER_DIVIDER_MASK;
> +               dev_dbg(&cdce->client->dev,
> +                       "%s: i: %u, parent: %u, div: %u\n", __func__, i,
> +                       cdce->divider[i].parent, cdce->divider[i].div);
> +       }
> +
> +       ret = cdce706_register_hw(cdce, cdce->divider,
> +                                 ARRAY_SIZE(cdce->divider),
> +                                 cdce706_divider_name, &init);
> +       return ret;
> +}
> +
> +static int cdce706_register_clkouts(struct cdce706_dev_data *cdce)
> +{
> +       struct clk_init_data init = {
> +               .ops = &cdce706_clkout_ops,
> +               .parent_names = cdce706_divider_name,
> +               .num_parents = ARRAY_SIZE(cdce706_divider_name),
> +               .flags = CLK_SET_RATE_PARENT,
> +       };
> +       unsigned i;
> +       int ret;
> +
> +       for (i = 0; i < ARRAY_SIZE(cdce->clkout); ++i) {
> +               unsigned val;
> +
> +               ret = cdce706_reg_read(cdce, CDCE706_CLKOUT(i), &val);
> +               if (ret < 0)
> +                       return ret;
> +               cdce->clkout[i].parent = val & CDCE706_CLKOUT_DIVIDER_MASK;
> +               dev_dbg(&cdce->client->dev,
> +                       "%s: i: %u, parent: %u\n", __func__, i,
> +                       cdce->clkout[i].parent);
> +       }
> +
> +       ret = cdce706_register_hw(cdce, cdce->clkout,
> +                                 ARRAY_SIZE(cdce->clkout),
> +                                 cdce706_clkout_name, &init);
> +       for (i = 0; i < ARRAY_SIZE(cdce->clkout); ++i)
> +               cdce->clks[i] = cdce->clkout[i].clk;
> +
> +       return ret;
> +}
> +
> +static int cdce706_probe(struct i2c_client *client,
> +                        const struct i2c_device_id *id)
> +{
> +       struct i2c_adapter *adapter = to_i2c_adapter(client->dev.parent);
> +       struct cdce706_dev_data *cdce;
> +       int ret;
> +
> +       if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA))
> +               return -EIO;
> +
> +       cdce = devm_kzalloc(&client->dev, sizeof(*cdce), GFP_KERNEL);
> +       if (!cdce)
> +               return -ENOMEM;
> +
> +       cdce->client = client;
> +       cdce->regmap = devm_regmap_init_i2c(client, &cdce706_regmap_config);
> +       if (IS_ERR(cdce->regmap)) {
> +               dev_err(&client->dev, "Failed to initialize regmap\n");
> +               return -EINVAL;
> +       }
> +
> +       i2c_set_clientdata(client, cdce);
> +
> +       ret = cdce706_register_clkin(cdce);
> +       if (ret < 0)
> +               return ret;
> +       ret = cdce706_register_plls(cdce);
> +       if (ret < 0)
> +               return ret;
> +       ret = cdce706_register_dividers(cdce);
> +       if (ret < 0)
> +               return ret;
> +       ret = cdce706_register_clkouts(cdce);
> +       if (ret < 0)
> +               return ret;
> +       cdce->onecell.clks = cdce->clks;
> +       cdce->onecell.clk_num = ARRAY_SIZE(cdce->clks);
> +       ret = of_clk_add_provider(client->dev.of_node, of_clk_src_onecell_get,
> +                                 &cdce->onecell);
> +
> +       return ret;
> +}
> +
> +static int cdce706_remove(struct i2c_client *client)
> +{
> +       return 0;
> +}
> +
> +
> +#ifdef CONFIG_OF
> +static const struct of_device_id cdce706_dt_match[] = {
> +       { .compatible = "ti,cdce706" },
> +       { },
> +};
> +MODULE_DEVICE_TABLE(of, cdce706_dt_match);
> +#endif
> +
> +static const struct i2c_device_id cdce706_id[] = {
> +       { "cdce706", 0 },
> +       { }
> +};
> +MODULE_DEVICE_TABLE(i2c, cdce706_id);
> +
> +static struct i2c_driver cdce706_i2c_driver = {
> +       .driver = {
> +               .name   = "cdce706",
> +               .of_match_table = of_match_ptr(cdce706_dt_match),
> +       },
> +       .probe          = cdce706_probe,
> +       .remove         = cdce706_remove,
> +       .id_table       = cdce706_id,
> +};
> +module_i2c_driver(cdce706_i2c_driver);
> +
> +MODULE_AUTHOR("Max Filippov <jcmvbkbc@...il.com>");
> +MODULE_DESCRIPTION("TI CDCE 706 clock synthesizer driver");
> +MODULE_LICENSE("GPL");
> -- 
> 1.8.1.4
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ