All of lore.kernel.org
 help / color / mirror / Atom feed
From: sashiko-bot@kernel.org
To: "Benjamin Hahn" <B.Hahn@phytec.de>
Cc: devicetree@vger.kernel.org, imx@lists.linux.dev, robh@kernel.org,
	conor+dt@kernel.org, Frank.Li@kernel.org
Subject: Re: [PATCH v2 3/5] arm64: dts: imx95-phyflex-libra-rdk: Add overlays
Date: Fri, 13 Nov 2026 16:16:50 +0000	[thread overview]
Message-ID: <sashiko-outbox-213180@kernel.org> (raw)
In-Reply-To: <20261113-wip-bhahn-imx95_upstreaming-v2-3-3ef58af5e689@phytec.de>

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Medium] arm64: dts: imx95-phyflex-libra-rdk: missing OF graph link in
  BT overlay
--

commit 924f1473173fca2dbc62cd020eeba54504832201
Author: Benjamin Hahn <B.Hahn@phytec.de>

arm64: dts: imx95-phyflex-libra-rdk: Add overlays

This commit adds devicetree overlays for the SoM temperature sensors and
for M.2 Key-E Wi-Fi/Bluetooth cards using the nxp,88w8987-bt binding on
the phyFLEX-i.MX 95 Libra RDK.

> diff --git a/arch/arm64/boot/dts/freescale/imx95-phyflex-libra-rdk-bluetooth-88w8987.dtso b/arch/arm64/boot/dts/freescale/imx95-phyflex-libra-rdk-bluetooth-88w8987.dtso
> new file mode 100644
> index 0000000000000..1601e04b3940f
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/imx95-phyflex-libra-rdk-bluetooth-88w8987.dtso
> @@ -0,0 +1,24 @@
> +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
> +/*
> + * Copyright (C) 2026 PHYTEC Messtechnik GmbH
> + */
> +/dts-v1/;
> +/plugin/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +
> +&lpuart8 {
> +	bluetooth {
> +		compatible = "nxp,88w8987-bt";
> +		fw-init-baudrate = <115200>;
> +	};
> +};

[Severity: Medium]
Does this overlay also need OF graph endpoints linking lpuart8 and
port@3 of m2-key-e-connector?

In imx95-phyflex-libra-rdk.dts, m2-key-e-connector uses
compatible = "pcie-m2-e-connector" and claims w-disable2-gpios.  When
CONFIG_POWER_SEQUENCING_PCIE_M2 is enabled, pwrseq_pcie_m2_probe()
requests w-disable2-gpios with GPIOD_OUT_HIGH, holding the active-low
W_nDISABLE2 line asserted until a consumer enables the "uart" power
sequencer target.

In drivers/bluetooth/btnxpuart.c, nxp_serdev_probe() checks
nxp_m2_connector_is_available() and calls pwrseq_get(..., "uart"), both
of which require an OF graph link between the UART controller and the
M.2 connector; without it, W_nDISABLE2 remains asserted and the M.2
Bluetooth radio stays disabled.  Adding the OF graph endpoints in this
overlay rather than the base DTB also keeps lpuart8 exposed as a normal
TTY device when the Bluetooth overlay is not applied.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261113-wip-bhahn-imx95_upstreaming-v2-0-3ef58af5e689@phytec.de?part=3

  reply	other threads:[~2026-11-13 16:16 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-13 16:00 [PATCH v2 0/5] arm64: dts: update imx95 Libra for the newest hardware version Benjamin Hahn
2026-11-13 16:00 ` [PATCH v2 1/5] dt-bindings: arm: fsl: Add phyFLEX-i.MX 95 Libra RDK Benjamin Hahn
2026-11-13 16:48   ` Conor Dooley
2026-11-13 16:00 ` [PATCH v2 2/5] arm64: dts: imx95-phyflex-libra-rdk: " Benjamin Hahn
2026-11-13 16:42   ` sashiko-bot
2026-11-13 16:00 ` [PATCH v2 3/5] arm64: dts: imx95-phyflex-libra-rdk: Add overlays Benjamin Hahn
2026-11-13 16:16   ` sashiko-bot [this message]
2026-11-13 16:32   ` Frank Li
2026-11-13 16:00 ` [PATCH v2 4/5] arm64: dts: imx95-libra-rdk-fpsc: remove devicetree for old Libra Benjamin Hahn
2026-11-13 16:00 ` [PATCH v2 5/5] dt-bindings: arm: fsl: remove outdated PHYTEC i.MX95 FPSC Benjamin Hahn
2026-11-13 16:48   ` Conor Dooley

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=sashiko-outbox-213180@kernel.org \
    --to=sashiko-bot@kernel.org \
    --cc=B.Hahn@phytec.de \
    --cc=Frank.Li@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=imx@lists.linux.dev \
    --cc=robh@kernel.org \
    --cc=sashiko-reviews@lists.linux.dev \
    /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.