[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <5a5e97f1-6ae1-46d2-a602-613e1b550ba5@kernel.org>
Date: Tue, 29 Apr 2025 08:13:56 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: barnabas.czeman@...nlining.org
Cc: Neil Armstrong <neil.armstrong@...aro.org>,
Jessica Zhang <quic_jesszhan@...cinc.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, dri-devel@...ts.freedesktop.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] dt-bindings: display: panel: Add BOE TD4320
On 28/04/2025 20:13, barnabas.czeman@...nlining.org wrote:
>>
>>> +
>>> +required:
>>> + - compatible
>>> + - reg
>>> + - reset-gpios
>>
>> No supplies? This looks really incomplete.
> It works without supplies because BL is enabling them and there is no
> qpnp-lcdb driver yet.
If something enables the supplies, means they exist. Binding must be
complete. Otherwise how bootloader using this binding can enable
supplies, if you do not allow them in the DTS the bootloader is going to
use?
> I do not see worth to add them at them moment but I can if you want.
>>
>>
>> Best regards,
>> Krzysztof
Best regards,
Krzysztof
Powered by blists - more mailing lists