[PATCH 8/9] s3c-fb: Add SHADOWCON shadow register locking support for S5PV210
From: m.szyprowski@samsung.com (Marek Szyprowski)
Date: 2010-06-16 09:34:51
Also in:
linux-fbdev, linux-samsung-soc
Subsystem:
arm port, framebuffer layer, the rest · Maintainers:
Russell King, Helge Deller, Linus Torvalds
From: Pawel Osciak <redacted> S5PV210 allows per-window locking of register value updates from shadow registers. Signed-off-by: Pawel Osciak <redacted> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> --- arch/arm/plat-samsung/include/plat/regs-fb.h | 3 ++ drivers/video/s3c-fb.c | 46 +++++++++++++++++++++++-- 2 files changed, 45 insertions(+), 4 deletions(-)
diff --git a/arch/arm/plat-samsung/include/plat/regs-fb.h b/arch/arm/plat-samsung/include/plat/regs-fb.h
index 5bcdd09..da54b64 100644
--- a/arch/arm/plat-samsung/include/plat/regs-fb.h
+++ b/arch/arm/plat-samsung/include/plat/regs-fb.h@@ -218,6 +218,9 @@ #define WINCON1_BPPMODE_25BPP_A1888 (0xd << 2) #define WINCON1_BPPMODE_28BPP_A4888 (0xd << 2) +/* S5PV210 */ +#define SHADOWCON (0x34) +#define SHADOWCON_WINx_PROTECT(_win) (1 << (10 + _win)) #define VIDOSDxA_TOPLEFT_X_MASK (0x7ff << 11) #define VIDOSDxA_TOPLEFT_X_SHIFT (11)
diff --git a/drivers/video/s3c-fb.c b/drivers/video/s3c-fb.c
index 0a93fca..5e14f0a 100644
--- a/drivers/video/s3c-fb.c
+++ b/drivers/video/s3c-fb.c@@ -79,6 +79,7 @@ struct s3c_fb; * @osd: The base for the OSD registers. * @palette: Address of palette memory, or 0 if none. * @has_prtcon: Set if has PRTCON register. + * @has_shadowcon: Set if has SHADOWCON register. */ struct s3c_fb_variant { unsigned int is_2443:1;
@@ -95,6 +96,7 @@ struct s3c_fb_variant { unsigned short palette[S3C_FB_MAX_WIN]; unsigned int has_prtcon:1; + unsigned int has_shadowcon:1; }; /**
@@ -620,6 +622,43 @@ static inline unsigned int chan_to_field(unsigned int chan, } /** + * shadow_protect() - disable updating values from shadow registers@vsync + * + * @sfb: The hardware information + * @win_no: Window number to protect registers for + */ +static void shadow_protect(struct s3c_fb_win *win) +{ + struct s3c_fb *sfb = win->parent; + + if (sfb->variant.has_prtcon) + writel(PRTCON_PROTECT, sfb->regs + PRTCON); + else if (sfb->variant.has_shadowcon) + writel(SHADOWCON_WINx_PROTECT(win->index), + sfb->regs + SHADOWCON); +} + +/** + * shadow_noprotect() - re-enable updating values from shadow registers at vsync + * + * @sfb: The hardware information + * @win_no: Window number to re-enable updates from shadow registers + */ +static void shadow_noprotect(struct s3c_fb_win *win) +{ + struct s3c_fb *sfb = win->parent; + u32 reg; + + if (sfb->variant.has_prtcon) { + writel(0, sfb->regs + PRTCON); + } else if (sfb->variant.has_shadowcon) { + reg = readl(sfb->regs + SHADOWCON); + writel(reg & ~SHADOWCON_WINx_PROTECT(win->index), + sfb->regs + SHADOWCON); + } +} + +/** * s3c_fb_setcolreg() - framebuffer layer request to change palette. * @regno: The palette index to change. * @red: The red field for the palette data.
@@ -811,16 +850,14 @@ static int s3c_fb_pan_display(struct fb_var_screeninfo *var, /* Temporarily turn off per-vsync update from shadow registers until * both start and end addresses are updated to prevent corruption */ - if (sfb->variant.has_prtcon) - writel(PRTCON_PROTECT, sfb->regs + PRTCON); + shadow_protect(win); writel(info->fix.smem_start + start_byte_offset, buf + sfb->variant.buf_start); writel(info->fix.smem_start + end_byte_offset, buf + sfb->variant.buf_end); - if (sfb->variant.has_prtcon) - writel(0, sfb->regs + PRTCON); + shadow_noprotect(win); return 0; }
@@ -1534,6 +1571,7 @@ static struct s3c_fb_driverdata s3c_fb_data_s5pv210 __devinitdata = { [4] = 0x3400, }, + .has_shadowcon = 1, }, .win[0] = &s3c_fb_data_64xx_wins[0], .win[1] = &s3c_fb_data_64xx_wins[1],
--
1.7.1.240.g225c