DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:date:message-id:subject:from:to:cc:content-type
         :content-transfer-encoding;
        b=gyh8mw3yXEZOySbF2TVHfqRXe/y49uAo4YFTWz5UAMgrb9jumq5xRN20ju7NC5aKYw
         pB5pYJgjhPRoEvDHeVXNHo+AsW6vxbbM7XrDYZqa+zKhG6NVh7Yi06jzQw5773+8n8Eg
         7tJy+6BzaZjBKIfTUi1/zUdw0KOgC0hNHs19k=
MIME-Version: 1.0
Date: Mon, 8 Jun 2009 13:53:32 +0900
Message-ID: <90b950fc0906072153j6308059eq4d6f864c97a2003d@mail.gmail.com>
Subject: Samsung SoC Framebuffer Driver issue.
From: InKi Dae <daeinki@gmail.com>
To: ben@simtec.co.uk
Cc: linux-kernel@vger.kernel.org, linux-fbdev-devel@lists.sourceforge.net,
       Kyungmin Park <kmpark@infradead.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1887
Lines: 51

Hello,  Ben  Dooks.

I am trying to develop Samsung SoC Framebuffer Driver for S5PC100 but
I faced some problems about design. so I have some questions about those.

there are some differences between LCD Controller registers of S3C64XX
and S5PC100 like the following.

. regiters not used by S3C64XX were added to S5PC100 also BPP MODE.
. WindowX Palette Data Index registers were removed from lcd
controller of S3C64XX and Palette Memory Registers were added to one
of S5PC100


1.
Your framebuffer driver, drivers/video/s3c-fb.c, is related to only S3C64XX.
and I'd like to divide framebuffer driver into common logical driver
for supporting S3C64XX and S5PC100 and S5PC100 specific driver.
but your driver needs to modifying because calling register control
commands in s3c-fb.c directly.

How do you think about that ?

2.
S3C64XX LCD Controller's registers are similar with S5PC100's, so I
wouldn't modify register prefix.
but some registers should added to definition file
(arch/arm/mach-s3c6400/include/mach/regs-fb.h).

which way do you prefer adding some registers to
arch/arm/mach-s3c6400/include/mach/regs-fb.h or creating new
definition file for S5PC100 ?

If you give me your comments for the issues above, I will modify
Samsung SoC Framebuffer Driver to be supported for S3C64XX and
S5PC100.
In the result, I expect that your driver will be divided into logical
driver and specific driver supporting S3C64XX and S5PC100 or either
S3C64XX or S5PC100.
If the driver would be completed, logical driver, s3c-fb.c, would
support S3C64XX, S5PC100 and also S5PC110.

Please give me your comments and advices.

Best Regards,
InKi Dae.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/