From: Bjorn Andersson <andersson@kernel.org>
To: Rahul Samana <rahul.samana@oss.qualcomm.com>
Cc: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>,
Bartosz Golaszewski <brgl@kernel.org>,
Marcel Holtmann <marcel@holtmann.org>,
Luiz Augusto von Dentz <luiz.dentz@gmail.com>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Balakrishna Godavarthi <quic_bgodavar@quicinc.com>,
Rocky Liao <quic_rjliao@quicinc.com>,
Manivannan Sadhasivam <mani@kernel.org>,
Konrad Dybcio <konradybcio@kernel.org>,
Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>,
Bartosz Golaszewski <brgl@bgdev.pl>,
linux-arm-msm@vger.kernel.org, linux-bluetooth@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-pci@vger.kernel.org, linux-pm@vger.kernel.org,
quic_mohamull@quicinc.com, quic_hbandi@quicinc.com,
quic_anubhavg@quicinc.com
Subject: Re: [PATCH v2 6/6] arm64: dts: qcom: rb3gen2: add Industrial BT UART overlay
Date: Sat, 1 Aug 2026 22:38:33 -0500 [thread overview]
Message-ID: <am6572YrbAULTa88@baldur> (raw)
In-Reply-To: <b5afd21c-4516-4a8c-bc8f-6598ec385d27@oss.qualcomm.com>
On Fri, Jul 31, 2026 at 11:20:07PM +0530, Rahul Samana wrote:
>
>
> On 31-07-2026 21:19, Konrad Dybcio wrote:
> > On 7/31/26 4:48 PM, Rahul Samana wrote:
> >>
> >>
> >> On 31-07-2026 18:20, Konrad Dybcio wrote:
> >>> On 7/31/26 8:53 AM, Rahul Samana wrote:
> >>>>
> >>>>
> >>>> On 29-07-2026 18:01, Konrad Dybcio wrote:
> >>>>> On 7/27/26 5:45 PM, Rahul Samana wrote:
> >>>>>> The reworked RB3 Gen 2 Industrial mezzanine keeps the common Industrial
> >>>>>> mezzanine hardware description but routes QCC2072 Bluetooth over UART4
> >>>>>> instead of the default Bluetooth-over-USB path.
> >
> > I only noticed this now - are there 2 variants of the industrial
> > mezz being sold concurrently? Is the already-in-kernel one some
> > sort of a prototype SKU? i.e. should we support both?
> >
> >>>>>>
> >>>>>> Build this variant by applying the common Industrial mezzanine overlay
> >>>>>> first, followed by the BT UART overlay. The overlay models the M.2 E-key
> >>>>>> connector graph endpoints for PCIe and UART, and disables the on-board
> >>>>>> WCN6750 PMU and UART7 path so the M.2 QCC2072 Bluetooth controller can be
> >>>>>> used instead.
> >>>>>
> >>>>> So is the onboard module disabled? Can we not just use two in parallel?
> >>>>>
> >>>>
> >>>> The Industrial mezzanine variants do not support the on-board WCN6750
> >>>> wireless path. The common Industrial mezzanine overlay already disables
> >>>> the on-board Wi-Fi node.
> >>>
> >>> What does 'do not support' it mean here? They are physically present
> >>> as part of the SoM, so unless the lanes are somehow diverted away,
> >>> why is it not?
> >>>
> >>> Konrad
> >>
> >> Hi Konrad,
> >>
> >> The WCN6750 module is physically present on the SoM. What I meant is that
> >> the current Industrial mezzanine devicetree already disables the on-board
> >> Wi-Fi path, and this BT UART variant followed the same board-level policy
> >> for the on-board Bluetooth UART/PMU path.
> >
> > Okay, and do we know why that's the case in the first place?
> >
> >> For the endpoint labels, I can move the UART endpoint to the SoC DTSI,
> >> kodiak.dtsi, as uart4_ep.
> >
> > Please do
> >
> >> For the PCIe endpoint, the PCIe path to the M.2 QCC2072 device is not the
> >> PCIe0 root port in kodiak.dtsi. In the composed devicetree, it is
> >> behind the Industrial mezzanine PCIe switch, under:
> >>
> >> /soc@0/pcie@1c00000/pcie@0/pcie@0,0/pcie@2,0
> >>
> >>
> >> So placing pcie0_port0_ep in kodiak.dtsi would not describe the actual
> >> topology.
> >
> > Yes, that was an omission from my side
> >
> >
> >> I tried adding a generic label/endpoint to that downstream port in the
> >> Industrial mezzanine overlay and referencing it from the BT UART overlay.
> >> However, when the overlays are compiled separately and then composed, that
> >> label is not available while applying the BT UART overlay, so the composed
> >> DTB build fails with FDT_ERR_NOTFOUND.
> >
> > I think this generic description is simply not achievable today without
> > a bigger plan in place
> >
> > Konrad
>
> Hi Konrad, Dmitry,
>
> Yes, both Industrial Kit variants need to be supported.
>
> The default Industrial Kit uses the common Industrial mezzanine hardware
> description and continues to use the existing Bluetooth-over-USB path.
"Kit uses the common" what do you even mean?! The mezzanine is a
physical thing, it doesn't _use_ anything! The mezzanine exists and the
DeviceTree describe what it is.
> The
> reworked Industrial Kit variant keeps the rest of the Industrial mezzanine
> hardware the same, but routes Bluetooth over UART4 instead.
>
Does "reworked mezzanine" mean what it usually does? I.e. that you have
taken the mezzanine and modified it?
[..]
>
> I am open to other suggestions if there is a better way to model this within
> the current overlay structure.
>
I don't think it's possible to give you other suggestions as you haven't
described what this thing is, you just explaining how you're hacking
something together and ask us to help adjust your hack without knowing
what it is you're trying to describe with your DeviceTree.
Regards,
Bjorn
> Thanks,
> Rahul
>
prev parent reply other threads:[~2026-08-02 3:38 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-27 15:45 [PATCH v2 0/6] Bluetooth: qca: enable QCC2072 on RB3 Gen 2 Industrial Rahul Samana
2026-07-27 15:45 ` [PATCH v2 1/6] dt-bindings: bluetooth: qca: add QCC2072 Rahul Samana
2026-07-31 9:46 ` Krzysztof Kozlowski
2026-07-31 14:45 ` Rahul Samana
2026-07-31 15:04 ` Krzysztof Kozlowski
2026-07-31 15:51 ` Rahul Samana
2026-08-01 14:48 ` Krzysztof Kozlowski
2026-08-01 15:31 ` Rahul Samana
2026-08-01 15:49 ` Krzysztof Kozlowski
2026-07-27 15:45 ` [PATCH v2 2/6] Bluetooth: qca: add QCC2072 support Rahul Samana
2026-07-31 15:09 ` Dmitry Baryshkov
2026-07-31 17:51 ` Rahul Samana
2026-07-27 15:45 ` [PATCH v2 3/6] power: sequencing: pwrseq-pcie-m2: add QCC2072 Rahul Samana
2026-07-27 15:45 ` [PATCH v2 4/6] arm64: dts: qcom: qcs6490-rb3gen2: label BT PMU and M.2 PCI node Rahul Samana
2026-07-31 15:30 ` Rahul Samana
2026-07-27 15:45 ` [PATCH v2 5/6] arm64: dts: qcom: kodiak: mark PCIe root port as bridge Rahul Samana
2026-07-27 15:57 ` Rahul Samana
2026-07-29 12:29 ` Konrad Dybcio
2026-07-27 15:45 ` [PATCH v2 6/6] arm64: dts: qcom: rb3gen2: add Industrial BT UART overlay Rahul Samana
2026-07-29 12:31 ` Konrad Dybcio
2026-07-31 6:53 ` Rahul Samana
2026-07-31 12:50 ` Konrad Dybcio
2026-07-31 14:48 ` Rahul Samana
2026-07-31 15:49 ` Konrad Dybcio
2026-07-31 17:50 ` Rahul Samana
2026-08-02 3:38 ` 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=am6572YrbAULTa88@baldur \
--to=andersson@kernel.org \
--cc=brgl@bgdev.pl \
--cc=brgl@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.baryshkov@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-bluetooth@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=luiz.dentz@gmail.com \
--cc=mani@kernel.org \
--cc=marcel@holtmann.org \
--cc=quic_anubhavg@quicinc.com \
--cc=quic_bgodavar@quicinc.com \
--cc=quic_hbandi@quicinc.com \
--cc=quic_mohamull@quicinc.com \
--cc=quic_rjliao@quicinc.com \
--cc=rahul.samana@oss.qualcomm.com \
--cc=robh@kernel.org \
/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