From: Krzysztof Kozlowski <krzk@kernel.org>
To: Arpit Saini <arpit.saini@oss.qualcomm.com>,
dmitry.baryshkov@oss.qualcomm.com
Cc: Neil Armstrong <neil.armstrong@linaro.org>,
Jessica Zhang <jesszhan0024@gmail.com>,
David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Thomas Zimmermann <tzimmermann@suse.de>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
Ayushi Makhija <ayushi.makhija@oss.qualcomm.com>,
rajeevny@qti.qualcomm.com
Subject: Re: [PATCH 1/2] dt-bindings: display: panel: add Ilitek ILI7807S panel controller
Date: Wed, 20 May 2026 15:19:34 +0200 [thread overview]
Message-ID: <d8faa1aa-e752-4d67-b6a3-20883ceb6eb0@kernel.org> (raw)
In-Reply-To: <dd065ccd-d7cb-45b5-8733-64b4f6571b3d@oss.qualcomm.com>
On 20/05/2026 14:40, Arpit Saini wrote:
> Hi Krzysztof , Dmitry
>
> On 5/19/2026 4:02 PM, Krzysztof Kozlowski wrote:
>> On Mon, May 18, 2026 at 04:34:12PM +0530, Arpit Saini wrote:
>>> ILI7807S is a DSI display controller used to drive MIPI-DSI panels.
>>> The DLC DLC0697 1080x1920 LCD panel is based on this controller.
>>>
>>> The panel requires a reset GPIO, backlight enable GPIO, I/O voltage
>>
>> If panel requires it, so should the binding.
>>
> Ack, I will update.
>> ...
>>
>>> + reg:
>>> + maxItems: 1
>>> + description: DSI virtual channel
>>> +
>>> + reset-gpios: true
>>> +
>>> + backlight-en-gpios:
>>> + description: Backlight enable GPIO (active high)
>>
>> What is the name of the pin in ili7807s device?
>>
> Display daughter card has WLED driver as well as LCD bias driver.
>
> The WLED driver's Enable is coming from WLED_P1_EN and PWM is coming
> from LCD_CABC output of the panel.
WLED driver is not ili7807s. LCD bias driver also sounds like something
else than ili7807s.
So my question stays.
Best regards,
Krzysztof
next prev parent reply other threads:[~2026-05-20 13:19 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-18 11:04 [PATCH 0/2] drm/panel: Add driver for DLC DLC0697 DSI panel Arpit Saini
2026-05-18 11:04 ` [PATCH 1/2] dt-bindings: display: panel: add Ilitek ILI7807S panel controller Arpit Saini
2026-05-18 16:23 ` Dmitry Baryshkov
2026-05-20 12:54 ` Arpit Saini
2026-06-06 12:16 ` Dmitry Baryshkov
2026-05-19 10:32 ` Krzysztof Kozlowski
2026-05-20 12:40 ` Arpit Saini
2026-05-20 13:19 ` Krzysztof Kozlowski [this message]
2026-05-20 13:26 ` Dmitry Baryshkov
2026-05-26 10:32 ` Arpit Saini
2026-06-06 12:13 ` Dmitry Baryshkov
2026-05-18 11:04 ` [PATCH 2/2] drm/panel: add Ilitek ILI7807S panel driver Arpit Saini
2026-05-18 13:50 ` Neil Armstrong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d8faa1aa-e752-4d67-b6a3-20883ceb6eb0@kernel.org \
--to=krzk@kernel.org \
--cc=airlied@gmail.com \
--cc=arpit.saini@oss.qualcomm.com \
--cc=ayushi.makhija@oss.qualcomm.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.baryshkov@oss.qualcomm.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=jesszhan0024@gmail.com \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=rajeevny@qti.qualcomm.com \
--cc=robh@kernel.org \
--cc=simona@ffwll.ch \
--cc=tzimmermann@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox