diff mbox

[1/2] clk: samsung: Add GPLL freq table for exynos5250 SoC

Message ID 1376301734-21847-2-git-send-email-vikas.sajjan@linaro.org (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Vikas C Sajjan Aug. 12, 2013, 10:02 a.m. UTC
Adds GPLL freq table for exynos5250 SoC.

Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org>
---
 drivers/clk/samsung/clk-exynos5250.c |   19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

Comments

Tomasz Figa Aug. 28, 2013, 8:06 a.m. UTC | #1
Hi Vikas,

On Monday 12 of August 2013 15:32:13 Vikas Sajjan wrote:
> Adds GPLL freq table for exynos5250 SoC.
> 
> Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org>
> ---
>  drivers/clk/samsung/clk-exynos5250.c |   19 ++++++++++++++++++-
>  1 file changed, 18 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/samsung/clk-exynos5250.c
> b/drivers/clk/samsung/clk-exynos5250.c index a9916a4..c400e82 100644
> --- a/drivers/clk/samsung/clk-exynos5250.c
> +++ b/drivers/clk/samsung/clk-exynos5250.c
> @@ -494,6 +494,21 @@ static struct samsung_gate_clock
> exynos5250_gate_clks[] __initdata = { GATE(g2d, "g2d", "aclk200",
> GATE_IP_ACP, 3, 0, 0),
>  };
> 
> +static struct samsung_pll_rate_table gpll_24mhz_tbl[] __initdata = {
> +	/* sorted in descending order */
> +	/* PLL_35XX_RATE(rate, m, p, s) */
> +	PLL_35XX_RATE(1400000000, 175, 3, 0), /* for 466MHz */
> +	PLL_35XX_RATE(800000000, 100, 3, 0),  /* for 400MHz, 200MHz */
> +	PLL_35XX_RATE(667000000, 389, 7, 1),  /* for 333MHz, 222MHz, 
166MHz */

Frequency generated by this entry is not exactly 667 MHz, but rather 
666857142 Hz. This must be reflected by the rate field or PLL rate setting 
won't work correctly otherwise.

> +	PLL_35XX_RATE(600000000, 200, 4, 1),  /* for 300MHz, 200MHz, 
150MHz
> */ +	PLL_35XX_RATE(533000000, 533, 12, 1), /* for 533MHz, 266MHz,
> 133MHz */ +	PLL_35XX_RATE(450000000, 450, 12, 1), /* for 450Hz */
> +	PLL_35XX_RATE(400000000, 100, 3, 1),
> +	PLL_35XX_RATE(333000000, 222, 4, 2),
> +	PLL_35XX_RATE(200000000, 100, 3, 2),
> +	{ },
> +};
> +
>  static struct samsung_pll_rate_table vpll_24mhz_tbl[] __initdata = {
>  	/* sorted in descending order */
>  	/* PLL_36XX_RATE(rate, m, p, s, k) */
> @@ -565,8 +580,10 @@ static void __init exynos5250_clk_init(struct
> device_node *np)
> 
>  	fin_pll_rate = _get_rate("fin_pll");
> 
> -	if (fin_pll_rate == 24 * MHZ)
> +	if (fin_pll_rate == 24 * MHZ) {
>  		exynos5250_plls[epll].rate_table = epll_24mhz_tbl;
> +		exynos5250_plls[gpll].rate_table = gpll_24mhz_tbl;
> +	}

Also you could rebase this series on my patches[1] cleaning several things 
up, to simplify this table setting code a bit.

[1] - http://www.spinics.net/lists/arm-kernel/msg269848.html

Best regards,
Tomasz

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/clk/samsung/clk-exynos5250.c b/drivers/clk/samsung/clk-exynos5250.c
index a9916a4..c400e82 100644
--- a/drivers/clk/samsung/clk-exynos5250.c
+++ b/drivers/clk/samsung/clk-exynos5250.c
@@ -494,6 +494,21 @@  static struct samsung_gate_clock exynos5250_gate_clks[] __initdata = {
 	GATE(g2d, "g2d", "aclk200", GATE_IP_ACP, 3, 0, 0),
 };
 
+static struct samsung_pll_rate_table gpll_24mhz_tbl[] __initdata = {
+	/* sorted in descending order */
+	/* PLL_35XX_RATE(rate, m, p, s) */
+	PLL_35XX_RATE(1400000000, 175, 3, 0), /* for 466MHz */
+	PLL_35XX_RATE(800000000, 100, 3, 0),  /* for 400MHz, 200MHz */
+	PLL_35XX_RATE(667000000, 389, 7, 1),  /* for 333MHz, 222MHz, 166MHz */
+	PLL_35XX_RATE(600000000, 200, 4, 1),  /* for 300MHz, 200MHz, 150MHz */
+	PLL_35XX_RATE(533000000, 533, 12, 1), /* for 533MHz, 266MHz, 133MHz */
+	PLL_35XX_RATE(450000000, 450, 12, 1), /* for 450Hz */
+	PLL_35XX_RATE(400000000, 100, 3, 1),
+	PLL_35XX_RATE(333000000, 222, 4, 2),
+	PLL_35XX_RATE(200000000, 100, 3, 2),
+	{ },
+};
+
 static struct samsung_pll_rate_table vpll_24mhz_tbl[] __initdata = {
 	/* sorted in descending order */
 	/* PLL_36XX_RATE(rate, m, p, s, k) */
@@ -565,8 +580,10 @@  static void __init exynos5250_clk_init(struct device_node *np)
 
 	fin_pll_rate = _get_rate("fin_pll");
 
-	if (fin_pll_rate == 24 * MHZ)
+	if (fin_pll_rate == 24 * MHZ) {
 		exynos5250_plls[epll].rate_table = epll_24mhz_tbl;
+		exynos5250_plls[gpll].rate_table = gpll_24mhz_tbl;
+	}
 
 	vpllsrc = __clk_lookup("mout_vpllsrc");
 	if (vpllsrc)