Re: [PATCH] clk: qoriq: fix a register offset error

From: Scott Wood
Date: Mon Aug 15 2016 - 12:32:16 EST


On Mon, 2016-08-15 at 15:28 +0800, yuantian.tang@xxxxxxx wrote:
> From: Tang Yuantian <Yuantian.Tang@xxxxxxx>
>
> The offset of Core Cluster clock control/status register
> on cluster group V3 version is different from others, and
> should be plus 0x70000.
>
> Signed-off-by: Tang Yuantian <yuantian.tang@xxxxxxx>
> ---
> Âdrivers/clk/clk-qoriq.c | 6 +++++-
> Â1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/clk/clk-qoriq.c b/drivers/clk/clk-qoriq.c
> index f4c455a..05e416c 100644
> --- a/drivers/clk/clk-qoriq.c
> +++ b/drivers/clk/clk-qoriq.c
> @@ -774,7 +774,11 @@ static struct clk * __init create_one_cmux(struct
> clockgen *cg, int idx)
> Â if (!hwc)
> Â return NULL;
> Â
> - hwc->reg = cg->regs + 0x20 * idx;
> + if (cg->info.flags & CG_VER3)
> + hwc->reg = cg->regs + 0x70000 + 0x20 * idx;
> + else
> + hwc->reg = cg->regs + 0x20 * idx;
> +
> Â hwc->info = cg->info.cmux_groups[cg->info.cmux_to_group[idx]];

Reviewed-by: Scott Wood <oss@xxxxxxxxxxxx>

-Scott