Re: [PATCH 3/4] arm64: dts: qcom: qcs6490-rb3gen2-industrial-mezzanine: Add QCC2072 Bluetooth M.2 overlay

[Dmitry Baryshkov]

On Fri, May 29, 2026 at 11:32:34PM +0530, Yepuri Siddu wrote:
> Add a DTSO overlay for the M.2 E-key slot on the QCS6490 RB3 Gen2
> industrial mezzanine board when populated with a QCC2072 Bluetooth
> module.
> 
> The overlay configures UART4 (gpio16-19) for Bluetooth communication
> at up to 3.2 Mbps. Sleep pin states are defined for all four UART
> lines to minimize power consumption when Bluetooth is inactive. Host
> wakeup is handled via a GPIO interrupt on the RX line (gpio19), and
> SW_CTRL is mapped to gpio86 for power sequencing.
> 
> Disable UART7 in the base industrial mezzanine overlay as it is
> not used when the M.2 slot is populated.
> 
> Signed-off-by: Yepuri Siddu <yepuri.siddu@xxxxxxxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/qcom/Makefile             |   2 +
>  ...3gen2-industrial-mezzanine-m2-cologne.dtso | 110 ++++++++++++++++++
>  .../qcs6490-rb3gen2-industrial-mezzanine.dtso |   4 +
>  3 files changed, 116 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/qcom/qcs6490-rb3gen2-industrial-mezzanine-m2-cologne.dtso
> 
> +&uart4 {
> +	status = "okay";

Status should be the last, empty line before it.

> +	/delete-property/ interrupts;
> +	interrupts-extended = <&intc GIC_SPI 605 IRQ_TYPE_LEVEL_HIGH>,
> +				<&tlmm 19 IRQ_TYPE_EDGE_FALLING>;

Align on '<'

> +	pinctrl-names = "default", "sleep";
> +	pinctrl-1 = <&qup_uart4_sleep_cts>, <&qup_uart4_sleep_rts>,
> +			<&qup_uart4_sleep_tx>, <&qup_uart4_sleep_rx>;

The same.

> +
> +	bluetooth: bluetooth {
> +		compatible = "qcom,qcc2072-bt";

If it is M.2 slot, then it can have any BT card, not necessarily
QCC2072.

> +		max-speed = <3200000>;
> +	};
> +};

-- 
With best wishes
Dmitry