From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Neil Armstrong <neil.armstrong@linaro.org>,
Sam Ravnborg <sam@ravnborg.org>, David Airlie <airlied@gmail.com>,
Daniel Vetter <daniel@ffwll.ch>, Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
Jianhua Lu <lujianhua000@gmail.com>,
Del Regno <angelogioacchino.delregno@somainline.org>,
Thierry Reding <thierry.reding@gmail.com>,
dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 1/3] dt-bindings: display: panel: add common dual-display schema
Date: Wed, 23 Aug 2023 13:07:02 +0300 [thread overview]
Message-ID: <20230823100702.GK4143@pendragon.ideasonboard.com> (raw)
In-Reply-To: <61ae58eb-d56c-59c1-81d7-b51322468680@linaro.org>
On Wed, Aug 23, 2023 at 11:35:24AM +0200, Krzysztof Kozlowski wrote:
> On 23/08/2023 10:34, Laurent Pinchart wrote:
> > Hi Krzysztof,
> >
> > Thank you for the patch.
> >
> > On Wed, Aug 23, 2023 at 10:14:58AM +0200, Krzysztof Kozlowski wrote:
> >> Add schema with common properties shared among dual display panel ICs.
> >>
> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >>
> >> ---
> >>
> >> v2: https://lore.kernel.org/all/20230502120036.47165-1-krzysztof.kozlowski@linaro.org/
> >> v1: https://lore.kernel.org/all/20230416153929.356330-1-krzysztof.kozlowski@linaro.org/
> >>
> >> Changes since v2:
> >> 1. New Patch
> >> ---
> >> .../display/panel/panel-common-dual.yaml | 46 +++++++++++++++++++
> >> 1 file changed, 46 insertions(+)
> >> create mode 100644 Documentation/devicetree/bindings/display/panel/panel-common-dual.yaml
> >>
> >> diff --git a/Documentation/devicetree/bindings/display/panel/panel-common-dual.yaml b/Documentation/devicetree/bindings/display/panel/panel-common-dual.yaml
> >> new file mode 100644
> >> index 000000000000..83fcd643b5f5
> >> --- /dev/null
> >> +++ b/Documentation/devicetree/bindings/display/panel/panel-common-dual.yaml
> >> @@ -0,0 +1,46 @@
> >> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> >> +%YAML 1.2
> >> +---
> >> +$id: http://devicetree.org/schemas/display/panel/panel-common-dual.yaml#
> >> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> >> +
> >> +title: Common Properties for Dual-Display Panels
> >> +
> >> +maintainers:
> >> + - Thierry Reding <thierry.reding@gmail.com>
> >> + - Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> >> +
> >> +description:
> >> + Properties common for Panel IC supporting dual display panels.
> >
> > Could you elaborate on what you mean by dual display panels ? From what
> > I understand of the patch series, this seems to be what I have called
> > dual port panels, that is panels that have two input ports to double the
> > bandwidth, but still operate as one display.
>
> Indeed, looks like that. I don't have the datasheets so I am trying to
> get some pieces of information. Judging by the usage of these panels -
> Qualcomm DSI with qcom,dual-dsi-mode - it indeed is one panel with two
> links.
>
> I would need to rephrase it.
Thanks :-)
Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
may be of interest to you, it's a bridge with two LVDS inputs. On the
other side, I've used
Documentation/devicetree/bindings/display/bridge/renesas,lvds.yaml which
is an LVDS encoder with a single output, but with the ability to operate
in dual link mode by coupling two encoder instances.
--
Regards,
Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Neil Armstrong <neil.armstrong@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
devicetree@vger.kernel.org, Sam Ravnborg <sam@ravnborg.org>,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
Rob Herring <robh+dt@kernel.org>,
Jianhua Lu <lujianhua000@gmail.com>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Del Regno <angelogioacchino.delregno@somainline.org>,
Thierry Reding <thierry.reding@gmail.com>
Subject: Re: [PATCH v3 1/3] dt-bindings: display: panel: add common dual-display schema
Date: Wed, 23 Aug 2023 13:07:02 +0300 [thread overview]
Message-ID: <20230823100702.GK4143@pendragon.ideasonboard.com> (raw)
In-Reply-To: <61ae58eb-d56c-59c1-81d7-b51322468680@linaro.org>
On Wed, Aug 23, 2023 at 11:35:24AM +0200, Krzysztof Kozlowski wrote:
> On 23/08/2023 10:34, Laurent Pinchart wrote:
> > Hi Krzysztof,
> >
> > Thank you for the patch.
> >
> > On Wed, Aug 23, 2023 at 10:14:58AM +0200, Krzysztof Kozlowski wrote:
> >> Add schema with common properties shared among dual display panel ICs.
> >>
> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >>
> >> ---
> >>
> >> v2: https://lore.kernel.org/all/20230502120036.47165-1-krzysztof.kozlowski@linaro.org/
> >> v1: https://lore.kernel.org/all/20230416153929.356330-1-krzysztof.kozlowski@linaro.org/
> >>
> >> Changes since v2:
> >> 1. New Patch
> >> ---
> >> .../display/panel/panel-common-dual.yaml | 46 +++++++++++++++++++
> >> 1 file changed, 46 insertions(+)
> >> create mode 100644 Documentation/devicetree/bindings/display/panel/panel-common-dual.yaml
> >>
> >> diff --git a/Documentation/devicetree/bindings/display/panel/panel-common-dual.yaml b/Documentation/devicetree/bindings/display/panel/panel-common-dual.yaml
> >> new file mode 100644
> >> index 000000000000..83fcd643b5f5
> >> --- /dev/null
> >> +++ b/Documentation/devicetree/bindings/display/panel/panel-common-dual.yaml
> >> @@ -0,0 +1,46 @@
> >> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> >> +%YAML 1.2
> >> +---
> >> +$id: http://devicetree.org/schemas/display/panel/panel-common-dual.yaml#
> >> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> >> +
> >> +title: Common Properties for Dual-Display Panels
> >> +
> >> +maintainers:
> >> + - Thierry Reding <thierry.reding@gmail.com>
> >> + - Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> >> +
> >> +description:
> >> + Properties common for Panel IC supporting dual display panels.
> >
> > Could you elaborate on what you mean by dual display panels ? From what
> > I understand of the patch series, this seems to be what I have called
> > dual port panels, that is panels that have two input ports to double the
> > bandwidth, but still operate as one display.
>
> Indeed, looks like that. I don't have the datasheets so I am trying to
> get some pieces of information. Judging by the usage of these panels -
> Qualcomm DSI with qcom,dual-dsi-mode - it indeed is one panel with two
> links.
>
> I would need to rephrase it.
Thanks :-)
Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
may be of interest to you, it's a bridge with two LVDS inputs. On the
other side, I've used
Documentation/devicetree/bindings/display/bridge/renesas,lvds.yaml which
is an LVDS encoder with a single output, but with the ability to operate
in dual link mode by coupling two encoder instances.
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2023-08-23 10:07 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-23 8:14 [PATCH v3 1/3] dt-bindings: display: panel: add common dual-display schema Krzysztof Kozlowski
2023-08-23 8:14 ` [PATCH v3 2/3] dt-bindings: display: novatek,nt35950: define ports Krzysztof Kozlowski
2023-08-23 9:08 ` Rob Herring
2023-08-23 9:08 ` Rob Herring
2023-08-23 9:53 ` Krzysztof Kozlowski
2023-08-23 9:53 ` Krzysztof Kozlowski
2023-08-23 12:48 ` Rob Herring
2023-08-23 12:48 ` Rob Herring
2023-08-24 17:20 ` Rob Herring
2023-08-24 17:20 ` Rob Herring
2023-08-23 8:15 ` [PATCH v3 3/3] dt-bindings: display: novatek,nt36523: " Krzysztof Kozlowski
2023-08-24 17:21 ` Rob Herring
2023-08-24 17:21 ` Rob Herring
2023-08-23 8:34 ` [PATCH v3 1/3] dt-bindings: display: panel: add common dual-display schema Laurent Pinchart
2023-08-23 8:34 ` Laurent Pinchart
2023-08-23 9:35 ` Krzysztof Kozlowski
2023-08-23 9:35 ` Krzysztof Kozlowski
2023-08-23 10:07 ` Laurent Pinchart [this message]
2023-08-23 10:07 ` Laurent Pinchart
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=20230823100702.GK4143@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=airlied@gmail.com \
--cc=angelogioacchino.delregno@somainline.org \
--cc=conor+dt@kernel.org \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lujianhua000@gmail.com \
--cc=neil.armstrong@linaro.org \
--cc=robh+dt@kernel.org \
--cc=sam@ravnborg.org \
--cc=thierry.reding@gmail.com \
/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.