Skip to content

asmc: Add support for MacbookPro11,4.#1366

Closed
MegaManSec wants to merge 1 commit intofreebsd:mainfrom
MegaManSec:asmc
Closed

asmc: Add support for MacbookPro11,4.#1366
MegaManSec wants to merge 1 commit intofreebsd:mainfrom
MegaManSec:asmc

Conversation

@MegaManSec
Copy link
Contributor

This patch adds support for asmc with the Macbookpro 11.4.

SMCs were (mostly) taken from https://logi.wiki/index.php/SMC_Sensor_Codes.

Three errors occur when running sysctl:
asmc0: asmc_key_read for key F1Sf failed 10 times, giving up
asmc0: asmc_key_read for key F0Sf failed 10 times, giving up
asmc0: asmc_key_read for key F0Sf failed 10 times, giving up

These correspond to:
dev.asmc.0.fan.1.safespeed
dev.asmc.0.fan.0.safespeed

but cannot be disabled without severely altering the driver, so I've left them as-is.

Keyboard backlight is working nicely, and fan and temp reporting is also working.

Copy link
Member

@markjdb markjdb left a comment

Choose a reason for hiding this comment

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

Thank you. Please squash the commits together.

Signed-off-by: Joshua Rogers <Joshua@Joshua.Hu>
@markjdb markjdb self-assigned this Aug 23, 2024
@markjdb markjdb added the ready Seems ready to go, subject to final sanity check label Aug 23, 2024
freebsd-git pushed a commit that referenced this pull request Aug 26, 2024
Signed-off-by: Joshua Rogers <Joshua@Joshua.Hu>

MFC after:	2 weeks
Pull Request:	#1366
@markjdb markjdb added the merged Closed commit that's been merged label Aug 26, 2024
@markjdb
Copy link
Member

markjdb commented Aug 26, 2024

Pushed to main, thank you.

@markjdb markjdb closed this Aug 26, 2024
freebsd-git pushed a commit that referenced this pull request Sep 20, 2024
Signed-off-by: Joshua Rogers <Joshua@Joshua.Hu>

MFC after:	2 weeks
Pull Request:	#1366

(cherry picked from commit 49a5fe1)
bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this pull request Dec 3, 2024
Signed-off-by: Joshua Rogers <Joshua@Joshua.Hu>

MFC after:	2 weeks
Pull Request:	freebsd/freebsd-src#1366
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged Closed commit that's been merged ready Seems ready to go, subject to final sanity check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants