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@xxxxxxxxxx> >> >> --- >> >> v2: https://lore.kernel.org/all/20230502120036.47165-1-krzysztof.kozlowski@xxxxxxxxxx/ >> v1: https://lore.kernel.org/all/20230416153929.356330-1-krzysztof.kozlowski@xxxxxxxxxx/ >> >> 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@xxxxxxxxx> >> + - Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> >> + >> +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. Best regards, Krzysztof