lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Fri, 17 May 2024 10:11:29 +0800
From: Liu Ying <victor.liu@....com>
To: neil.armstrong@...aro.org, Aradhya Bhatia <a-bhatia1@...com>,
 Jessica Zhang <quic_jesszhan@...cinc.com>, Sam Ravnborg <sam@...nborg.org>,
 David Airlie <airlied@...il.com>, Daniel Vetter <daniel@...ll.ch>,
 Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Thierry Reding
 <thierry.reding@...il.com>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 Tomi Valkeinen <tomi.valkeinen@...asonboard.com>,
 Laurent Pinchart <laurent.pinchart@...asonboard.com>
Cc: DRI Development List <dri-devel@...ts.freedesktop.org>,
 Devicetree List <devicetree@...r.kernel.org>,
 Linux Kernel List <linux-kernel@...r.kernel.org>, Nishanth Menon
 <nm@...com>, Vignesh Raghavendra <vigneshr@...com>,
 Praneeth Bajjuri <praneeth@...com>, Udit Kumar <u-kumar1@...com>,
 Devarsh Thakkar <devarsht@...com>, Jai Luthra <j-luthra@...com>
Subject: Re: [PATCH v3 3/6] dt-bindings: display: simple: Add Microtips &
 Lincolntech Dual-LVDS Panels

On 5/16/24 20:36, Neil Armstrong wrote:
> On 16/05/2024 13:06, Aradhya Bhatia wrote:
>> Hi Liu,
>>
>> Thanks for reviewing the patch.
>>
>> On 16/05/24 07:49, Liu Ying wrote:
>>> On 5/15/24 17:51, Aradhya Bhatia wrote:
>>>> Add the Microtips Technology USA's MF-101HIEBCAF0 10.1"[0] panel,
>>>> MF-103HIEB0GA0 10.25"[1] panel, and Lincoln Technology Solutions'
>>>> LCD185-101CT 10.1"[2] panel.
>>>>
>>>> Thes are all dual-lvds panels.
>>>>
>>>> Panel Links:
>>>> [0]: https://simplespec.microtipsusa.com/uploads/spec/datasheetFile/2588/13-101HIEBCAF0-S_V1.1_20221104.pdf
>>>> [1]: https://simplespec.microtipsusa.com/uploads/spec/datasheetFile/2660/13-103HIEB0GA0-S_V1.0_20211206.pdf
>>>
>>> This one mentions some controls in '3. PIN DESCRIPTION' which
>>> don't comply with this binding, like RL, TB, STBYB and RESET.
>>> Note this binding only allows compatible, ports, backlight,
>>> enable-gpios and power-supply properties, nothing more.
>>>
>>
>> Maybe the internal module _can_ support these control options but the
>> actual panel hardware does not expose any of these controls on its
>> connector pin-out, and hence has no possibility for additional SW
>> control. Even for the usage, the device-tree node for the panel only has
>> "compatible" and "ports" properties defined.

At least, power-supply is required by this binding doc.

>>
>> The panel is being used within the confines of a simple panel.
> 
> If somehow some modules with those panels would need more control lines,
> we'll simply move them out of simple panel bindings & drivers if
> required.
> 
> Those are perfectly fine as is, we don't want useless description and dead code.

It would be better to specify module vendor and panel type in
the compatible property instead of using the generic one in this
binding doc.

If TI is the module vendor, it could be something like "ti,panel-type-x".
See ili9881c based panel "bananapi,lhr050h41" as an example.

Regards,
Liu Ying

> 
> Neil
> 
>>
>> Regards
>> Aradhya
>>
>>>
>>>> [2]: https://lincolntechsolutions.com/wp-content/uploads/2023/04/LCD185-101CTL1ARNTT_DS_R1.3.pdf
>>>>
>>>> Signed-off-by: Aradhya Bhatia <a-bhatia1@...com>
>>>> ---
>>>>   .../display/panel/panel-simple-lvds-dual-ports.yaml         | 6 ++++++
>>>>   1 file changed, 6 insertions(+)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple-lvds-dual-ports.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple-lvds-dual-ports.yaml
>>>> index 716ece5f3978..e78160d1aa24 100644
>>>> --- a/Documentation/devicetree/bindings/display/panel/panel-simple-lvds-dual-ports.yaml
>>>> +++ b/Documentation/devicetree/bindings/display/panel/panel-simple-lvds-dual-ports.yaml
>>>> @@ -41,6 +41,12 @@ properties:
>>>>         - auo,g190ean01
>>>>           # Kaohsiung Opto-Electronics Inc. 10.1" WUXGA (1920 x 1200) LVDS TFT LCD panel
>>>>         - koe,tx26d202vm0bwa
>>>> +        # Lincoln Technology Solutions, LCD185-101CT 10.1" TFT 1920x1200
>>>> +      - lincolntech,lcd185-101ct
>>>> +        # Microtips Technology MF-101HIEBCAF0 10.1" WUXGA (1920x1200) TFT LCD panel
>>>> +      - microtips,mf-101hiebcaf0
>>>> +        # Microtips Technology MF-103HIEB0GA0 10.25" 1920x720 TFT LCD panel
>>>> +      - microtips,mf-103hieb0ga0
>>>>           # NLT Technologies, Ltd. 15.6" FHD (1920x1080) LVDS TFT LCD panel
>>>>         - nlt,nl192108ac18-02d
>>>>   
>>>
> 


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