Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1339351 > unrolled thread
| Started by | Jitao Shi <jitao.shi@mediatek.com> |
|---|---|
| First post | 2016-02-22 12:10 +0100 |
| Last post | 2016-02-22 12:50 +0100 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input Jitao Shi <jitao.shi@mediatek.com> - 2016-02-22 12:10 +0100
Re: [PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input Thierry Reding <thierry.reding@gmail.com> - 2016-02-22 12:50 +0100
| From | Jitao Shi <jitao.shi@mediatek.com> |
|---|---|
| Date | 2016-02-22 12:10 +0100 |
| Subject | [PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input |
| Message-ID | <r4Wxk-4RK-33@gated-at.bofh.it> |
The LG lp120up1 TFT LCD panel with eDP interface is a 12.0" 1920x1280
panel, which can be supported by the simple panel driver
Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
---
Changes since v1:
- Add eDP panel type in comment msg
- Fixed comment msg with 72 characters width
---
drivers/gpu/drm/panel/panel-simple.c | 26 ++++++++++++++++++++++++++
1 file changed, 26 insertions(+)
diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
index f88a631..2030c37 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -982,6 +982,29 @@ static const struct panel_desc lg_lb070wv8 = {
.bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG,
};
+static const struct drm_display_mode lg_lp120up1_mode = {
+ .clock = 162300,
+ .hdisplay = 1920,
+ .hsync_start = 1920 + 40,
+ .hsync_end = 1920 + 40 + 40,
+ .htotal = 1920 + 40 + 40+ 80,
+ .vdisplay = 1280,
+ .vsync_start = 1280 + 4,
+ .vsync_end = 1280 + 4 + 4,
+ .vtotal = 1280 + 4 + 4 + 12,
+ .vrefresh = 60,
+};
+
+static const struct panel_desc lg_lp120up1 = {
+ .modes = &lg_lp120up1_mode,
+ .num_modes = 1,
+ .bpc = 8,
+ .size = {
+ .width = 267,
+ .height = 183,
+ },
+};
+
static const struct drm_display_mode lg_lp129qe_mode = {
.clock = 285250,
.hdisplay = 2560,
@@ -1256,6 +1279,9 @@ static const struct of_device_id platform_of_match[] = {
.compatible = "lg,lb070wv8",
.data = &lg_lb070wv8,
}, {
+ .compatible = "lg,lp120up1",
+ .data = &lg_lp120up1,
+ }, {
.compatible = "lg,lp129qe",
.data = &lg_lp129qe,
}, {
--
1.7.9.5
[toc] | [next] | [standalone]
| From | Thierry Reding <thierry.reding@gmail.com> |
|---|---|
| Date | 2016-02-22 12:50 +0100 |
| Subject | Re: [PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input |
| Message-ID | <r4Xa2-5ds-9@gated-at.bofh.it> |
| In reply to | #1339351 |
[Multipart message — attachments visible in raw view] — view raw
On Mon, Feb 22, 2016 at 04:42:54PM +0530, sanjeev sharma wrote:
> Hello Jitao,
>
> Can't we add this panel information in device tree file instead inside the
> device driver ?
We could, but that would be redundant. Panels need to be represented by
a specific compatible string anyway, and that compatible string implies
the video timings, bits-per-color, width and height.
Also there's more to panels than just the timings or dimensions. Power
up and power down sequences are also implied by the compatible stirng.
Describing all of that in the device tree was at some point attempted,
but in the end it turned out too complicated and we ended up with what
we have now.
Thierry
> On Mon, Feb 22, 2016 at 4:31 PM, Jitao Shi <jitao.shi@mediatek.com> wrote:
>
> > The LG lp120up1 TFT LCD panel with eDP interface is a 12.0" 1920x1280
> > panel, which can be supported by the simple panel driver
> >
> > Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
> > ---
> > Changes since v1:
> > - Add eDP panel type in comment msg
> > - Fixed comment msg with 72 characters width
> > ---
> > drivers/gpu/drm/panel/panel-simple.c | 26 ++++++++++++++++++++++++++
> > 1 file changed, 26 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/panel/panel-simple.c
> > b/drivers/gpu/drm/panel/panel-simple.c
> > index f88a631..2030c37 100644
> > --- a/drivers/gpu/drm/panel/panel-simple.c
> > +++ b/drivers/gpu/drm/panel/panel-simple.c
> > @@ -982,6 +982,29 @@ static const struct panel_desc lg_lb070wv8 = {
> > .bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG,
> > };
> >
> > +static const struct drm_display_mode lg_lp120up1_mode = {
> > + .clock = 162300,
> > + .hdisplay = 1920,
> > + .hsync_start = 1920 + 40,
> > + .hsync_end = 1920 + 40 + 40,
> > + .htotal = 1920 + 40 + 40+ 80,
> > + .vdisplay = 1280,
> > + .vsync_start = 1280 + 4,
> > + .vsync_end = 1280 + 4 + 4,
> > + .vtotal = 1280 + 4 + 4 + 12,
> > + .vrefresh = 60,
> > +};
> > +
> > +static const struct panel_desc lg_lp120up1 = {
> > + .modes = &lg_lp120up1_mode,
> > + .num_modes = 1,
> > + .bpc = 8,
> > + .size = {
> > + .width = 267,
> > + .height = 183,
> > + },
> > +};
> > +
> > static const struct drm_display_mode lg_lp129qe_mode = {
> > .clock = 285250,
> > .hdisplay = 2560,
> > @@ -1256,6 +1279,9 @@ static const struct of_device_id platform_of_match[]
> > = {
> > .compatible = "lg,lb070wv8",
> > .data = &lg_lb070wv8,
> > }, {
> > + .compatible = "lg,lp120up1",
> > + .data = &lg_lp120up1,
> > + }, {
> > .compatible = "lg,lp129qe",
> > .data = &lg_lp129qe,
> > }, {
> > --
> > 1.7.9.5
> >
> >
> > _______________________________________________
> > linux-arm-kernel mailing list
> > linux-arm-kernel@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> >
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web