All of lore.kernel.org
 help / color / mirror / Atom feed
From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
To: Pengyu Luo <mitltlatltl@gmail.com>
Cc: aliceryhl@google.com, andersson@kernel.org, conor+dt@kernel.org,
	devicetree@vger.kernel.org, ebiggers@google.com,
	ilpo.jarvinen@linux.intel.com, joel.granados@kernel.org,
	konradybcio@kernel.org, krzk+dt@kernel.org, len.brown@intel.com,
	linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
	lossin@kernel.org, mingo@kernel.org, ojeda@kernel.org,
	robh@kernel.org, sfr@canb.auug.org.au, vanyang@smail.nju.edu.cn,
	viro@zeniv.linux.org.uk
Subject: Re: [PATCH 3/4] arm64: dts: qcom: sc8280xp: Add initial support for Ntmer TW220
Date: Wed, 30 Jul 2025 22:32:51 +0200	[thread overview]
Message-ID: <9efafa16-e6db-4227-9b47-5803f5933a7d@oss.qualcomm.com> (raw)
In-Reply-To: <CAH2e8h6XWAz-pqmuvzK8JqOb=ggiDGb2U3TvAR2+43D_zdsZpQ@mail.gmail.com>

On 7/30/25 5:29 PM, Pengyu Luo wrote:
> On Wed, Jul 30, 2025 at 8:53 PM Konrad Dybcio
> <konrad.dybcio@oss.qualcomm.com> wrote:
>>
>> On 7/6/25 5:43 AM, Pengyu Luo wrote:
>>> On Sat, Jun 28, 2025 at 3:48 AM Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> wrote:
>>>> On 6/17/25 11:29 AM, Pengyu Luo wrote:
>>>>> The Ntmer TW220 is a WOS tablet based on the Qualcomm SC8280XP platform,
>>>>> also known as the Robo&Kala 2-in-1 Laptop. Thanks to Hong for providing
>>>>> the unlocked device and early development work. This patch adds an
>>>>> initial device tree to enable basic functionality.
>>>>>
>>>>> Currently supported components include:
>>>>> - Bluetooth & Wi-Fi (board file regeneration required)
>>>>> - Battery charging (up to 15V/3A fixed PDO) and reporting via pmic-glink
>>>>> - Flash LEDs (front and rear)
>>>>> - Hall sensor (lid detection)
>>>>> - Keyboard (via Bluetooth or USB)
>>>>> - NVMe SSD
>>>>> - Power and volume keys
>>>>> - Simple-framebuffer
>>>>> - Sound (playback and capture; top-left DMIC only, top-right works only
>>>>>   on Windows)
>>>>> - Touchscreen and stylus (requires GPI DMA support [1] and stylus support [2])
>>>>> - USB Type-C ports
>>>>>
>>>>> The following components are currently non-functional:
>>>>> - Cameras (GalaxyCore GC5035; only sensor ID is detectable, no frames in libcamera;
>>>>>   partial driver can be found on LKML archives)
>>>>> - DSI display (blank screen with `dsi_err_worker: status=4`; primary DSI register
>>>>>   dump included below)
>>>>> - Stylus wireless charger (CPS4035)
>>>>> - UCSI over GLINK
>>>>>
>>>>> [1]: https://lore.kernel.org/linux-arm-msm/20250617090032.1487382-3-mitltlatltl@gmail.com
>>>>> [2]: https://lore.kernel.org/linux-input/20250605054855.403487-2-mitltlatltl@gmail.com
>>>>>
>>>>> Note: This series does **not** include any confidential material. Those
>>>>> who wish to run Linux on this device should contact Ntmer, as the
>>>>> bootloader is locked via secure boot.
>>>>>
>>>>> Co-developed-by: Hong Zhu <vanyang@smail.nju.edu.cn>
>>>>> Signed-off-by: Hong Zhu <vanyang@smail.nju.edu.cn>
>>>>> Signed-off-by: Pengyu Luo <mitltlatltl@gmail.com>
>>>>>
>>>>> dsi_ctrl, reg = <0 0x0ae94000 0 0x400>;
>>>>> 0xae94000 20050001 000001f3 0000000b dddd1011
>>>>
>>>> This is not something we want in the commit log
>>>>
>>>
>>> I will remove it. I need help, then I attached it, two of my sc8280xp
>>> devices require dsi to work. Reversing and guessing wasted a lot of
>>> time. I will appreciate it if qcom could support it.
>>
>> There should be nothing interesting compared to what's on SM8350 when it
>> comes to DSI on 8280.
>>
>> I would assume something's wrong with how your panel is being set up.
> 
> Just to mention it. According to some public sources, the Adreno 1095 DPU
> is included in the sm8350, while the Adreno 1199 DPU is included in the
> sc8280xp, they are slightly different. Here is a quick comparison
> 
>                    dpu_version     dsi_version     dsi_phy_version
> Adreno 1095 DPU:     7.0            2.5.0           4.2
> Adreno 1199 DPU:     8.0            2.5.1           4.2

