31 comments:
File src/mainboard/apple/macbookpro8_1/Makefile.inc:
You would want this to build with current master: […]
Done
File src/mainboard/apple/macbookpro8_1/cmos.layout:
Patch Set #8, Line 55: 408 1 e 1 nmi
Doesn't have a default
Done
Patch Set #8, Line 56: 409 2 e 7 power_on_after_fail
Doesn't have a default
Done
# coreboot config options: EC
#411 1 e 8 first_battery
#412 1 e 1 bluetooth
#413 1 e 1 wwan
#414 1 e 1 touchpad
#415 1 e 1 wlan
#416 1 e 1 trackpoint
#417 1 e 1 fn_ctrl_swap
#418 1 e 1 sticky_fn
#419 2 e 13 usb_always_on
#421 1 e 9 sata_mode
#422 2 e 10 backlight
# coreboot config options: cpu
#424 8 r 0 unused
# coreboot config options: northbridge
#432 5 e 11 gfx_uma_size
#437 3 r 0 unused
#440 8 h 0 volume
This is all commented-out
Done
2 0 Enable
2 1 Disable
Unused
Done
File src/mainboard/apple/macbookpro8_1/devicetree.cb:
Patch Set #8, Line 1: FIXME: check gfx.ndid and gfx.did
You might as well drop them
Done
Please "pick up" these `end` (put them on the previous line)
Done
Patch Set #8, Line 35: device domain 0x0 on
Right after this line, add: […]
Done
Patch Set #8, Line 38: register "docking_supported" = "0"
Can drop this
Done
Patch Set #8, Line 44: register "pcie_hotplug_map" = "{ 0, 0, 0, 0, 0, 0, 0, 0 }"
Can drop this
Done
Patch Set #8, Line 57: Unsupported PCI device 8086:1c2c
I think you can drop this
Done
Patch Set #8, Line 60: Audio controller
You can drop this redundant part
Done
Patch Set #8, Line 77: Unsupported PCI device 8086:1c27
I think you can drop this
Done
Patch Set #8, Line 81: PCI-LPC bridge
This can be dropped too
Done
Patch Set #8, Line 93: Host bridge
Redundant
Done
device pci 00.0 on # Host bridge Host bridge
subsystemid 0x106b 0x00db
end
device pci 01.0 off # PCIe Bridge for discrete graphics
subsystemid 0x106b 0x00db
end
device pci 02.0 on # Internal graphics VGA controller
subsystemid 0x106b 0x00db
end
This should go right after this line: […]
Done
Patch Set #8, Line 102: device pci 01.1 on
What's this?
Was added by autoport. Usually discrete graphics. Removed.
Patch Set #8, Line 105: device pci 1a.7 on
What's this?
No idea. I don't see any issues after removing.
Patch Set #8, Line 108: device pci 1d.7 on
What's this?
Same.
File src/mainboard/apple/macbookpro8_1/dsdt.asl:
Patch Set #8, Line 28: /* Some generic macros */
Please drop this comment
Done
File src/mainboard/apple/macbookpro8_1/hda_verb.c:
Patch Set #8, Line 21: Cirrus
Cirrus... […]
Yes.
Patch Set #8, Line 24: 0x0000000b
Replace with `12`. […]
But 0b is 11.
Patch Set #8, Line 25: /* NID 0x01: Subsystem ID. */
You can drop these comments
Done
Refers to codec number 0, so it should be decimal
Done
Same for all of these
Done
Patch Set #8, Line 57: 0x80862805, /* Codec Vendor / Device ID: Intel */
You might want to add some spacing between the two codecs
Done
Patch Set #8, Line 60: 0x00000004
Same as above
Done
Same as above
Done
Same as above
Done
File src/mainboard/apple/macbookpro8_1/romstage.c:
This was renamed to early_init. […]
Done
I would use the already-existing macros to do this
I guess it can be removed now.
To view, visit change 33151. To unsubscribe, or for help writing mail filters, visit settings.