Skip to content

fix(etc.d/modprobe.d): drop bt_coex_active=0 option for iwlwifi#48

Merged
BLumia merged 1 commit into
masterfrom
bai/drop-iwlwifi-bt_coex_active
Feb 24, 2025
Merged

fix(etc.d/modprobe.d): drop bt_coex_active=0 option for iwlwifi#48
BLumia merged 1 commit into
masterfrom
bai/drop-iwlwifi-bt_coex_active

Conversation

@MingcongBai

Copy link
Copy Markdown
Contributor

This was introduced at least 6 years ago and causes boot errors on devices with newer Intel Wireless cards:

iwlmvm doesn't allow to disable BT Coex, check bt_coex_active module parameter

Drop this antiquated (and probably unneeded version).

@github-actions

github-actions Bot commented Jul 30, 2024

Copy link
Copy Markdown

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@MingcongBai
MingcongBai force-pushed the bai/drop-iwlwifi-bt_coex_active branch from 64b67df to 0683c2b Compare July 30, 2024 01:45
This was introduced at least 6 years ago and causes boot errors on devices
with newer Intel Wireless cards:

iwlmvm doesn't allow to disable BT Coex, check bt_coex_active module parameter

Drop this antiquated (and probably unneeded version).
@MingcongBai
MingcongBai force-pushed the bai/drop-iwlwifi-bt_coex_active branch from 0683c2b to 8a17ef8 Compare July 30, 2024 01:46
@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

关键摘要:

  • 移除了bt_coex_active选项,需要确认这是否是有意为之。
  • 代码格式化:选项之间添加了空格,提高了可读性。

是否建议立即修改:

  • 是,如果移除bt_coex_active选项是无意的,则需要立即修复。
  • 否,如果这是一个有意的更改,并且已经过充分讨论和审查,则可以不立即修改。

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: justforlxz, MingcongBai
Once this PR has been reviewed and has the lgtm label, please assign hudeng-go for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@deepin-bot

deepin-bot Bot commented Sep 2, 2024

Copy link
Copy Markdown
Contributor

TAG Bot

New tag: 2024.08.13
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #49

@Zeno-sole

Copy link
Copy Markdown
Contributor

/integrate

@github-actions

github-actions Bot commented Sep 3, 2024

Copy link
Copy Markdown

AutoIntegrationPr Bot
auto integrate with pr url: deepin-community/Repository-Integration#1960
PrNumber: 1960
PrBranch: auto-integration-10675110462

@deepin-bot

deepin-bot Bot commented Nov 25, 2024

Copy link
Copy Markdown
Contributor

TAG Bot

New tag: 2024.11.25
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #54

@deepin-bot

deepin-bot Bot commented Nov 29, 2024

Copy link
Copy Markdown
Contributor

TAG Bot

New tag: 2024.11.29
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #56

@deepin-bot

deepin-bot Bot commented Dec 12, 2024

Copy link
Copy Markdown
Contributor

TAG Bot

New tag: 2024.12.12
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #58

@deepin-bot

deepin-bot Bot commented Dec 14, 2024

Copy link
Copy Markdown
Contributor

TAG Bot

New tag: 2024.12.13
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #59

@BLumia
BLumia merged commit 5fe08ea into master Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants