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)
	Thu, Jul 2, 12:39 PM

	Unknown Object (File)
	Wed, Jul 1, 12:18 AM

	Unknown Object (File)
	Tue, Jun 30, 7:53 AM

	Unknown Object (File)
	Sun, Jun 28, 7:33 PM

	Unknown Object (File)
	Tue, Jun 23, 11:37 PM

	Unknown Object (File)
	Tue, Jun 23, 10:31 PM

	Unknown Object (File)
	Sun, Jun 21, 9:52 PM

	Unknown Object (File)
	May 21 2026, 9:49 AM

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