2015-02-23 21:30:32

by Stephen Boyd

Subject: [PATCH] clk: qcom: Fix i2c frequency table

PXO is 25MHz, not 27MHz. Fix the table.

Fixes: 24d8fba44af3 "clk: qcom: Add support for IPQ8064's global
clock controller (GCC)"

Signed-off-by: Stephen Boyd <[email protected]>
---
drivers/clk/qcom/gcc-ipq806x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/qcom/gcc-ipq806x.c b/drivers/clk/qcom/gcc-ipq806x.c
index cbdc31dea7f4..a015bb06c09b 100644
--- a/drivers/clk/qcom/gcc-ipq806x.c
+++ b/drivers/clk/qcom/gcc-ipq806x.c
@@ -525,8 +525,8 @@ static struct freq_tbl clk_tbl_gsbi_qup[] = {
{ 10800000, P_PXO, 1, 2, 5 },
{ 15060000, P_PLL8, 1, 2, 51 },
{ 24000000, P_PLL8, 4, 1, 4 },
+ { 25000000, P_PXO, 1, 0, 0 },
{ 25600000, P_PLL8, 1, 1, 15 },
- { 27000000, P_PXO, 1, 0, 0 },
{ 48000000, P_PLL8, 4, 1, 2 },
{ 51200000, P_PLL8, 1, 2, 15 },
{ }
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

2015-02-26 20:42:00

by Andy Gross

Subject: Re: [PATCH] clk: qcom: Fix i2c frequency table

On Mon, Feb 23, 2015 at 01:30:28PM -0800, Stephen Boyd wrote:
> PXO is 25MHz, not 27MHz. Fix the table.
>
> Fixes: 24d8fba44af3 "clk: qcom: Add support for IPQ8064's global
> clock controller (GCC)"
>
> Signed-off-by: Stephen Boyd <[email protected]>

Reviewed-by: Andy Gross <[email protected]>
Tested-by: Andy Gross <[email protected]>

--
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

2015-02-27 01:15:59

by Mike Turquette

Subject: Re: [PATCH] clk: qcom: Fix i2c frequency table

Quoting Stephen Boyd (2015-02-23 13:30:28)
> PXO is 25MHz, not 27MHz. Fix the table.
>
> Fixes: 24d8fba44af3 "clk: qcom: Add support for IPQ8064's global
> clock controller (GCC)"
>
> Signed-off-by: Stephen Boyd <[email protected]>

Applied to clk-next.

Regards,
Mike

> ---
> drivers/clk/qcom/gcc-ipq806x.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/clk/qcom/gcc-ipq806x.c b/drivers/clk/qcom/gcc-ipq806x.c
> index cbdc31dea7f4..a015bb06c09b 100644
> --- a/drivers/clk/qcom/gcc-ipq806x.c
> +++ b/drivers/clk/qcom/gcc-ipq806x.c
> @@ -525,8 +525,8 @@ static struct freq_tbl clk_tbl_gsbi_qup[] = {
> { 10800000, P_PXO, 1, 2, 5 },
> { 15060000, P_PLL8, 1, 2, 51 },
> { 24000000, P_PLL8, 4, 1, 4 },
> + { 25000000, P_PXO, 1, 0, 0 },
> { 25600000, P_PLL8, 1, 1, 15 },
> - { 27000000, P_PXO, 1, 0, 0 },
> { 48000000, P_PLL8, 4, 1, 2 },
> { 51200000, P_PLL8, 1, 2, 15 },
> { }
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>