14 comments:
though
Done
Patch Set #35, Line 18: Even tho it has a 16MB chip equipped (W25Q128.V) only the first 8MB are used and mapped. The rest should be left empty (0xFF)
These lines are still too long
Done
Patch Set #35, Line 19: ports
port's
Done
Not going to implement:
Does that mean never, or not in this commit, or something in between?
Changed to "Not part of this commit"
File src/mainboard/razer/blade_stealth_kbl/acpi/battery.asl:
Should be tabs everywhere btw
Done
dito
Done
File src/mainboard/razer/blade_stealth_kbl/acpi/ec.asl:
dito, for most of the file.
Done
Patch Set #35, Line 75: // Store Lids -> \Lids should be working...
Can you explain this comment in more detail, please? What works, what doesn't?
Was a leftover. Lid works.
File src/mainboard/razer/blade_stealth_kbl/board_info.txt:
Patch Set #35, Line 1: razer
This is inconsistent with the string in Kconfig (RAZER vs. razer); not sure if intended.
Done
File src/mainboard/razer/blade_stealth_kbl/devicetree.cb:
Patch Set #35, Line 274: subsystemid 0x1a58 0x6752
There is a way to set the subsystem ID once and for all PCI devices, which makes sense *if* the vend […]
Works without it. Discarded.
File src/mainboard/razer/blade_stealth_kbl/hda_verb.c:
The empty comment lines on this file could be removed
Done
The indentation is weird here (tab+space)
Done
File src/mainboard/razer/blade_stealth_kbl/mainboard.c:
Patch Set #35, Line 38: if (rdev_readat(&cbfs_region, serial_number, 0, serial_len)
[bikeshedding-grade] seeing this line being split makes me suffer. […]
Ack
File src/mainboard/razer/blade_stealth_kbl/spd/Makefile.inc:
Patch Set #35, Line 21: b0001
I think this should rather be index 0 (0b0000) and index 1 (0b0001)
Thanks
To view, visit change 34475. To unsubscribe, or for help writing mail filters, visit settings.