Skip to content

Add workaround for BCM4377b Wi-Fi failing after lid close#688

Open
herzj-tcd wants to merge 2 commits intot2linux:masterfrom
herzj-tcd:master
Open

Add workaround for BCM4377b Wi-Fi failing after lid close#688
herzj-tcd wants to merge 2 commits intot2linux:masterfrom
herzj-tcd:master

Conversation

@herzj-tcd
Copy link

Adds documentation for a known issue on T2 Macs with the BCM4377b Wi-Fi chip (confirmed on MacBookAir9,1) where brcmfmac fails to enter D3 power state during suspend, leaving Wi-Fi and Bluetooth unresponsive after the lid is opened.

The root cause is a Broadcom firmware bug that has been reported upstream but has no fix available. The workaround involves forcing s2idle sleep and using a post-resume systemd script to rebind the driver and reload the firmware.

```

!!! Warning "Do not unload the module pre-suspend"
Unloading `brcmfmac` before suspend (e.g. with `modprobe -r`) causes a hard crash on
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is weird. I've seen many people simply unloading and suspending.

Comment on lines +245 to +246
known Broadcom firmware bug [reported to Broadcom](https://github.com/t2linux/T2-Debian-and-Ubuntu-Kernel/issues)
with no upstream fix available. The workaround below resolves it.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you used the wrong link here

`GRUB_CMDLINE_LINUX_DEFAULT`, then update GRUB:
```bash
sudo update-grub
```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the correct way to change kernel parameters on all distros. On Fedora you should use grubby --args="mem_sleep_default=s2idle" --update-kernel=ALL, and some people with arch use systemd-boot which means you have to edit /etc/kernel/cmdline and then regenerate the BLS files.

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.

3 participants