From: Stephan Gerhold <stephan.gerhold@linaro.org>
To: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Cc: Paul Sajna <sajattack@postmarketos.org>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konradybcio@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
David Heidelberg <david@ixit.cz>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, phone-devel@vger.kernel.org,
Amir Dahan <system64fumo@protonmail.com>,
Christopher Brown <crispybrown@gmail.com>
Subject: Re: [PATCH v2 11/13] arm64: dts: qcom: sdm845-lg-common: Add bootloader-compatible board and MSM IDs
Date: Wed, 17 Sep 2025 16:02:29 +0200 [thread overview]
Message-ID: <aMq_dbLRShTVLlc8@linaro.org> (raw)
In-Reply-To: <acb4f8a2-ff08-4a90-a7ad-7b25a7f4b6b0@oss.qualcomm.com>
On Wed, Sep 17, 2025 at 03:55:47PM +0200, Konrad Dybcio wrote:
> On 9/17/25 3:09 AM, Paul Sajna wrote:
> > The original bootloader (fastboot) requires board and MSM IDs in order
> > to boot the kernel, so these have been added. When re-using the same
> > files to chainload U-Boot, these compatibles are also needed to ensure
> > proper booting.
> >
> > Signed-off-by: Paul Sajna <sajattack@postmarketos.org>
> > ---
> > arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi | 2 ++
> > arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts | 1 +
> > 2 files changed, 3 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> > index 1baec758c7d881026bf6001a0a305687d7a14b40..a513cd931b3a984443183ee4e8df7b0e45732630 100644
> > --- a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> > @@ -5,6 +5,7 @@
> > * Copyright (c) 2022, The Linux Foundation. All rights reserved.
> > */
> >
> > +#include <dt-bindings/arm/qcom,ids.h>
> > #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/leds/common.h>
> > #include <dt-bindings/pinctrl/qcom,pmic-gpio.h>
> > @@ -29,6 +30,7 @@
> >
> > / {
> > chassis-type = "handset";
> > + qcom,msm-id = <QCOM_ID_SDM845 0x20001>;
> >
> > aliases {
> > serial0 = &uart9;
> > diff --git a/arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts b/arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts
> > index 8c1692f86e6ceea7b718361965e78f95d39373bb..beb1372dc6c17f5f06c2044412ee1c8165858cd1 100644
> > --- a/arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts
> > +++ b/arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts
> > @@ -12,6 +12,7 @@
> > / {
> > model = "LG G7 ThinQ";
> > compatible = "lg,judyln", "qcom,sdm845";
> > + qcom,board-id = <0x020C0008 0>;
>
> This is almost QCOM_BOARD_ID(MTP, 12, 0), except for the odd BIT(25)
> which the dt-binding promises should be left unused..
>
> +Stephan maybe you know more?
>
I'm not sure what BIT(25) is exactly, but I think you can generally give
up any hope of trying to make sense of the board IDs of large vendors
like LG, Samsung, Motorola etc. It's a pity that Qualcomm has never
created a board ID scheme with enough room for customization for
vendors. The qcom,board-id scheme seems to be entirely focused just
around Qualcomm's own boards (MTP, QRD, SBC etc). That is why we ended
up with every vendor using their own incompatible approach. :-(
Sorry, I wish I had a better explanation. :-)
Stephan
next prev parent reply other threads:[~2025-09-17 14:02 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-17 1:09 [PATCH v2 00/13] arm64: dts: qcom: sdm845-lg-{common, judyln}: Improve HW support in dts Paul Sajna
2025-09-17 1:09 ` [PATCH v2 01/13] arm64: dts: qcom: sdm845-lg-common: Sort nodes and properties Paul Sajna
2025-09-17 13:38 ` Konrad Dybcio
2025-09-18 3:13 ` Dmitry Baryshkov
2025-09-17 1:09 ` [PATCH v2 02/13] arm64: dts: qcom: sdm845-lg-common: Add uarts and Bluetooth Paul Sajna
2025-09-17 13:40 ` Konrad Dybcio
2025-09-18 2:17 ` Paul Sajna
2025-09-17 1:09 ` [PATCH v2 03/13] arm64: dts: qcom: sdm845-lg-judyln: Add battery and charger Paul Sajna
2025-09-17 13:47 ` Konrad Dybcio
2025-09-18 1:44 ` Chris Brown
[not found] ` <CABN5=wFgARbrhqyNbgQ_iMKPt8M7ixZPEhvXVd6AY+Jv+7=YWw@mail.gmail.com>
2025-10-07 13:52 ` Konrad Dybcio
2025-10-23 9:12 ` Pavel Machek
2025-09-17 1:09 ` [PATCH v2 04/13] arm64: dts: qcom: sdm845-lg-common: Add leds Paul Sajna
2025-09-17 13:42 ` Konrad Dybcio
2025-10-23 9:10 ` Pavel Machek
2025-09-17 1:09 ` [PATCH v2 05/13] arm64: dts: qcom: sdm845-lg-judyln: Add display panel Paul Sajna
2025-09-17 13:50 ` Konrad Dybcio
2025-09-18 1:51 ` Paul Sajna
[not found] ` <da2f17a144374f326de9e0c5bec638829a31cd1e@postmarketos.org>
2025-10-07 13:53 ` Konrad Dybcio
2025-09-17 1:09 ` [PATCH v2 06/13] arm64: dts: qcom: sdm845-lg-judyln: Add firmware nodes Paul Sajna
2025-09-17 13:51 ` Konrad Dybcio
2025-09-18 1:52 ` Paul Sajna
2025-09-17 1:09 ` [PATCH v2 07/13] arm64: dts: qcom: sdm845-lg-{common, judyln}: Add wifi node Paul Sajna
2025-09-17 13:44 ` Konrad Dybcio
2025-09-17 1:09 ` [PATCH v2 08/13] arm64: dts: qcom: sdm845-lg-common: Add chassis-type Paul Sajna
2025-09-17 13:44 ` Konrad Dybcio
2025-09-17 1:09 ` [PATCH v2 09/13] arm64: dts: qcom: sdm845-lg-judyln: Add fb_panel dimensions Paul Sajna
2025-09-17 13:59 ` Konrad Dybcio
2025-09-17 14:54 ` Alexey Minnekhanov
2025-10-07 13:55 ` Konrad Dybcio
2025-10-09 18:02 ` Alexey Minnekhanov
2025-09-17 1:09 ` [PATCH v2 10/13] arm64: dts: qcom: sdm845-lg-common: Add camera flash Paul Sajna
2025-09-17 13:45 ` Konrad Dybcio
2025-09-17 1:09 ` [PATCH v2 11/13] arm64: dts: qcom: sdm845-lg-common: Add bootloader-compatible board and MSM IDs Paul Sajna
2025-09-17 13:55 ` Konrad Dybcio
2025-09-17 14:02 ` Stephan Gerhold [this message]
2025-09-17 1:09 ` [PATCH v2 12/13] arm64: dts: qcom: sdm845-judyln-common: Remove framebuffer reserved-mem Paul Sajna
2025-09-17 13:58 ` Konrad Dybcio
2025-09-17 1:09 ` [PATCH v2 13/13] arm64: dts: qcom: sdm845-lg-judyln: Add lab/ibb Paul Sajna
2025-09-17 13:46 ` Konrad Dybcio
2025-09-17 15:00 ` [PATCH v2 00/13] arm64: dts: qcom: sdm845-lg-{common, judyln}: Improve HW support in dts 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=aMq_dbLRShTVLlc8@linaro.org \
--to=stephan.gerhold@linaro.org \
--cc=andersson@kernel.org \
--cc=conor+dt@kernel.org \
--cc=crispybrown@gmail.com \
--cc=david@ixit.cz \
--cc=devicetree@vger.kernel.org \
--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=phone-devel@vger.kernel.org \
--cc=robh@kernel.org \
--cc=sajattack@postmarketos.org \
--cc=system64fumo@protonmail.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 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.