linux-qcom-next: Enable BT_RFCOMM, BT_BNEP and UHID

[WeiDeng-weiden]

Enable as modules the BT_BNEP and BT_RFCOMM drivers which are required for Bluetooth profiles such as PAN and SPP. Without these modules, PAN and SPP functionality will not be available. Enable UHID support to allow HID over GATT operations. Without this, user-space cannot create virtual HID devices, which impacts HID profile usage over Bluetooth.

[lumag]

s/ config//

[github-actions]

Test run workflow

Test jobs for commit 7ff739f

• Job 121904 on qcs9100-rb8 - boot test (iq-9075-evk) 19900184782
• Job 121896 on qcs9100 - boot test (qcs9100-ride-sx) 19900184782
• Job 121903 on dragonboard-820c - boot test (dragonboard-820c) 19900184782
• Job 121894 on qcs6490 - boot test (rb3gen2-core-kit) 19900184782
• Job 121892 on dragonboard-410c - boot test (dragonboard-410c) 19900184782
• Job 121899 on qrb2210-rb1 - boot test (rb1-core-kit) 19900184782
• Job 121901 on qcs9100-rb8 - boot test (iq-9075-evk) 19900184782
• Job 121900 on qcs8300 - boot test (qcs8300-ride-sx) 19900184782
• Job 121898 on dragonboard-820c - boot test (dragonboard-820c) 19900184782
• Job 121893 on dragonboard-410c - boot test (dragonboard-410c) 19900184782
• Job 121895 on qrb2210-rb1 - boot test (rb1-core-kit) 19900184782
• Job 121902 on qcs8300 - boot test (qcs8300-ride-sx) 19900184782
• Job 121891 on qcs6490 - boot test (rb3gen2-core-kit) 19900184782
• Job 121897 on qcs9100 - boot test (qcs9100-ride-sx) 19900184782

[test-reporting-app]

Test Results

 14 files  ± 0   28 suites  +1   38m 25s ⏱️ - 36m 56s
 50 tests  -  3   50 ✅ + 2  0 💤 ±0  0 ❌  - 5 
304 runs  +13  304 ✅ +18  0 💤 ±0  0 ❌  - 5 

Results for commit 0211486. ± Comparison against base commit f1e6892.

This pull request removes 3 tests.

lava ‑ downloads
lava ‑ lava-overlay
lava ‑ test-definition

♻️ This comment has been updated with latest results.

[WeiDeng-weiden]

s/ config//

Thanks for the review.

For the comment s/ config//, could you please clarify which config you would like me to remove?

[lumag]

s/ config//

Thanks for the review.

For the comment s/ config//, could you please clarify which config you would like me to remove?

I see only one place where the regexp is applicable.

[WeiDeng-weiden]

s/ config//

Thanks for the review.
For the comment s/ config//, could you please clarify which config you would like me to remove?

I see only one place where the regexp is applicable.

Here is my current diff:
Could you please point out the exact line (file and line number) where you’d like me to remove config, or let me know if this comment can be ignored with the latest patch set?

diff --git a/recipes-kernel/linux/linux-qcom-next/configs/qcom.cfg b/recipes-kernel/linux/linux-qcom-next/configs/qcom.cfg
index b5a294f..837a6aa 100644
--- a/recipes-kernel/linux/linux-qcom-next/configs/qcom.cfg
+++ b/recipes-kernel/linux/linux-qcom-next/configs/qcom.cfg
@@ -308,3 +308,11 @@ CONFIG_NFT_TPROXY=m
CONFIG_NFT_TUNNEL=m
CONFIG_PACKET_DIAG=y
CONFIG_VETH=m
+
+# Enable BT_RFCOMM, BT_BNEP and UHID
+CONFIG_BT_BNEP=m
+CONFIG_BT_BNEP_MC_FILTER=y
+CONFIG_BT_BNEP_PROTO_FILTER=y
+CONFIG_BT_RFCOMM=m
+CONFIG_BT_RFCOMM_TTY=y
+CONFIG_UHID=m

[lumag]

s/ config//

Thanks for the review.
For the comment s/ config//, could you please clarify which config you would like me to remove?

I see only one place where the regexp is applicable.

Here is my current diff: Could you please point out the exact line (file and line number) where you’d like me to remove config, or let me know if this comment can be ignored with the latest patch set?

You've posted a diff, but it's not a full patch. There is an extra " config" in the commit subject.

[WeiDeng-weiden]

s/ config//

Thanks for the review.
For the comment s/ config//, could you please clarify which config you would like me to remove?

I see only one place where the regexp is applicable.

Here is my current diff: Could you please point out the exact line (file and line number) where you’d like me to remove config, or let me know if this comment can be ignored with the latest patch set?

You've posted a diff, but it's not a full patch. There is an extra " config" in the commit subject.

OK, will remove config in the commit subject.

[ricardosalveti]

Please rebase, build failure is related to the mesa updates (fixed in master).

[WeiDeng-weiden]

Please rebase, build failure is related to the mesa updates (fixed in master).

Rebased and re-submitted.

[github-actions]

Test run workflow

Test jobs for commit 0211486

• Job 122419 on qrb2210-rb1 - boot test (rb1-core-kit) 19952632699
• Job 122417 on qcs9100 - boot test (qcs9100-ride-sx) 19952632699
• Job 122411 on dragonboard-820c - boot test (dragonboard-820c) 19952632699
• Job 122414 on qcs6490 - boot test (rb3gen2-core-kit) 19952632699
• Job 122408 on dragonboard-410c - boot test (dragonboard-410c) 19952632699
• Job 122413 on qcs6490 - boot test (rb3gen2-core-kit) 19952632699
• Job 122416 on qrb2210-rb1 - boot test (rb1-core-kit) 19952632699
• Job 122412 on qcs8300 - boot test (qcs8300-ride-sx) 19952632699
• Job 122407 on qcs9100 - boot test (qcs9100-ride-sx) 19952632699
• Job 122418 on qcs9100-rb8 - boot test (iq-9075-evk) 19952632699
• Job 122415 on dragonboard-820c - boot test (dragonboard-820c) 19952632699
• Job 122410 on qcs9100-rb8 - boot test (iq-9075-evk) 19952632699
• Job 122420 on qcs8300 - boot test (qcs8300-ride-sx) 19952632699
• Job 122409 on dragonboard-410c - boot test (dragonboard-410c) 19952632699

[lumag]

At a second consideration: please send these changes to the upstream Linux kernel, then backport them to qcom-next as usual. There is nothing distro-specific nor BSP-specific in these Kconfig entries.

[WeiDeng-weiden]

At a second consideration: please send these changes to the upstream Linux kernel, then backport them to qcom-next as usual. There is nothing distro-specific nor BSP-specific in these Kconfig entries.

Sure, will send to upstream then get back.