Inki Dae wrote:

Hello, Mr. Kukjin.

Hi,

I know we had a discussion about your patch below.
after that, did you have some discussion about that?
If not, Paul advised to use clkdev lookup.

Please, refer to below mail.

http://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg03448.html

I think it's different with your approach even though the purpose is similar
and I know, Mr. Han explained about clock configurable by call.

Paul, how do you think about this?

Personally, if we want to support S5PV310 FIMD now, this patch is good to
support it and to us now.
Of course, I think, we can upgrade it later.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim [off-list ref], Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

quoted

-----Original Message-----
From: linux-fbdev-owner@vger.kernel.org [mailto:linux-fbdev-
owner@vger.kernel.org] On Behalf Of Kukjin Kim
Sent: Tuesday, January 04, 2011 5:06 PM
To: linux-samsung-soc@vger.kernel.org; linux-fbdev@vger.kernel.org
Cc: ben-linux@fluff.org; akpm@linux-foundation.org; lethal@linux-sh.org;
Jonghun Han; Sangbeom Kim; InKi Dae; Kukjin Kim
Subject: [PATCH RE-SEND] s3c-fb: Add support S5PV310 FIMD

From: Jonghun Han <redacted>

This patch adds struct s3c_fb_driverdata s3c_fb_data_s5pv310 for S5PV310
and S5PC210. The clk_type is added to distinguish clock type in it and
lcd_clk is added in structure s3c_fb to calculate divider for lcd panel.

Please refer to below diagrams about clocks of FIMD IP. FIMD driver

needs

quoted

two clocks for FIMD IP and LCD pixel clock. Actually, the LCD pixel

clock

quoted

can be selected from 1.clk 'lcd' and 2.SCLK_FIMD before S5PV310. But

from

quoted

S5PV310, the 2.SCLK_FIMD can be used only for source of LCD pixel clock.

FIMD_CLK_TYPE0:
           ------------------------------------
                      dsys bus
           ----------------+-------------------
                           |
                           |1.clk 'lcd'
                           |
                           | FIMD block
                       +---+-----------+
4.mout_mpll |\         |   |           |
    --------|m|        | +-+-+ +----+  |
            |u|-+      | |   +-+core|  |
            |x| |      | |     +----+  |
            |/  |      | | |\          |
                |      | +-|m|  +---+  |
                |      |   |u|--+div|  |
                +------+---|x|  +---+  |
           2.SCLK_FIMD |   |/     |    |
                       |          |    |
                       +----------+----+
                                  |
           inside of SoC          |
           -----------------------+--------------------------
           outside of SoC         |
                                  | 3.LCD pixel clock
                                  |
                          +--------------+
                          | LCD module   |
                          +--------------+

FIMD_CLK_TYPE1:
           ------------------------------------
                      dsys bus
           ----------------+-------------------
                           |
                           |1.clk 'fimd'
                           |
                           | FIMD block
                       +---+-----------+
4.mout_mpll |\         |   |           |
    --------|m|        |   |   +----+  |
            |u|-+      |   +---+core|  |
            |x| |      |       +----+  |
            |/  |      |               |
                |      |        +---+  |
                |      |     +--+div|  |
                +------+-----+  +---+  |
           2.SCLK_FIMD |          |    |
                       |          |    |
                       +----------+----+
                                  |
           inside of SoC          |
           -----------------------+--------------------------
           outside of SoC         |
                                  | 3.LCD pixel clock
                                  |
                          +--------------+
                          | LCD module   |
                          +--------------+

Signed-off-by: Jonghun Han <redacted>
Signed-off-by: Sangbeom Kim <redacted>
Cc: InKi Dae <inki.dae@samsung.com>
Cc: Ben Dooks <ben-linux@fluff.org>
Signed-off-by: Kukjin Kim <redacted>
---
Hi Paul,

I and Mr. Han, Jonghun sent below patch several weeks ago.
But I couldn't find it in your tree...
(Re-made against on your latest fbdev-2.6.git #master)

I think this should be merged for supporting S5PV310/S5PC210 frame

buffer.

quoted

So could you please let me know your opinion or plan about this?

NOTE: Needs following platform device patches for S5PV310/S5PC210 frame
buffer.
And I already applied that in my tree as S5P SoCs architecture

maintainer.

quoted

0001-ARM-S5PV310-Add-FIMD-resource-definition.patch
0002-ARM-S5PV310-Add-platform-device-and-helper-functio.patch
0004-ARM-S5PV310-Add-support-FIMD0-and-LTE480WV-LCD-for.patch

 drivers/video/Kconfig  |    2 +-
 drivers/video/s3c-fb.c |  125
+++++++++++++++++++++++++++++++++++++++++++-----
 2 files changed, 113 insertions(+), 14 deletions(-)