From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v7 2/2] video: backlight: support s6e8ax0 panel driver based on MIPI DSI
Date: Thu, 26 Jan 2012 15:52:59 +0000 [thread overview]
Message-ID: <1327593179.2843.5.camel@lappy> (raw)
In-Reply-To: <4F17AA14.8030100@samsung.com>
[-- Attachment #1: Type: text/plain, Size: 1331 bytes --]
Hi,
On Thu, 2012-01-19 at 14:28 +0900, Donghwa Lee wrote:
> This patch is amoled panel driver based MIPI DSI interface.
> S6E8AX0 means it may includes many other ldi controllers, for example,
> S6E8AA0, S6E8AB0, and so on.
>
> This patch can be modified depending on each panel properites. For example,
> second parameter of panel condition register can be changed depending on
> ldi controller or amoled type.
>
> Changes since v6:
> - change data type from const to static const
>
> Signed-off-by: Donghwa Lee <dh09.lee@samsung.com>
> Signed-off-by: Inki Dae <inki.dae@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> ---
> drivers/video/backlight/Kconfig | 7 +
> drivers/video/backlight/Makefile | 1 +
> drivers/video/backlight/s6e8ax0.c | 898 +++++++++++++++++++++++++++++++++++++
> drivers/video/backlight/s6e8ax0.h | 21 +
> 4 files changed, 927 insertions(+), 0 deletions(-)
> create mode 100644 drivers/video/backlight/s6e8ax0.c
> create mode 100644 drivers/video/backlight/s6e8ax0.h
Why is this in backlight directory? It's a panel driver, not a backlight
driver.
And the driver is s5p specific, wouldn't a directory like
drivers/video/s5p/, where both the s5p DSI driver from the previous
patch and this panel driver could be?
Tomi
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: tomi.valkeinen@ti.com (Tomi Valkeinen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v7 2/2] video: backlight: support s6e8ax0 panel driver based on MIPI DSI
Date: Thu, 26 Jan 2012 17:52:59 +0200 [thread overview]
Message-ID: <1327593179.2843.5.camel@lappy> (raw)
In-Reply-To: <4F17AA14.8030100@samsung.com>
Hi,
On Thu, 2012-01-19 at 14:28 +0900, Donghwa Lee wrote:
> This patch is amoled panel driver based MIPI DSI interface.
> S6E8AX0 means it may includes many other ldi controllers, for example,
> S6E8AA0, S6E8AB0, and so on.
>
> This patch can be modified depending on each panel properites. For example,
> second parameter of panel condition register can be changed depending on
> ldi controller or amoled type.
>
> Changes since v6:
> - change data type from const to static const
>
> Signed-off-by: Donghwa Lee <dh09.lee@samsung.com>
> Signed-off-by: Inki Dae <inki.dae@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> ---
> drivers/video/backlight/Kconfig | 7 +
> drivers/video/backlight/Makefile | 1 +
> drivers/video/backlight/s6e8ax0.c | 898 +++++++++++++++++++++++++++++++++++++
> drivers/video/backlight/s6e8ax0.h | 21 +
> 4 files changed, 927 insertions(+), 0 deletions(-)
> create mode 100644 drivers/video/backlight/s6e8ax0.c
> create mode 100644 drivers/video/backlight/s6e8ax0.h
Why is this in backlight directory? It's a panel driver, not a backlight
driver.
And the driver is s5p specific, wouldn't a directory like
drivers/video/s5p/, where both the s5p DSI driver from the previous
patch and this panel driver could be?
Tomi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120126/73d361ca/attachment.sig>
next prev parent reply other threads:[~2012-01-26 15:52 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-19 5:28 [PATCH v7 2/2] video: backlight: support s6e8ax0 panel driver based on MIPI DSI Donghwa Lee
2012-01-19 5:28 ` Donghwa Lee
2012-01-26 15:52 ` Tomi Valkeinen [this message]
2012-01-26 15:52 ` Tomi Valkeinen
2012-01-27 2:07 ` Inki Dae
2012-01-27 2:07 ` Inki Dae
2012-01-27 3:00 ` Jingoo Han
2012-01-27 3:00 ` Jingoo Han
2012-02-01 6:11 ` Tomi Valkeinen
2012-02-01 6:11 ` Tomi Valkeinen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1327593179.2843.5.camel@lappy \
--to=tomi.valkeinen@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.