Re: [PATCH v2 1/2] dt-bindings: display: panel: Add Novatek NT37802

[Krzysztof Kozlowski <krzk@kernel.org>]

On 12/11/2026 10:37, Jayant Shekhar wrote:
> 
> On 09-11-2026 18:48, Krzysztof Kozlowski wrote:
>> On 09/11/2026 07:27, Jayant Shekhar wrote:
>>> Document the Novatek NT37802 AMOLED DSI command-mode panel.
>>>
>>> Signed-off-by: Jayant Shekhar <jayant.shekhar@oss.qualcomm.com>
>>> ---
>>>   .../bindings/display/panel/novatek,nt37801.yaml    | 49 ++++++++++++++++++++--
>>>   1 file changed, 45 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt37801.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt37801.yaml
>>> index 1b38c1d0af68..be318cc84d52 100644
>>> --- a/Documentation/devicetree/bindings/display/panel/novatek,nt37801.yaml
>>> +++ b/Documentation/devicetree/bindings/display/panel/novatek,nt37801.yaml
>>> @@ -4,26 +4,30 @@
>>>   $id: http://devicetree.org/schemas/display/panel/novatek,nt37801.yaml#
>>>   $schema: http://devicetree.org/meta-schemas/core.yaml#
>>>   
>>> -title: Novatek NT37801 AMOLED DSI Panel
>>> +title: Novatek NT37801 and NT37802 AMOLED DSI Panels
>>>   
>>>   maintainers:
>>>     - Krzysztof Kozlowski <krzk@kernel.org>
>>>   
>>>   description:
>>> -  Naming is inconclusive and different sources claim this is either Novatek
>>> -  NT37801 or NT37810 AMOLED DSI Panel.
>>> +  Naming is inconclusive and different sources claim the NT37801 panel is
>>> +  either Novatek NT37801 or NT37810. The NT37802 panel uses the same common
>>> +  DSI interface and regulator supplies, and additionally requires EXVDD.
>> What is EXVDD supply? The example DTS suggests it is something fake.
> 
> 
> Its not fake, it would be something like below:

OK, how is the pin named on NT37802 side?

> 
>      display_panel_vdd: regulator-display-panel-vdd {
>          compatible = "regulator-fixed";
>          regulator-name = "display-panel-vdd";
>          regulator-min-microvolt = <1100000>;
>          regulator-max-microvolt = <1100000>;
>          startup-delay-us = <233>;
>          gpio = <&pmd8028_e1_gpios 2 GPIO_ACTIVE_HIGH>;
>          enable-active-high;
>          regulator-boot-on;
>          pinctrl-0 = <&sde_disp0_vdd_active>;
>          pinctrl-names = "default";

Looks like something for backlight honestly.

Best regards,
Krzysztof