Attention is currently required from: Paul Menzel, Keith Hui.
10 comments:
File src/mainboard/asus/p2b/suspend.asl:
Patch Set #2, Line 7: * ACPI message indicator.
Dot at the end not needed?
Gone
Patch Set #2, Line 33: /*SLST = Arg0*/
Add spaces around the comment characters?
Done
Patch Set #2, Line 63: Return(Package(0x02){0x00, 0x00})
Add a space at least before {?
Done
File src/mainboard/asus/p2b/suspend.asl:
Patch Set #4, Line 2: /* This file is part of the coreboot project. */
please remove
Gone
File src/mainboard/asus/p2b/variants/p3b-f/suspend.asl:
Patch Set #2, Line 169: * Prepare To Sleep,
Remove , at the end?
Gone
Patch Set #2, Line 175: SLST = Arg0
No idea if Firefox/Gerrit is broken, but there are no tabs shown here for indentation. […]
Done
File src/mainboard/asus/p2b/variants/p3b-f/suspend.asl:
Patch Set #4, Line 2: * This file is part of the coreboot project. *
please remove
Gone
Patch Set #4, Line 35: Store
you used ASL2. […]
Done
File src/mainboard/asus/p2b/variants/p3b-f/suspend.asl:
Patch Set #6, Line 217: Method (_MSG, 1, NotSerialized)
AAAAA! Spaces!
Done
double blank line
Done
To view, visit change 41098. To unsubscribe, or for help writing mail filters, visit settings.