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)
	Sun, Jul 26, 5:05 PM

	Unknown Object (File)
	Sat, Jul 25, 12:17 AM

	Unknown Object (File)
	Fri, Jul 10, 7:21 PM

	Unknown Object (File)
	Jul 2 2026, 12:39 PM

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

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

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

	Unknown Object (File)
	Jun 23 2026, 11:37 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