From: t <jason.pettit@oss.qualcomm.com>
To: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Cc: Jason Pettit <jason.pettit@oss.qualcomm.com>,
Bjorn Andersson <andersson@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
Akhil P Oommen <akhilpo@oss.qualcomm.com>,
Mahadevan P <mahadevan.p@oss.qualcomm.com>,
Sibi Sankar <sibi.sankar@oss.qualcomm.com>,
Jingyi Wang <jingyi.wang@oss.qualcomm.com>,
Ananthu C V <ananthu.cv@oss.qualcomm.com>
Subject: Re: [PATCH v2 2/3] arm64: dts: qcom: Add HP EliteBook X G2q 14 AI
Date: Sat, 27 Jun 2026 11:55:06 -0700 [thread overview]
Message-ID: <20260627185506.26974-1-t@t.t> (raw)
In-Reply-To: <e6c95342-b3a9-46bd-aeb2-a1384234476d@oss.qualcomm.com>
From: Jason Pettit <jason.pettit@oss.qualcomm.com>
On Fri, 26 Jun 2026 13:26:14 +0200, Konrad Dybcio wrote:
> I would assume it's simply not present (as every SMB2370 is a separate
> physical chip) - perhaps we should just add status = "disabled" in the
> DTSI file and enable the nodes on an as-needed basis
I'm not sure if the chip is populated, but I tested whether it enumerates.
With smb2370_l enabled, the SID-11 SPMI device (2-0b on the SPMI-2 bus)
fails to probe:
spmi spmi-2: pmic_arb_check_chnl_status_v1: 0xb 0x104: transaction failed (0x3)
pmic-spmi 2-0b: probe with driver pmic-spmi failed with error -5
The SPMI PMIC arbiter can't complete a transaction to SID 0xb, so pmic-spmi
can't read it and the eusb2 repeater doesn't bind pmic@b (only pmic@9 and
pmic@a come up). I keep it disabled.
Defaulting that in smb2370.dtsi would touch the shared file instead of just
this board, so I've left the override here for now. The comment in v3 will
be updated to read:
/*
* smb2370.dtsi enables all three repeaters by default; SID-11
* (smb2370_l) does not enumerate on this board, so override it back
* to disabled.
*/
Thanks,
Jason
next prev parent reply other threads:[~2026-06-27 18:55 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-26 5:47 [PATCH v2 0/3] X2 Elite HP EliteBook X G2q support Jason Pettit
2026-06-26 5:47 ` [PATCH v2 1/3] dt-bindings: arm: qcom: Add HP EliteBook X G2q 14 AI Jason Pettit
2026-06-26 10:20 ` Krzysztof Kozlowski
2026-06-26 5:47 ` [PATCH v2 2/3] arm64: dts: " Jason Pettit
2026-06-26 6:00 ` sashiko-bot
2026-06-26 11:26 ` Konrad Dybcio
2026-06-27 18:55 ` t [this message]
2026-06-27 19:06 ` Jason Pettit
2026-06-29 9:27 ` Konrad Dybcio
2026-06-26 11:27 ` Konrad Dybcio
2026-06-27 18:55 ` t
2026-06-27 19:06 ` Jason Pettit
2026-06-28 13:02 ` Dmitry Baryshkov
2026-06-28 16:29 ` Jason Pettit
2026-06-29 9:47 ` Konrad Dybcio
2026-06-29 14:19 ` Jason Pettit
2026-06-30 11:04 ` Konrad Dybcio
2026-06-26 5:47 ` [PATCH v2 3/3] firmware: qcom: scm: Allow QSEECOM on " Jason Pettit
2026-06-28 13:02 ` Dmitry Baryshkov
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=20260627185506.26974-1-t@t.t \
--to=jason.pettit@oss.qualcomm.com \
--cc=akhilpo@oss.qualcomm.com \
--cc=ananthu.cv@oss.qualcomm.com \
--cc=andersson@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jingyi.wang@oss.qualcomm.com \
--cc=konrad.dybcio@oss.qualcomm.com \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mahadevan.p@oss.qualcomm.com \
--cc=robh@kernel.org \
--cc=sibi.sankar@oss.qualcomm.com \
/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