Patchlevel/step versions are usually not supposed to break any software
interfaces, so I'd be *really* surprised if this is the case

DPU is rather different (the one on 8280 is way larger than 8350's), but
ultimately they're built of the same bricks

> 
> The panel setup has been tweaked and tested countless times, I really don't
> think I screwed it up. I had already brought up a lcd video panel with dual
> dsi and dual dsc on the sm8650. On the sc8280xp, I’ve worked with two
> different panels:
> 
> - One LCD video panel with dual DSI and dual DSC
> - One OLED video panel with dual DSI only
> 
> Both fail to display anything and complain about DSI FIFO errors
> (dsi_err_worker: status=4). Someone also encountered a similar issue on
> the SC8180X([1]), another WoA platform.
> 
> Anyway, I had quit. This doesn’t bother me anymore. Just mentioning the
> info for reference.

Sad to see you go.

If you still want to, I saw a fix for a sneaky bug earlier today on
LKML that may actually be related to your issue, given you seem to
have a really high res panel..

https://lore.kernel.org/linux-arm-msm/20250730123938.1038640-1-quic_amakhija@quicinc.com/

Konrad

  reply	other threads:[~2025-07-30 20:32 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-17  9:29 [PATCH 0/4] arm64: dts: qcom: Introduce Ntmer TW220 Pengyu Luo
2025-06-17  9:29 ` [PATCH 1/4] dt-bindings: vendor-prefixes: Add Ntmer Pengyu Luo
2025-06-27 19:20   ` Rob Herring (Arm)
2025-06-17  9:29 ` [PATCH 2/4] dt-bindings: arm: qcom: Add Ntmer TW220 Pengyu Luo
2025-06-27 19:20   ` Rob Herring (Arm)
2025-06-17  9:29 ` [PATCH 3/4] arm64: dts: qcom: sc8280xp: Add initial support for " Pengyu Luo
2025-06-25  6:17   ` kernel test robot
2025-06-27 19:48   ` Konrad Dybcio
2025-07-06  3:43     ` Pengyu Luo
2025-07-30 12:53       ` Konrad Dybcio
2025-07-30 15:29         ` Pengyu Luo
2025-07-30 20:32           ` Konrad Dybcio [this message]
2025-07-31  3:05             ` Pengyu Luo
2025-08-01 12:29               ` Konrad Dybcio
2025-08-01 14:27                 ` Pengyu Luo
2025-08-01 17:27                   ` Dmitry Baryshkov
2025-08-03  3:36                     ` Pengyu Luo
2025-06-17  9:29 ` [PATCH 4/4] firmware: qcom: scm: Allow QSEECOM on " Pengyu Luo
2025-06-17 14:21 ` [PATCH 0/4] arm64: dts: qcom: Introduce " 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=9efafa16-e6db-4227-9b47-5803f5933a7d@oss.qualcomm.com \
    --to=konrad.dybcio@oss.qualcomm.com \
    --cc=aliceryhl@google.com \
    --cc=andersson@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=ebiggers@google.com \
    --cc=ilpo.jarvinen@linux.intel.com \
    --cc=joel.granados@kernel.org \
    --cc=konradybcio@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=len.brown@intel.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lossin@kernel.org \
    --cc=mingo@kernel.org \
    --cc=mitltlatltl@gmail.com \
    --cc=ojeda@kernel.org \
    --cc=robh@kernel.org \
    --cc=sfr@canb.auug.org.au \
    --cc=vanyang@smail.nju.edu.cn \
    --cc=viro@zeniv.linux.org.uk \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.