Linux ARM-MSM sub-architecture
 help / color / mirror / Atom feed
From: David Heidelberg <david@ixit.cz>
To: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Jessica Zhang <jessica.zhang@oss.qualcomm.com>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Maxime Ripard <mripard@kernel.org>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Thierry Reding <thierry.reding@gmail.com>,
	Sam Ravnborg <sam@ravnborg.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konradybcio@kernel.org>,
	Casey Connolly <casey.connolly@linaro.org>
Cc: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	phone-devel@vger.kernel.org
Subject: Re: [PATCH 0/8] Add OnePlus 6T display (Samsung S6E3FC2X01 DDIC with AMS641RW panel)
Date: Thu, 25 Sep 2025 15:41:34 +0200	[thread overview]
Message-ID: <7282a888-e550-4d75-838c-0fb242fefdf6@ixit.cz> (raw)
In-Reply-To: <a078c6a0-5a54-4ad5-8e3a-c676783e68a3@oss.qualcomm.com>

On 25/09/2025 11:37, Konrad Dybcio wrote:
> On 9/25/25 11:12 AM, David Heidelberg via B4 Relay wrote:
>> This patchset enables display on the OnePlus 6T smartphone.
>>
>> Minor adjust to the device-tree of OnePlus 6 had to be done
>> to properly document reset GPIO used. Also same adjustments
>> had been done to the sofef00 panel driver (used by OnePlus 6).
>>
>> In the last step new DDIC driver is introduced together with AMS641RW
>> panel sequences.
> 
> I think you skipped the elephant in the room - some of these patches
> break panel for the phone intermittently..

Thank you for the review Konrad. I plan to incorporate all your 
suggested changes in the next revision.

To clarify, while sofef00 has been introduced as supporting OnePlus 6T 
too, but it never did (and the compatible was removed later).

All the changes CAN'T BREAK anything for OnePlus 6T display as it's not 
supported yet by mainline.

All the changes SHOULDN'T CHANGE anything for the OnePlus 6, as both 
sides (device-tree and the driver) is changed at once to correct reset 
polarity.

David

> 
> Konrad

-- 
David Heidelberg


  reply	other threads:[~2025-09-25 13:49 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-25  9:12 [PATCH 0/8] Add OnePlus 6T display (Samsung S6E3FC2X01 DDIC with AMS641RW panel) David Heidelberg via B4 Relay
2025-09-25  9:12 ` [PATCH 1/8] dt-bindings: panel: Add Samsung S6E3FC2X01 DDIC with panel David Heidelberg via B4 Relay
2025-10-02  1:00   ` Rob Herring (Arm)
2025-10-02  1:06   ` Rob Herring
2025-09-25  9:12 ` [PATCH 2/8] dt-bindings: display: panel-simple-dsi: Remove Samsung S6E3FC2 compatible David Heidelberg via B4 Relay
2025-09-25 20:51   ` Dmitry Baryshkov
2025-10-02  1:05     ` Rob Herring
2025-09-25  9:12 ` [PATCH 3/8] arm64: dts: qcom: sdm845-oneplus: Describe panel vci and poc supplies David Heidelberg via B4 Relay
2025-09-25  9:20   ` Konrad Dybcio
2025-09-25  9:12 ` [PATCH 4/8] arm64: dts: qcom: sdm845-oneplus-fajita: Reflect used panel in compatible David Heidelberg via B4 Relay
2025-09-25  9:12 ` [PATCH 5/8] arm64: dts: qcom: sdm845-oneplus: Correct panel reset gpio polarity David Heidelberg via B4 Relay
2025-09-25 19:34   ` Jens Reidel
2025-09-25  9:12 ` [PATCH 6/8] arm64: dts: qcom: sdm845-oneplus: Document TE gpio David Heidelberg via B4 Relay
2025-09-25  9:21   ` Konrad Dybcio
2025-09-25  9:12 ` [PATCH 7/8] drm/panel: samsung-sofef00: Invert reset gpio polarity David Heidelberg via B4 Relay
2025-09-25 21:00   ` Dmitry Baryshkov
2025-09-26 11:45     ` David Heidelberg
2025-09-25  9:12 ` [PATCH 8/8] drm/panel: Add Samsung S6E3FC2X01 DDIC with AMS641RW panel David Heidelberg via B4 Relay
2025-09-27 19:37   ` kernel test robot
2025-09-25  9:37 ` [PATCH 0/8] Add OnePlus 6T display (Samsung S6E3FC2X01 DDIC with AMS641RW panel) Konrad Dybcio
2025-09-25 13:41   ` David Heidelberg [this message]
2025-09-25 14:12 ` Rob Herring (Arm)

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=7282a888-e550-4d75-838c-0fb242fefdf6@ixit.cz \
    --to=david@ixit.cz \
    --cc=airlied@gmail.com \
    --cc=andersson@kernel.org \
    --cc=casey.connolly@linaro.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jessica.zhang@oss.qualcomm.com \
    --cc=konrad.dybcio@oss.qualcomm.com \
    --cc=konradybcio@kernel.org \
    --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=phone-devel@vger.kernel.org \
    --cc=robh@kernel.org \
    --cc=sam@ravnborg.org \
    --cc=simona@ffwll.ch \
    --cc=thierry.reding@gmail.com \
    --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