summaryrefslogtreecommitdiff
path: root/src/soc/intel/jasperlake/acpi
diff options
context:
space:
mode:
authorV Sowmya <v.sowmya@intel.com>2020-05-28 22:29:52 +0530
committerPatrick Georgi <pgeorgi@google.com>2020-06-02 07:49:51 +0000
commit8e6fa57e523c594d278afa34fe0cc00d3e82b606 (patch)
tree5ef0859f6ec99e9ef2254421cfe6c6bb163483e1 /src/soc/intel/jasperlake/acpi
parent226715877611ab2307945f872b1cf324a4d2bf64 (diff)
soc/intel/jasperlake: Update gpio_op.asl to ASL2.0 syntax
This change updates gpio_op.asl to use ASL2.0 syntax. This increases the readability of the ASL code. TEST=Verified using --timeless option to abuild that the resulting coreboot.rom is same as without the ASL2.0 syntax changes for wdoo. Change-Id: I3ec442ad85f408135642a112873231ce7d39524e Signed-off-by: V Sowmya <v.sowmya@intel.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/41864 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Diffstat (limited to 'src/soc/intel/jasperlake/acpi')
-rw-r--r--src/soc/intel/jasperlake/acpi/gpio_op.asl32
1 files changed, 16 insertions, 16 deletions
diff --git a/src/soc/intel/jasperlake/acpi/gpio_op.asl b/src/soc/intel/jasperlake/acpi/gpio_op.asl
index 16e7690565..683686f3ca 100644
--- a/src/soc/intel/jasperlake/acpi/gpio_op.asl
+++ b/src/soc/intel/jasperlake/acpi/gpio_op.asl
@@ -11,7 +11,7 @@ Method (GRXS, 1, Serialized)
{
VAL0, 32
}
- And (PAD_CFG0_RX_STATE, ShiftRight (VAL0, PAD_CFG0_RX_STATE_BIT), Local0)
+ Local0 = PAD_CFG0_RX_STATE & (VAL0 >> PAD_CFG0_RX_STATE_BIT)
Return (Local0)
}
@@ -27,7 +27,7 @@ Method (GTXS, 1, Serialized)
{
VAL0, 32
}
- And (PAD_CFG0_TX_STATE, VAL0, Local0)
+ Local0 = PAD_CFG0_TX_STATE & VAL0
Return (Local0)
}
@@ -43,7 +43,7 @@ Method (STXS, 1, Serialized)
{
VAL0, 32
}
- Or (PAD_CFG0_TX_STATE, VAL0, VAL0)
+ VAL0 = PAD_CFG0_TX_STATE | VAL0
}
/*
@@ -57,7 +57,7 @@ Method (CTXS, 1, Serialized)
{
VAL0, 32
}
- And (Not (PAD_CFG0_TX_STATE), VAL0, VAL0)
+ VAL0 = ~PAD_CFG0_TX_STATE & VAL0
}
/*
@@ -76,10 +76,10 @@ Method (GPMO, 2, Serialized)
{
VAL0, 32
}
- Store (VAL0, Local0)
- And (Not (PAD_CFG0_MODE_MASK), Local0, Local0)
- And (ShiftLeft (Arg1, PAD_CFG0_MODE_SHIFT, Arg1), PAD_CFG0_MODE_MASK, Arg1)
- Or (Local0, Arg1, VAL0)
+ Local0 = VAL0
+ Local0 = ~PAD_CFG0_MODE_MASK & Local0
+ Arg1 = (Arg1 <<= PAD_CFG0_MODE_SHIFT) & PAD_CFG0_MODE_MASK
+ VAL0 = Local0 | Arg1
}
/*
@@ -97,10 +97,10 @@ Method (GTXE, 2, Serialized)
VAL0, 32
}
- If (LEqual (Arg1, 1)) {
- And (Not (PAD_CFG0_TX_DISABLE), VAL0, VAL0)
- } ElseIf (LEqual (Arg1, 0)){
- Or (PAD_CFG0_TX_DISABLE, VAL0, VAL0)
+ If (Arg1 == 1) {
+ VAL0 = ~PAD_CFG0_TX_DISABLE & VAL0
+ } ElseIf (Arg1 == 0){
+ VAL0 = PAD_CFG0_TX_DISABLE | VAL0
}
}
@@ -119,9 +119,9 @@ Method (GRXE, 2, Serialized)
VAL0, 32
}
- If (LEqual (Arg1, 1)) {
- And (Not (PAD_CFG0_RX_DISABLE), VAL0, VAL0)
- } ElseIf (LEqual (Arg1, 0)){
- Or (PAD_CFG0_RX_DISABLE, VAL0, VAL0)
+ If (Arg1 == 1) {
+ VAL0 = ~PAD_CFG0_RX_DISABLE & VAL0
+ } ElseIf (Arg1 == 0){
+ VAL0 = PAD_CFG0_RX_DISABLE | VAL0
}
}