Prashant Malani has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
ec/google/chromeec/acpi: Move ECPD under CREC
Move the ECPD (GOOG0003) device under CREC (GOOG0004) so that the ECPD AP device drivers can access the parent EC device to communicate with the EC. Also, update the Notify() call to reflect the new location of the ECPD device.
Signed-off-by: Prashant Malani pmalani@chromium.org Change-Id: I830b030c7a063506f50f9cd51df3a5018e248fc2 --- M src/ec/google/chromeec/acpi/cros_ec.asl M src/ec/google/chromeec/acpi/ec.asl 2 files changed, 5 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39469/1
diff --git a/src/ec/google/chromeec/acpi/cros_ec.asl b/src/ec/google/chromeec/acpi/cros_ec.asl index d41071e..bcf38d2 100644 --- a/src/ec/google/chromeec/acpi/cros_ec.asl +++ b/src/ec/google/chromeec/acpi/cros_ec.asl @@ -60,6 +60,10 @@ Name (_DDN, "EC Base Switch Device") } #endif + +#ifdef EC_ENABLE_PD_MCU_DEVICE + #include "pd.asl" +#endif Method(_STA, 0) { Return (0xB) diff --git a/src/ec/google/chromeec/acpi/ec.asl b/src/ec/google/chromeec/acpi/ec.asl index bf792d3..fa5eca6 100644 --- a/src/ec/google/chromeec/acpi/ec.asl +++ b/src/ec/google/chromeec/acpi/ec.asl @@ -344,7 +344,7 @@ Method (_Q16, 0, NotSerialized) { Store ("EC: GOT PD EVENT", Debug) - Notify (ECPD, 0x80) + Notify (_SB.PCI0.LPCB.EC0.CREC.ECPD, 0x80) } #endif
@@ -558,10 +558,6 @@ #include "keyboard_backlight.asl" #endif
-#ifdef EC_ENABLE_PD_MCU_DEVICE - #include "pd.asl" -#endif - #ifdef EC_ENABLE_TBMC_DEVICE #include "tbmc.asl" #endif
Prashant Malani has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
Patch Set 1:
Hey Furquan, PTAL.
Thanks!
Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
Patch Set 1: Code-Review+2
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
Patch Set 1: Code-Review+2
Prashant Malani has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
Patch Set 1:
Thanks for the quick reviews.
Since I lack V+1 or Submit privileges in Coreboot, could someone kindly flip those bits for this CL?
Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
Patch Set 1:
V+1 got set by the bot so it is good there. I can submit after giving 24h for the community to poke at it.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
Patch Set 1:
Patch Set 1:
Thanks for the quick reviews.
Since I lack V+1 or Submit privileges in Coreboot, could someone kindly flip those bits for this CL?
We need to wait for 24 hours before the change can be submitted. Patrick typically takes care of submitting the CL and cherry-picking to chromium tree.
Prashant Malani has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
Patch Set 1:
Patch Set 1:
Patch Set 1:
Thanks for the quick reviews.
Since I lack V+1 or Submit privileges in Coreboot, could someone kindly flip those bits for this CL?
We need to wait for 24 hours before the change can be submitted. Patrick typically takes care of submitting the CL and cherry-picking to chromium tree.
Got it. Thanks!
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
Patch Set 1: Code-Review+1
Duncan Laurie has submitted this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
ec/google/chromeec/acpi: Move ECPD under CREC
Move the ECPD (GOOG0003) device under CREC (GOOG0004) so that the ECPD AP device drivers can access the parent EC device to communicate with the EC. Also, update the Notify() call to reflect the new location of the ECPD device.
Signed-off-by: Prashant Malani pmalani@chromium.org Change-Id: I830b030c7a063506f50f9cd51df3a5018e248fc2 Reviewed-on: https://review.coreboot.org/c/coreboot/+/39469 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Duncan Laurie dlaurie@chromium.org Reviewed-by: Furquan Shaikh furquan@google.com Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net --- M src/ec/google/chromeec/acpi/cros_ec.asl M src/ec/google/chromeec/acpi/ec.asl 2 files changed, 5 insertions(+), 5 deletions(-)
Approvals: build bot (Jenkins): Verified Duncan Laurie: Looks good to me, approved Paul Menzel: Looks good to me, but someone else must approve Furquan Shaikh: Looks good to me, approved
diff --git a/src/ec/google/chromeec/acpi/cros_ec.asl b/src/ec/google/chromeec/acpi/cros_ec.asl index d41071e..bcf38d2 100644 --- a/src/ec/google/chromeec/acpi/cros_ec.asl +++ b/src/ec/google/chromeec/acpi/cros_ec.asl @@ -60,6 +60,10 @@ Name (_DDN, "EC Base Switch Device") } #endif + +#ifdef EC_ENABLE_PD_MCU_DEVICE + #include "pd.asl" +#endif Method(_STA, 0) { Return (0xB) diff --git a/src/ec/google/chromeec/acpi/ec.asl b/src/ec/google/chromeec/acpi/ec.asl index bf792d3..fa5eca6 100644 --- a/src/ec/google/chromeec/acpi/ec.asl +++ b/src/ec/google/chromeec/acpi/ec.asl @@ -344,7 +344,7 @@ Method (_Q16, 0, NotSerialized) { Store ("EC: GOT PD EVENT", Debug) - Notify (ECPD, 0x80) + Notify (_SB.PCI0.LPCB.EC0.CREC.ECPD, 0x80) } #endif
@@ -558,10 +558,6 @@ #include "keyboard_backlight.asl" #endif
-#ifdef EC_ENABLE_PD_MCU_DEVICE - #include "pd.asl" -#endif - #ifdef EC_ENABLE_TBMC_DEVICE #include "tbmc.asl" #endif
9elements QA has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39469 )
Change subject: ec/google/chromeec/acpi: Move ECPD under CREC ......................................................................
Patch Set 2:
Automatic boot test returned (PASS/FAIL/TOTAL): 3/0/3 Emulation targets: EMULATION_QEMU_X86_Q35 using payload TianoCore : SUCCESS : https://lava.9esec.io/r/1340 EMULATION_QEMU_X86_Q35 using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/1339 EMULATION_QEMU_X86_I440FX using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/1338
Please note: This test is under development and might not be accurate at all!