Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1204634 > unrolled thread

Re: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support

Started byBjorn Andersson <bjorn.andersson@sonymobile.com>
First post2015-08-11 01:30 +0200
Last post2015-08-11 02:50 +0200
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.


Contents

  Re: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-08-11 01:30 +0200
    Re: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support Rob Clark <robdclark@gmail.com> - 2015-08-11 02:50 +0200

#1204634 — Re: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support

FromBjorn Andersson <bjorn.andersson@sonymobile.com>
Date2015-08-11 01:30 +0200
SubjectRe: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support
Message-ID<pW4FY-4dI-11@gated-at.bofh.it>
On Tue 28 Jul 05:54 PDT 2015, Srinivas Kandagatla wrote:

> This patch adds LVDS panel for IFC6410.
> 

My ifc6410 doesn't have a LVDS display, so I don't think these nodes
belong in this dts file.

I was thinking we could have a variant including this dts file adding
the panel, but I assume that you can mount other panels than the auo
panel defined below?


Either way, I think we should keep this file to define the stuff that's
on the IFC6410 itself and then figure out how to aid people in adding
the rest of their additions...

Regards,
Bjorn
--
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/

[toc] | [next] | [standalone]


#1204653

FromRob Clark <robdclark@gmail.com>
Date2015-08-11 02:50 +0200
Message-ID<pW5Vn-5Vd-7@gated-at.bofh.it>
In reply to#1204634
On Mon, Aug 10, 2015 at 7:25 PM, Bjorn Andersson
<bjorn.andersson@sonymobile.com> wrote:
> On Tue 28 Jul 05:54 PDT 2015, Srinivas Kandagatla wrote:
>
>> This patch adds LVDS panel for IFC6410.
>>
>
> My ifc6410 doesn't have a LVDS display, so I don't think these nodes
> belong in this dts file.

fwiw, my hope was that we could implement a panel-picker for lvds+ddc,
in a similar way to handling devices that could have multiple
different dsi panels, to make this a bit more automagically
plug-and-play.. srini has done some experiments with probing panel's
edid, so we just need to figure out if vendor+device-id in edid is
reliable enough to have a global table mapping them back to panel
compat strings or whether that table needs to go in dt..

BR,
-R

> I was thinking we could have a variant including this dts file adding
> the panel, but I assume that you can mount other panels than the auo
> panel defined below?
>
>
> Either way, I think we should keep this file to define the stuff that's
> on the IFC6410 itself and then figure out how to aid people in adding
> the rest of their additions...
>
> Regards,
> Bjorn
--
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/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web