[coreboot-gerrit] New patch to review for coreboot: glados: use macros for magic numbers in ASL
Aaron Durbin (adurbin@chromium.org)
gerrit at coreboot.org
Fri Aug 14 16:07:46 CEST 2015
Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11234
-gerrit
commit 2a7ab589aed6e400612b4f2ea47cee7c6144c680
Author: Aaron Durbin <adurbin at chromium.org>
Date: Thu Aug 13 03:06:02 2015 -0500
glados: use macros for magic numbers in ASL
The skylake SoC code now has macros for the previously
hard-code numbers for IRQs and GPEs. Switch over to using
those as they bring a little more clarity.
BUG=chrome-os-partner:43522
BRANCH=None
TEST=Built and booted glados.
Original-Change-Id: Ic8fcc59d680cdddec9dfbc3bf679731f6d786793
Original-Signed-off-by: Aaron Durbin <adurbin at chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/293411
Original-Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
Change-Id: I594907005372100a3c9d17dda9d17769844ad272
Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
src/mainboard/google/glados/acpi/mainboard.asl | 6 ++++--
src/mainboard/google/glados/ec.h | 7 +++----
2 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/src/mainboard/google/glados/acpi/mainboard.asl b/src/mainboard/google/glados/acpi/mainboard.asl
index 53001e4..f8bc462 100644
--- a/src/mainboard/google/glados/acpi/mainboard.asl
+++ b/src/mainboard/google/glados/acpi/mainboard.asl
@@ -17,8 +17,10 @@
* Foundation, Inc.
*/
+#include <soc/gpio.h>
+
#define BOARD_TRACKPAD_I2C_ADDR 0x15
-#define BOARD_TRACKPAD_IRQ 0x33
+#define BOARD_TRACKPAD_IRQ GPP_B3_IRQ
Scope (\_SB)
{
@@ -31,7 +33,7 @@ Scope (\_SB)
}
/* EC wake is LAN_WAKE# which is a special DeepSX wake pin */
- Name (_PRW, Package(){ 112, 5 }) /* LAN_WAKE_EN */
+ Name (_PRW, Package(){ GPE0_LAN_WAK, 5 })
}
Device (PWRB)
diff --git a/src/mainboard/google/glados/ec.h b/src/mainboard/google/glados/ec.h
index d43b57c..bfe4a11 100644
--- a/src/mainboard/google/glados/ec.h
+++ b/src/mainboard/google/glados/ec.h
@@ -23,11 +23,10 @@
#include <ec/google/chromeec/ec_commands.h>
#include <soc/gpio.h>
+#include <soc/gpe.h>
-/* GPP_E16 is EC_SCI_L, however the EC_SCI_GPI needs to be a bit
- * number relative to the GPE0 block. GPP_E is routed as the dword 2
- * in the GPE0 block. Therefore, 16 + 2 * 32 = 80. */
-#define EC_SCI_GPI 80
+/* GPP_E16 is EC_SCI_L. GPP_E group is routed to dword 2 in the GPE0 block. */
+#define EC_SCI_GPI GPE0_DW2_16
#define EC_SMI_GPI GPP_E15
#define MAINBOARD_EC_SCI_EVENTS \
More information about the coreboot-gerrit
mailing list