Thread (24 messages) 24 messages, 6 authors, 2012-08-01

[PATCH 1/3] Move FIMD register headers to include/video/

From: m.szyprowski@samsung.com (Marek Szyprowski)
Date: 2012-07-31 06:28:13
Also in: dri-devel, linux-fbdev, linux-samsung-soc

Hello,

On Tuesday, July 31, 2012 2:48 AM Jingoo Han wrote:
On Monday, July 30, 2012 8:16 PM, Leela Krishna Amudala wrote:
quoted
Hello Jingoo Han,

On Mon, Jul 30, 2012 at 2:23 PM, Jingoo Han [off-list ref] wrote:
quoted
On Monday, July 30, 2012 5:45 PM, Leela Krishna Amudala wrote:
quoted
Moved the contents of regs-fb-v4.h and regs-fb.h from arch side
to include/video/samsung_fimd.h

Signed-off-by: Leela Krishna Amudala <redacted>
---
 arch/arm/plat-samsung/include/plat/regs-fb-v4.h |  159 -------
 arch/arm/plat-samsung/include/plat/regs-fb.h    |  403 -----------------
 include/video/samsung_fimd.h                    |  533 +++++++++++++++++++++++
 3 files changed, 533 insertions(+), 562 deletions(-)
 delete mode 100644 arch/arm/plat-samsung/include/plat/regs-fb-v4.h
 delete mode 100644 arch/arm/plat-samsung/include/plat/regs-fb.h
 create mode 100644 include/video/samsung_fimd.h

+*/
+
+/*FIMD V8 REG OFFSET */
+#define FIMD_V8_VIDTCON0     (0x20010)
+#define FIMD_V8_VIDTCON1     (0x20014)
+#define FIMD_V8_VIDTCON2     (0x20018)
+#define FIMD_V8_VIDTCON3     (0x2001C)
+#define FIMD_V8_VIDCON1              (0x20004)

How about using soc_is_exynos5250()?

+#define VIDTCON0				(soc_is_exynos5250() ? \
+						(0x20010) : (0x10))

In this case, the FIMD driver does not need to change.
Also, one binary is available.
Please don't mix two methods of runtime detection. FIMD driver (s3c-fb) already
has runtime hw detection based on platform device id. Adding such detection for 
exynos5 to DRM FIMD driver should not be a big issue too.

Best regards
-- 
Marek Szyprowski
Samsung Poland R&D Center
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help