[coreboot-gerrit] Change in coreboot[master]: mb/acer: Add Acer Aspire VN7-572G

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35523 )

Change subject: mb/acer: Add Acer Aspire VN7-572G
......................................................................

Patch Set 99: Code-Review+1

(1 comment)

https://review.coreboot.org/c/coreboot/+/35523/99/src/mainboard/acer/aspire_... 
File src/mainboard/acer/aspire_vn7_572g/acpi/mainboard.asl:

https://review.coreboot.org/c/coreboot/+/35523/99/src/mainboard/acer/aspire_... 
PS99, Line 96: Device (SLPB)
             : 	{
             : 		Name (_HID, EisaId ("PNP0C0E") /* Sleep Button Device */)  // _HID: Hardware ID
             : 		Name (_PRW, Package (0x02)  // _PRW: Power Resources for Wake
             : 		{
             : 			0x0A,
             : 			0x03
             : 		})
             : 	}
> dmesg mentions a SLPB device just once. But would there have been a second? […]

I haven’t fully understood the sleep button stuff yet. There should be a fixed one too. You could just try to remove this, and see if the button still works with your port. Your call.