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)
	Mon, Aug 31, 3:26 AM

	Unknown Object (File)
	Sat, Aug 22, 5:25 PM

	Unknown Object (File)
	Fri, Aug 21, 7:54 AM

	Unknown Object (File)
	Tue, Aug 18, 10:30 AM

	Unknown Object (File)
	Mon, Aug 10, 7:14 AM

	Unknown Object (File)
	Sun, Aug 9, 10:27 PM

	Unknown Object (File)
	Sun, Aug 9, 4:48 PM

	Unknown Object (File)
	Thu, Aug 6, 8:05 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