Commit de1121fd authored by Daniel Stone's avatar Daniel Stone Committed by Tony Lindgren

ARM: OMAP2: Fix definition of SGX clock register bits

The GFX/SGX functional and interface clocks have different masks, for
some unknown reason, so split EN_SGX_SHIFT into one each for fclk and
iclk.

Correct according to the TRM and the far more important 'does this
actually work at all?' metric.
Signed-off-by: default avatarDaniel Stone <daniel.stone@nokia.com>
Signed-off-by: default avatarTony Lindgren <tony@atomide.com>
parent 2fe67a56
...@@ -1378,7 +1378,7 @@ static struct clk sgx_fck = { ...@@ -1378,7 +1378,7 @@ static struct clk sgx_fck = {
.name = "sgx_fck", .name = "sgx_fck",
.init = &omap2_init_clksel_parent, .init = &omap2_init_clksel_parent,
.enable_reg = _OMAP34XX_CM_REGADDR(OMAP3430ES2_SGX_MOD, CM_FCLKEN), .enable_reg = _OMAP34XX_CM_REGADDR(OMAP3430ES2_SGX_MOD, CM_FCLKEN),
.enable_bit = OMAP3430ES2_EN_SGX_SHIFT, .enable_bit = OMAP3430ES2_CM_FCLKEN_SGX_EN_SGX_SHIFT,
.clksel_reg = _OMAP34XX_CM_REGADDR(OMAP3430ES2_SGX_MOD, CM_CLKSEL), .clksel_reg = _OMAP34XX_CM_REGADDR(OMAP3430ES2_SGX_MOD, CM_CLKSEL),
.clksel_mask = OMAP3430ES2_CLKSEL_SGX_MASK, .clksel_mask = OMAP3430ES2_CLKSEL_SGX_MASK,
.clksel = sgx_clksel, .clksel = sgx_clksel,
...@@ -1391,7 +1391,7 @@ static struct clk sgx_ick = { ...@@ -1391,7 +1391,7 @@ static struct clk sgx_ick = {
.name = "sgx_ick", .name = "sgx_ick",
.parent = &l3_ick, .parent = &l3_ick,
.enable_reg = _OMAP34XX_CM_REGADDR(OMAP3430ES2_SGX_MOD, CM_ICLKEN), .enable_reg = _OMAP34XX_CM_REGADDR(OMAP3430ES2_SGX_MOD, CM_ICLKEN),
.enable_bit = OMAP3430ES2_EN_SGX_SHIFT, .enable_bit = OMAP3430ES2_CM_ICLKEN_SGX_EN_SGX_SHIFT,
.flags = CLOCK_IN_OMAP3430ES2, .flags = CLOCK_IN_OMAP3430ES2,
.clkdm = { .name = "sgx_clkdm" }, .clkdm = { .name = "sgx_clkdm" },
.recalc = &followparent_recalc, .recalc = &followparent_recalc,
......
...@@ -340,8 +340,12 @@ ...@@ -340,8 +340,12 @@
#define OMAP3430ES1_CLKACTIVITY_GFX_MASK (1 << 0) #define OMAP3430ES1_CLKACTIVITY_GFX_MASK (1 << 0)
/* CM_FCLKEN_SGX */ /* CM_FCLKEN_SGX */
#define OMAP3430ES2_EN_SGX_SHIFT 1 #define OMAP3430ES2_CM_FCLKEN_SGX_EN_SGX_SHIFT 1
#define OMAP3430ES2_EN_SGX_MASK (1 << 1) #define OMAP3430ES2_CM_FCLKEN_SGX_EN_SGX_MASK (1 << 1)
/* CM_ICLKEN_SGX */
#define OMAP3430ES2_CM_ICLKEN_SGX_EN_SGX_SHIFT 0
#define OMAP3430ES2_CM_ICLKEN_SGX_EN_SGX_MASK (1 << 0)
/* CM_CLKSEL_SGX */ /* CM_CLKSEL_SGX */
#define OMAP3430ES2_CLKSEL_SGX_SHIFT 0 #define OMAP3430ES2_CLKSEL_SGX_SHIFT 0
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment