From: sashiko-bot@kernel.org
To: erebion <erebion@erebion.eu>
Cc: conor+dt@kernel.org, dri-devel@lists.freedesktop.org,
robh@kernel.org, devicetree@vger.kernel.org
Subject: Re: [PATCH v2 5/8] arm64: dts: qcom: sm6350-lena: Enable Bluetooth
Date: Sat, 26 Sep 2026 14:11:30 +0000 [thread overview]
Message-ID: <20260926141130.A47171F000FF@smtp.kernel.org> (raw)
In-Reply-To: <20260926-pdx213-upstream-3-v2-5-b4612e3eb536@erebion.eu>
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] dt-bindings: missing swctrl-gpios schema definition causes dtbs_check failure
--
commit 2895c104a8a87260afe61b3db500946505dbefb6
Author: erebion <erebion@erebion.eu>
Subject: arm64: dts: qcom: sm6350-lena: Enable Bluetooth
Configure the required regulators for Bluetooth and enable it on the
Xperia 10 III.
> diff --git a/arch/arm64/boot/dts/qcom/sm6350-sony-xperia-lena-pdx213.dts b/arch/arm64/boot/dts/qcom/sm6350-sony-xperia-lena-pdx213.dts
> index f3ed6445c7d35..9be0e7ceccb55 100644
> --- a/arch/arm64/boot/dts/qcom/sm6350-sony-xperia-lena-pdx213.dts
> +++ b/arch/arm64/boot/dts/qcom/sm6350-sony-xperia-lena-pdx213.dts
[ ... ]
> @@ -424,6 +466,73 @@ ts_int_default: ts-int-default-state {
[ ... ]
> +&uart1 {
> + /delete-property/ interrupts;
> + interrupts-extended = <&intc GIC_SPI 602 IRQ_TYPE_LEVEL_HIGH>,
> + <&tlmm 64 IRQ_TYPE_EDGE_FALLING>;
> +
> + pinctrl-names = "default", "sleep";
> + pinctrl-1 = <&qup_uart1_sleep_cts>, <&qup_uart1_sleep_rts>, <&qup_uart1_sleep_tx>, <&qup_uart1_sleep_rx>;
> +
> + status = "okay";
> +
> + bluetooth {
> + compatible = "qcom,wcn3991-bt";
> + vddio-supply = <&pm6350_l11>;
> + vddxo-supply = <&pm6350_l7>;
> + vddrf-supply = <&pm6150l_l2>;
> + vddch0-supply = <&pm6150l_l10>;
> + swctrl-gpios = <&tlmm 69 GPIO_ACTIVE_HIGH>;
[Severity: Low]
Does this property need to be added to the dt-bindings?
When running make dtbs_check, the swctrl-gpios property here fails validation
because it isn't documented in the qcom,wcn3990-bt.yaml schema. Since the
schema enforces unevaluatedProperties: false, the build generates a warning.
The hci_qca driver parses this property correctly at runtime, so there
isn't a functional issue, but should the binding schema be updated to avoid
the validation warning?
> + firmware-name = "crnv32u.bin";
> + };
> };
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260926-pdx213-upstream-3-v2-0-b4612e3eb536@erebion.eu?part=5
next prev parent reply other threads:[~2026-09-26 14:11 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-26 14:00 [PATCH v2 0/8] Enable more nodes for the Xperia 10 III erebion
2026-09-26 14:00 ` [PATCH v2 1/8] arm64: dts: qcom: sm6350-lena: Add flash LED erebion
2026-09-27 21:00 ` Luca Weiss
2026-09-28 1:42 ` Bjorn Andersson
2026-09-28 17:33 ` Luca Weiss
2026-09-26 14:00 ` [PATCH v2 2/8] arm64: dts: qcom: sm6350-lena: Add qcom,rmtfs-mem node erebion
2026-09-26 14:00 ` [PATCH v2 3/8] arm64: dts: qcom: sm6350-lena: Enable ADSP, CDSP & MPSS erebion
2026-09-26 14:00 ` [PATCH v2 4/8] arm64: dts: qcom: sm6350-lena: Enable IPA erebion
2026-09-26 14:00 ` [PATCH v2 5/8] arm64: dts: qcom: sm6350-lena: Enable Bluetooth erebion
2026-09-26 14:11 ` sashiko-bot [this message]
2026-09-26 14:00 ` [PATCH v2 6/8] arm64: dts: qcom: sm6350-lena: Enable wifi erebion
2026-09-26 14:00 ` [PATCH v2 7/8] arm64: dts: qcom: sm6350-lena: Enable PM7250B erebion
2026-09-26 14:00 ` [PATCH v2 8/8] arm64: dts: qcom: sm6350-lena: Enable PMK8350 erebion
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=20260926141130.A47171F000FF@smtp.kernel.org \
--to=sashiko-bot@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=erebion@erebion.eu \
--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.