Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34801 )
Change subject: ec/google/chromeec: Depend on events_copy_b to identify the wake source. ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/34801/4/src/ec/google/chromeec/ec.c File src/ec/google/chromeec/ec.c:
https://review.coreboot.org/c/coreboot/+/34801/4/src/ec/google/chromeec/ec.c... PS4, Line 431: u64 nit: uint64_t
https://review.coreboot.org/c/coreboot/+/34801/4/src/ec/google/chromeec/ec.c... PS4, Line 439: /* : * If the EC supports unified wake masks, then there is no need to set : * wake mask before reading out the host events. : */ : if (google_chromeec_check_feature(EC_FEATURE_UNIFIED_WAKE_MASKS) != 1) { : wake_mask = google_chromeec_get_wake_mask(); : google_chromeec_set_wake_mask(mask); : restore_wake_mask = true; : } : Now that we are relying on events_copy_b there is no need to do this setting/restoring of wake masks any more. That was only required when reading the host events using the ACPI command. Can you please confirm if this is required now?