search

zeule / asus-ec-sensors

Linux HWMON sensors driver for ASUS motherboards to get sensor readings from the embedded controller
GNU General Public License v2.0
45 stars 21 forks source link

Add support for Maximus XI Hero (WiFi) #24

Closed kyndigen closed 2 years ago

kyndigen commented 2 years ago

I have a patch to add support for the Maximus XI Hero (first discussed on kernel.org bug #204807). Based on feedback in that thread I've modified it to adjust the MAX_IDENTICAL_BOARD_VARIATIONS, and it works fine. I want to do a few more experiments to understand why it worked without that extra fix before I actually submit the pull request.

Expect to see something early next week. I figured I'd open an issue here to track it and acknowledge the conversation on the other bug thread.

Thanks, Kyndigen

kyndigen commented 2 years ago

Ah, the existing .board_names initializer is missing a comma, causing the first two names to get concatenated. I'll include a fix with my patch:

Jun 10 18:18:43 arrakis kernel: asus-ec-sensors PNP0C09:00: board_names[0] = ROG CROSSHAIR VIII FORMULAROG CROSSHAIR VIII HERO Jun 10 18:18:43 arrakis kernel: asus-ec-sensors PNP0C09:00: board_names[1] = ROG CROSSHAIR VIII HERO (WI-FI) Jun 10 18:18:43 arrakis kernel: asus-ec-sensors PNP0C09:00: board_names[2] = ROG MAXIMUS XI HERO (WI-FI)

kyndigen commented 2 years ago

dsdt.dsl.gz

Attached DSDT dump for the board.