Re: [PATCH v3 1/3] dt-bindings: display: panel: add common dual-display schema

From: Krzysztof Kozlowski
Date: Wed Aug 23 2023 - 05:41:54 EST


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