Linux bluetooth development
 help / color / mirror / Atom feed
From: Rahul Samana <rahul.samana@oss.qualcomm.com>
To: Bjorn Andersson <andersson@kernel.org>
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: Wed, 5 Aug 2026 11:48:48 +0530	[thread overview]
Message-ID: <a86124c9-a5ff-4265-8a14-68b25f010577@oss.qualcomm.com> (raw)
In-Reply-To: <am6572YrbAULTa88@baldur>



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.

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

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.

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-05  6:18 UTC|newest]

Thread overview: 33+ 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 [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=a86124c9-a5ff-4265-8a14-68b25f010577@oss.qualcomm.com \
    --to=rahul.samana@oss.qualcomm.com \
    --cc=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=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