[PATCHv4 5/7] clk: samsung: exynos3250: Add clocks using common clock framework

Chanwoo Choi cw00.choi at samsung.com
Tue May 13 23:57:58 PDT 2014


Hi Tomasz,

On 05/14/2014 01:28 AM, Tomasz Figa wrote:
> Hi Chanwoo,
> 
> On 13.05.2014 13:49, Chanwoo Choi wrote:
>> Hi Tomasz,
>>
>> On 04/26/2014 09:39 AM, Tomasz Figa wrote:
>>> Hi Chanwoo,
>>>
>>> On 25.04.2014 03:16, Chanwoo Choi wrote:
>>>> From: Tomasz Figa <t.figa at samsung.com>
>>>>
>>>> This patch add new the clock drvier of Exynos3250 SoC based on Cortex-A7
>>>> using common clock framework. The CMU (Clock Management Unit) of Exynos3250
>>>> control PLLs(Phase Locked Loops) and generate system clocks for CPU, buses,
>>>> and function clocks for individual IPs.
>>>>
>>>> The CMU of Exynos3250 includes following clock doamins:
>>>> - CPU block for Cortex-A7 MPCore processor
>>>> - LEFTBUS/RIGHTBUS block
>>>> - TOP block for G3D/MFC/LCD0/ISP/CAM/FSYS/MFC/PERIL/PERIR
>>>
>>> In original driver present in our internal tree I have separated several CMUs to account for certain factors caused by hardware design, which require such separation. Is there any reason why they were merged together into a single CMU again?
>>
>> This patch just include clocks in CMU clocks without CMU_DMC/CMU_ISP.
>> I'll send a further patches to support CMU_DMC/CMU_ISP after verifying it.
>>
>>>
>>>>
>>>> Cc: Mike Turquette <mturquette at linaro.org>
>>>> Cc: Kukjin Kim <kgene.kim at samsung.com>
>>>> Cc: Rob Herring <robh+dt at kernel.org>
>>>> Cc: Pawel Moll <pawel.moll at arm.com>
>>>> Cc: Mark Rutland <mark.rutland at arm.com>
>>>> Cc: Ian Campbell <ijc+devicetree at hellion.org.uk>
>>>> Cc: Kumar Gala <galak at codeaurora.org>
>>>> Signed-off-by: Tomasz Figa <t.figa at samsung.com>
>>>> Signed-off-by: Chanwoo Choi <cw00.choi at samsung.com>
>>>> Signed-off-by: Hyunhee Kim <hyunhee.kim at samsung.com>
>>>> Signed-off-by: Sylwester Nawrocki <s.nawrocki at samsung.com>
>>>> Signed-off-by: Inki Dae <inki.dae at samsung.com>
>>>> Signed-off-by: Seung-Woo Kim <sw0312.kim at samsung.com>
>>>> Signed-off-by: Jaehoon Chung <jh80.chung at samsung.com>
>>>> Signed-off-by: Karol Wrona <k.wrona at samsung.com>
>>>> Signed-off-by: YoungJun Cho <yj44.cho at samsung.com>
>>>> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
>>>> ---
>>>>   drivers/clk/samsung/Makefile           |   1 +
>>>>   drivers/clk/samsung/clk-exynos3250.c   | 785 +++++++++++++++++++++++++++++++++
>>>>   include/dt-bindings/clock/exynos3250.h | 256 +++++++++++
>>>>   3 files changed, 1042 insertions(+)
>>>>   create mode 100644 drivers/clk/samsung/clk-exynos3250.c
>>>>   create mode 100644 include/dt-bindings/clock/exynos3250.h
>>>>
>>>> diff --git a/drivers/clk/samsung/Makefile b/drivers/clk/samsung/Makefile
>>>> index 8eb4799..d120797 100644
>>>> --- a/drivers/clk/samsung/Makefile
>>>> +++ b/drivers/clk/samsung/Makefile
>>>> @@ -3,6 +3,7 @@
>>>>   #
>>>>
>>>>   obj-$(CONFIG_COMMON_CLK)    += clk.o clk-pll.o
>>>> +obj-$(CONFIG_SOC_EXYNOS3250)    += clk-exynos3250.o
>>>>   obj-$(CONFIG_ARCH_EXYNOS4)    += clk-exynos4.o
>>>>   obj-$(CONFIG_SOC_EXYNOS5250)    += clk-exynos5250.o
>>>>   obj-$(CONFIG_SOC_EXYNOS5420)    += clk-exynos5420.o
>>>> diff --git a/drivers/clk/samsung/clk-exynos3250.c b/drivers/clk/samsung/clk-exynos3250.c
>>>> new file mode 100644
>>>> index 0000000..0574a76
>>>> --- /dev/null
>>>> +++ b/drivers/clk/samsung/clk-exynos3250.c
>>>> @@ -0,0 +1,785 @@
>>>> +/*
>>>> + * Copyright (c) 2014 Samsung Electronics Co., Ltd.
>>>> + *
>>>> + * 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.
>>>> + *
>>>> + * Common Clock Framework support for Exynos3250 SoC.
>>>> + */
>>>> +
>>>> +#include <linux/clk.h>
>>>> +#include <linux/clkdev.h>
>>>> +#include <linux/clk-provider.h>
>>>> +#include <linux/of.h>
>>>> +#include <linux/of_address.h>
>>>> +#include <linux/platform_device.h>
>>>> +#include <linux/syscore_ops.h>
>>>> +
>>>> +#include <dt-bindings/clock/exynos3250.h>
>>>> +
>>>> +#include "clk.h"
>>>> +#include "clk-pll.h"
>>>> +
>>>> +#define SRC_LEFTBUS        0x4200
>>>> +#define DIV_LEFTBUS        0x4500
>>>> +#define GATE_IP_LEFTBUS        0x4800
>>>> +#define SRC_RIGHTBUS        0x8200
>>>> +#define DIV_RIGHTBUS        0x8500
>>>> +#define GATE_IP_RIGHTBUS    0x8800
>>>> +#define GATE_IP_PERIR        0x8960
>>>> +#define MPLL_LOCK        0xc010
>>>> +#define MPLL_CON0        0xc110
>>>> +#define VPLL_LOCK        0xc020
>>>> +#define VPLL_CON0        0xc120
>>>> +#define UPLL_LOCK        0xc030
>>>> +#define UPLL_CON0        0xc130
>>>> +#define SRC_TOP0        0xc210
>>>> +#define SRC_TOP1        0xc214
>>>> +#define SRC_CAM            0xc220
>>>> +#define SRC_MFC            0xc228
>>>> +#define SRC_G3D            0xc22c
>>>> +#define SRC_LCD            0xc234
>>>> +#define SRC_ISP            0xc238
>>>> +#define SRC_FSYS        0xc240
>>>> +#define SRC_PERIL0        0xc250
>>>> +#define SRC_PERIL1        0xc254
>>>> +#define SRC_MASK_TOP        0xc310
>>>> +#define SRC_MASK_CAM        0xc320
>>>> +#define SRC_MASK_LCD        0xc334
>>>> +#define SRC_MASK_ISP        0xc338
>>>> +#define SRC_MASK_FSYS        0xc340
>>>> +#define SRC_MASK_PERIL0        0xc350
>>>> +#define SRC_MASK_PERIL1        0xc354
>>>> +#define DIV_TOP            0xc510
>>>> +#define DIV_CAM            0xc520
>>>> +#define DIV_MFC            0xc528
>>>> +#define DIV_G3D            0xc52c
>>>> +#define DIV_LCD            0xc534
>>>> +#define DIV_ISP            0xc538
>>>> +#define DIV_FSYS0        0xc540
>>>> +#define DIV_FSYS1        0xc544
>>>> +#define DIV_FSYS2        0xc548
>>>> +#define DIV_PERIL0        0xc550
>>>> +#define DIV_PERIL1        0xc554
>>>> +#define DIV_PERIL3        0xc55c
>>>> +#define DIV_PERIL4        0xc560
>>>> +#define DIV_PERIL5        0xc564
>>>> +#define DIV_CAM1        0xc568
>>>> +#define CLKDIV2_RATIO        0xc580
>>>> +#define GATE_SCLK_CAM        0xc820
>>>> +#define GATE_SCLK_MFC        0xc828
>>>> +#define GATE_SCLK_G3D        0xc82c
>>>> +#define GATE_SCLK_LCD        0xc834
>>>> +#define GATE_SCLK_ISP_TOP    0xc838
>>>> +#define GATE_SCLK_FSYS        0xc840
>>>> +#define GATE_SCLK_PERIL        0xc850
>>>> +#define GATE_IP_CAM        0xc920
>>>> +#define GATE_IP_MFC        0xc928
>>>> +#define GATE_IP_G3D        0xc92c
>>>> +#define GATE_IP_LCD        0xc934
>>>> +#define GATE_IP_ISP        0xc938
>>>> +#define GATE_IP_FSYS        0xc940
>>>> +#define GATE_IP_PERIL        0xc950
>>>> +#define GATE_BLOCK        0xc970
>>>> +#define APLL_LOCK        0x14000
>>>> +#define APLL_CON0        0x14100
>>>> +#define SRC_CPU            0x14200
>>>> +#define DIV_CPU0        0x14500
>>>> +#define DIV_CPU1        0x14504
>>>> +
>>>> +/* list of PLLs to be registered */
>>>> +enum exynos3250_plls {
>>>> +    apll, mpll, vpll, upll,
>>>> +    nr_plls
>>>> +};
>>>> +
>>>> +void __iomem *reg_base;
>>>
>>> static void
>>
>> I'll fix it.
>>
> 
> OK.
> 
>>>
>>>> +
>>>> +/*
>>>> + * Support for CMU save/restore across system suspends
>>>> + */
>>>> +#ifdef CONFIG_PM_SLEEP
>>>> +static struct samsung_clk_reg_dump *exynos3250_clk_regs;
>>>> +
>>>> +static unsigned long exynos3250_cmu_clk_regs[] __initdata = {
>>>> +    SRC_LEFTBUS,
>>>> +    DIV_LEFTBUS,
>>>> +    GATE_IP_LEFTBUS,
>>>> +    SRC_RIGHTBUS,
>>>> +    DIV_RIGHTBUS,
>>>> +    GATE_IP_RIGHTBUS,
>>>> +    GATE_IP_PERIR,
>>>> +    MPLL_LOCK,
>>>> +    MPLL_CON0,
>>>> +    VPLL_LOCK,
>>>> +    VPLL_CON0,
>>>> +    UPLL_LOCK,
>>>> +    UPLL_CON0,
>>>> +    SRC_TOP0,
>>>> +    SRC_TOP1,
>>>> +    SRC_CAM,
>>>> +    SRC_MFC,
>>>> +    SRC_G3D,
>>>> +    SRC_LCD,
>>>> +    SRC_ISP,
>>>> +    SRC_FSYS,
>>>> +    SRC_PERIL0,
>>>> +    SRC_PERIL1,
>>>> +    SRC_MASK_TOP,
>>>> +    SRC_MASK_CAM,
>>>> +    SRC_MASK_LCD,
>>>> +    SRC_MASK_ISP,
>>>> +    SRC_MASK_FSYS,
>>>> +    SRC_MASK_PERIL0,
>>>> +    SRC_MASK_PERIL1,
>>>> +    DIV_TOP,
>>>> +    DIV_CAM,
>>>> +    DIV_MFC,
>>>> +    DIV_G3D,
>>>> +    DIV_LCD,
>>>> +    DIV_ISP,
>>>> +    DIV_FSYS0,
>>>> +    DIV_FSYS1,
>>>> +    DIV_FSYS2,
>>>> +    DIV_PERIL0,
>>>> +    DIV_PERIL1,
>>>> +    DIV_PERIL3,
>>>> +    DIV_PERIL4,
>>>> +    DIV_PERIL5,
>>>> +    DIV_CAM1,
>>>> +    CLKDIV2_RATIO,
>>>> +    GATE_SCLK_CAM,
>>>> +    GATE_SCLK_MFC,
>>>> +    GATE_SCLK_G3D,
>>>> +    GATE_SCLK_LCD,
>>>> +    GATE_SCLK_ISP_TOP,
>>>> +    GATE_SCLK_FSYS,
>>>> +    GATE_SCLK_PERIL,
>>>> +    GATE_IP_CAM,
>>>> +    GATE_IP_MFC,
>>>> +    GATE_IP_G3D,
>>>> +    GATE_IP_LCD,
>>>> +    GATE_IP_ISP,
>>>> +    GATE_IP_FSYS,
>>>> +    GATE_IP_PERIL,
>>>> +    GATE_BLOCK,
>>>> +    APLL_LOCK,
>>>> +    SRC_CPU,
>>>> +    DIV_CPU0,
>>>> +    DIV_CPU1,
>>>> +};
>>>> +
>>>> +static int exynos3250_clk_suspend(void)
>>>> +{
>>>> +    samsung_clk_save(reg_base, exynos3250_clk_regs,
>>>> +                ARRAY_SIZE(exynos3250_cmu_clk_regs));
>>>> +    return 0;
>>>> +}
>>>> +
>>>> +static void exynos3250_clk_resume(void)
>>>> +{
>>>> +    samsung_clk_restore(reg_base, exynos3250_clk_regs,
>>>> +                ARRAY_SIZE(exynos3250_cmu_clk_regs));
>>>> +}
>>>> +
>>>> +static struct syscore_ops exynos3250_clk_syscore_ops = {
>>>> +    .suspend = exynos3250_clk_suspend,
>>>> +    .resume = exynos3250_clk_resume,
>>>> +};
>>>> +
>>>> +static void exynos3250_clk_sleep_init(void)
>>>> +{
>>>> +    exynos3250_clk_regs =
>>>> +        samsung_clk_alloc_reg_dump(exynos3250_cmu_clk_regs,
>>>> +                       ARRAY_SIZE(exynos3250_cmu_clk_regs));
>>>> +    if (!exynos3250_clk_regs) {
>>>> +        pr_warn("%s: Failed to allocate sleep save data\n", __func__);
>>>> +        goto err;
>>>> +    }
>>>> +
>>>> +    register_syscore_ops(&exynos3250_clk_syscore_ops);
>>>> +    return;
>>>> +err:
>>>> +    kfree(exynos3250_clk_regs);
>>>> +}
>>>> +#else
>>>> +static inline void exynos3250_clk_sleep_init(void) { }
>>>> +#endif
>>>> +
>>>> +/* list of all parent clock list */
>>>> +PNAME(mout_vpllsrc_p)        = { "fin_pll", };
>>>> +
>>>> +PNAME(mout_apll_p)        = { "fin_pll", "fout_apll", };
>>>> +PNAME(mout_mpll_p)        = { "fin_pll", "fout_mpll", };
>>>> +PNAME(mout_vpll_p)        = { "fin_pll", "fout_vpll", };
>>>> +PNAME(mout_upll_p)        = { "fin_pll", "fout_upll", };
>>>> +
>>>> +PNAME(mout_mpll_user_p)        = { "fin_pll", "div_mpll_pre", };
>>>> +PNAME(mout_epll_user_p)        = { "fin_pll", "mout_epll", };
>>>> +PNAME(mout_core_p)        = { "mout_apll", "mout_mpll_user_c", };
>>>> +PNAME(mout_hpm_p)        = { "mout_apll", "mout_mpll_user_c", };
>>>> +
>>>> +PNAME(mout_ebi_p)        = { "div_aclk_200", "div_aclk_160", };
>>>> +PNAME(mout_ebi_1_p)        = { "mout_ebi", "mout_vpll", };
>>>> +
>>>> +PNAME(mout_gdl_p)        = { "mout_mpll_user_l", };
>>>> +PNAME(mout_gdr_p)        = { "mout_mpll_user_r", };
>>>> +
>>>> +PNAME(mout_aclk_400_mcuisp_sub_p)
>>>> +                = { "fin_pll", "div_aclk_400_mcuisp", };
>>>> +PNAME(mout_aclk_266_0_p)    = { "div_mpll_pre", "mout_vpll", };
>>>> +PNAME(mout_aclk_266_1_p)    = { "mout_epll_user", };
>>>> +PNAME(mout_aclk_266_p)        = { "mout_aclk_266_0", "mout_aclk_266_1", };
>>>> +PNAME(mout_aclk_266_sub_p)    = { "fin_pll", "div_aclk_266", };
>>>> +
>>>> +PNAME(group_div_mpll_pre_p)    = { "div_mpll_pre", };
>>>> +PNAME(group_epll_vpll_p)    = { "mout_epll_user", "mout_vpll" };
>>>> +PNAME(group_sclk_p)        = { "xxti", "xusbxti",
>>>> +                    "none", "none",
>>>> +                    "none", "none", "div_mpll_pre",
>>>> +                    "mout_epll_user", "mout_vpll", };
>>>> +PNAME(group_sclk_audio_p)    = { "audiocdclk", "none",
>>>> +                    "none", "none",
>>>> +                    "xxti", "xusbxti",
>>>> +                    "div_mpll_pre", "mout_epll_user",
>>>> +                    "mout_vpll", };
>>>> +PNAME(group_sclk_cam_blk_p)    = { "xxti", "xusbxti",
>>>> +                    "none", "none", "none",
>>>> +                    "none", "div_mpll_pre",
>>>> +                    "mout_epll_user", "mout_vpll",
>>>> +                    "div_cam_blk_320", };
>>>> +PNAME(group_sclk_fimd0_p)    = { "xxti", "xusbxti",
>>>> +                    "m_bitclkhsdiv4_2l", "none",
>>>> +                    "none", "none", "div_mpll_pre",
>>>> +                    "mout_epll_user", "mout_vpll",
>>>> +                    "none", "none", "none",
>>>> +                    "div_lcd_blk_145", };
>>>> +
>>>> +PNAME(mout_mfc_p)        = { "mout_mfc_0", "mout_mfc_1" };
>>>> +PNAME(mout_g3d_p)        = { "mout_g3d_0", "mout_g3d_1" };
>>>> +
>>>> +static struct samsung_fixed_rate_clock fixed_rate_clks[] __initdata = {
>>>> +};
>>>
>>> Any reason to have this empty array at all?
>>
>> OK, I'll delete empty array.
>>
> 
> OK.
> 
>>>
>>>> +
>>>> +static struct samsung_fixed_factor_clock fixed_factor_clks[] __initdata = {
>>>> +    FFACTOR(0, "sclk_mpll_1600", "mout_mpll", 1, 1, 0),
>>>> +    FFACTOR(0, "sclk_mpll_mif", "mout_mpll", 1, 2, 0),
>>>> +    FFACTOR(0, "sclk_bpll", "fout_bpll", 1, 2, 0),
>>>> +    FFACTOR(0, "div_cam_blk_320", "sclk_mpll_1600", 1, 5, 0),
>>>> +    FFACTOR(0, "div_lcd_blk_145", "sclk_mpll_1600", 1, 11, 0),
>>>> +
>>>> +    FFACTOR(CLK_FIN_PLL, "fin_pll", "xusbxti", 1, 1, 0),
>>>
>>> This is a hack. The real "fin_pll" is a mux driven by XOM pin which selects which clock input is used as main system oscillator. It should be implemented properly in this driver or at least appropriate comment should be added stressing the fact that it is a hack.
>>
>> OK, I'll add comment about it and if you ok, I'll send a further patchset to set relation
>> between fin_pll and xusbxti/xxti by reading XOM pin of chipid register.
>>
> 
> OK.
> 
>>>
>>>> +};
>>>> +
>>>> +static struct samsung_mux_clock mux_clks[] __initdata = {
>>>> +    /*
>>>> +     * NOTE: Following table is sorted by register address in ascending
>>>> +     * order and then bitfield shift in descending order, as it is done
>>>> +     * in the User's Manual. When adding new entries, please make sure
>>>> +     * that the order is preserved, to avoid merge conflicts and make
>>>> +     * further work with defined data easier.
>>>> +     */
>>>> +
>>>> +    /* SRC_LEFTBUS */
>>>> +    MUX(CLK_MOUT_MPLL_USER_L, "mout_mpll_user_l", mout_mpll_user_p,
>>>> +        SRC_LEFTBUS, 4, 1),
>>>> +    MUX(CLK_MOUT_GDL, "mout_gdl", mout_gdl_p, SRC_LEFTBUS, 0, 1),
>>>> +
>>>> +    /* SRC_RIGHTBUS */
>>>> +    MUX(CLK_MOUT_MPLL_USER_R, "mout_mpll_user_r", mout_mpll_user_p,
>>>> +        SRC_RIGHTBUS, 4, 1),
>>>> +    MUX(CLK_MOUT_GDR, "mout_gdr", mout_gdr_p, SRC_RIGHTBUS, 0, 1),
>>>> +
>>>> +    /* SRC_TOP0 */
>>>> +    MUX(CLK_MOUT_EBI, "mout_ebi", mout_ebi_p, SRC_TOP0, 28, 1),
>>>> +    MUX(CLK_MOUT_ACLK_200, "mout_aclk_200", group_div_mpll_pre_p,SRC_TOP0, 24, 1),
>>>> +    MUX(CLK_MOUT_ACLK_160, "mout_aclk_160", group_div_mpll_pre_p, SRC_TOP0, 20, 1),
>>>> +    MUX(CLK_MOUT_ACLK_100, "mout_aclk_100", group_div_mpll_pre_p, SRC_TOP0, 16, 1),
>>>> +    MUX(CLK_MOUT_ACLK_266_1, "mout_aclk_266_1", mout_aclk_266_1_p, SRC_TOP0, 14, 1),
>>>> +    MUX(CLK_MOUT_ACLK_266_0, "mout_aclk_266_0", mout_aclk_266_0_p, SRC_TOP0, 13, 1),
>>>> +    MUX(CLK_MOUT_ACLK_266, "mout_aclk_266", mout_aclk_266_p, SRC_TOP0, 12, 1),
>>>> +    MUX(CLK_MOUT_VPLL, "mout_vpll", mout_vpll_p, SRC_TOP0, 8, 1),
>>>> +    MUX(CLK_MOUT_EPLL_USER, "mout_epll_user", mout_epll_user_p, SRC_TOP0, 4, 1),
>>>> +    MUX(CLK_MOUT_EBI_1, "mout_ebi_1", mout_ebi_1_p, SRC_TOP0, 0, 1),
>>>> +
>>>> +    /* SRC_TOP1 */
>>>> +    MUX(CLK_MOUT_UPLL, "mout_upll", mout_upll_p, SRC_TOP1, 28, 1),
>>>> +    MUX(CLK_MOUT_ACLK_400_MCUISP_SUB, "mout_aclk_400_mcuisp_sub", mout_aclk_400_mcuisp_sub_p,
>>>> +        SRC_TOP1, 24, 1),
>>>> +    MUX(CLK_MOUT_ACLK_266_SUB, "mout_aclk_266_sub", mout_aclk_266_sub_p, SRC_TOP1, 20, 1),
>>>> +    MUX(CLK_MOUT_MPLL, "mout_mpll", mout_mpll_p, SRC_TOP1, 12, 1),
>>>> +    MUX(CLK_MOUT_ACLK_400_MCUISP, "mout_aclk_400_mcuisp", group_div_mpll_pre_p, SRC_TOP1, 8, 1),
>>>> +    MUX(CLK_MOUT_VPLLSRC, "mout_vpllsrc", mout_vpllsrc_p, SRC_TOP1, 0, 1),
>>>> +
>>>> +    /* SRC_CAM */
>>>> +    MUX(CLK_MOUT_CAM1, "mout_cam1", group_sclk_p, SRC_CAM, 20, 4),
>>>> +    MUX(CLK_MOUT_CAM_BLK, "mout_cam_blk", group_sclk_cam_blk_p, SRC_CAM, 0, 4),
>>>> +
>>>> +    /* SRC_MFC */
>>>> +    MUX(CLK_MOUT_MFC, "mout_mfc", mout_mfc_p, SRC_MFC, 8, 1),
>>>> +    MUX(CLK_MOUT_MFC_1, "mout_mfc_1", group_epll_vpll_p, SRC_MFC, 4, 1),
>>>> +    MUX(CLK_MOUT_MFC_0, "mout_mfc_0", group_div_mpll_pre_p, SRC_MFC, 0, 1),
>>>> +
>>>> +    /* SRC_G3D */
>>>> +    MUX(CLK_MOUT_G3D, "mout_g3d", mout_g3d_p, SRC_G3D, 8, 1),
>>>> +    MUX(CLK_MOUT_G3D_1, "mout_g3d_1", group_epll_vpll_p, SRC_G3D, 4, 1),
>>>> +    MUX(CLK_MOUT_G3D_0, "mout_g3d_0", group_div_mpll_pre_p, SRC_G3D, 0, 1),
>>>> +
>>>> +    /* SRC_LCD */
>>>> +    MUX(CLK_MOUT_MIPI0, "mout_mipi0", group_sclk_p, SRC_LCD, 12, 4),
>>>> +    MUX(CLK_MOUT_FIMD0, "mout_fimd0", group_sclk_fimd0_p, SRC_LCD, 0, 4),
>>>> +
>>>> +    /* SRC_ISP */
>>>> +    MUX(CLK_MOUT_UART_ISP, "mout_uart_isp", group_sclk_p, SRC_ISP, 12, 4),
>>>> +    MUX(CLK_MOUT_SPI1_ISP, "mout_spi1_isp", group_sclk_p, SRC_ISP, 8, 4),
>>>> +    MUX(CLK_MOUT_SPI0_ISP, "mout_spi0_isp", group_sclk_p, SRC_ISP, 4, 4),
>>>> +
>>>> +    /* SRC_FSYS */
>>>> +    MUX(CLK_MOUT_TSADC, "mout_tsadc", group_sclk_p, SRC_FSYS, 28, 4),
>>>> +    MUX(CLK_MOUT_MMC1, "mout_mmc1", group_sclk_p, SRC_FSYS, 4, 3),
>>>> +    MUX(CLK_MOUT_MMC0, "mout_mmc0", group_sclk_p, SRC_FSYS, 0, 3),
>>>> +
>>>> +    /* SRC_PERIL0 */
>>>> +    MUX(CLK_MOUT_UART1, "mout_uart1", group_sclk_p, SRC_PERIL0, 4, 4),
>>>> +    MUX(CLK_MOUT_UART0, "mout_uart0", group_sclk_p, SRC_PERIL0, 0, 4),
>>>> +
>>>> +    /* SRC_PERIL1 */
>>>> +    MUX(CLK_MOUT_SPI1, "mout_spi1", group_sclk_p, SRC_PERIL1, 20, 4),
>>>> +    MUX(CLK_MOUT_SPI0, "mout_spi0", group_sclk_p, SRC_PERIL1, 16, 4),
>>>> +    MUX(CLK_MOUT_AUDIO, "mout_audio", group_sclk_audio_p, SRC_PERIL1, 4, 4),
>>>> +
>>>> +    /* SRC_CPU */
>>>> +    MUX(CLK_MOUT_MPLL_USER_C, "mout_mpll_user_c", mout_mpll_user_p,
>>>> +        SRC_CPU, 24, 1),
>>>> +    MUX(CLK_MOUT_HPM, "mout_hpm", mout_hpm_p, SRC_CPU, 20, 1),
>>>> +    MUX(CLK_MOUT_CORE, "mout_core", mout_core_p, SRC_CPU, 16, 1),
>>>> +    MUX(CLK_MOUT_APLL, "mout_apll", mout_apll_p, SRC_CPU, 0, 1),
>>>> +};
>>>> +
>>>> +static struct samsung_div_clock div_clks[] __initdata = {
>>>> +    /*
>>>> +     * NOTE: Following table is sorted by register address in ascending
>>>> +     * order and then bitfield shift in descending order, as it is done
>>>> +     * in the User's Manual. When adding new entries, please make sure
>>>> +     * that the order is preserved, to avoid merge conflicts and make
>>>> +     * further work with defined data easier.
>>>> +     */
>>>> +
>>>> +    /* DIV_LEFTBUS */
>>>> +    DIV(CLK_DIV_GPL, "div_gpl", "div_gdl", DIV_LEFTBUS, 4, 3),
>>>> +    DIV(CLK_DIV_GDL, "div_gdl", "mout_gdl", DIV_LEFTBUS, 0, 4),
>>>> +
>>>> +    /* DIV_RIGHTBUS */
>>>> +    DIV(CLK_DIV_GPR, "div_gpr", "div_gdr", DIV_RIGHTBUS, 4, 3),
>>>> +    DIV(CLK_DIV_GDR, "div_gdr", "mout_gdr", DIV_RIGHTBUS, 0, 4),
>>>> +
>>>> +    /* DIV_TOP */
>>>> +    DIV(CLK_DIV_MPLL_PRE, "div_mpll_pre", "sclk_mpll_mif", DIV_TOP, 28, 2),
>>>> +    DIV(CLK_DIV_ACLK_400_MCUISP, "div_aclk_400_mcuisp",
>>>> +        "mout_aclk_400_mcuisp", DIV_TOP, 24, 3),
>>>> +    DIV(CLK_DIV_EBI, "div_ebi", "mout_ebi_1", DIV_TOP, 16, 3),
>>>> +    DIV(CLK_DIV_ACLK_200, "div_aclk_200", "mout_aclk_200", DIV_TOP, 12, 3),
>>>> +    DIV(CLK_DIV_ACLK_160, "div_aclk_160", "mout_aclk_160", DIV_TOP, 8, 3),
>>>> +    DIV(CLK_DIV_ACLK_100, "div_aclk_100", "mout_aclk_100", DIV_TOP, 4, 4),
>>>> +    DIV(CLK_DIV_ACLK_266, "div_aclk_266", "mout_aclk_266", DIV_TOP, 0, 3),
>>>> +
>>>> +    /* DIV_CAM */
>>>> +    DIV(CLK_DIV_CAM1, "div_cam1", "mout_cam1", DIV_CAM, 20, 4),
>>>> +    DIV(CLK_DIV_CAM_BLK, "div_cam_blk", "mout_cam_blk", DIV_CAM, 0, 4),
>>>> +
>>>> +    /* DIV_MFC */
>>>> +    DIV(CLK_DIV_MFC, "div_mfc", "mout_mfc", DIV_MFC, 0, 4),
>>>> +
>>>> +    /* DIV_G3D */
>>>> +    DIV(CLK_DIV_G3D, "div_g3d", "mout_g3d", DIV_G3D, 0, 4),
>>>> +
>>>> +    /* DIV_LCD */
>>>> +    DIV_F(CLK_DIV_MIPI0_PRE, "div_mipi0_pre", "div_mipi0", DIV_LCD, 20, 4,
>>>> +        CLK_SET_RATE_PARENT, 0),
>>>> +    DIV(CLK_DIV_MIPI0, "div_mipi0", "mout_mipi0", DIV_LCD, 16, 4),
>>>> +    DIV(CLK_DIV_FIMD0, "div_fimd0", "mout_fimd0", DIV_LCD, 0, 4),
>>>> +
>>>> +    /* DIV_ISP */
>>>> +    DIV(CLK_DIV_UART_ISP, "div_uart_isp", "mout_uart_isp", DIV_ISP, 28, 4),
>>>> +    DIV_F(CLK_DIV_SPI1_ISP_PRE, "div_spi1_isp_pre", "div_spi1_isp",
>>>> +        DIV_ISP, 20, 8, CLK_SET_RATE_PARENT, 0),
>>>> +    DIV(CLK_DIV_SPI1_ISP, "div_spi1_isp", "mout_spi1_isp", DIV_ISP, 16, 4),
>>>> +    DIV_F(CLK_DIV_SPI0_ISP_PRE, "div_spi0_isp_pre", "div_spi0_isp",
>>>> +        DIV_ISP, 8, 8, CLK_SET_RATE_PARENT, 0),
>>>> +    DIV(CLK_DIV_SPI0_ISP, "div_spi0_isp", "mout_spi0_isp", DIV_ISP, 0, 4),
>>>> +
>>>> +    /* DIV_FSYS0 */
>>>> +    DIV_F(CLK_DIV_TSADC_PRE, "div_tsadc_pre", "div_tsadc", DIV_FSYS0, 8, 8,
>>>> +        CLK_SET_RATE_PARENT, 0),
>>>> +    DIV(CLK_DIV_TSADC, "div_tsadc", "mout_tsadc", DIV_FSYS0, 0, 4),
>>>> +
>>>> +    /* DIV_FSYS1 */
>>>> +    DIV_F(CLK_DIV_MMC1_PRE, "div_mmc1_pre", "div_mmc1", DIV_FSYS1, 24, 8,
>>>> +        CLK_SET_RATE_PARENT, 0),
>>>> +    DIV(CLK_DIV_MMC1, "div_mmc1", "mout_mmc1", DIV_FSYS1, 16, 4),
>>>> +    DIV_F(CLK_DIV_MMC0_PRE, "div_mmc0_pre", "div_mmc0", DIV_FSYS1, 8, 8,
>>>> +        CLK_SET_RATE_PARENT, 0),
>>>> +    DIV(CLK_DIV_MMC0, "div_mmc0", "mout_mmc0", DIV_FSYS1, 0, 4),
>>>> +
>>>> +    /* DIV_PERIL0 */
>>>> +    DIV(CLK_DIV_UART1, "div_uart1", "mout_uart1", DIV_PERIL0, 4, 4),
>>>> +    DIV(CLK_DIV_UART0, "div_uart0", "mout_uart0", DIV_PERIL0, 0, 4),
>>>> +
>>>> +    /* DIV_PERIL1 */
>>>> +    DIV_F(CLK_DIV_SPI1_PRE, "div_spi1_pre", "div_spi1", DIV_PERIL1, 24, 8,
>>>> +        CLK_SET_RATE_PARENT, 0),
>>>> +    DIV(CLK_DIV_SPI1, "div_spi1", "mout_spi1", DIV_PERIL1, 16, 4),
>>>> +    DIV_F(CLK_DIV_SPI0_PRE, "div_spi0_pre", "div_spi0", DIV_PERIL1, 8, 8,
>>>> +        CLK_SET_RATE_PARENT, 0),
>>>> +    DIV(CLK_DIV_SPI0, "div_spi0", "mout_spi0", DIV_PERIL1, 0, 4),
>>>> +
>>>> +    /* DIV_PERIL4 */
>>>> +    DIV(CLK_DIV_PCM, "div_pcm", "div_audio", DIV_PERIL4, 20, 8),
>>>> +    DIV(CLK_DIV_AUDIO, "div_audio", "mout_audio", DIV_PERIL4, 16, 4),
>>>> +
>>>> +    /* DIV_PERIL5 */
>>>> +    DIV(CLK_DIV_I2S, "div_i2s", "div_audio", DIV_PERIL5, 8, 6),
>>>> +
>>>> +    /* DIV_CPU0 */
>>>> +    DIV(CLK_DIV_CORE2, "div_core2", "div_core", DIV_CPU0, 28, 3),
>>>> +    DIV(CLK_DIV_APLL, "div_apll", "mout_apll", DIV_CPU0, 24, 3),
>>>> +    DIV(CLK_DIV_PCLK_DBG, "div_pclk_dbg", "div_core2", DIV_CPU0, 20, 3),
>>>> +    DIV(CLK_DIV_ATB, "div_atb", "div_core2", DIV_CPU0, 16, 3),
>>>> +    DIV(CLK_DIV_COREM, "div_corem", "div_core2", DIV_CPU0, 4, 3),
>>>> +    DIV(CLK_DIV_CORE, "div_core", "mout_core", DIV_CPU0, 0, 3),
>>>> +
>>>> +    /* DIV_CPU1 */
>>>> +    DIV(CLK_DIV_HPM, "div_hpm", "div_copy", DIV_CPU1, 4, 3),
>>>> +    DIV(CLK_DIV_COPY, "div_copy", "mout_hpm", DIV_CPU1, 0, 3),
>>>> +};
>>>> +
>>>> +static struct samsung_gate_clock gate_clks[] __initdata = {
>>>> +    /*
>>>> +     * NOTE: Following table is sorted by register address in ascending
>>>> +     * order and then bitfield shift in descending order, as it is done
>>>> +     * in the User's Manual. When adding new entries, please make sure
>>>> +     * that the order is preserved, to avoid merge conflicts and make
>>>> +     * further work with defined data easier.
>>>> +     */
>>>> +
>>>> +    /* GATE_IP_LEFTBUS */
>>>> +    GATE(CLK_ASYNC_G3D, "async_g3d", "div_aclk_100", GATE_IP_LEFTBUS, 6,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_ASYNC_MFCL, "async_mfcl", "div_aclk_100", GATE_IP_LEFTBUS, 4,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PPMULEFT, "ppmuleft", "div_aclk_100", GATE_IP_LEFTBUS, 1,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_GPIO_LEFT, "gpio_left", "div_aclk_100", GATE_IP_LEFTBUS, 0,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +
>>>> +    /* GATE_IP_RIGHTBUS */
>>>> +    GATE(CLK_ASYNC_ISPMX, "async_ispmx", "div_aclk_100",
>>>> +        GATE_IP_RIGHTBUS, 9, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_ASYNC_FSYSD, "async_fsysd", "div_aclk_100",
>>>> +        GATE_IP_RIGHTBUS, 5, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_ASYNC_LCD0X, "async_lcd0x", "div_aclk_100",
>>>> +        GATE_IP_RIGHTBUS, 3, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_ASYNC_CAMX, "async_camx", "div_aclk_100", GATE_IP_RIGHTBUS, 2,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PPMURIGHT, "ppmuright", "div_aclk_100", GATE_IP_RIGHTBUS, 1,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_GPIO_RIGHT, "gpio_right", "div_aclk_100", GATE_IP_RIGHTBUS, 0,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +
>>>> +    /* GATE_IP_PERIR */
>>>> +    GATE(CLK_MONOCNT, "monocnt", "div_aclk_100", GATE_IP_PERIR, 22,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_TZPC6, "tzpc6", "div_aclk_100", GATE_IP_PERIR, 21,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PROVISIONKEY1, "provisionkey1", "div_aclk_100",
>>>> +        GATE_IP_PERIR, 20, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PROVISIONKEY0, "provisionkey0", "div_aclk_100",
>>>> +        GATE_IP_PERIR, 19, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_CMU_ISPPART, "cmu_isppart", "div_aclk_100", GATE_IP_PERIR, 18,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_TMU_APBIF, "tmu_apbif", "div_aclk_100",
>>>> +        GATE_IP_PERIR, 17, 0, 0),
>>>> +    GATE(CLK_KEYIF, "keyif", "div_aclk_100", GATE_IP_PERIR, 16, 0, 0),
>>>> +    GATE(CLK_RTC, "rtc", "div_aclk_100", GATE_IP_PERIR, 15, 0, 0),
>>>> +    GATE(CLK_WDT, "wdt", "div_aclk_100", GATE_IP_PERIR, 14, 0, 0),
>>>> +    GATE(CLK_MCT, "mct", "div_aclk_100", GATE_IP_PERIR, 13, 0, 0),
>>>> +    GATE(CLK_SECKEY, "seckey", "div_aclk_100", GATE_IP_PERIR, 12,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_TZPC5, "tzpc5", "div_aclk_100", GATE_IP_PERIR, 10,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_TZPC4, "tzpc4", "div_aclk_100", GATE_IP_PERIR, 9,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_TZPC3, "tzpc3", "div_aclk_100", GATE_IP_PERIR, 8,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_TZPC2, "tzpc2", "div_aclk_100", GATE_IP_PERIR, 7,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_TZPC1, "tzpc1", "div_aclk_100", GATE_IP_PERIR, 6,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_TZPC0, "tzpc0", "div_aclk_100", GATE_IP_PERIR, 5,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_CMU_COREPART, "cmu_corepart", "div_aclk_100", GATE_IP_PERIR, 4,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_CMU_TOPPART, "cmu_toppart", "div_aclk_100", GATE_IP_PERIR, 3,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PMU_APBIF, "pmu_apbif", "div_aclk_100", GATE_IP_PERIR, 2,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_SYSREG, "sysreg", "div_aclk_100", GATE_IP_PERIR, 1,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_CHIP_ID, "chip_id", "div_aclk_100", GATE_IP_PERIR, 0,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +
>>>> +    /* GATE_SCLK_CAM */
>>>> +    GATE(CLK_SCLK_JPEG, "sclk_jpeg", "div_cam_blk",
>>>> +        GATE_SCLK_CAM, 8, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_M2MSCALER, "sclk_m2mscaler", "div_cam_blk",
>>>> +        GATE_SCLK_CAM, 2, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_GSCALER1, "sclk_gscaler1", "div_cam_blk",
>>>> +        GATE_SCLK_CAM, 1, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_GSCALER0, "sclk_gscaler0", "div_cam_blk",
>>>> +        GATE_SCLK_CAM, 0, CLK_SET_RATE_PARENT, 0),
>>>> +
>>>> +    /* GATE_SCLK_MFC */
>>>> +    GATE(CLK_SCLK_MFC, "sclk_mfc", "div_mfc",
>>>> +        GATE_SCLK_MFC, 0, CLK_SET_RATE_PARENT, 0),
>>>> +
>>>> +    /* GATE_SCLK_G3D */
>>>> +    GATE(CLK_SCLK_G3D, "sclk_g3d", "div_g3d",
>>>> +        GATE_SCLK_G3D, 0, CLK_SET_RATE_PARENT, 0),
>>>> +
>>>> +    /* GATE_SCLK_LCD */
>>>> +    GATE(CLK_SCLK_MIPIDPHY2L, "sclk_mipidphy2l", "div_mipi0",
>>>> +        GATE_SCLK_LCD, 4, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_MIPI0, "sclk_mipi0", "div_mipi0_pre",
>>>> +        GATE_SCLK_LCD, 3, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_FIMD0, "sclk_fimd0", "div_fimd0",
>>>> +        GATE_SCLK_LCD, 0, CLK_SET_RATE_PARENT, 0),
>>>> +
>>>> +    /* GATE_SCLK_ISP_TOP */
>>>> +    GATE(CLK_SCLK_CAM1, "sclk_cam1", "div_cam1",
>>>> +        GATE_SCLK_ISP_TOP, 4, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_UART_ISP, "sclk_uart_isp", "div_uart_isp",
>>>> +        GATE_SCLK_ISP_TOP, 3, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_SPI1_ISP, "sclk_spi1_isp", "div_spi1_isp",
>>>> +        GATE_SCLK_ISP_TOP, 2, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_SPI0_ISP, "sclk_spi0_isp", "div_spi0_isp",
>>>> +        GATE_SCLK_ISP_TOP, 1, CLK_SET_RATE_PARENT, 0),
>>>> +
>>>> +    /* GATE_SCLK_FSYS */
>>>> +    GATE(CLK_SCLK_UPLL, "sclk_upll", "mout_upll", GATE_SCLK_FSYS, 10, 0, 0),
>>>> +    GATE(CLK_SCLK_TSADC, "sclk_tsadc", "div_tsadc_pre",
>>>> +        GATE_SCLK_FSYS, 9, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_EBI, "sclk_ebi", "div_ebi",
>>>> +        GATE_SCLK_FSYS, 6, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_MMC1, "sclk_mmc1", "div_mmc1_pre",
>>>> +        GATE_SCLK_FSYS, 1, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_MMC0, "sclk_mmc0", "div_mmc0_pre",
>>>> +        GATE_SCLK_FSYS, 0, CLK_SET_RATE_PARENT, 0),
>>>> +
>>>> +    /* GATE_SCLK_PERIL */
>>>> +    GATE(CLK_SCLK_I2S, "sclk_i2s", "div_i2s",
>>>> +        GATE_SCLK_PERIL, 18, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_PCM, "sclk_pcm", "div_pcm",
>>>> +        GATE_SCLK_PERIL, 16, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_SPI1, "sclk_spi1", "div_spi1_pre",
>>>> +        GATE_SCLK_PERIL, 7, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_SPI0, "sclk_spi0", "div_spi0_pre",
>>>> +        GATE_SCLK_PERIL, 6, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_UART1, "sclk_uart1", "div_uart1",
>>>> +        GATE_SCLK_PERIL, 1, CLK_SET_RATE_PARENT, 0),
>>>> +    GATE(CLK_SCLK_UART0, "sclk_uart0", "div_uart0",
>>>> +        GATE_SCLK_PERIL, 0, CLK_SET_RATE_PARENT, 0),
>>>> +
>>>> +    /* GATE_IP_CAM */
>>>> +    GATE(CLK_QEJPEG, "qejpeg", "div_cam_blk_320", GATE_IP_CAM, 19,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PIXELASYNCM1, "pixelasyncm1", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 18, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PIXELASYNCM0, "pixelasyncm0", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 17, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PPMUCAMIF, "ppmucamif", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 16, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_QEM2MSCALER, "qem2mscaler", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 14, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_QEGSCALER1, "qegscaler1", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 13, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_QEGSCALER0, "qegscaler0", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 12, CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_SMMUJPEG, "smmujpeg", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 11, 0, 0),
>>>> +    GATE(CLK_SMMUM2M2SCALER, "smmum2m2scaler", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 9, 0, 0),
>>>> +    GATE(CLK_SMMUGSCALER1, "smmugscaler1", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 8, 0, 0),
>>>> +    GATE(CLK_SMMUGSCALER0, "smmugscaler0", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 7, 0, 0),
>>>> +    GATE(CLK_JPEG, "jpeg", "div_cam_blk_320", GATE_IP_CAM, 6, 0, 0),
>>>> +    GATE(CLK_M2MSCALER, "m2mscaler", "div_cam_blk_320",
>>>> +        GATE_IP_CAM, 2, 0, 0),
>>>> +    GATE(CLK_GSCALER1, "gscaler1", "div_cam_blk_320", GATE_IP_CAM, 1, 0, 0),
>>>> +    GATE(CLK_GSCALER0, "gscaler0", "div_cam_blk_320", GATE_IP_CAM, 0, 0, 0),
>>>> +
>>>> +    /* GATE_IP_MFC */
>>>> +    GATE(CLK_QEMFC, "qemfc", "div_aclk_200", GATE_IP_MFC, 5,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PPMUMFC_L, "ppmumfc_l", "div_aclk_200", GATE_IP_MFC, 3,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_SMMUMFC_L, "smmumfc_l", "div_aclk_200", GATE_IP_MFC, 1, 0, 0),
>>>> +    GATE(CLK_MFC, "mfc", "div_aclk_200", GATE_IP_MFC, 0, 0, 0),
>>>> +
>>>> +    /* GATE_IP_G3D */
>>>> +    GATE(CLK_SMMUG3D, "smmug3d", "div_aclk_200", GATE_IP_G3D, 3, 0, 0),
>>>> +    GATE(CLK_QEG3D, "qeg3d", "div_aclk_200", GATE_IP_G3D, 2,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PPMUG3D, "ppmug3d", "div_aclk_200", GATE_IP_G3D, 1,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_G3D, "g3d", "div_aclk_200", GATE_IP_G3D, 0, 0, 0),
>>>> +
>>>> +    /* GATE_IP_LCD */
>>>> +    GATE(CLK_QE_CH1_LCD, "qe_ch1_lcd", "div_aclk_160", GATE_IP_LCD, 7,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_QE_CH0_LCD, "qe_ch0_lcd", "div_aclk_160", GATE_IP_LCD, 6,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_PPMULCD0, "ppmulcd0", "div_aclk_160", GATE_IP_LCD, 5,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_SMMUFIMD0, "smmufimd0", "div_aclk_160", GATE_IP_LCD, 4, 0, 0),
>>>> +    GATE(CLK_DSIM0, "dsim0", "div_aclk_160", GATE_IP_LCD, 3, 0, 0),
>>>> +    GATE(CLK_SMIES, "smies", "div_aclk_160", GATE_IP_LCD, 2, 0, 0),
>>>> +    GATE(CLK_FIMD0, "fimd0", "div_aclk_160", GATE_IP_LCD, 0, 0, 0),
>>>> +
>>>> +    /* GATE_IP_ISP */
>>>> +    GATE(CLK_CAM1, "cam1", "mout_aclk_266_sub", GATE_IP_ISP, 5, 0, 0),
>>>> +    GATE(CLK_UART_ISP_TOP, "uart_isp_top", "mout_aclk_266_sub",
>>>> +        GATE_IP_ISP, 3, 0, 0),
>>>> +    GATE(CLK_SPI1_ISP_TOP, "spi1_isp_top", "mout_aclk_266_sub",
>>>> +        GATE_IP_ISP, 2, 0, 0),
>>>> +    GATE(CLK_SPI0_ISP_TOP, "spi0_isp_top", "mout_aclk_266_sub",
>>>> +        GATE_IP_ISP, 1, 0, 0),
>>>> +
>>>> +    /* GATE_IP_FSYS */
>>>> +    GATE(CLK_TSADC, "tsadc", "div_aclk_200", GATE_IP_FSYS, 20, 0, 0),
>>>> +    GATE(CLK_PPMUFILE, "ppmufile", "div_aclk_200", GATE_IP_FSYS, 17,
>>>> +        CLK_IGNORE_UNUSED, 0),
>>>> +    GATE(CLK_USBOTG, "usbotg", "div_aclk_200", GATE_IP_FSYS, 13, 0, 0),
>>>> +    GATE(CLK_USBHOST, "usbhost", "div_aclk_200", GATE_IP_FSYS, 12, 0, 0),
>>>> +    GATE(CLK_SROMC, "sromc", "div_aclk_200", GATE_IP_FSYS, 11, 0, 0),
>>>> +    GATE(CLK_SDMMC1, "sdmmc1", "div_aclk_200", GATE_IP_FSYS, 6, 0, 0),
>>>> +    GATE(CLK_SDMMC0, "sdmmc0", "div_aclk_200", GATE_IP_FSYS, 5, 0, 0),
>>>> +    GATE(CLK_PDMA1, "pdma1", "div_aclk_200", GATE_IP_FSYS, 1, 0, 0),
>>>> +    GATE(CLK_PDMA0, "pdma0", "div_aclk_200", GATE_IP_FSYS, 0, 0, 0),
>>>> +
>>>> +    /* GATE_IP_PERIL */
>>>> +    GATE(CLK_PWM, "pwm", "div_aclk_100", GATE_IP_PERIL, 24, 0, 0),
>>>> +    GATE(CLK_PCM, "pcm", "div_aclk_100", GATE_IP_PERIL, 23, 0, 0),
>>>> +    GATE(CLK_I2S, "i2s", "div_aclk_100", GATE_IP_PERIL, 21, 0, 0),
>>>> +    GATE(CLK_SPI1, "spi1", "div_aclk_100", GATE_IP_PERIL, 17, 0, 0),
>>>> +    GATE(CLK_SPI0, "spi0", "div_aclk_100", GATE_IP_PERIL, 16, 0, 0),
>>>> +    GATE(CLK_I2C7, "i2c7", "div_aclk_100", GATE_IP_PERIL, 13, 0, 0),
>>>> +    GATE(CLK_I2C6, "i2c6", "div_aclk_100", GATE_IP_PERIL, 12, 0, 0),
>>>> +    GATE(CLK_I2C5, "i2c5", "div_aclk_100", GATE_IP_PERIL, 11, 0, 0),
>>>> +    GATE(CLK_I2C4, "i2c4", "div_aclk_100", GATE_IP_PERIL, 10, 0, 0),
>>>> +    GATE(CLK_I2C3, "i2c3", "div_aclk_100", GATE_IP_PERIL, 9, 0, 0),
>>>> +    GATE(CLK_I2C2, "i2c2", "div_aclk_100", GATE_IP_PERIL, 8, 0, 0),
>>>> +    GATE(CLK_I2C1, "i2c1", "div_aclk_100", GATE_IP_PERIL, 7, 0, 0),
>>>> +    GATE(CLK_I2C0, "i2c0", "div_aclk_100", GATE_IP_PERIL, 6, 0, 0),
>>>> +    GATE(CLK_UART1, "uart1", "div_aclk_100", GATE_IP_PERIL, 1, 0, 0),
>>>> +    GATE(CLK_UART0, "uart0", "div_aclk_100", GATE_IP_PERIL, 0, 0, 0),
>>>> +
>>>> +    /* GATE_BLOCK */
>>>> +    GATE(CLK_BLOCK_LCD, "block_lcd", "div_aclk_160", GATE_BLOCK, 4, 0, 0),
>>>> +    GATE(CLK_BLOCK_G3D, "block_g3d", "div_aclk_200", GATE_BLOCK, 3, 0, 0),
>>>
>>> Are there only 2 gate block clocks? By the way, how are they going to be handled by respective drivers? There is no mainline support for them right now, but you should be aware that adding them will cause common clock framework to disable them if not claimed by any driver.
>>
>> OK, I'll add remaing clock gate of GATE_BLOCK as following.
>> - CLK_BLOCK_MFC MFC_BLK
>> - CLK_BLOCK_CAM CAM_BLK
>>
> 
> I agree that in the end the block gates will have to be added. However
> currently drivers do not request block gates and enable them.
> Considering that common clock framework disables all unused clocks by
> default, this will lead to all the gate block clocks being disabled,
> which is not desired.

You're right.

> 
> My opinion on this is that block gate clocks should be added in separate
> patch along with patches adding code to get and enable them.

OK, I'll remove the clocks of GATE_BLOCK on next posting(v6)

> 
>>>
>>>> +};
>>>> +
>>>> +/* APLL & MPLL & BPLL & UPLL */
>>>> +static struct samsung_pll_rate_table exynos3250_pll_rates[] = {
>>>> +    PLL_35XX_RATE(1200000000, 400, 4, 1),
>>>> +    PLL_35XX_RATE(1100000000, 275, 3, 1),
>>>> +    PLL_35XX_RATE(1066000000, 533, 6, 1),
>>>> +    PLL_35XX_RATE(1000000000, 250, 3, 1),
>>>> +    PLL_35XX_RATE( 960000000, 320, 4, 1),
>>>> +    PLL_35XX_RATE( 900000000, 300, 4, 1),
>>>> +    PLL_35XX_RATE( 850000000, 425, 6, 1),
>>>> +    PLL_35XX_RATE( 800000000, 200, 3, 1),
>>>> +    PLL_35XX_RATE( 700000000, 175, 3, 1),
>>>> +    PLL_35XX_RATE( 667000000, 667, 12, 1),
>>>> +    PLL_35XX_RATE( 600000000, 400, 4, 2),
>>>> +    PLL_35XX_RATE( 533000000, 533, 6, 2),
>>>> +    PLL_35XX_RATE( 520000000, 260, 3, 2),
>>>> +    PLL_35XX_RATE( 500000000, 250, 3, 2),
>>>> +    PLL_35XX_RATE( 400000000, 200, 3, 2),
>>>> +    PLL_35XX_RATE( 200000000, 200, 3, 3),
>>>> +    PLL_35XX_RATE( 100000000, 200, 3, 4),
>>>> +    { /* sentinel */ }
>>>> +};
>>>> +
>>>> +/* VPLL */
>>>> +static struct samsung_pll_rate_table exynos3250_vpll_rates[] = {
>>>> +    PLL_36XX_RATE(600000000, 100, 2, 1,     0),
>>>> +    PLL_36XX_RATE(533000000, 267, 3, 2, 32668),
> 
> The TRM actually lists this as 267, 3, 2, 32768, and according to the
> equation it will be 535000015 Hz. Looks like a typo in the data sheet,
> as 266, 3, 2, 32768 gives 533000015, which is almost exactly 533 MHz.
> 
>>>> +    PLL_36XX_RATE(519231000, 173, 2, 2,  5046),
> 
> 519230991
> 
>>>> +    PLL_36XX_RATE(500000000, 250, 3, 2,     0),
>>>> +    PLL_36XX_RATE(445500000, 149, 2, 2, 32768),
> 
> 448500022
> 
> Also looks like a typo in the TRM, as 148, 2, 2, 32768 gives 445500022,
> which is almost exactly 445.5 MHz.
> 
> 
>>>> +    PLL_36XX_RATE(445055000, 148, 2, 2, 23047),
> 
> 445055024
> 
>>>> +    PLL_36XX_RATE(400000000, 200, 3, 2,     0),
>>>> +    PLL_36XX_RATE(371250000, 124, 2, 2, 49512),
> 
> The TRM lists this as 124, 2, 2, 49152 and calculated frequency is
> 374250034. This one also looks like a typo. 123, 2, 2, 49512 would give
> 371250034.

When I calculated fout with following data:
- 124, 2, 2, 49512 would give 374266514.1
- 123, 2, 2, 49512 would give 371266514.1.

I think below value is proper. 
- 123, 2, 2, 49512 would give 371266514.1.

> 
>>>> +    PLL_36XX_RATE(370879000, 185, 3, 2, 28803),
> 
> 370879011
> 
>>>> +    PLL_36XX_RATE(340000000, 170, 3, 2,     0),
>>>> +    PLL_36XX_RATE(335000000, 112, 2, 2, 43691),
> 
> 338000045
> 
> 111, 2, 2, 43691 would give 335000045. A typo in TRM?
> 
>>>> +    PLL_36XX_RATE(333000000, 111, 2, 2,     0),
>>>> +    PLL_36XX_RATE(330000000, 110, 2, 2,     0),
>>>> +    PLL_36XX_RATE(320000000, 107, 2, 2, 43691),
> 
> 323000045
> 
> 106, 2, 2, 43691 would give 320000045.
> 
>>>> +    PLL_36XX_RATE(300000000, 100, 2, 2,     0),
>>>> +    PLL_36XX_RATE(275000000, 275, 3, 3,     0),
>>>> +    PLL_36XX_RATE(222750000, 149, 2, 3, 32768),
> 
> 224250011
> 
> 148, 2, 3, 32768 would give 222750011.
> 
>>>> +    PLL_36XX_RATE(222528000, 148, 2, 3, 23069),
> 
> 222528015
> 
>>>> +    PLL_36XX_RATE(160000000, 160, 3, 3,     0),
>>>> +    PLL_36XX_RATE(148500000,  99, 2, 3,     0),
>>>> +    PLL_36XX_RATE(148352000,  99, 2, 3, 59070),
> 
> 149852025
> 
> 98, 2, 3, 59070 would give 148352025.
> 
>>>> +    PLL_36XX_RATE(108000000, 144, 2, 4,     0),
>>>> +    PLL_36XX_RATE( 74250000,  99, 2, 4,     0),
>>>> +    PLL_36XX_RATE( 74176000,  99, 3, 4, 59070),
> 
> The TRM seems to list this as 99, 2, 4 and calculated frequency will be
> 74926012, but 98, 2, 4, 59070 would give 74176012.
> 
>>>> +    PLL_36XX_RATE( 54054000, 216, 3, 5, 14156),
> 
> 54054001
> 
>>>> +    PLL_36XX_RATE( 54000000, 144, 2, 5,     0),
>>>
>>> Are all these frequencies above calculated exactly? For correct operation of rate setting code, it is necessary for frequency values specified in these arrays to be exact, not rounded.
>>
>> When I implemnted exynos3250_vpll_rates array, I used 'VPLL PMS Value' in Exynos3250 TRM without modification.
>> This rate value of exynos3250_vpll_rates is correct.
> 
> Well, after checking the values using PLL equation for VPLL, as
> specified in TRM and used by clk-pll.c, the values don't match.
> 
> Keep in mind that the values must be exact, _not_ rounded, while in the
> TRM they are rounded. Moreover it looks like several frequencies in TRM
> are off by 1 in M coefficient. Please see above.

Thanks for your point out.

So, I calculated fout of VPLL using PLL equation in Exynos3250 TRM.

Following table show fout(Recalc rate) with fin_pll/mdiv/pdiv/sdiv/kdiv:
fin_pll		Recalc rate	TRM rate	mdiv	pdiv	sdiv	kdiv
24000000	600000000	600000000	100	2	1	0
24000000	533000015.3	533000000	266	3	2	32768
24000000	519230991.1	519231000	173	2	2	5046
24000000	500000000	500000000	250	3	2	0
24000000	445500022.9	445500000	148	2	2	32768
24000000	445055024	445055000	148	2	2	23047
24000000	400000000	400000000	200	3	2	0
24000000	371266514.1	371250000	123	2	2	49512
24000000	370879011.2	370879000	185	3	2	28803
24000000	340000000	340000000	170	3	2	0
24000000	335000045.8	335000000	111	2	2	43691
24000000	333000000	333000000	111	2	2	0
24000000	330000000	330000000	110	2	2	0
24000000	320000045.8	320000000	106	2	2	43691
24000000	300000000	300000000	100	2	2	0
24000000	275000000	275000000	275	3	3	0
24000000	222750011.4	222750000	148	2	3	32768
24000000	222528015.6	222528000	148	2	3	23069
24000000	160000000	160000000	160	3	3	0
24000000	148500000	148500000	99	2	3	0
24000000	148352025.6	148352000	98	2	3	59070
24000000	108000000	108000000	144	2	4	0
24000000	74250000	74250000	99	2	4	0
24000000	74176012.82	74176000	98	2	4	59070
24000000	54054001.68	54054000	216	3	5	14156
24000000	54000000	54000000	144	2	5	0


If you ok, I'll modify vpll_rates table as following:
+static struct samsung_pll_rate_table exynos3250_vpll_rates[] = {
+	PLL_36XX_RATE(600000000, 100, 2, 1,     0),
+	PLL_36XX_RATE(533000000, 266, 3, 2, 32768),
+	PLL_36XX_RATE(519230991, 173, 2, 2,  5046),
+	PLL_36XX_RATE(500000000, 250, 3, 2,     0),
+	PLL_36XX_RATE(445500022, 148, 2, 2, 32768),
+	PLL_36XX_RATE(445055024, 148, 2, 2, 23047),
+	PLL_36XX_RATE(400000000, 200, 3, 2,     0),
+	PLL_36XX_RATE(371266514, 123, 2, 2, 49512),
+	PLL_36XX_RATE(370879011, 185, 3, 2, 28803),
+	PLL_36XX_RATE(340000000, 170, 3, 2,     0),
+	PLL_36XX_RATE(335000045, 111, 2, 2, 43691),
+	PLL_36XX_RATE(333000000, 111, 2, 2,     0),
+	PLL_36XX_RATE(330000000, 110, 2, 2,     0),
+	PLL_36XX_RATE(320000045, 106, 2, 2, 43691),
+	PLL_36XX_RATE(300000000, 100, 2, 2,     0),
+	PLL_36XX_RATE(275000000, 275, 3, 3,     0),
+	PLL_36XX_RATE(222750011, 148, 2, 3, 32768),
+	PLL_36XX_RATE(222528015, 148, 2, 3, 23069),
+	PLL_36XX_RATE(160000000, 160, 3, 3,     0),
+	PLL_36XX_RATE(148500000,  99, 2, 3,     0),
+	PLL_36XX_RATE(148352025,  98, 2, 3, 59070),
+	PLL_36XX_RATE(108000000, 144, 2, 4,     0),
+	PLL_36XX_RATE( 74250000,  99, 2, 4,     0),
+	PLL_36XX_RATE( 74176012,  98, 2, 4, 59070),
+	PLL_36XX_RATE( 54054012, 216, 3, 5, 14156),
+	PLL_36XX_RATE( 54000000, 144, 2, 5,     0),
+	{ /* sentinel */ }
+};

Best Regards,
Chanwoo Choi






More information about the linux-arm-kernel mailing list