From: barnabas.czeman@mainlining.org
To: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Cc: Lee Jones <lee@kernel.org>, Daniel Thompson <danielt@kernel.org>,
Jingoo Han <jingoohan1@gmail.com>,
Pavel Machek <pavel@kernel.org>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Kiran Gunda <quic_kgunda@quicinc.com>,
Helge Deller <deller@gmx.de>, Luca Weiss <luca@lucaweiss.eu>,
Konrad Dybcio <konradybcio@kernel.org>,
Eugene Lepshy <fekz115@gmail.com>,
Gianluca Boiano <morf3089@gmail.com>,
Alejandro Tafalla <atafalla@dnyon.com>,
dri-devel@lists.freedesktop.org, linux-leds@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Daniel Thompson <daniel.thompson@linaro.org>,
linux-arm-msm@vger.kernel.org, linux-fbdev@vger.kernel.org
Subject: Re: [PATCH v2 1/7] dt-bindings: backlight: qcom-wled: Document ovp values for PMI8994
Date: Thu, 08 Jan 2026 10:30:16 +0100 [thread overview]
Message-ID: <1d1601c3b871e67c76e4f3572400b2d7@mainlining.org> (raw)
In-Reply-To: <52778327-69bb-4f6c-8d64-094f33809480@oss.qualcomm.com>
On 2026-01-08 10:22, Konrad Dybcio wrote:
> On 1/8/26 10:17 AM, barnabas.czeman@mainlining.org wrote:
>> On 2026-01-08 09:54, Konrad Dybcio wrote:
>>> On 1/8/26 4:43 AM, Barnabás Czémán wrote:
>>>> Document ovp values supported by wled found in PMI8994.
>>>>
>>>> Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
>>>
>>> I reviewed a version of this that said PMI8950, which is very much
>>> not the same..
>>>
>>> Let me try and get some more docs to confirm or deny what you're
>>> saying..
>> I have sent it for the previous revision msm-4.4 contains the
>> documentation.
>> https://lore.kernel.org/all/95f7c89fdf9b856f567a498cc56ecfa5@mainlining.org/T/#u
>
> Right, but I didn't send any reply acknowledging that.
>
> On v1, my review tag meant "I concur this is the case for PMI8950"
Yes, sorry. Since they are pretty much same i have considered it is a
minor change i have only changed everything to pmi8994 because it
was introduced before pmi8950 at upstream. I will be more careful next
time.
>
> Because you carried it in v2, it appears as if I said "I concur this
> is the case for PMI8994", which is not what I then said.
>
> Konrad
next prev parent reply other threads:[~2026-01-08 9:30 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-08 3:43 [PATCH v2 0/7] Fix PMI8950 WLED ovp values and more Barnabás Czémán
2026-01-08 3:43 ` [PATCH v2 1/7] dt-bindings: backlight: qcom-wled: Document ovp values for PMI8994 Barnabás Czémán
2026-01-08 8:54 ` Konrad Dybcio
2026-01-08 9:17 ` barnabas.czeman
2026-01-08 9:22 ` Konrad Dybcio
2026-01-08 9:30 ` barnabas.czeman [this message]
2026-01-08 9:21 ` Konrad Dybcio
2026-01-08 10:09 ` Krzysztof Kozlowski
2026-01-08 3:43 ` [PATCH v2 2/7] backlight: qcom-wled: Support " Barnabás Czémán
2026-01-08 8:55 ` Konrad Dybcio
2026-01-08 9:22 ` Konrad Dybcio
2026-01-08 11:28 ` Daniel Thompson
2026-01-09 6:36 ` barnabas.czeman
2026-01-09 11:09 ` Konrad Dybcio
2026-01-09 13:33 ` Daniel Thompson
2026-01-09 15:42 ` barnabas.czeman
2026-01-09 15:45 ` barnabas.czeman
2026-01-08 3:43 ` [PATCH v2 3/7] dt-bindings: backlight: qcom-wled: Document ovp values for PMI8950 Barnabás Czémán
2026-01-08 9:22 ` Konrad Dybcio
2026-01-08 10:17 ` Krzysztof Kozlowski
2026-01-08 3:43 ` [PATCH v2 4/7] backlight: qcom-wled: Fix " Barnabás Czémán
2026-01-08 9:22 ` Konrad Dybcio
2026-01-08 3:43 ` [PATCH v2 5/7] arm64: dts: qcom: msm8953-xiaomi-vince: correct wled ovp value Barnabás Czémán
2026-01-08 3:43 ` [PATCH v2 6/7] arm64: dts: qcom: msm8937-xiaomi-land: " Barnabás Czémán
2026-01-08 3:43 ` [PATCH v2 7/7] arm64: dts: qcom: msm8953-xiaomi-daisy: fix backlight Barnabás Czémán
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=1d1601c3b871e67c76e4f3572400b2d7@mainlining.org \
--to=barnabas.czeman@mainlining.org \
--cc=andersson@kernel.org \
--cc=atafalla@dnyon.com \
--cc=conor+dt@kernel.org \
--cc=daniel.thompson@linaro.org \
--cc=danielt@kernel.org \
--cc=deller@gmx.de \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=fekz115@gmail.com \
--cc=jingoohan1@gmail.com \
--cc=konrad.dybcio@oss.qualcomm.com \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=lee@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-leds@vger.kernel.org \
--cc=luca@lucaweiss.eu \
--cc=morf3089@gmail.com \
--cc=pavel@kernel.org \
--cc=quic_kgunda@quicinc.com \
--cc=robh@kernel.org \
/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