Support up to 12 fans on PowerMacs
ClosedPublic
Actions

Authored by tuexen on Oct 29 2018, 8:03 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 5, 5:28 AM

	Unknown Object (File)
	Wed, Dec 3, 5:34 AM

	Unknown Object (File)
	Fri, Nov 28, 3:54 PM

	Unknown Object (File)
	Nov 17 2025, 7:02 AM

	Unknown Object (File)
	Nov 9 2025, 11:48 PM

	Unknown Object (File)
	Nov 3 2025, 1:38 PM

	Unknown Object (File)
	Nov 3 2025, 1:37 PM

	Unknown Object (File)
	Nov 3 2025, 1:36 PM

View All Files

Subscribers

imp

jhibbits

Details

Reviewers

andreast
nwhitehorn
jhibbits

Commits

rS340101: MFC r339914:
rS339914: Bump the number of fans supported from 8 to 12.

Summary

The current code supports a hard limit of 8 fans on PowerMacs. However, the liquid cooled PowerMac7,3 has 9 fans and the last one is not registered successfully. Bump the parameters such that up to 12 fans are supported.

Test Plan

Boot a PowerMac with 9 fans and verify that all fans are registered successfully.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tuexen created this revision.Oct 29 2018, 8:03 PM

Herald added subscribers: jhibbits, imp. · View Herald TranscriptOct 29 2018, 8:03 PM

Harbormaster completed remote builds in B20503: Diff 49765.Oct 29 2018, 8:03 PM

tuexen added a reviewer: jhibbits.Oct 29 2018, 8:46 PM

Tested on my 7,3. Thanks Michael!

This revision is now accepted and ready to land.Oct 29 2018, 9:09 PM

Closed by commit rS339914: Bump the number of fans supported from 8 to 12. (authored by tuexen). · Explain WhyOct 30 2018, 11:51 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

powerpc/

powermac/

fcu.c

6 lines

Diff 49791

View Options

head/sys/powerpc/powermac/fcu.c

Support up to 12 fans on PowerMacsClosedPublicActions