All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pankaj Dubey <pankaj.dubey-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
To: Chanwoo Choi <cw00.choi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
	s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org,
	tomasz.figa-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org
Cc: kgene-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	inki.dae-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org,
	chanho61.park-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org,
	sw0312.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org,
	linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [v3,1/9] clk: samsung: exynos5433: Add clocks for CMU_APOLLO domain
Date: Thu, 05 Feb 2015 13:14:39 +0530	[thread overview]
Message-ID: <54D31F67.1050701@samsung.com> (raw)
In-Reply-To: <1422922437-7414-2-git-send-email-cw00.choi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>

Hi Chanwoo,

On Tuesday 03 February 2015 05:43 AM, Chanwoo Choi wrote:
> This patch adds the mux/divider/gate clocks for CMU_APOLLO domain which
> generates the clocks for Cortex-A53 Quad-core processsor.
>
> Cc: Sylwester Nawrocki <s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
> Cc: Tomasz Figa <tomasz.figa-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> Signed-off-by: Chanwoo Choi <cw00.choi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
> Acked-by: Inki Dae <inki.dae-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
>
> ---
> .../devicetree/bindings/clock/exynos5433-clock.txt |  15 ++
>   drivers/clk/samsung/clk-exynos5433.c               | 193 +++++++++++++++++++++
>   include/dt-bindings/clock/exynos5433.h             |  37 ++++
>   3 files changed, 245 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/clock/exynos5433-clock.txt b/Documentation/devicetree/bindings/clock/exynos5433-clock.txt
> index 9e7ed2d..0a71468 100644
> --- a/Documentation/devicetree/bindings/clock/exynos5433-clock.txt
> +++ b/Documentation/devicetree/bindings/clock/exynos5433-clock.txt
> @@ -32,6 +32,8 @@ Required Properties:
>       which generates clocks for 3D Graphics Engine IP.
>     - "samsung,exynos5433-cmu-gscl"  - clock controller compatible for CMU_GSCL
>       which generates clocks for GSCALER IPs.
> +  - "samsung,exynos5433-cmu-apollo"- clock controller compatible for CMU_APOLLO
> +    which generates clocks for Cortex-A53 Quad-core processor.
>
>   - reg: physical base address of the controller and length of memory mapped
>     region.
> @@ -105,6 +107,10 @@ Required Properties:
>   		- aclk_gscl_111
>   		- aclk_gscl_333
>
> +	Input clocks for apollo clock controller:
> +		- oscclk
> +		- sclk_bus_pll_apollo
> +
>   Each clock is assigned an identifier and client nodes can use this identifier
>   to specify the clock which they consume.
>
> @@ -289,6 +295,15 @@ Example 2: Examples of clock controller nodes are listed below.
>   			<&cmu_top CLK_ACLK_GSCL_333>;
>   	};
>
> +	cmu_apollo: clock-controller@11900000 {
> +		compatible = "samsung,exynos5433-cmu-apollo";
> +		reg = <0x11900000 0x1088>;
> +		#clock-cells = <1>;
> +
> +		clock-names = "oscclk", "sclk_bus_pll_apollo";
> +		clocks = <&xxti>, <&cmu_mif CLK_SCLK_BUS_PLL_APOLLO>;
> +	};
> +
>   Example 3: UART controller node that consumes the clock generated by the clock
>   	   controller.
>
> diff --git a/drivers/clk/samsung/clk-exynos5433.c b/drivers/clk/samsung/clk-exynos5433.c
> index 8ae9c48..9a5d33c 100644
> --- a/drivers/clk/samsung/clk-exynos5433.c
> +++ b/drivers/clk/samsung/clk-exynos5433.c
> @@ -3393,3 +3393,196 @@ static void __init exynos5433_cmu_gscl_init(struct device_node *np)
>   }
>   CLK_OF_DECLARE(exynos5433_cmu_gscl, "samsung,exynos5433-cmu-gscl",
>   		exynos5433_cmu_gscl_init);
> +
> +/*
> + * Register offset definitions for CMU_APOLLO
> + */
> +#define APOLLO_PLL_LOCK				0x0000
> +#define APOLLO_PLL_CON0				0x0100
> +#define APOLLO_PLL_CON1				0x0104
> +#define APOLLO_PLL_FREQ_DET			0x010c
> +#define MUX_SEL_APOLLO0				0x0200
> +#define MUX_SEL_APOLLO1				0x0204
> +#define MUX_SEL_APOLLO2				0x0208
> +#define MUX_ENABLE_APOLLO0			0x0300
> +#define MUX_ENABLE_APOLLO1			0x0304
> +#define MUX_ENABLE_APOLLO2			0x0308
> +#define MUX_STAT_APOLLO0			0x0400
> +#define MUX_STAT_APOLLO1			0x0404
> +#define MUX_STAT_APOLLO2			0x0408
> +#define DIV_APOLLO0				0x0600
> +#define DIV_APOLLO1				0x0604
> +#define DIV_APOLLO_PLL_FREQ_DET			0x0608
> +#define DIV_STAT_APOLLO0			0x0700
> +#define DIV_STAT_APOLLO1			0x0704
> +#define DIV_STAT_APOLLO_PLL_FREQ_DET		0x0708
> +#define ENABLE_ACLK_APOLLO			0x0800
> +#define ENABLE_PCLK_APOLLO			0x0900
> +#define ENABLE_SCLK_APOLLO			0x0a00
> +#define ENABLE_IP_APOLLO0			0x0b00
> +#define ENABLE_IP_APOLLO1			0x0b04
> +#define CLKOUT_CMU_APOLLO			0x0c00
> +#define CLKOUT_CMU_APOLLO_DIV_STAT		0x0c04
> +#define ARMCLK_STOPCTRL				0x1000
> +#define APOLLO_PWR_CTRL				0x1020
> +#define APOLLO_PWR_CTRL2			0x1024
> +#define APOLLO_INTR_SPREAD_ENABLE		0x1080
> +#define APOLLO_INTR_SPREAD_USE_STANDBYWFI	0x1084
> +#define APOLLO_INTR_SPREAD_BLOCKING_DURATION	0x1088
> +
> +static unsigned long apollo_clk_regs[] __initdata = {
> +	APOLLO_PLL_LOCK,
> +	APOLLO_PLL_CON0,
> +	APOLLO_PLL_CON1,
> +	APOLLO_PLL_FREQ_DET,
> +	MUX_SEL_APOLLO0,
> +	MUX_SEL_APOLLO1,
> +	MUX_SEL_APOLLO2,
> +	MUX_ENABLE_APOLLO0,
> +	MUX_ENABLE_APOLLO1,
> +	MUX_ENABLE_APOLLO2,
> +	MUX_STAT_APOLLO0,
> +	MUX_STAT_APOLLO1,
> +	MUX_STAT_APOLLO2,
> +	DIV_APOLLO0,
> +	DIV_APOLLO1,
> +	DIV_APOLLO_PLL_FREQ_DET,
> +	DIV_STAT_APOLLO0,
> +	DIV_STAT_APOLLO1,
> +	DIV_STAT_APOLLO_PLL_FREQ_DET,
> +	ENABLE_ACLK_APOLLO,
> +	ENABLE_PCLK_APOLLO,
> +	ENABLE_SCLK_APOLLO,
> +	ENABLE_IP_APOLLO0,
> +	ENABLE_IP_APOLLO1,
> +	CLKOUT_CMU_APOLLO,
> +	CLKOUT_CMU_APOLLO_DIV_STAT,
> +	ARMCLK_STOPCTRL,
> +	APOLLO_PWR_CTRL,
> +	APOLLO_PWR_CTRL2,
> +	APOLLO_INTR_SPREAD_ENABLE,
> +	APOLLO_INTR_SPREAD_USE_STANDBYWFI,
> +	APOLLO_INTR_SPREAD_BLOCKING_DURATION,
> +};
> +
> +/* list of all parent clock list */
> +PNAME(mout_apollo_pll_p)		= { "oscclk", "fout_apollo_pll", };
> +PNAME(mout_bus_pll_apollo_user_p)	= { "oscclk", "sclk_bus_pll_apollo", };
> +PNAME(mout_apollo_p)			= { "mout_apollo_pll",
> +					    "mout_bus_pll_apollo_user", };
> +
> +static struct samsung_pll_clock apollo_pll_clks[] __initdata = {
> +	PLL(pll_35xx, CLK_FOUT_APOLLO_PLL, "fout_apollo_pll", "oscclk",
> +		APOLLO_PLL_LOCK, APOLLO_PLL_CON0, exynos5443_pll_rates),
> +};
> +
> +static struct samsung_mux_clock apollo_mux_clks[] __initdata = {
> +	/* MUX_SEL_APOLLO0 */
> +	MUX_F(CLK_MOUT_APOLLO_PLL, "mout_apollo_pll", mout_apollo_pll_p,
> +			MUX_SEL_APOLLO0, 0, 1, 0, CLK_MUX_READ_ONLY),
> +
> +	/* MUX_SEL_APOLLO1 */
> +	MUX(CLK_MOUT_BUS_PLL_APOLLO_USER, "mout_bus_pll_apollo_user",
> +			mout_bus_pll_apollo_user_p, MUX_SEL_APOLLO1, 0, 1),
> +
> +	/* MUX_SEL_APOLLO2 */
> +	MUX_F(CLK_MOUT_APOLLO, "mout_apollo", mout_apollo_p, MUX_SEL_APOLLO2,
> +			0, 1, 0, CLK_MUX_READ_ONLY),
> +};
> +
> +static struct samsung_div_clock apollo_div_clks[] __initdata = {
> +	/* DIV_APOLLO0 */
> +	DIV_F(CLK_DIV_CNTCLK_APOLLO, "div_cntclk_apollo", "div_apollo2",
> +			DIV_APOLLO0, 24, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_PCLK_DBG_APOLLO, "div_pclk_dbg_apollo", "div_apollo2",
> +			DIV_APOLLO0, 20, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_ATCLK_APOLLO, "div_atclk_apollo", "div_apollo2",
> +			DIV_APOLLO0, 16, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_PCLK_APOLLO, "div_pclk_apollo", "div_apollo2",
> +			DIV_APOLLO0, 12, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_ACLK_APOLLO, "div_aclk_apollo", "div_apollo2",
> +			DIV_APOLLO0, 8, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_APOLLO2, "div_apollo2", "div_apollo1",
> +			DIV_APOLLO0, 4, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_APOLLO1, "div_apollo1", "mout_apollo",
> +			DIV_APOLLO0, 0, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +
> +	/* DIV_APOLLO1 */
> +	DIV_F(CLK_DIV_SCLK_HPM_APOLLO, "div_sclk_hpm_apollo", "mout_apollo",
> +			DIV_APOLLO1, 4, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_APOLLO_PLL, "div_apollo_pll", "mout_apollo",
> +			DIV_APOLLO1, 0, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +};
> +
> +static struct samsung_gate_clock apollo_gate_clks[] __initdata = {
> +	/* ENABLE_ACLK_APOLLO */
> +	GATE(CLK_ACLK_ASATBSLV_APOLLO_3_CSSYS, "aclk_asatbslv_apollo_3_cssys",
> +			"div_atclk_apollo", ENABLE_ACLK_APOLLO,
> +			6, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_ASATBSLV_APOLLO_2_CSSYS, "aclk_asatbslv_apollo_2_cssys",
> +			"div_atclk_apollo", ENABLE_ACLK_APOLLO,
> +			5, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_ASATBSLV_APOLLO_1_CSSYS, "aclk_asatbslv_apollo_1_cssys",
> +			"div_atclk_apollo", ENABLE_ACLK_APOLLO,
> +			4, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_ASATBSLV_APOLLO_0_CSSYS, "aclk_asatbslv_apollo_0_cssys",
> +			"div_atclk_apollo", ENABLE_ACLK_APOLLO,
> +			3, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_ASYNCACES_APOLLO_CCI, "aclk_asyncaces_apollo_cci",
> +			"div_aclk_apollo", ENABLE_ACLK_APOLLO,
> +			2, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_AHB2APB_APOLLOP, "aclk_ahb2apb_apollop",
> +			"div_pclk_apollo", ENABLE_ACLK_APOLLO,
> +			1, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_APOLLONP_200, "aclk_apollonp_200",
> +			"div_pclk_apollo", ENABLE_ACLK_APOLLO,
> +			0, CLK_IGNORE_UNUSED, 0),
> +
> +	/* ENABLE_PCLK_APOLLO */
> +	GATE(CLK_PCLK_ASAPBMST_CSSYS_APOLLO, "pclk_asapbmst_cssys_apollo",
> +			"div_pclk_dbg_apollo", ENABLE_PCLK_APOLLO,
> +			2, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_PCLK_PMU_APOLLO, "pclk_pmu_apollo", "div_pclk_apollo",
> +			ENABLE_PCLK_APOLLO, 1, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_PCLK_SYSREG_APOLLO, "pclk_pmu_sysreg_apollo",

Isn't "pclk_pmu_sysreg_apollo" be named as "pclk_sysreg_apollo" to match 
with UM?


Rest changes in this file are OK. I confirmed clock tree and clock bits 
against UM.

Thanks,
Pankaj Dubey
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: Pankaj Dubey <pankaj.dubey@samsung.com>
To: Chanwoo Choi <cw00.choi@samsung.com>,
	s.nawrocki@samsung.com, tomasz.figa@gmail.com,
	mturquette@linaro.org
Cc: kgene@kernel.org, inki.dae@samsung.com,
	chanho61.park@samsung.com, sw0312.kim@samsung.com,
	linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
	devicetree@vger.kernel.org
Subject: Re: [v3,1/9] clk: samsung: exynos5433: Add clocks for CMU_APOLLO domain
Date: Thu, 05 Feb 2015 13:14:39 +0530	[thread overview]
Message-ID: <54D31F67.1050701@samsung.com> (raw)
In-Reply-To: <1422922437-7414-2-git-send-email-cw00.choi@samsung.com>

Hi Chanwoo,

On Tuesday 03 February 2015 05:43 AM, Chanwoo Choi wrote:
> This patch adds the mux/divider/gate clocks for CMU_APOLLO domain which
> generates the clocks for Cortex-A53 Quad-core processsor.
>
> Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>
> Cc: Tomasz Figa <tomasz.figa@gmail.com>
> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
> Acked-by: Inki Dae <inki.dae@samsung.com>
>
> ---
> .../devicetree/bindings/clock/exynos5433-clock.txt |  15 ++
>   drivers/clk/samsung/clk-exynos5433.c               | 193 +++++++++++++++++++++
>   include/dt-bindings/clock/exynos5433.h             |  37 ++++
>   3 files changed, 245 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/clock/exynos5433-clock.txt b/Documentation/devicetree/bindings/clock/exynos5433-clock.txt
> index 9e7ed2d..0a71468 100644
> --- a/Documentation/devicetree/bindings/clock/exynos5433-clock.txt
> +++ b/Documentation/devicetree/bindings/clock/exynos5433-clock.txt
> @@ -32,6 +32,8 @@ Required Properties:
>       which generates clocks for 3D Graphics Engine IP.
>     - "samsung,exynos5433-cmu-gscl"  - clock controller compatible for CMU_GSCL
>       which generates clocks for GSCALER IPs.
> +  - "samsung,exynos5433-cmu-apollo"- clock controller compatible for CMU_APOLLO
> +    which generates clocks for Cortex-A53 Quad-core processor.
>
>   - reg: physical base address of the controller and length of memory mapped
>     region.
> @@ -105,6 +107,10 @@ Required Properties:
>   		- aclk_gscl_111
>   		- aclk_gscl_333
>
> +	Input clocks for apollo clock controller:
> +		- oscclk
> +		- sclk_bus_pll_apollo
> +
>   Each clock is assigned an identifier and client nodes can use this identifier
>   to specify the clock which they consume.
>
> @@ -289,6 +295,15 @@ Example 2: Examples of clock controller nodes are listed below.
>   			<&cmu_top CLK_ACLK_GSCL_333>;
>   	};
>
> +	cmu_apollo: clock-controller@11900000 {
> +		compatible = "samsung,exynos5433-cmu-apollo";
> +		reg = <0x11900000 0x1088>;
> +		#clock-cells = <1>;
> +
> +		clock-names = "oscclk", "sclk_bus_pll_apollo";
> +		clocks = <&xxti>, <&cmu_mif CLK_SCLK_BUS_PLL_APOLLO>;
> +	};
> +
>   Example 3: UART controller node that consumes the clock generated by the clock
>   	   controller.
>
> diff --git a/drivers/clk/samsung/clk-exynos5433.c b/drivers/clk/samsung/clk-exynos5433.c
> index 8ae9c48..9a5d33c 100644
> --- a/drivers/clk/samsung/clk-exynos5433.c
> +++ b/drivers/clk/samsung/clk-exynos5433.c
> @@ -3393,3 +3393,196 @@ static void __init exynos5433_cmu_gscl_init(struct device_node *np)
>   }
>   CLK_OF_DECLARE(exynos5433_cmu_gscl, "samsung,exynos5433-cmu-gscl",
>   		exynos5433_cmu_gscl_init);
> +
> +/*
> + * Register offset definitions for CMU_APOLLO
> + */
> +#define APOLLO_PLL_LOCK				0x0000
> +#define APOLLO_PLL_CON0				0x0100
> +#define APOLLO_PLL_CON1				0x0104
> +#define APOLLO_PLL_FREQ_DET			0x010c
> +#define MUX_SEL_APOLLO0				0x0200
> +#define MUX_SEL_APOLLO1				0x0204
> +#define MUX_SEL_APOLLO2				0x0208
> +#define MUX_ENABLE_APOLLO0			0x0300
> +#define MUX_ENABLE_APOLLO1			0x0304
> +#define MUX_ENABLE_APOLLO2			0x0308
> +#define MUX_STAT_APOLLO0			0x0400
> +#define MUX_STAT_APOLLO1			0x0404
> +#define MUX_STAT_APOLLO2			0x0408
> +#define DIV_APOLLO0				0x0600
> +#define DIV_APOLLO1				0x0604
> +#define DIV_APOLLO_PLL_FREQ_DET			0x0608
> +#define DIV_STAT_APOLLO0			0x0700
> +#define DIV_STAT_APOLLO1			0x0704
> +#define DIV_STAT_APOLLO_PLL_FREQ_DET		0x0708
> +#define ENABLE_ACLK_APOLLO			0x0800
> +#define ENABLE_PCLK_APOLLO			0x0900
> +#define ENABLE_SCLK_APOLLO			0x0a00
> +#define ENABLE_IP_APOLLO0			0x0b00
> +#define ENABLE_IP_APOLLO1			0x0b04
> +#define CLKOUT_CMU_APOLLO			0x0c00
> +#define CLKOUT_CMU_APOLLO_DIV_STAT		0x0c04
> +#define ARMCLK_STOPCTRL				0x1000
> +#define APOLLO_PWR_CTRL				0x1020
> +#define APOLLO_PWR_CTRL2			0x1024
> +#define APOLLO_INTR_SPREAD_ENABLE		0x1080
> +#define APOLLO_INTR_SPREAD_USE_STANDBYWFI	0x1084
> +#define APOLLO_INTR_SPREAD_BLOCKING_DURATION	0x1088
> +
> +static unsigned long apollo_clk_regs[] __initdata = {
> +	APOLLO_PLL_LOCK,
> +	APOLLO_PLL_CON0,
> +	APOLLO_PLL_CON1,
> +	APOLLO_PLL_FREQ_DET,
> +	MUX_SEL_APOLLO0,
> +	MUX_SEL_APOLLO1,
> +	MUX_SEL_APOLLO2,
> +	MUX_ENABLE_APOLLO0,
> +	MUX_ENABLE_APOLLO1,
> +	MUX_ENABLE_APOLLO2,
> +	MUX_STAT_APOLLO0,
> +	MUX_STAT_APOLLO1,
> +	MUX_STAT_APOLLO2,
> +	DIV_APOLLO0,
> +	DIV_APOLLO1,
> +	DIV_APOLLO_PLL_FREQ_DET,
> +	DIV_STAT_APOLLO0,
> +	DIV_STAT_APOLLO1,
> +	DIV_STAT_APOLLO_PLL_FREQ_DET,
> +	ENABLE_ACLK_APOLLO,
> +	ENABLE_PCLK_APOLLO,
> +	ENABLE_SCLK_APOLLO,
> +	ENABLE_IP_APOLLO0,
> +	ENABLE_IP_APOLLO1,
> +	CLKOUT_CMU_APOLLO,
> +	CLKOUT_CMU_APOLLO_DIV_STAT,
> +	ARMCLK_STOPCTRL,
> +	APOLLO_PWR_CTRL,
> +	APOLLO_PWR_CTRL2,
> +	APOLLO_INTR_SPREAD_ENABLE,
> +	APOLLO_INTR_SPREAD_USE_STANDBYWFI,
> +	APOLLO_INTR_SPREAD_BLOCKING_DURATION,
> +};
> +
> +/* list of all parent clock list */
> +PNAME(mout_apollo_pll_p)		= { "oscclk", "fout_apollo_pll", };
> +PNAME(mout_bus_pll_apollo_user_p)	= { "oscclk", "sclk_bus_pll_apollo", };
> +PNAME(mout_apollo_p)			= { "mout_apollo_pll",
> +					    "mout_bus_pll_apollo_user", };
> +
> +static struct samsung_pll_clock apollo_pll_clks[] __initdata = {
> +	PLL(pll_35xx, CLK_FOUT_APOLLO_PLL, "fout_apollo_pll", "oscclk",
> +		APOLLO_PLL_LOCK, APOLLO_PLL_CON0, exynos5443_pll_rates),
> +};
> +
> +static struct samsung_mux_clock apollo_mux_clks[] __initdata = {
> +	/* MUX_SEL_APOLLO0 */
> +	MUX_F(CLK_MOUT_APOLLO_PLL, "mout_apollo_pll", mout_apollo_pll_p,
> +			MUX_SEL_APOLLO0, 0, 1, 0, CLK_MUX_READ_ONLY),
> +
> +	/* MUX_SEL_APOLLO1 */
> +	MUX(CLK_MOUT_BUS_PLL_APOLLO_USER, "mout_bus_pll_apollo_user",
> +			mout_bus_pll_apollo_user_p, MUX_SEL_APOLLO1, 0, 1),
> +
> +	/* MUX_SEL_APOLLO2 */
> +	MUX_F(CLK_MOUT_APOLLO, "mout_apollo", mout_apollo_p, MUX_SEL_APOLLO2,
> +			0, 1, 0, CLK_MUX_READ_ONLY),
> +};
> +
> +static struct samsung_div_clock apollo_div_clks[] __initdata = {
> +	/* DIV_APOLLO0 */
> +	DIV_F(CLK_DIV_CNTCLK_APOLLO, "div_cntclk_apollo", "div_apollo2",
> +			DIV_APOLLO0, 24, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_PCLK_DBG_APOLLO, "div_pclk_dbg_apollo", "div_apollo2",
> +			DIV_APOLLO0, 20, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_ATCLK_APOLLO, "div_atclk_apollo", "div_apollo2",
> +			DIV_APOLLO0, 16, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_PCLK_APOLLO, "div_pclk_apollo", "div_apollo2",
> +			DIV_APOLLO0, 12, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_ACLK_APOLLO, "div_aclk_apollo", "div_apollo2",
> +			DIV_APOLLO0, 8, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_APOLLO2, "div_apollo2", "div_apollo1",
> +			DIV_APOLLO0, 4, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_APOLLO1, "div_apollo1", "mout_apollo",
> +			DIV_APOLLO0, 0, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +
> +	/* DIV_APOLLO1 */
> +	DIV_F(CLK_DIV_SCLK_HPM_APOLLO, "div_sclk_hpm_apollo", "mout_apollo",
> +			DIV_APOLLO1, 4, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +	DIV_F(CLK_DIV_APOLLO_PLL, "div_apollo_pll", "mout_apollo",
> +			DIV_APOLLO1, 0, 3, CLK_GET_RATE_NOCACHE,
> +			CLK_DIVIDER_READ_ONLY),
> +};
> +
> +static struct samsung_gate_clock apollo_gate_clks[] __initdata = {
> +	/* ENABLE_ACLK_APOLLO */
> +	GATE(CLK_ACLK_ASATBSLV_APOLLO_3_CSSYS, "aclk_asatbslv_apollo_3_cssys",
> +			"div_atclk_apollo", ENABLE_ACLK_APOLLO,
> +			6, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_ASATBSLV_APOLLO_2_CSSYS, "aclk_asatbslv_apollo_2_cssys",
> +			"div_atclk_apollo", ENABLE_ACLK_APOLLO,
> +			5, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_ASATBSLV_APOLLO_1_CSSYS, "aclk_asatbslv_apollo_1_cssys",
> +			"div_atclk_apollo", ENABLE_ACLK_APOLLO,
> +			4, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_ASATBSLV_APOLLO_0_CSSYS, "aclk_asatbslv_apollo_0_cssys",
> +			"div_atclk_apollo", ENABLE_ACLK_APOLLO,
> +			3, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_ASYNCACES_APOLLO_CCI, "aclk_asyncaces_apollo_cci",
> +			"div_aclk_apollo", ENABLE_ACLK_APOLLO,
> +			2, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_AHB2APB_APOLLOP, "aclk_ahb2apb_apollop",
> +			"div_pclk_apollo", ENABLE_ACLK_APOLLO,
> +			1, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_ACLK_APOLLONP_200, "aclk_apollonp_200",
> +			"div_pclk_apollo", ENABLE_ACLK_APOLLO,
> +			0, CLK_IGNORE_UNUSED, 0),
> +
> +	/* ENABLE_PCLK_APOLLO */
> +	GATE(CLK_PCLK_ASAPBMST_CSSYS_APOLLO, "pclk_asapbmst_cssys_apollo",
> +			"div_pclk_dbg_apollo", ENABLE_PCLK_APOLLO,
> +			2, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_PCLK_PMU_APOLLO, "pclk_pmu_apollo", "div_pclk_apollo",
> +			ENABLE_PCLK_APOLLO, 1, CLK_IGNORE_UNUSED, 0),
> +	GATE(CLK_PCLK_SYSREG_APOLLO, "pclk_pmu_sysreg_apollo",

Isn't "pclk_pmu_sysreg_apollo" be named as "pclk_sysreg_apollo" to match 
with UM?


Rest changes in this file are OK. I confirmed clock tree and clock bits 
against UM.

Thanks,
Pankaj Dubey

  parent reply	other threads:[~2015-02-05  7:44 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-03  0:13 [PATCH v3 0/9] clk: samsung: Add clocks for remaining domains of Exynos5433 Chanwoo Choi
     [not found] ` <1422922437-7414-1-git-send-email-cw00.choi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2015-02-03  0:13   ` [PATCH v3 1/9] clk: samsung: exynos5433: Add clocks for CMU_APOLLO domain Chanwoo Choi
2015-02-03  0:13     ` Chanwoo Choi
     [not found]     ` <1422922437-7414-2-git-send-email-cw00.choi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2015-02-05  7:44       ` Pankaj Dubey [this message]
2015-02-05  7:44         ` [v3,1/9] " Pankaj Dubey
2015-02-05  9:56         ` Sylwester Nawrocki
     [not found]           ` <54D33E66.30909-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2015-02-05 13:42             ` Chanwoo Choi
2015-02-05 13:42               ` Chanwoo Choi
2015-02-04  1:12   ` [PATCH] clk: samsung: exynos5433: Move CLK_SCLK_HDMI_SPDIF_DISP clock to CMU_TOP domain Chanwoo Choi
2015-02-04  1:12     ` Chanwoo Choi
2015-02-03  0:13 ` [PATCH v3 2/9] clk: samsung: exynos5433: Add clocks for CMU_ATLAS domain Chanwoo Choi
     [not found]   ` <1422922437-7414-3-git-send-email-cw00.choi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2015-02-05  8:13     ` [v3,2/9] " Pankaj Dubey
2015-02-05  8:13       ` Pankaj Dubey
2015-02-03  0:13 ` [PATCH v3 3/9] clk: samsung: exynos5433: Add clocks for CMU_MSCL domain Chanwoo Choi
     [not found]   ` <1422922437-7414-4-git-send-email-cw00.choi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2015-02-05  8:30     ` [v3,3/9] " Pankaj Dubey
2015-02-05  8:30       ` Pankaj Dubey
2015-02-03  0:13 ` [PATCH v3 4/9] clk: samsung: exynos5433: Add clocks for CMU_MFC domain Chanwoo Choi
2015-02-03  0:13 ` [PATCH v3 5/9] clk: samsung: exynos5433: Add clocks for CMU_HEVC domain Chanwoo Choi
2015-02-03  0:13 ` [PATCH v3 6/9] clk: samsung: exynos5433: Add clocks for CMU_ISP domain Chanwoo Choi
2015-02-03  0:13 ` [PATCH v3 7/9] clk: samsung: exynos5433: Add clocks for CMU_CAM0 domain Chanwoo Choi
2015-02-03  0:13 ` [PATCH v3 8/9] clk: samsung: exynos5433: Add clocks for CMU_CAM1 domain Chanwoo Choi
2015-02-03  0:13 ` [PATCH v3 9/9] clk: samsung: Add CLKOUT driver support for Exynos5433 SoC Chanwoo Choi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54D31F67.1050701@samsung.com \
    --to=pankaj.dubey-sze3o3uu22jbdgjk7y7tuq@public.gmane.org \
    --cc=chanho61.park-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=cw00.choi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=inki.dae-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=kgene-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=sw0312.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=tomasz.figa-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.