Felix Singer has submitted this change. ( https://review.coreboot.org/c/coreboot/+/60501 )
Change subject: ec/smsc/mec1308/acpi: Replace Subtract(a,b) with ASL 2.0 syntax ......................................................................
ec/smsc/mec1308/acpi: Replace Subtract(a,b) with ASL 2.0 syntax
Replace `Subtract (a, b)` with `a - b`.
Change-Id: Ia736b0080848e029eaeae094f1d2cc12aace3730 Signed-off-by: Felix Singer felixsinger@posteo.net Reviewed-on: https://review.coreboot.org/c/coreboot/+/60501 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: HAOUAS Elyes ehaouas@noos.fr --- M src/ec/smsc/mec1308/acpi/battery.asl 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified HAOUAS Elyes: Looks good to me, approved
diff --git a/src/ec/smsc/mec1308/acpi/battery.asl b/src/ec/smsc/mec1308/acpi/battery.asl index bbbcfe2..603b1d4 100644 --- a/src/ec/smsc/mec1308/acpi/battery.asl +++ b/src/ec/smsc/mec1308/acpi/battery.asl @@ -157,7 +157,7 @@
// See if within ~3% of full ShiftRight (Local2, 5, Local3) - If (LGreater (Local1, Subtract (Local2, Local3)) && + If (LGreater (Local1, Local2 - Local3) && LLess (Local1, Add (Local2, Local3))) { Store (Local2, Local1)
2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.