From: Bjorn Andersson <andersson@kernel.org>
To: pabloyoyoista@postmarketos.org
Cc: Konrad Dybcio <konradybcio@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,
Richard Acayan <mailingradian@gmail.com>,
~postmarketos/upstreaming@lists.sr.ht
Subject: Re: [PATCH v2 0/3] Add support and enable the debug UART in the Pixel 3a and Pixel 3a XL
Date: Mon, 27 Apr 2026 14:07:13 -0500 [thread overview]
Message-ID: <ae-zV6gtMA8LLTex@baldur> (raw)
In-Reply-To: <20260328-pabloyoyoista-debug-uart-on-rdacayan-next-v2-0-53abd9db8f0a@postmarketos.org>
On Sat, Mar 28, 2026 at 11:58:39PM +0100, Pablo Correa Gómez via B4 Relay wrote:
> In order to get logs through the SBU pins in Google Pixel devices, it is
> necessary to add support to the devicetree, and enable the corresponding
> UART. With this code and an USB-Cereal board, I was able to get full kernel
> logs through serial.
>
> This series depends on the patch adding a sdm670-google-common dtsi for
> dealing with both the Pixel 3a and Pixel 3a XL:
> https://lore.kernel.org/all/20260310002606.16413-5-mailingradian@gmail.com/
Please resubmit when this dependency has been accepted, or perhaps work
together with Richard to send the two series together, if possible.
>
> This is my first patch sent to the kernel mailing list, so apologies ahead
> if I missed something in the documentation.
>
Once the patches are applied, the "Depends on" statement in patch 3
doesn't carry any value, so please drop that.
Other than that, I think this looks good.
Thank,
Bjorn
> Signed-off-by: Pablo Correa Gómez <pabloyoyoista@postmarketos.org>
> ---
> Changes in v2:
> - Fix typo in (3/3) commit message
> - Link to v1: https://patch.msgid.link/20260328-pabloyoyoista-debug-uart-on-rdacayan-next-v1-0-0babf584acdb@postmarketos.org
>
> ---
> Pablo Correa Gómez (3):
> arm64: dts: qcom: sdm670: add default uart pinctrl nodes
> arm64: dts: qcom: sdm670: add debug uart soc node
> arm64: dts: qcom: sdm670-google-common: enable debug uart
>
> arch/arm64/boot/dts/qcom/sdm670-google-common.dtsi | 18 +-
> arch/arm64/boot/dts/qcom/sdm670.dtsi | 207 +++++++++++++++++++++
> 2 files changed, 224 insertions(+), 1 deletion(-)
> ---
> base-commit: 54f966f63b379d0c62bb044b7903319776443a4a
> change-id: 20260328-pabloyoyoista-debug-uart-on-rdacayan-next-69274358cef4
> prerequisite-message-id: <20260217002738.133534-1-mailingradian@gmail.com>
> prerequisite-patch-id: aeaea0c438bff770d0c4c33d937db5706d642daf
> prerequisite-patch-id: 11ce654ef61d48fdebf1c4e1d9d184cfb7bcaa59
> prerequisite-patch-id: c6203c8a71fe103b983b6ba5fc99a22948118fb4
> prerequisite-patch-id: f9ff17c7e4b4396850f51e45249d2f6e35df3725
> prerequisite-patch-id: faab09f9fe421a6358e5512c6050dd70d3271273
> prerequisite-patch-id: adb88e3ddcbe33db1e3fc60ea4cb195dac18a8b4
> prerequisite-patch-id: 0c8f7fc0835cdda72529fcfa7b654bb97109fae4
> prerequisite-message-id: <20260310002606.16413-1-mailingradian@gmail.com>
> prerequisite-patch-id: eb2d2be6edae3d09973b49f3ff6c48fa0845e7e0
> prerequisite-patch-id: e6d9e56eaeceddee68d56e2ec8ac20997bff52c9
> prerequisite-patch-id: d5e2975adf904bf656827100df7989218ac9e963
> prerequisite-patch-id: 04478351b1304dfe1357d176c8d0c235346ca40f
> prerequisite-patch-id: a54db56ccdf1491c46158a84ac1cae70f74bb36c
>
> Best regards,
> --
> Pablo Correa Gómez <pabloyoyoista@postmarketos.org>
>
>
prev parent reply other threads:[~2026-04-27 19:07 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-28 22:58 [PATCH v2 0/3] Add support and enable the debug UART in the Pixel 3a and Pixel 3a XL Pablo Correa Gómez via B4 Relay
2026-03-28 22:58 ` [PATCH v2 1/3] arm64: dts: qcom: sdm670: add default uart pinctrl nodes Pablo Correa Gómez via B4 Relay
2026-03-28 22:58 ` [PATCH v2 2/3] arm64: dts: qcom: sdm670: add debug uart soc node Pablo Correa Gómez via B4 Relay
2026-03-29 9:55 ` Dmitry Baryshkov
2026-03-30 9:33 ` Konrad Dybcio
2026-03-28 22:58 ` [PATCH v2 3/3] arm64: dts: qcom: sdm670-google-common: enable debug uart Pablo Correa Gómez via B4 Relay
2026-03-29 9:55 ` Dmitry Baryshkov
2026-03-30 9:34 ` Konrad Dybcio
2026-04-27 19:07 ` Bjorn Andersson [this message]
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=ae-zV6gtMA8LLTex@baldur \
--to=andersson@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mailingradian@gmail.com \
--cc=pabloyoyoista@postmarketos.org \
--cc=robh@kernel.org \
--cc=~postmarketos/upstreaming@lists.sr.ht \
/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