On Wed, Apr 01, 2020 at 12:49:29PM +0530, Amit Singh Tomar wrote:
> This patch converts S900 clock driver to something common that can
> be used for other SoCs, for instance S700(few of clk registers are same).
> 
> Reviewed-by: Andre Przywara <andre.przyw...@arm.com>
> Signed-off-by: Amit Singh Tomar <amittome...@gmail.com>
> ---
> Changes since v8:
>         * Fixed the bubblegum-96 boot issue by introducing
>           dev_get_driver_data API to read driver data.
> Changes since v7:
>         * No changes.
> Changes since v6:
>         * Fixed the bug that breaks S900(missing break in switch statement).
> Changes since v5:
>         * No changes.
> Changes since v4:
>         * This patch is re-ordered, moved from 08/11 to 04/11.
> Changes since v3:
>         * Replaced dtsi with dts in subject line along with arm:dts:
>           to the prefix.
>         * Added reviewed-by tag.
> Changes since v2:
>         * Newly added patch, not there in v2/v1.
> ---
>  arch/arm/Kconfig                          |   2 +
>  arch/arm/include/asm/arch-owl/clk_s900.h  |  57 -----------
>  arch/arm/include/asm/arch-owl/regs_s700.h |  56 +++++++++++
>  configs/bubblegum_96_defconfig            |   3 -
>  drivers/clk/owl/Kconfig                   |   8 +-
>  drivers/clk/owl/Makefile                  |   2 +-
>  drivers/clk/owl/clk_owl.c                 | 155 
> ++++++++++++++++++++++++++++++
>  drivers/clk/owl/clk_owl.h                 |  64 ++++++++++++
>  drivers/clk/owl/clk_s900.c                | 137 --------------------------
>  9 files changed, 280 insertions(+), 204 deletions(-)
>  delete mode 100644 arch/arm/include/asm/arch-owl/clk_s900.h
>  create mode 100644 arch/arm/include/asm/arch-owl/regs_s700.h
>  create mode 100644 drivers/clk/owl/clk_owl.c
>  create mode 100644 drivers/clk/owl/clk_owl.h
>  delete mode 100644 drivers/clk/owl/clk_s900.c
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index de6b7f9..05658cf 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -876,6 +876,8 @@ config ARCH_OWL
>       select DM
>       select DM_SERIAL
>       select OWL_SERIAL
> +     select CLK
> +     select CLK_OWL
>       select OF_CONTROL
>       imply CMD_DM
>  
> diff --git a/arch/arm/include/asm/arch-owl/clk_s900.h 
> b/arch/arm/include/asm/arch-owl/clk_s900.h
> deleted file mode 100644
> index 88e88f7..0000000
> --- a/arch/arm/include/asm/arch-owl/clk_s900.h
> +++ /dev/null
> @@ -1,57 +0,0 @@
> -/* SPDX-License-Identifier: GPL-2.0+ */
> -/*
> - * Actions Semi S900 Clock Definitions
> - *
> - * Copyright (C) 2015 Actions Semi Co., Ltd.
> - * Copyright (C) 2018 Manivannan Sadhasivam 
> <manivannan.sadhasi...@linaro.org>
> - *
> - */
> -
> -#ifndef _OWL_CLK_S900_H_
> -#define _OWL_CLK_S900_H_
> -
> -#include <clk-uclass.h>
> -
> -struct owl_clk_priv {
> -     phys_addr_t base;
> -};
> -
> -/* BUSCLK register definitions */
> -#define CMU_PDBGDIV_8                7
> -#define CMU_PDBGDIV_SHIFT    26
> -#define CMU_PDBGDIV_DIV              (CMU_PDBGDIV_8 << CMU_PDBGDIV_SHIFT)
> -#define CMU_PERDIV_8         7
> -#define CMU_PERDIV_SHIFT     20
> -#define CMU_PERDIV_DIV               (CMU_PERDIV_8 << CMU_PERDIV_SHIFT)
> -#define CMU_NOCDIV_2         1
> -#define CMU_NOCDIV_SHIFT     19
> -#define CMU_NOCDIV_DIV               (CMU_NOCDIV_2 << CMU_NOCDIV_SHIFT)
> -#define CMU_DMMCLK_SRC_APLL  2
> -#define CMU_DMMCLK_SRC_SHIFT 10
> -#define CMU_DMMCLK_SRC               (CMU_DMMCLK_SRC_APLL << 
> CMU_DMMCLK_SRC_SHIFT)
> -#define CMU_APBCLK_DIV               BIT(8)
> -#define CMU_NOCCLK_SRC               BIT(7)
> -#define CMU_AHBCLK_DIV               BIT(4)
> -#define CMU_CORECLK_MASK     3
> -#define CMU_CORECLK_CPLL     BIT(1)
> -#define CMU_CORECLK_HOSC     BIT(0)
> -
> -/* COREPLL register definitions */
> -#define CMU_COREPLL_EN               BIT(9)
> -#define CMU_COREPLL_HOSC_EN  BIT(8)
> -#define CMU_COREPLL_OUT              (1104 / 24)
> -
> -/* DEVPLL register definitions */
> -#define CMU_DEVPLL_CLK               BIT(12)
> -#define CMU_DEVPLL_EN                BIT(8)
> -#define CMU_DEVPLL_OUT               (660 / 6)
> -
> -/* UARTCLK register definitions */
> -#define CMU_UARTCLK_SRC_DEVPLL       BIT(16)
> -
> -/* DEVCLKEN1 register definitions */
> -#define CMU_DEVCLKEN1_UART5  BIT(21)
> -
> -#define PLL_STABILITY_WAIT_US        50
> -
> -#endif
> diff --git a/arch/arm/include/asm/arch-owl/regs_s700.h 
> b/arch/arm/include/asm/arch-owl/regs_s700.h
> new file mode 100644
> index 0000000..2f21c15
> --- /dev/null
> +++ b/arch/arm/include/asm/arch-owl/regs_s700.h
> @@ -0,0 +1,56 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */
> +/*
> + * Actions Semi S700 Register Definitions
> + *
> + */
> +
> +#ifndef _OWL_REGS_S700_H_
> +#define _OWL_REGS_S700_H_
> +
> +#define CMU_COREPLL          0x0000
> +#define CMU_DEVPLL           0x0004
> +#define CMU_DDRPLL           0x0008
> +#define CMU_NANDPLL          0x000C
> +#define CMU_DISPLAYPLL               0x0010
> +#define CMU_AUDIOPLL         0x0014
> +#define CMU_TVOUTPLL         0x0018
> +#define CMU_BUSCLK           0x001C
> +#define CMU_SENSORCLK                0x0020
> +#define CMU_LCDCLK           0x0024
> +#define CMU_DSIPLLCLK                0x0028
> +#define CMU_CSICLK           0x002C
> +#define CMU_DECLK            0x0030
> +#define CMU_SICLK            0x0034
> +#define CMU_BUSCLK1          0x0038
> +#define CMU_HDECLK           0x003C
> +#define CMU_VDECLK           0x0040
> +#define CMU_VCECLK           0x0044
> +#define CMU_NANDCCLK         0x004C
> +#define CMU_SD0CLK           0x0050
> +#define CMU_SD1CLK           0x0054
> +#define CMU_SD2CLK           0x0058
> +#define CMU_UART0CLK         0x005C
> +#define CMU_UART1CLK         0x0060
> +#define CMU_UART2CLK         0x0064
> +#define CMU_UART3CLK         0x0068
> +#define CMU_UART4CLK         0x006C
> +#define CMU_UART5CLK         0x0070
> +#define CMU_UART6CLK         0x0074
> +#define CMU_PWM0CLK          0x0078
> +#define CMU_PWM1CLK          0x007C
> +#define CMU_PWM2CLK          0x0080
> +#define CMU_PWM3CLK          0x0084
> +#define CMU_PWM4CLK          0x0088
> +#define CMU_PWM5CLK          0x008C
> +#define CMU_GPU3DCLK         0x0090
> +#define CMU_CORECTL          0x009C
> +#define CMU_DEVCLKEN0                0x00A0
> +#define CMU_DEVCLKEN1                0x00A4
> +#define CMU_DEVRST0          0x00A8
> +#define CMU_DEVRST1          0x00AC
> +#define CMU_USBPLL           0x00B0
> +#define CMU_ETHERNETPLL              0x00B4
> +#define CMU_CVBSPLL          0x00B8
> +#define CMU_SSTSCLK          0x00C0
> +
> +#endif
> diff --git a/configs/bubblegum_96_defconfig b/configs/bubblegum_96_defconfig
> index 8c94def..e76e9a2 100644
> --- a/configs/bubblegum_96_defconfig
> +++ b/configs/bubblegum_96_defconfig
> @@ -17,6 +17,3 @@ CONFIG_CMD_CACHE=y
>  CONFIG_CMD_TIMER=y
>  CONFIG_DEFAULT_DEVICE_TREE="bubblegum_96"
>  CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> -CONFIG_CLK=y
> -CONFIG_CLK_OWL=y
> -CONFIG_CLK_S900=y
> diff --git a/drivers/clk/owl/Kconfig b/drivers/clk/owl/Kconfig
> index 661f198..c6afef9 100644
> --- a/drivers/clk/owl/Kconfig
> +++ b/drivers/clk/owl/Kconfig
> @@ -3,10 +3,6 @@ config CLK_OWL
>          depends on CLK && ARCH_OWL
>          help
>            Enable support for clock managemet unit present in Actions Semi
> -       OWL SoCs.
> +       Owl series S900/S700 SoCs.
> +
>  
> -config CLK_S900
> -        bool "Actions Semi S900 clock driver"
> -        depends on CLK_OWL && ARM64
> -        help
> -          Enable support for the clocks in Actions Semi S900 SoC.
> diff --git a/drivers/clk/owl/Makefile b/drivers/clk/owl/Makefile
> index 63ab573..5218b6b 100644
> --- a/drivers/clk/owl/Makefile
> +++ b/drivers/clk/owl/Makefile
> @@ -1,3 +1,3 @@
>  # SPDX-License-Identifier: GPL-2.0+
>  
> -obj-$(CONFIG_CLK_S900) += clk_s900.o
> +obj-$(CONFIG_CLK_OWL) += clk_owl.o
> diff --git a/drivers/clk/owl/clk_owl.c b/drivers/clk/owl/clk_owl.c
> new file mode 100644
> index 0000000..96a5593
> --- /dev/null
> +++ b/drivers/clk/owl/clk_owl.c
> @@ -0,0 +1,155 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Common clock driver for Actions Semi SoCs.
> + *
> + * Copyright (C) 2015 Actions Semi Co., Ltd.
> + * Copyright (C) 2018 Manivannan Sadhasivam 
> <manivannan.sadhasi...@linaro.org>
> + */
> +
> +#include <common.h>
> +#include <dm.h>
> +#include "clk_owl.h"
> +#include <asm/io.h>
> +#if defined(CONFIG_MACH_S900)
> +#include <asm/arch-owl/regs_s900.h>
> +#include <dt-bindings/clock/actions,s900-cmu.h>
> +#elif defined(CONFIG_MACH_S700)
> +#include <asm/arch-owl/regs_s700.h>
> +#include <dt-bindings/clock/actions,s700-cmu.h>
> +#endif
> +
> +void owl_clk_init(struct owl_clk_priv *priv)
> +{
> +     u32 bus_clk = 0, core_pll, dev_pll;
> +
> +#if defined(CONFIG_MACH_S900)
> +     /* Enable ASSIST_PLL */
> +     setbits_le32(priv->base + CMU_ASSISTPLL, BIT(0));
> +     udelay(PLL_STABILITY_WAIT_US);
> +#endif
> +
> +     /* Source HOSC to DEV_CLK */
> +     clrbits_le32(priv->base + CMU_DEVPLL, CMU_DEVPLL_CLK);
> +
> +     /* Configure BUS_CLK */
> +     bus_clk |= (CMU_PDBGDIV_DIV | CMU_PERDIV_DIV | CMU_NOCDIV_DIV |
> +                     CMU_DMMCLK_SRC | CMU_APBCLK_DIV | CMU_AHBCLK_DIV |
> +                     CMU_NOCCLK_SRC | CMU_CORECLK_HOSC);
> +     writel(bus_clk, priv->base + CMU_BUSCLK);
> +
> +     udelay(PLL_STABILITY_WAIT_US);
> +
> +     /* Configure CORE_PLL */
> +     core_pll = readl(priv->base + CMU_COREPLL);
> +     core_pll |= (CMU_COREPLL_EN | CMU_COREPLL_HOSC_EN | CMU_COREPLL_OUT);
> +     writel(core_pll, priv->base + CMU_COREPLL);
> +
> +     udelay(PLL_STABILITY_WAIT_US);
> +
> +     /* Configure DEV_PLL */
> +     dev_pll = readl(priv->base + CMU_DEVPLL);
> +     dev_pll |= (CMU_DEVPLL_EN | CMU_DEVPLL_OUT);
> +     writel(dev_pll, priv->base + CMU_DEVPLL);
> +
> +     udelay(PLL_STABILITY_WAIT_US);
> +
> +     /* Source CORE_PLL for CORE_CLK */
> +     clrsetbits_le32(priv->base + CMU_BUSCLK, CMU_CORECLK_MASK,
> +                     CMU_CORECLK_CPLL);
> +
> +     /* Source DEV_PLL for DEV_CLK */
> +     setbits_le32(priv->base + CMU_DEVPLL, CMU_DEVPLL_CLK);
> +
> +     udelay(PLL_STABILITY_WAIT_US);
> +}
> +
> +int owl_clk_enable(struct clk *clk)
> +{
> +     struct owl_clk_priv *priv = dev_get_priv(clk->dev);
> +     enum owl_soc model = dev_get_driver_data(clk->dev);
> +
> +     switch (clk->id) {
> +     case CLK_UART5:
> +             if (model == S700)
> +                     return -EINVAL;

This doesn't look right to me. What will happen if S500 is added some day? Will
you extend this to (model == S700 || model == S500)? NO...

Please check for the relevant SoC instead as,

if (model != S900)
        return -EINVAL;

On a side note, I really hope that only the serial ports associated w/ console
will only be used :)

Thanks,
Mani

> +             /* Source HOSC for UART5 interface */
> +             clrbits_le32(priv->base + CMU_UART5CLK, CMU_UARTCLK_SRC_DEVPLL);
> +             /* Enable UART5 interface clock */
> +             setbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_UART5);
> +             break;
> +     case CLK_UART3:
> +             if (model == S900)
> +                     return -EINVAL;
> +             /* Source HOSC for UART3 interface */
> +             clrbits_le32(priv->base + CMU_UART3CLK, CMU_UARTCLK_SRC_DEVPLL);
> +             /* Enable UART3 interface clock */
> +             setbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_UART3);
> +             break;
> +     default:
> +             return -EINVAL;
> +     }
> +
> +     return 0;
> +}
> +
> +int owl_clk_disable(struct clk *clk)
> +{
> +     struct owl_clk_priv *priv = dev_get_priv(clk->dev);
> +     enum owl_soc model = dev_get_driver_data(clk->dev);
> +
> +     switch (clk->id) {
> +     case CLK_UART5:
> +             if (model == S700)
> +                     return -EINVAL;
> +             /* Disable UART5 interface clock */
> +             clrbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_UART5);
> +             break;
> +     case CLK_UART3:
> +             if (model == S900)
> +                     return -EINVAL;
> +             /* Disable UART3 interface clock */
> +             clrbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_UART3);
> +             break;
> +     default:
> +             return -EINVAL;
> +     }
> +
> +     return 0;
> +}
> +
> +static int owl_clk_probe(struct udevice *dev)
> +{
> +     struct owl_clk_priv *priv = dev_get_priv(dev);
> +
> +     priv->base = dev_read_addr(dev);
> +     if (priv->base == FDT_ADDR_T_NONE)
> +             return -EINVAL;
> +
> +     /* setup necessary clocks */
> +     owl_clk_init(priv);
> +
> +     return 0;
> +}
> +
> +static const struct clk_ops owl_clk_ops = {
> +     .enable = owl_clk_enable,
> +     .disable = owl_clk_disable,
> +};
> +
> +static const struct udevice_id owl_clk_ids[] = {
> +#if defined(CONFIG_MACH_S900)
> +     { .compatible = "actions,s900-cmu", .data = S900 },
> +#elif defined(CONFIG_MACH_S700)
> +     { .compatible = "actions,s700-cmu", .data = S700 },
> +#endif
> +     { }
> +};
> +
> +U_BOOT_DRIVER(clk_owl) = {
> +     .name           = "clk_owl",
> +     .id             = UCLASS_CLK,
> +     .of_match       = owl_clk_ids,
> +     .ops            = &owl_clk_ops,
> +     .priv_auto_alloc_size = sizeof(struct owl_clk_priv),
> +     .probe          = owl_clk_probe,
> +};
> diff --git a/drivers/clk/owl/clk_owl.h b/drivers/clk/owl/clk_owl.h
> new file mode 100644
> index 0000000..b8d3362
> --- /dev/null
> +++ b/drivers/clk/owl/clk_owl.h
> @@ -0,0 +1,64 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */
> +/*
> + * Actions Semi SoCs Clock Definitions
> + *
> + * Copyright (C) 2015 Actions Semi Co., Ltd.
> + * Copyright (C) 2018 Manivannan Sadhasivam 
> <manivannan.sadhasi...@linaro.org>
> + *
> + */
> +
> +#ifndef _OWL_CLK_H_
> +#define _OWL_CLK_H_
> +
> +#include <clk-uclass.h>
> +
> +enum owl_soc {
> +     S700,
> +     S900,
> +};
> +
> +struct owl_clk_priv {
> +     phys_addr_t base;
> +};
> +
> +/* BUSCLK register definitions */
> +#define CMU_PDBGDIV_8                7
> +#define CMU_PDBGDIV_SHIFT    26
> +#define CMU_PDBGDIV_DIV              (CMU_PDBGDIV_8 << CMU_PDBGDIV_SHIFT)
> +#define CMU_PERDIV_8         7
> +#define CMU_PERDIV_SHIFT     20
> +#define CMU_PERDIV_DIV               (CMU_PERDIV_8 << CMU_PERDIV_SHIFT)
> +#define CMU_NOCDIV_2         1
> +#define CMU_NOCDIV_SHIFT     19
> +#define CMU_NOCDIV_DIV               (CMU_NOCDIV_2 << CMU_NOCDIV_SHIFT)
> +#define CMU_DMMCLK_SRC_APLL  2
> +#define CMU_DMMCLK_SRC_SHIFT 10
> +#define CMU_DMMCLK_SRC               (CMU_DMMCLK_SRC_APLL << 
> CMU_DMMCLK_SRC_SHIFT)
> +#define CMU_APBCLK_DIV               BIT(8)
> +#define CMU_NOCCLK_SRC               BIT(7)
> +#define CMU_AHBCLK_DIV               BIT(4)
> +#define CMU_CORECLK_MASK     3
> +#define CMU_CORECLK_CPLL     BIT(1)
> +#define CMU_CORECLK_HOSC     BIT(0)
> +
> +/* COREPLL register definitions */
> +#define CMU_COREPLL_EN               BIT(9)
> +#define CMU_COREPLL_HOSC_EN  BIT(8)
> +#define CMU_COREPLL_OUT              (1104 / 24)
> +
> +/* DEVPLL register definitions */
> +#define CMU_DEVPLL_CLK               BIT(12)
> +#define CMU_DEVPLL_EN                BIT(8)
> +#define CMU_DEVPLL_OUT               (660 / 6)
> +
> +/* UARTCLK register definitions */
> +#define CMU_UARTCLK_SRC_DEVPLL       BIT(16)
> +
> +#define PLL_STABILITY_WAIT_US        50
> +
> +#define CMU_DEVCLKEN1_UART5  BIT(21)
> +#define CMU_DEVCLKEN1_UART3  BIT(11)
> +
> +#define CMU_DEVCLKEN1_ETH_S700       BIT(23)
> +
> +#endif
> diff --git a/drivers/clk/owl/clk_s900.c b/drivers/clk/owl/clk_s900.c
> deleted file mode 100644
> index d60f199..0000000
> --- a/drivers/clk/owl/clk_s900.c
> +++ /dev/null
> @@ -1,137 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0+
> -/*
> - * Actions Semi S900 clock driver
> - *
> - * Copyright (C) 2015 Actions Semi Co., Ltd.
> - * Copyright (C) 2018 Manivannan Sadhasivam 
> <manivannan.sadhasi...@linaro.org>
> - */
> -
> -#include <common.h>
> -#include <dm.h>
> -#include <asm/arch-owl/clk_s900.h>
> -#include <asm/arch-owl/regs_s900.h>
> -#include <asm/io.h>
> -
> -#include <dt-bindings/clock/actions,s900-cmu.h>
> -
> -void owl_clk_init(struct owl_clk_priv *priv)
> -{
> -     u32 bus_clk = 0, core_pll, dev_pll;
> -
> -     /* Enable ASSIST_PLL */
> -     setbits_le32(priv->base + CMU_ASSISTPLL, BIT(0));
> -
> -     udelay(PLL_STABILITY_WAIT_US);
> -
> -     /* Source HOSC to DEV_CLK */
> -     clrbits_le32(priv->base + CMU_DEVPLL, CMU_DEVPLL_CLK);
> -
> -     /* Configure BUS_CLK */
> -     bus_clk |= (CMU_PDBGDIV_DIV | CMU_PERDIV_DIV | CMU_NOCDIV_DIV |
> -                     CMU_DMMCLK_SRC | CMU_APBCLK_DIV | CMU_AHBCLK_DIV |
> -                     CMU_NOCCLK_SRC | CMU_CORECLK_HOSC);
> -     writel(bus_clk, priv->base + CMU_BUSCLK);
> -
> -     udelay(PLL_STABILITY_WAIT_US);
> -
> -     /* Configure CORE_PLL */
> -     core_pll = readl(priv->base + CMU_COREPLL);
> -     core_pll |= (CMU_COREPLL_EN | CMU_COREPLL_HOSC_EN | CMU_COREPLL_OUT);
> -     writel(core_pll, priv->base + CMU_COREPLL);
> -
> -     udelay(PLL_STABILITY_WAIT_US);
> -
> -     /* Configure DEV_PLL */
> -     dev_pll = readl(priv->base + CMU_DEVPLL);
> -     dev_pll |= (CMU_DEVPLL_EN | CMU_DEVPLL_OUT);
> -     writel(dev_pll, priv->base + CMU_DEVPLL);
> -
> -     udelay(PLL_STABILITY_WAIT_US);
> -
> -     /* Source CORE_PLL for CORE_CLK */
> -     clrsetbits_le32(priv->base + CMU_BUSCLK, CMU_CORECLK_MASK,
> -                     CMU_CORECLK_CPLL);
> -
> -     /* Source DEV_PLL for DEV_CLK */
> -     setbits_le32(priv->base + CMU_DEVPLL, CMU_DEVPLL_CLK);
> -
> -     udelay(PLL_STABILITY_WAIT_US);
> -}
> -
> -void owl_uart_clk_enable(struct owl_clk_priv *priv)
> -{
> -     /* Source HOSC for UART5 interface */
> -     clrbits_le32(priv->base + CMU_UART5CLK, CMU_UARTCLK_SRC_DEVPLL);
> -
> -     /* Enable UART5 interface clock */
> -     setbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_UART5);
> -}
> -
> -void owl_uart_clk_disable(struct owl_clk_priv *priv)
> -{
> -     /* Disable UART5 interface clock */
> -     clrbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_UART5);
> -}
> -
> -int owl_clk_enable(struct clk *clk)
> -{
> -     struct owl_clk_priv *priv = dev_get_priv(clk->dev);
> -
> -     switch (clk->id) {
> -     case CLK_UART5:
> -             owl_uart_clk_enable(priv);
> -             break;
> -     default:
> -             return 0;
> -     }
> -
> -     return 0;
> -}
> -
> -int owl_clk_disable(struct clk *clk)
> -{
> -     struct owl_clk_priv *priv = dev_get_priv(clk->dev);
> -
> -     switch (clk->id) {
> -     case CLK_UART5:
> -             owl_uart_clk_disable(priv);
> -             break;
> -     default:
> -             return 0;
> -     }
> -
> -     return 0;
> -}
> -
> -static int owl_clk_probe(struct udevice *dev)
> -{
> -     struct owl_clk_priv *priv = dev_get_priv(dev);
> -
> -     priv->base = dev_read_addr(dev);
> -     if (priv->base == FDT_ADDR_T_NONE)
> -             return -EINVAL;
> -
> -     /* setup necessary clocks */
> -     owl_clk_init(priv);
> -
> -     return 0;
> -}
> -
> -static struct clk_ops owl_clk_ops = {
> -     .enable = owl_clk_enable,
> -     .disable = owl_clk_disable,
> -};
> -
> -static const struct udevice_id owl_clk_ids[] = {
> -     { .compatible = "actions,s900-cmu" },
> -     { }
> -};
> -
> -U_BOOT_DRIVER(clk_owl) = {
> -     .name           = "clk_s900",
> -     .id             = UCLASS_CLK,
> -     .of_match       = owl_clk_ids,
> -     .ops            = &owl_clk_ops,
> -     .priv_auto_alloc_size = sizeof(struct owl_clk_priv),
> -     .probe          = owl_clk_probe,
> -};
> -- 
> 2.7.4
> 

Reply via email to