Re: [PATCH 1/2] dt-bindings: display: panel: add optional wled-supply

[Krzysztof Kozlowski]

On Tue, Sep 29, 2026 at 06:42:20PM +0530, Arpit Saini wrote:
> Some boards drive the ILI7807S panel's backlight from an external
> WLED driver whose enable input is wired to a GPIO, typically modeled
> as a fixed regulator (e.g. vreg_wled).

You describe something else. What's fixed regulator should not matter
here. Which pin is it in ILI7807S?

It seems you just want to represent GPIO with a regulator. This is just
confusing and typical downstream workaround.

> 
> Document the optional wled-supply property for the ILI7807S panel
> binding. When present, the supply can be used to control the enable
> input of the external WLED driver.
> 
> Platforms without an external WLED enable supply can omit this
> property and continue to use the existing backlight handling.
> 
> Fixes: 447a76a99e24 ("dt-bindings: display: panel: add Ilitek ILI7807S panel controller")
> Signed-off-by: Arpit Saini <arpit.saini@xxxxxxxxxxxxxxxx>
> ---
>  .../devicetree/bindings/display/panel/ilitek,ili7807s.yaml   | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili7807s.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili7807s.yaml
> index ba8c5bbf8ffc..8d56b74a962a 100644
> --- a/Documentation/devicetree/bindings/display/panel/ilitek,ili7807s.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili7807s.yaml
> @@ -34,6 +34,10 @@ properties:
>      description: Negative LCD bias supply (AVEE), typically -5.5V
>        (range -6.3V to -4.5V)
>  
> +  wled-supply:
> +    description: Optional Supply for the WLED driver that provides the
> +      panel backlight
> +
>  required:
>    - compatible
>    - reg
> @@ -49,6 +53,13 @@ examples:
>    - |
>      #include <dt-bindings/gpio/gpio.h>
>  
> +    vreg_wled: regulator-wled {
> +        compatible = "regulator-fixed";
> +        regulator-name = "vreg_wled";
> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> +        enable-active-high;
> +    };

Irrelevant here

Best regards,
Krzysztof