Linux bluetooth development
 help / color / mirror / Atom feed
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: Thu, 6 Aug 2026 18:02:56 -0500	[thread overview]
Message-ID: <anURderHHz5WHIRe@baldur> (raw)
In-Reply-To: <a86124c9-a5ff-4265-8a14-68b25f010577@oss.qualcomm.com>

On Wed, Aug 05, 2026 at 11:48:48AM +0530, Rahul Samana wrote:
> 
> 
> On 02-08-2026 09:08, Bjorn Andersson wrote:
> > 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?
> > 
> 
> Hi Bjorn,
> 
> Let me clarify.
> 
> There are two physical RB3 Gen 2 Industrial mezzanine variants that need
> to be supported:
> 
> 1. The existing RB3 Gen 2 Industrial mezzanine variant currently
>    described in-tree, where the M.2 E-key interface routes Bluetooth over
>    USB.
> 
> 2. A reworked RB3 Gen 2 Industrial mezzanine variant, where the
>    mezzanine board was reworked to make the existing M.2 E-key interface
>    route Bluetooth over UART4.
> 

Reworked as in: "there exist a dozen such boards in the world" or is
this an actually supported configuration that can be purchased?


The fact that you call this mezzanine "the BT-UART mezzanine" forces me
to assume that this isn't an actual product, prove me wrong please.

> The reworked variant is a physical mezzanine board variant, not a
> software-only configuration of the same board.
> 

That's obvious.

> The intent of this overlay is to describe only that board-level
> difference while reusing the common Industrial mezzanine description for
> the hardware that remains unchanged.
> 

But then you need to clearly describe what the difference between the
two boards is and model that in a clear way.

Regards,
Bjorn

> Thanks,
> Rahul
> 
> 
> > [..]
> >>
> >> 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
> >>
> 

      reply	other threads:[~2026-08-06 23:03 UTC|newest]

Thread overview: 34+ 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-27 16:32   ` Bluetooth: qca: enable QCC2072 on RB3 Gen 2 Industrial bluez.test.bot
2026-07-31  9:46   ` [PATCH v2 1/6] dt-bindings: bluetooth: qca: add QCC2072 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-08-05 12:09                 ` Manivannan Sadhasivam
2026-08-05 12:33                   ` Krzysztof Kozlowski
2026-08-05 14:12                     ` Manivannan Sadhasivam
2026-08-05 14:36                       ` Krzysztof Kozlowski
2026-08-05 14:52                         ` Manivannan Sadhasivam
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
2026-08-05  6:18                 ` Rahul Samana
2026-08-06 23:02                   ` 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=anURderHHz5WHIRe@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